Skip to content

Clarify calicoctl datastore migrate-policy-names help text (release-v3.32) - #13303

Merged
caseydavenport merged 1 commit into
projectcalico:release-v3.32from
caseydavenport:casey-migrate-policy-names-help-v332
Jul 28, 2026
Merged

Clarify calicoctl datastore migrate-policy-names help text (release-v3.32)#13303
caseydavenport merged 1 commit into
projectcalico:release-v3.32from
caseydavenport:casey-migrate-policy-names-help-v332

Conversation

@caseydavenport

Copy link
Copy Markdown
Member

release-v3.32 equivalent of #13302.

The migrate-policy-names one-line help said it drops "the tier prefix", but the command only rewrites default-tier policy names. Updated the summary to name the legacy "default." prefix specifically, matching the fuller description the subcommand already prints.

This is a separate change from #13302 rather than an auto-pick, because calicoctl still uses docopt on this branch (docopt was removed on master in #12624), so the wording lives in a different place here.

Companion docs update: tigera/docs#2877.

None

The command only rewrites default-tier policy names, so the one-line
summary now names the legacy "default." prefix specifically, matching
the fuller description the subcommand already prints.
Copilot AI review requested due to automatic review settings July 23, 2026 15:48
@caseydavenport
caseydavenport requested a review from a team as a code owner July 23, 2026 15:48
@caseydavenport caseydavenport added docs-not-required Docs not required for this change release-note-not-required Change has no user-facing impact labels Jul 23, 2026
@marvin-tigera marvin-tigera added this to the Calico v3.32.2 milestone Jul 23, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates calicoctl’s datastore migrate-policy-names one-line help text on the release-v3.32 branch to accurately describe what the command rewrites (the legacy default. tier prefix) rather than implying it drops an arbitrary tier prefix. This aligns the top-level datastore command summary with the subcommand’s more detailed description.

Changes:

  • Refine migrate-policy-names summary text to specify the legacy "default." tier prefix.
  • Improve consistency between the datastore command list and the subcommand help description.

@caseydavenport
caseydavenport merged commit 2231f46 into projectcalico:release-v3.32 Jul 28, 2026
4 checks passed
@caseydavenport
caseydavenport deleted the casey-migrate-policy-names-help-v332 branch July 28, 2026 23:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs-not-required Docs not required for this change release-note-not-required Change has no user-facing impact

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants