Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # 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
Signed-off-by: Andre Baptista Aguas <andre.aguas@protonmail.com>
@abaguas abaguas force-pushed the annotation/referenced branch from 998d571 to 7c9b5df Compare August 12, 2024 18:47
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant