-
Notifications
You must be signed in to change notification settings - Fork 162
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
beta to stable #6377
beta to stable #6377
Conversation
zalando-teapot-robot
commented
Aug 24, 2023
•
edited
Loading
edited
- Validation webhook for ingress (#6341)
- Make it possible to setup a custom etcd stack (#6375)
- skipper: configure terminationMessagePolicy (#6376)
- Include externalIngress port in stack definition. (#6381)
- skipper: add config to enable ingress validation webhook (#6384)
Signed-off-by: Mustafa Abdelrahman <mustafa.abdelrahman@zalando.de>
Signed-off-by: Mustafa Abdelrahman <mustafa.abdelrahman@zalando.de>
Signed-off-by: Mustafa Abdelrahman <mustafa.abdelrahman@zalando.de>
add initial deletion remove condition for webhook to work always with ingress at least Signed-off-by: Mustafa Abdelrahman <mustafa.abdelrahman@zalando.de>
Signed-off-by: Mustafa Abdelrahman <mustafa.abdelrahman@zalando.de>
Signed-off-by: Mustafa Abdelrahman <mustafa.abdelrahman@zalando.de>
…r-ingress Validation webhook for ingress
Signed-off-by: Mikkel Oscar Lyderik Larsen <mikkel.larsen@zalando.de>
* configure terminationMessagePolicy for hostname-credentials CronJobs to have last log messages in the pod status * update versions Signed-off-by: Alexander Yastrebov <alexander.yastrebov@zalando.de>
Make it possible to setup a custom etcd stack
Users reported a potential issue with the validation. We need to understand if this is an issue or not before rolling forward. |
Signed-off-by: Rodrigo Reis <rodrigo.gargravarr@gmail.com>
Signed-off-by: Rodrigo Reis <rodrigo.gargravarr@gmail.com>
…minationMessagePolicy skipper: configure terminationMessagePolicy
Similar to existing routegroups_validation. Tested that it works also when both are disabled: ``` ~$ kubectl get ValidatingWebhookConfiguration skipper-admitter.teapot.zalan.do NAME WEBHOOKS AGE skipper-admitter.teapot.zalan.do 0 27h ``` Followup on #6341 Signed-off-by: Alexander Yastrebov <alexander.yastrebov@zalando.de>
I propose we disable it #6384 and develop a better rollout plan. |
Include externalIngress port in stack definition.
Also do not update webhook version #6385 |
…ok-config skipper: add config to enable ingress validation webhook
Decided not to downgrade. |
👍 |
This validation has been disabled which is included now in the PR. So we're good to move forward. |
👍 |