-
Notifications
You must be signed in to change notification settings - Fork 579
MCO-1524: Promote PinnedImageSet feature gate to default feature set #2269
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
MCO-1524: Promote PinnedImageSet feature gate to default feature set #2269
Conversation
|
Hello @RishabhSaini! Some important instructions when contributing to openshift/api: |
|
@RishabhSaini: This pull request references MCO-1524 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.19.0" version, but no target version was set. 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 openshift-eng/jira-lifecycle-plugin repository. |
|
/test verify-feature-promotion |
|
/retest |
|
/hold This depends on the GA of the MachineConfigNode feature (#2288). |
|
/test verify-feature-promotion |
a25a302 to
75ea6e1
Compare
75ea6e1 to
7171625
Compare
|
/hold cancel #2288 has merged and this can go ahead. Note that much like #2288 (comment) , the test names were changed. The failures in verify-crd-schema seem to be related to legacy MCP fields since this changes that API. Perhaps we should override that since we don't want to change any legacy fields in this PR? |
|
Reviewing the promotion results, since the tests were renamed, we have the 14 required runs and there have been no failures, showing 100% across the board. Taking into account the renames, the worst pass rate is 94%, but since these results are older than a week, we can discount these failures now /lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: JoelSpeed, RishabhSaini 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 |
|
@JoelSpeed: Overrode contexts on behalf of JoelSpeed: ci/prow/verify-crd-schema, ci/prow/verify-feature-promotion 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-sigs/prow repository. |
|
@RishabhSaini: The following tests failed, say
Full PR test history. Your PR dashboard. 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. I understand the commands that are listed here. |
features: Lift feature gate for PIS