✨ alpha(update): add --squash, --preserve-path, --output-branch for PR-friendly upgrades #5002
+337
−62
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
k8s-ci-robot
added
the
cncf-cla: yes
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: camilamacedo86 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
k8s-ci-robot
added
approved
camilamacedo86
force-pushed
the
add-more-options-alpha
branch
2 times, most recently
from
5e5b396 to
e647ab5
Compare
k8s-ci-robot
added
size/XL
camilamacedo86
force-pushed
the
add-more-options-alpha
branch
2 times, most recently
from
88cb831 to
d986484
Compare
k8s-ci-robot
added
size/L
…friendly upgrades
This change makes `kubebuilder alpha update` produce a PR-ready branch and a
single squashed commit when requested, improving automation and review UX.
Key changes
-----------
• New `--squash` flag:
- Snapshots the exact tree of the temporary merge branch into ONE commit on a
stable branch: `kubebuilder-alpha-update-to-<to-version>`.
- Intended for opening/refreshing idempotent PRs.
- Gracefully handles "no changes" (git commit exits 1 → treated as no-op).
• New `--preserve-path` (repeatable):
- When squashing, restore given paths from the base branch (e.g.
`.github/workflows`) so CI/config files are kept as-is on the PR branch.
• New `--output-branch`:
- Overrides the default PR branch name created by `--squash`.
• Commit message used by `--squash`:
- `[kubebuilder-automated-update]: update scaffold from <from> to <to>; (squashed 3-way merge)`
• Behavior/ergonomics:
- Without `--force`: stops on conflicts on the temporary merge branch.
- With `--force`: commits conflict markers on the merge branch (automation-friendly).
- After merge, still best-effort run: `make manifests generate fmt vet lint-fix`.
Defaults / Compatibility
------------------------
- `--squash` is off by default (no behavior change unless opted-in).
- `--from-branch` defaults to `main`.
- `--preserve-path` is empty by default (no restores).
- Safe to run on projects scaffolded with v4.5.0+ (uses `alpha generate`).
Motivation
----------
Make upgrades PR-centric and automation-ready by producing a deterministic,
reviewable branch and a single squashed commit that mirrors the merge result.
Assisted-by: ChatGPT (OpenAI)
Co-authored-by: Vitor Floriano <[email protected]>
camilamacedo86
force-pushed
the
add-more-options-alpha
branch
from
d986484 to
0d40b9c
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This change makes
kubebuilder alpha updateproduce a PR-ready branch and a single squashed commit when requested, improving automation and review UX.Key changes
• New
--squashflag:kubebuilder-alpha-update-to-<to-version>.• New
--preserve-path(repeatable):.github/workflows) so CI/config files are kept as-is on the PR branch.• New
--output-branch:--squash.• Commit message used by
--squash:[kubebuilder-automated-update]: update scaffold from <from> to <to>; (squashed 3-way merge)• Behavior/ergonomics:
--force: stops on conflicts on the temporary merge branch.--force: commits conflict markers on the merge branch (automation-friendly).make manifests generate fmt vet lint-fix.Defaults / Compatibility
--squashis off by default (no behavior change unless opted-in).--from-branchdefaults tomain.--preserve-pathis empty by default (no restores).alpha generate).Motivation
Make upgrades PR-centric and automation-ready by producing a deterministic, reviewable branch and a single squashed commit that mirrors the merge result.
Assisted-by: ChatGPT (OpenAI)