-
Notifications
You must be signed in to change notification settings - Fork 526
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
Added Feature Gate for On Cluster Build #1696
Added Feature Gate for On Cluster Build #1696
Conversation
Hello @dkhater-redhat! Some important instructions when contributing to openshift/api: |
This is for openshift/enhancements#1515, are we cool with that enhancement still being in Draft for now, or does it need to be "proper open" to get a feature gate? |
/test e2e-gcp |
Implementation would be: openshift/machine-config-operator#4060 |
@dkhater-redhat: The following test 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/test-infra repository. I understand the commands that are listed here. |
/lgtm |
Can someone confirm for me why we are adding a feature gate this late in the cycle? Has the feature been merged and we now want to gate it because it isn't ready? Or is this a net new feature that isn't currently available at all, but will be added later? |
The feature we are gating was initially merged in 4.14. When we were discussing the feature then, we were ambivalent to gate it because you have to "opt-in" to the feature anyways. Furthermore, the feature in 4.14 is in dev preview so we didn't see it necessary to gate it (as it is unsupported anyways). You are definitely right on it being late in the cycle- but we were wanting to land it in 4.15 so we can get it in before tech preview. |
/lgtm Discussed in slack, gating for a feature that was in dev preview and undocumented, should be no existing users |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dkhater-redhat, JoelSpeed, mrunalp 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 |
Added Feature Gate for On Cluster Build Signed-off-by: Charlie Doern <[email protected]>
[ART PR BUILD NOTIFIER] This PR has been included in build ose-cluster-config-api-container-v4.16.0-202312071632.p0.g5f15541.assembly.stream for distgit ose-cluster-config-api. |
This is for openshift/enhancements#1515