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.

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

skipper admission: re-enable ingress validation #6525

Merged
merged 1 commit into from
Sep 27, 2023

Conversation

MustafaSaber
Copy link
Member

After some cleaning this shouldn't create problems as it did last time and we can work with people to fix their configuration or disable on demand.

Signed-off-by: Mustafa Abdelrahman <mustafa.abdelrahman@zalando.de>
@@ -6,9 +6,6 @@ pre_apply:

# everything defined under here will be deleted after applying the manifests
post_apply:
# TODO: remove later
Copy link
Member Author

Choose a reason for hiding this comment

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

Clean up for #6341

@AlexanderYastrebov
Copy link
Member

👍

1 similar comment
@szuecs
Copy link
Member

szuecs commented Sep 27, 2023

👍

@szuecs szuecs merged commit 12a0178 into dev Sep 27, 2023
9 checks passed
@szuecs szuecs deleted the enable-ingress-validation branch September 27, 2023 18:50
This was referenced Sep 27, 2023
@MustafaSaber MustafaSaber self-assigned this Jan 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants