self-node-remediation icon indicating copy to clipboard operation
self-node-remediation copied to clipboard

Support Multiple Same Kind Remediation

Open clobrano opened this issue 1 year ago • 3 comments

Why we need this PR

This PR will enable the user to select several SNR Templates (of the same kind) when using NHC escalating remediation.

This feature is dependent on the changes in NHC as well (which is available starting from NHC 0.8.0).

Changes made

  • Automatically add the MultipleTemplatesSupportedAnnotation that enables this feature via "Defaulting" (Mutating) webhook.
  • Support that Node's name might come from NodeNameAnnotation instead than from remediation's name (when the feature is enabled).
  • Updated Common package to the version shipping the new annotations needed by this feature.
  • Test coverage for both configurations (supported and unsupported multiple same kind remediation).

Which issue(s) this PR fixes

https://issues.redhat.com/browse/ECOPROJECT-1930

Test plan

clobrano avatar May 14 '24 09:05 clobrano

Skipping CI for Draft Pull Request. If you want CI signal for your change, please convert it to an actual PR. You can still manually trigger a test run with /test all

openshift-ci[bot] avatar May 14 '24 09:05 openshift-ci[bot]

/test 4.15-openshift-e2e

clobrano avatar May 14 '24 11:05 clobrano

/test 4.15-openshift-e2e

clobrano avatar May 20 '24 07:05 clobrano

/test 4.15-openshift-e2e

clobrano avatar May 20 '24 13:05 clobrano

/lgtm

mshitrit avatar May 28 '24 10:05 mshitrit

/retest

clobrano avatar May 28 '24 14:05 clobrano

/retest

clobrano avatar May 28 '24 17:05 clobrano

/retest

clobrano avatar May 29 '24 12:05 clobrano

/retest

clobrano avatar Jun 03 '24 07:06 clobrano

/test 4.15-openshift-e2e

clobrano avatar Jun 05 '24 13:06 clobrano

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: clobrano, slintes

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:
  • ~~OWNERS~~ [clobrano,slintes]

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

openshift-ci[bot] avatar Jun 11 '24 13:06 openshift-ci[bot]

/retest

clobrano avatar Jun 12 '24 09:06 clobrano

/test 4.13-openshift-e2e

clobrano avatar Jun 12 '24 11:06 clobrano

the problem on 4.16 does not depend on us /override "ci/prow/4.16-openshift-e2e"

clobrano avatar Jun 12 '24 13:06 clobrano

@clobrano: Overrode contexts on behalf of clobrano: ci/prow/4.16-openshift-e2e

In response to this:

the problem on 4.16 does not depend on us /override "ci/prow/4.16-openshift-e2e"

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.

openshift-ci[bot] avatar Jun 12 '24 13:06 openshift-ci[bot]