-
Notifications
You must be signed in to change notification settings - Fork 223
OCPBUGS-71234: Permit z-stream upgrades when custom featureGates are enabled #6083
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
Conversation
… upgrades add unit test for upgrade validation
|
@copejon: This pull request references Jira Issue OCPBUGS-71234, which is invalid:
Comment The bug has been updated to refer to the pull request using the external bug tracker. DetailsIn 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 openshift-eng/jira-lifecycle-plugin repository. |
pkg/admin/prerun/featuregate_lock.go
Outdated
| "4. Restart MicroShift: sudo systemctl restart microshift", | ||
| "- Roll back MicroShift to version %s, OR\n"+ | ||
| "- Run: sudo microshift-cleanup-data --all\n"+ | ||
| " and remove custom feature gates from /etc/microshift/config.yaml before restarting MicroShift.", |
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.
I prefer previous version of instruction with number points. From my point of view it's more readable.
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.
reverted
| lockFileVer: versionMetadata{Major: 5, Minor: 0, Patch: 0}, | ||
| currentVer: versionMetadata{Major: 4, Minor: 18, Patch: 0}, | ||
| wantErr: true, | ||
| description: "Major version downgrade (5.0.0 -> 4.18.0) should be blocked", |
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.
I don't know if I've understood this change correctly but these unit tests seems to be wrong or at least misleading because FeatureSet feature does not exist before 4.21 so I wouldn't use versions previous like 4.18 or 4.19. Also, I don't think we should test downgrade paths because they are not supported.
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.
Makes sense. The values themselves are unimportant so long as they represent different upgrade scenarios.
|
Pausing until we have a decision on MicroShift supporting specific tech-preview features /hold |
|
/unhold |
Co-authored-by: Alejandro Gullón <agullon@redhat.com>
|
/test e2e-aws-tests-bootc-arm |
|
@copejon: all tests passed! Full PR test history. Your PR dashboard. DetailsInstructions 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-sigs/prow repository. I understand the commands that are listed here. |
copejon
left a comment
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.
/jira refresh
pkg/admin/prerun/featuregate_lock.go
Outdated
| "4. Restart MicroShift: sudo systemctl restart microshift", | ||
| "- Roll back MicroShift to version %s, OR\n"+ | ||
| "- Run: sudo microshift-cleanup-data --all\n"+ | ||
| " and remove custom feature gates from /etc/microshift/config.yaml before restarting MicroShift.", |
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.
reverted
|
it looks good to me |
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: copejon, pmtk The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
/jira refresh |
|
@copejon: This pull request references Jira Issue OCPBUGS-71234, which is invalid:
Comment DetailsIn 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 openshift-eng/jira-lifecycle-plugin repository. |
|
/jira refresh |
|
@copejon: This pull request references Jira Issue OCPBUGS-71234, which is invalid:
Comment DetailsIn 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 openshift-eng/jira-lifecycle-plugin repository. |
|
/jira refresh |
|
@copejon: This pull request references Jira Issue OCPBUGS-71234, which is invalid:
Comment DetailsIn 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 openshift-eng/jira-lifecycle-plugin repository. |
|
/jira refresh |
|
@copejon: This pull request references Jira Issue OCPBUGS-71234, which is valid. The bug has been moved to the POST state. 3 validation(s) were run on this bug
Requesting review from QA contact: DetailsIn 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 openshift-eng/jira-lifecycle-plugin repository. |
|
/verified by jcope,ci |
|
@copejon: This PR has been marked as verified by DetailsIn 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 openshift-eng/jira-lifecycle-plugin repository. |
|
@copejon: Jira Issue OCPBUGS-71234: All pull requests linked via external trackers have merged: All linked pull requests have the DetailsIn 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 openshift-eng/jira-lifecycle-plugin repository. |
|
/cherrypick release-4.21 |
|
@agullon: new pull request created: #6161 DetailsIn 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-sigs/prow repository. |
|
@copejon: new pull request could not be created: failed to create pull request against openshift/microshift#release-4.21 from head openshift-cherrypick-robot:cherry-pick-6083-to-release-4.21: status code 422 not one of [201], body: {"message":"Validation Failed","errors":[{"resource":"PullRequest","code":"custom","message":"A pull request already exists for openshift-cherrypick-robot:cherry-pick-6083-to-release-4.21."}],"documentation_url":"https://docs.github.com/rest/pulls/pulls#create-a-pull-request","status":"422"} DetailsIn 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-sigs/prow repository. |
No description provided.