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

Create referenced ingress if GSLB is configured with Ingress annotations #1700

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

abaguas
Copy link
Collaborator

@abaguas abaguas commented Aug 11, 2024

Tackles one of the prerequisites outlined in #1698 to decommission embedded ingresses.

By default a GSLB with a referenced Ingress will be created if a user adds k8gb annotations to an Ingress resource.
To smoothen the user migration:

  • a warning is shown if an Ingress resource with k8gb annotations does not have any labels
  • it is still possible to create a GSLB with an embedded ingress by toggling the helm value annotationCreatesGSLBWithEmbeddedIngress

In v1.1 the option will be deleted and the code will be cleaned up to support only Referenced Ingresses.

@abaguas abaguas marked this pull request as draft August 11, 2024 14:15
@abaguas abaguas force-pushed the annotation/referenced branch 5 times, most recently from 494e215 to 998d571 Compare August 11, 2024 20:30
@abaguas abaguas force-pushed the annotation/referenced branch from 998d571 to 7c9b5df Compare August 12, 2024 18:47
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.

1 participant