-
Notifications
You must be signed in to change notification settings - Fork 905
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
Allow kubernetes service annotations #5471
Allow kubernetes service annotations #5471
Conversation
f3c9c87
to
530a34a
Compare
Allow setting annotations on the kubernetes service object. In GPC we need annotations to expose a service as a NEG which is required in order for the k8s controlplane to reach the webhook. This pr will add the ability to add annotations to the service object Signed-off-by: Atze de Vries <atze.wiebe.de.vries@coop.no>
Signed-off-by: Atze de Vries <atze.wiebe.de.vries@coop.no>
Signed-off-by: Atze de Vries <atze.wiebe.de.vries@coop.no>
33a3c24
to
3a9740c
Compare
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.
Looks good to me, thanks!
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.
Thanks!
@AtzeDeVries We typically only back port bug fixes. If it's easy for you to use the updated Helm chart with 1.15 until we release 1.16 that would be my preference. |
Description of your changes
Allow setting custom annotations on the service object. For GCP we require a
NEG when using webhooks. Setting up the NEG requires annotations on the
service object.
I cannot add labels, but prefer backport to 1.13
I have:
make reviewable
to ensure this PR is ready for review.Added or updated unit tests.Added or updated e2e tests.Linked a PR or a docs tracking issue to document this change.backport release-x.y
labels to auto-backport this PR.