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.

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

Add kyverno Policies endpoints #7106

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Add kyverno Policies endpoints #7106

wants to merge 8 commits into from
+16,598 −85

Conversation

ahmadhamzh
Copy link
Contributor

@ahmadhamzh ahmadhamzh commented Jan 29, 2025 •
edited
Loading

What this PR does / why we need it:
Creates endpoints for managing policy Templates and policy bindings

Which issue(s) this PR fixes:

Ref #7006

What type of PR is this?
/kind feature

Add new API endpoints for Kyverno integration.

Documentation:

TBD

@kubermatic-bot kubermatic-bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jan 29, 2025
@kubermatic-bot
Copy link
Contributor

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

@kubermatic-bot kubermatic-bot added dco-signoff: yes Denotes that all commits in the pull request have the valid DCO signoff message. do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. do-not-merge/docs-needed Indicates that a PR should not merge because it's missing one of the documentation labels. sig/api Denotes a PR or issue as being assigned to SIG API. do-not-merge/code-freeze Indicates that a PR should not merge because it has not been approved for code freeze yet. labels Jan 29, 2025
@kubermatic-bot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign ahmedwaleedmalik for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

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

@kubermatic-bot kubermatic-bot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Jan 29, 2025
@ahmadhamzh ahmadhamzh force-pushed the 7006-kyverno-integration branch from cf9ce91 to 89da532 Compare January 29, 2025 09:05
@ahmadhamzh ahmadhamzh marked this pull request as ready for review January 29, 2025 14:17
@kubermatic-bot kubermatic-bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jan 29, 2025
@ahmadhamzh ahmadhamzh changed the title Add kyverno Policies endpoints WIP Add kyverno Policies endpoints Jan 29, 2025
@kubermatic-bot kubermatic-bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jan 29, 2025
@csengerszabo csengerszabo added this to the KKP 2.27 milestone Feb 4, 2025
@ahmadhamzh ahmadhamzh force-pushed the 7006-kyverno-integration branch 4 times, most recently from 0744dec to 57f1009 Compare February 6, 2025 13:55
@ahmadhamzh ahmadhamzh changed the title WIP Add kyverno Policies endpoints Add kyverno Policies endpoints Feb 6, 2025
@kubermatic-bot kubermatic-bot added release-note Denotes a PR that will be considered when it comes time to generate release notes. docs/tbd Denotes a PR that needs documentation (change) that will be done later. and removed do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. do-not-merge/docs-needed Indicates that a PR should not merge because it's missing one of the documentation labels. labels Feb 6, 2025
@ahmadhamzh ahmadhamzh changed the title Add kyverno Policies endpoints WIP Add kyverno Policies endpoints Feb 6, 2025
@kubermatic-bot kubermatic-bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Feb 6, 2025
@ahmadhamzh ahmadhamzh force-pushed the 7006-kyverno-integration branch from 57f1009 to 6cbdc2f Compare February 6, 2025 15:52
@ahmadhamzh ahmadhamzh changed the title WIP Add kyverno Policies endpoints Add kyverno Policies endpoints Feb 6, 2025
@kubermatic-bot kubermatic-bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Feb 6, 2025
@ahmadhamzh
Copy link
Contributor Author

/retest

@ahmadhamzh ahmadhamzh force-pushed the 7006-kyverno-integration branch 5 times, most recently from e248d65 to 1d31eab Compare February 7, 2025 09:01
@ahmadhamzh
Copy link
Contributor Author

/retest

1 similar comment
@ahmadhamzh
Copy link
Contributor Author

/retest

@ahmadhamzh ahmadhamzh force-pushed the 7006-kyverno-integration branch from 1d31eab to c079ea2 Compare February 10, 2025 09:25
@ahmadhamzh ahmadhamzh force-pushed the 7006-kyverno-integration branch 2 times, most recently from 22320a8 to 6a17696 Compare February 10, 2025 11:04
@ahmadhamzh ahmadhamzh force-pushed the 7006-kyverno-integration branch from 6a17696 to a9931ad Compare February 10, 2025 13:09
@kubermatic-bot
Copy link
Contributor

@ahmadhamzh: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
pre-dashboard-api-lint a9931ad link true /test pre-dashboard-api-lint
pre-dashboard-api-verify a9931ad link true /test pre-dashboard-api-verify
pre-dashboard-api-unit a9931ad link true /test pre-dashboard-api-unit
pre-dashboard-api-e2e a9931ad link true /test pre-dashboard-api-e2e

Full PR test history

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. I understand the commands that are listed here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
dco-signoff: yes Denotes that all commits in the pull request have the valid DCO signoff message. do-not-merge/code-freeze Indicates that a PR should not merge because it has not been approved for code freeze yet. docs/tbd Denotes a PR that needs documentation (change) that will be done later. release-note Denotes a PR that will be considered when it comes time to generate release notes. sig/api Denotes a PR or issue as being assigned to SIG API. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Milestone
KKP 2.27
Development

Successfully merging this pull request may close these issues.
4 participants
@ahmadhamzh @kubermatic-bot @csengerszabo @ahmedwaleedmalik