-
Notifications
You must be signed in to change notification settings - Fork 46
Add ready check for webhook server #224
Add ready check for webhook server #224
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please also squash commits, thanks for your contribution!
@@ -46,6 +46,19 @@ spec: | |||
{{- if .Values.global.kubeconfig }} | |||
- --kubeconfig=/etc/gardener-extension-validator-vsphere/kubeconfig/kubeconfig | |||
{{- end }} | |||
- --health-bind-address=:{{ .Values.global.healthPort }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It makes sense to wrap this in case the values are not provided, see https://github.com/gardener/gardener/blob/e8e73842ad0acfe30c02eb7d17c8c7b2fd42a4e5/charts/gardener/gardener-resource-manager/templates/deployment.yaml#L62-L64
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It was suggested here not to add if gardener/gardener-extension-provider-openstack#421 (comment)
@@ -44,6 +44,7 @@ spec: | |||
- --webhook-config-server-port={{ .Values.webhookConfig.serverPort }} | |||
- --disable-controllers={{ .Values.disableControllers | join "," }} | |||
- --disable-webhooks={{ .Values.disableWebhooks | join "," }} | |||
- --health-bind-address=:{{ .Values.healthPort }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It makes sense to wrap this in case the values are not provided, see https://github.com/gardener/gardener/blob/e8e73842ad0acfe30c02eb7d17c8c7b2fd42a4e5/charts/gardener/gardener-resource-manager/templates/deployment.yaml#L62-L64
How to categorize this PR?
/area open-source
/kind enhancement
/platform vsphere
What this PR does / why we need it:
The PR adds ready check for webhook server. The ready check wait for 10 sec for webhook server to be ready, otherwise throws error.
Which issue(s) this PR fixes:
Part of gardener/gardener#4772
Special notes for your reviewer:
Release note: