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

(feat): added status based policy creation logic #212

Merged
merged 7 commits into from
Jul 5, 2024

Conversation

VedRatan
Copy link
Collaborator

@VedRatan VedRatan commented Jul 2, 2024

Description

  • In this feature we watch for the status of Kyveno policy to be in ready phase, only then we'll create a trigger for the existing resources to apply the mutation via Kyverno mutation policy.
  • Couple of bug fixes are also included in this PR.
  • The status logic is being written referring kyverno code-desc

Fixes #207
Does this PR introduce a breaking change?

Checklist

  • PR title follows the <type>: <description> convention
  • I use conventional commits in my commit messages
  • I have updated the documentation accordingly
  • I Keep It Small and Simple: The smaller the PR is, the easier it is to review and have it merged
  • I have performed a self-review of my code
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

Additional information for reviewer

Mention if this PR is part of any design or a continuation of previous PRs

Signed-off-by: Ved Ratan <vedratan8@gmail.com>
Signed-off-by: Ved Ratan <vedratan8@gmail.com>
Signed-off-by: Ved Ratan <vedratan8@gmail.com>
Signed-off-by: Ved Ratan <vedratan8@gmail.com>
Signed-off-by: Ved Ratan <vedratan8@gmail.com>
Signed-off-by: Ved Ratan <vedratan8@gmail.com>
@shivaccuknox shivaccuknox merged commit 21f617f into 5GSEC:main Jul 5, 2024
12 checks passed
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.

coco kyverno: When an coco intent is applied, the runtime class is not being mutated
2 participants