8000 feat: Init DeletingPolicy API and Compiler by fjogeleit · Pull Request #13008 · kyverno/kyverno · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

feat: Init DeletingPolicy API and Compiler #13008

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

feat: Init DeletingPolicy API and Compiler #13008

wants to merge 2 commits into from

Conversation

fjogeleit
Copy link
Member
@fjogeleit fjogeleit commented May 5, 2025 •
edited
Loading

Explanation

Init DeletingPolicy API

Related issue

#12839

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

@dosubot dosubot bot added the size:XXL This PR changes 1000+ lines, ignoring generated files. label May 5, 2025
@fjogeleit fjogeleit marked this pull request as draft May 5, 2025 08:38
@codecov Codecov
Copy link
codecov bot commented May 5, 2025 •
edited
Loading

Codecov Report

Attention: Patch coverage is 0% with 506 lines in your changes missing coverage. Please review.

Project coverage is 15.25%. Comparing base (b634cbd) to head (92e4a8d).

Files with missing lines Patch % Lines
...iesv1alpha1/deletingpolicies/resource.generated.go 0.00% 303 Missing ⚠️
pkg/cel/policies/dpol/compiler/compiler.go 0.00% 67 Missing ⚠️
pkg/cel/policies/dpol/compiler/policy.go 0.00% 62 Missing ⚠️
...ons/policies.kyverno.io/v1alpha1/deletingpolicy.go 0.00% 21 Missing ⚠️
...es.kyverno.io/v1alpha1/fake/fake_deletingpolicy.go 0.00% 16 Missing ⚠️
...ped/policies.kyverno.io/v1alpha1/deletingpolicy.go 0.00% 11 Missing ⚠️
pkg/cel/policies/dpol/compiler/eval.go 0.00% 9 Missing ⚠️
...ients/kyverno/policiesv1alpha1/client.generated.go 0.00% 7 Missing ⚠️
...o/v1alpha1/fake/fake_policies.kyverno.io_client.go 0.00% 2 Missing ⚠️
....kyverno.io/v1alpha1/policies.kyverno.io_client.go 0.00% 2 Missing ⚠️
... and 3 more
Additional details and impacted files 
@@            Coverage Diff             @@
##             main   #13008      +/-   ##
==========================================
- Coverage   15.33%   15.25%   -0.08%     
==========================================
  Files         923      931       +8     
  Lines      100275   100781     +506     
==========================================
+ Hits        15373    15376       +3     
- Misses      83189    83693     +504     
+ Partials     1713     1712       -1

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@fjogeleit fjogeleit force-pushed the cleanuppolicy branch 3 times, most recently from 009c9db to 87d2840 Compare May 9, 2025 06:55
@fjogeleit fjogeleit changed the title wip feat: Init DeletingPolicy API and Compiler May 9, 2025
@fjogeleit fjogeleit requested a review from MariamFahmy98 May 9, 2025 06:57
@fjogeleit fjogeleit force-pushed the cleanuppolicy branch 2 times, most recently from f94531c to 1b11305 Compare May 9, 2025 06:59
@fjogeleit
Implement DeletingPolicy API and Compiler
0594c10 
Signed-off-by: Frank Jogeleit <frank.jogeleit@web.de>
@fjogeleit fjogeleit marked this pull request as ready for review May 9, 2025 07:00
@fjogeleit fjogeleit added this to the Kyverno Release 1.15.0 milestone May 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eddycharly eddycharly Awaiting requested review from eddycharly eddycharly is a code owner

@realshuting realshuting Awaiting requested review from realshuting realshuting is a code owner

@MariamFahmy98 MariamFahmy98 Awaiting requested review from MariamFahmy98

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
cherry-pick-required milestone 1.15.0 size:XXL This PR changes 1000+ lines, ignoring generated files.
Projects
None yet
Milestone
Kyverno Release 1.15.0
Development

Successfully merging this pull request may close these issues.
1 participant
@fjogeleit
0