-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
🌱 Add custom upgrade option to e2e #7341
🌱 Add custom upgrade option to e2e #7341
Conversation
Hi @mboukhalfa. Thanks for your PR. I'm waiting for a kubernetes-sigs member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/ok-to-test |
Thx! /lgtm /cherry-pick release-1.2 |
@sbueringer: once the present PR merges, I will cherry-pick it on top of release-1.2 in a new PR and assign it to you. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/hold |
1b69f90
to
93324a7
Compare
88c7f88
to
da12861
Compare
@sbueringer @fabriziopandini I added a test commit to verify that these changes can help with custom upgrade to a pre-release can we run full e2e on that? |
Sure! good idea /test pull-cluster-api-e2e-full-main |
Hmm 0.4 to custom upgrade |
/test pull-cluster-api-e2e-full-main |
1 similar comment
/test pull-cluster-api-e2e-full-main |
/unhold |
aa36cbd
to
e153d11
Compare
fcef696
to
9845ba5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Last one from my side
Once we basically have lgtm from me and Fabrizio.
Do you want to do another test with setting providers and ensuring it still works after the changes?
9845ba5
to
82fbe79
Compare
Added a test commit I will run full e2e |
/test pull-cluster-api-e2e-full-main |
as the full e2e passed I will remove the test commit then the PR will ready to go in |
82fbe79
to
48ad7f5
Compare
Thank you! /lgtm |
Great work! |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: fabriziopandini 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 |
@sbueringer: #7341 failed to apply on top of branch "release-1.2":
In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
What this PR does / why we need it:
Add custom upgrade option to
UpgradeManagementClusterAndWait
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Fixes # 7311