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

Draft
wants to merge 9 commits into
base: main
Choose a base branch
from
Draft

Add kyverno Policies endpoints #7106

wants to merge 9 commits into from
+8,407 −2

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

1 similar comment
@ahmadhamzh
Copy link
Contributor Author

/retest

@ahmadhamzh ahmadhamzh force-pushed the 7006-kyverno-integration branch 5 times, most recently from a9931ad to 0a7f987 Compare February 11, 2025 21:39
@Waseem826
Copy link
Contributor

/test pre-dashboard-api-e2e

@ahmadhamzh ahmadhamzh force-pushed the 7006-kyverno-integration branch from 0a7f987 to b1ce74d Compare February 12, 2025 13:21
@ahmadhamzh ahmadhamzh changed the title Add kyverno Policies endpoints WIP Add kyverno Policies endpoints Feb 12, 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 12, 2025
@ahmadhamzh ahmadhamzh force-pushed the 7006-kyverno-integration branch 2 times, most recently from b2a0f75 to 2dc0315 Compare February 13, 2025 05:32
@ahmadhamzh ahmadhamzh force-pushed the 7006-kyverno-integration branch from 2dc0315 to a5852c6 Compare February 13, 2025 10:07
@ahmadhamzh
Copy link
Contributor Author

/retest

@ahmadhamzh ahmadhamzh changed the title WIP Add kyverno Policies endpoints Add kyverno Policies endpoints Feb 13, 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 13, 2025
@kubermatic-bot
Copy link
Contributor

@ahmadhamzh: The following test 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-e2e a5852c6 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.

@Waseem826 Waseem826 removed this from the KKP 2.27 milestone Feb 13, 2025
@ahmadhamzh ahmadhamzh marked this pull request as draft February 13, 2025 12:14
@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 13, 2025
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. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. 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
No milestone
Development

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