Skip to content
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

NDEV-20176 : backport allow changes for preexisting resources that violate policy #148

Open
wants to merge 4 commits into
base: release-1.10-n4k
Choose a base branch
from

Conversation

VedRatan
Copy link

Explanation

This PR backports the upstream fix, that allows the mutation of the resources which violates the policies.

Related issue

JIRA

Milestone of this PR

Documentation (required for features)

My PR contains new or altered behavior to Kyverno.

What type of PR is this

Proposed Changes

Proof Manifests

Checklist

  • I have read the contributing guidelines.
  • I have read the PR documentation guide and followed the process including adding proof manifests to this PR.
  • This is a bug fix and I have added unit tests that prove my fix is effective.
  • This is a feature and I have added CLI tests that are applicable.
  • My PR needs to be cherry picked to a specific release branch which is .
  • My PR contains new or altered behavior to Kyverno and
    • CLI support should be added and my PR doesn't contain that functionality.

Further Comments

Signed-off-by: Ved Ratan <[email protected]>
Signed-off-by: Ved Ratan <[email protected]>
@vishal-chdhry
Copy link

@VedRatan can you also add the kuttl test (translate them from chainsaw)

@anushkamittal2001
Copy link

@VedRatan the lint and unit test workflow is failing. Can you look at that as well please!

@@ -33,6 +34,7 @@ func (h mutateExistingHandler) Process(
resource unstructured.Unstructured,
rule kyvernov1.Rule,
contextLoader engineapi.EngineContextLoader,
exceptions []*v2alpha1.PolicyException,

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

exception changes are unrelated to the PR

Signed-off-by: Ved Ratan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants