Skip to content

Conversation

@olamilekan000
Copy link
Contributor

@olamilekan000 olamilekan000 commented Dec 5, 2025

Summary

Change adds support for cross-workspace implementation for ValidatingAdmissionPolicy.
This enables resource validation from a provider workspace. 
image

What Type of PR Is This?

/kind feature

Related Issue(s)

Fixes 3733

Release Notes

Added cross-workspace implementation for ValidatingAdmissionPolicy.

@kcp-ci-bot kcp-ci-bot added release-note Denotes a PR that will be considered when it comes time to generate release notes. kind/feature Categorizes issue or PR as related to a new feature. dco-signoff: yes Indicates the PR's author has signed the DCO. kind/api-change Categorizes issue or PR as related to adding, removing, or otherwise changing an API labels Dec 5, 2025
@olamilekan000 olamilekan000 requested a review from xmudrii December 5, 2025 12:27
@kcp-ci-bot kcp-ci-bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Dec 5, 2025
@olamilekan000 olamilekan000 requested a review from gman0 December 5, 2025 12:27
@olamilekan000 olamilekan000 force-pushed the implement-crossworskapce-validating-admission-policy branch 3 times, most recently from 2d86b89 to 70c4885 Compare December 5, 2025 16:04
@olamilekan000
Copy link
Contributor Author

olamilekan000 commented Dec 5, 2025

/retest

5 similar comments
@olamilekan000
Copy link
Contributor Author

/retest

@olamilekan000
Copy link
Contributor Author

/retest

@olamilekan000
Copy link
Contributor Author

/retest

@olamilekan000
Copy link
Contributor Author

/retest

@olamilekan000
Copy link
Contributor Author

/retest

Copy link
Contributor

@mjudeikis mjudeikis left a comment

Choose a reason for hiding this comment

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

Few questions. Mostrly why do we need to change delegate keys?

@olamilekan000 olamilekan000 force-pushed the implement-crossworskapce-validating-admission-policy branch 2 times, most recently from b6b1a86 to e6d3d6d Compare December 9, 2025 12:19
@mjudeikis
Copy link
Contributor

/retest
@gman0 flakes I mentioned

@olamilekan000 olamilekan000 force-pushed the implement-crossworskapce-validating-admission-policy branch from e6d3d6d to 65cc18f Compare December 9, 2025 12:55
Copy link
Contributor

@mjudeikis mjudeikis left a comment

Choose a reason for hiding this comment

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

/lgtm
/approve

@kcp-ci-bot kcp-ci-bot added the lgtm Indicates that a PR is ready to be merged. label Dec 9, 2025
@kcp-ci-bot
Copy link
Contributor

LGTM label has been added.

Git tree hash: e4eaf0518563864113f582b6d0a3597c023f3e4e

@kcp-ci-bot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mjudeikis

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

The pull request process is described 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

@kcp-ci-bot kcp-ci-bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Dec 9, 2025
@mjudeikis
Copy link
Contributor

/retest

@kcp-ci-bot kcp-ci-bot merged commit 1869e2c into kcp-dev:main Dec 9, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. dco-signoff: yes Indicates the PR's author has signed the DCO. kind/api-change Categorizes issue or PR as related to adding, removing, or otherwise changing an API kind/feature Categorizes issue or PR as related to a new feature. lgtm Indicates that a PR is ready to be merged. release-note Denotes a PR that will be considered when it comes time to generate release notes. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feature: Implement ValidatingAdmissionPolicy

3 participants