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
Bug 1868304: remove ValidatingWebhookConfiguration for downgrades #909
Bug 1868304: remove ValidatingWebhookConfiguration for downgrades #909
Conversation
@simonpasquier: This pull request references Bugzilla bug 1869301, which is invalid:
Comment 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. |
ping @bparees as discussed OOB to override the BZ rules here :-) |
4c308d2
to
0db47ab
Compare
@simonpasquier: This pull request references Bugzilla bug 1868304, which is invalid:
Comment 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. |
/lgtm |
https://bugzilla.redhat.com/show_bug.cgi?id=1868304 really should target 4.5.z since the change is to 4.5.z. i'm going to fix its target. i will also set https://bugzilla.redhat.com/show_bug.cgi?id=1869301 as a 4.6.0 target (which you've already closed as fixed, which is fine since it's not actually relevant to 4.6.0). |
/bugzilla refresh |
@bparees: This pull request references Bugzilla bug 1868304, which is invalid:
Comment 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. |
ok this should be able to merge now. |
/retest Please review the full test history for this PR and help us cut down flakes. |
1 similar comment
/retest Please review the full test history for this PR and help us cut down flakes. |
@simonpasquier i believe we need to add the delete permission to CMO:
|
/retest Please review the full test history for this PR and help us cut down flakes. |
3 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
10 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/hold |
When downgrading from 4.6 to 4.5, the operator needs to remove the ValidatingWebhookConfiguration resource used to validate rules in 4.6. Signed-off-by: Simon Pasquier <spasquie@redhat.com>
0db47ab
to
d590d3c
Compare
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: s-urbaniak, simonpasquier 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 |
/retest |
1 similar comment
/retest |
/hold cancel |
/retest Please review the full test history for this PR and help us cut down flakes. |
1 similar comment
/retest Please review the full test history for this PR and help us cut down flakes. |
@simonpasquier: All pull requests linked via external trackers have merged: openshift/cluster-monitoring-operator#909. Bugzilla bug 1868304 has been moved to the MODIFIED state. 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. |
When downgrading from 4.6 to 4.5, the operator needs to remove the
ValidatingWebhookConfiguration resource used to validate rules in 4.6.
Signed-off-by: Simon Pasquier spasquie@redhat.com