Closed p0lyn0mial closed 1 month ago
Hello @p0lyn0mial! Some important instructions when contributing to openshift/api: API design plays an important part in the user experience of OpenShift and as such API PRs are subject to a high level of scrutiny to ensure they follow our best practices. If you haven't already done so, please review the OpenShift API Conventions and ensure that your proposed changes are compliant. Following these conventions will help expedite the api review process for your PR.
@p0lyn0mial: This pull request references Jira Issue OCPBUGS-35906, which is invalid:
Comment /jira refresh
to re-evaluate validity if changes to the Jira bug are made, or edit the title of this pull request to link to a different bug.
The bug has been updated to refer to the pull request using the external bug tracker.
/assign @deads2k
add testing
@deads2k added the missing test cases, PTAL.
Setting different values caused failures, so this is ok.
/lgtm /approve
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: deads2k, p0lyn0mial
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/jira refresh
@p0lyn0mial: This pull request references Jira Issue OCPBUGS-35906, which is invalid:
Comment /jira refresh
to re-evaluate validity if changes to the Jira bug are made, or edit the title of this pull request to link to a different bug.
/jira refresh
@p0lyn0mial: This pull request references Jira Issue OCPBUGS-35906, which is valid. The bug has been moved to the POST state.
Requesting review from QA contact: /cc @gangwgr
/test e2e-aws-ovn-techpreview
/cherry-pick release-4.16
@p0lyn0mial: once the present PR merges, I will cherry-pick it on top of release-4.16 in a new PR and assign it to you.
/retest-required
Remaining retests: 0 against base HEAD c299a673ec729b7e63a6c363d67e961454218e09 and 2 for PR HEAD 213d2faf133a9b931e48d50a8e382a3c8c3e7809 in total
/retest-required
Remaining retests: 0 against base HEAD 1cd5e299324762879b66c5f5c1c279a98367ee60 and 1 for PR HEAD 213d2faf133a9b931e48d50a8e382a3c8c3e7809 in total
/retest-required
Remaining retests: 0 against base HEAD 868ddb3cbdfc21a20dc5e68f32626e624c816ab6 and 0 for PR HEAD 213d2faf133a9b931e48d50a8e382a3c8c3e7809 in total
@p0lyn0mial: all tests passed!
Full PR test history. Your PR dashboard.
@p0lyn0mial: Jira Issue OCPBUGS-35906: All pull requests linked via external trackers have merged:
Jira Issue OCPBUGS-35906 has been moved to the MODIFIED state.
@p0lyn0mial: new pull request created: #1985
[ART PR BUILD NOTIFIER]
Distgit: ose-cluster-config-api This PR has been included in build ose-cluster-config-api-container-v4.18.0-202408020343.p0.g0271697.assembly.stream.el9. All builds following this will include this PR.
In the past, there was an admission plugin for checking if the value of the featureSet field matched a known one.
With https://github.com/openshift/kubernetes/pull/1944, the validation was moved to CEL and the check was dropped. This PR adds the missing check.
NOTE that there was an additional check that allowed updates to FG even with an invalid featureSet if the field wasn't modified. I haven't found a way to move this to CEL for kube < 1.30.
The validation was moved in 4.16, and setting an invalid value breaks an operator and prevents upgrades. This PR allows transitioning from an invalid value to unblock upgrades.
For full backward compatibility we would have to bring back the admission plugin for 4.16 and use for 4.17 use
optionalOldSelf
.