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

Support for multi-ingress environments #278

Merged
merged 1 commit into from
May 3, 2022
Merged

Conversation

sivanov-nuodb
Copy link
Collaborator

Issue

If the test job is killed and HAProxy teardown doesn't execute, the IngressClass resource (which is cluster-scoped) will be left behind. This will cause all subsequent executions of the test to fail with the following message:

Error: INSTALLATION FAILED: rendered manifests contain a resource that already exists. Unable to continue with install: IngressClass "haproxy" in namespace "" exists and cannot be imported into the current release: invalid ownership metadata; annotation validation error: key "meta.helm.sh/release-name" must equal "haproxy-ooimr1": current value is "haproxy-mgyoxp"; annotation validation error: key "meta.helm.sh/release-namespace" must equal "testkubernetesingress-5czgy7": current value is "testkubernetesingress-s0uufa"

Changes

Added support for multi-ingress environments.

For more information, see https://github.com/haproxytech/helm-charts/tree/main/kubernetes-ingress#installing-in-multi-ingress-environment.

@sivanov-nuodb sivanov-nuodb added the meta Exclude from Changelog label May 3, 2022
@sivanov-nuodb sivanov-nuodb merged commit 46a3c85 into master May 3, 2022
@sivanov-nuodb sivanov-nuodb deleted the sivanov/fix-haproxy branch May 3, 2022 06:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
meta Exclude from Changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants