Skip to content

add ValidatingWebhook for preventing attachment of mantle-related PVs #138

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

Merged
merged 8 commits into from
Apr 28, 2025

Conversation

molpako
Copy link
Contributor

@molpako molpako commented Apr 11, 2025

No description provided.

@molpako molpako force-pushed the protect-second-pv branch 2 times, most recently from 603b8d8 to d1d8928 Compare April 14, 2025 02:54
@molpako molpako force-pushed the protect-second-pv branch 2 times, most recently from df61408 to bc12e6e Compare April 15, 2025 02:16
@molpako molpako force-pushed the protect-second-pv branch 3 times, most recently from 157d536 to a9393e0 Compare April 18, 2025 02:26
@molpako molpako changed the title temp add ValidatingWebhook for preventing attachment of mantle-related PVs Apr 18, 2025
@molpako molpako force-pushed the protect-second-pv branch from 940f871 to 9651d61 Compare April 18, 2025 06:49
@peng225 peng225 force-pushed the protect-second-pv branch from 825469c to b7b0700 Compare April 21, 2025 01:41
@peng225 peng225 force-pushed the protect-second-pv branch 2 times, most recently from 56066c0 to ec11b84 Compare April 22, 2025 06:58
@molpako molpako force-pushed the protect-second-pv branch 3 times, most recently from 6faca6d to 9e6404a Compare April 23, 2025 03:00
@peng225 peng225 marked this pull request as ready for review April 23, 2025 05:50
Copy link
Contributor

@peng225 peng225 left a comment

Choose a reason for hiding this comment

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

A commit with at title "move testutil to internal directory" contains the changes to the webhook test file. This is not what we expect.

Signed-off-by: Kyori Sakao <[email protected]>
Co-authored-by: Shinya Hayashi <[email protected]>
Co-authored-by: Yuji Ito <[email protected]>
@peng225 peng225 force-pushed the protect-second-pv branch from 9e6404a to d584c33 Compare April 23, 2025 06:46
peng225 and others added 7 commits April 28, 2025 06:27
…ed to a pod.

Signed-off-by: Kyori Sakao <[email protected]>
Co-authored-by: Shinya Hayashi <[email protected]>
Co-authored-by: Yuji Ito <[email protected]>
Signed-off-by: Kyori Sakao <[email protected]>
Co-authored-by: Shinya Hayashi <[email protected]>
Co-authored-by: Yuji Ito <[email protected]>
Signed-off-by: Kyori Sakao <[email protected]>
Co-authored-by: Shinya Hayashi <[email protected]>
Co-authored-by: Yuji Ito <[email protected]>
Signed-off-by: Kyori Sakao <[email protected]>
Co-authored-by: Shinya Hayashi <[email protected]>
Co-authored-by: Yuji Ito <[email protected]>
Signed-off-by: Shinya Hayashi <[email protected]>
Co-authored-by: Yuji Ito <[email protected]>
Co-authored-by: Kyori Sakao <[email protected]>
Signed-off-by: Shinya Hayashi <[email protected]>
Co-authored-by: Kyori Sakao <[email protected]>
Co-authored-by: Yuji Ito <[email protected]>
Signed-off-by: Kyori Sakao <[email protected]>
@satoru-takeuchi satoru-takeuchi merged commit 6b8ea41 into main Apr 28, 2025
9 checks passed
@satoru-takeuchi satoru-takeuchi deleted the protect-second-pv branch April 28, 2025 07:43
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.

4 participants