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.

Sign up for GitHub

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

Jump to bottom

Cannot disable nginx-ingress addon #5480

Closed
ialidzhikov opened this issue Feb 23, 2022 · 0 comments · Fixed by #5482
Closed

Cannot disable nginx-ingress addon #5480

ialidzhikov opened this issue Feb 23, 2022 · 0 comments · Fixed by #5482
Assignees
@ialidzhikov
Labels
area/quality Output qualification (tests, checks, scans, automation in general, etc.) related kind/bug Bug kind/regression Bug that hit us already in the past and that is reappearing/requires a proper solution

Comments

@ialidzhikov
Copy link
Member

ialidzhikov commented Feb 23, 2022 •
edited

/area quality
/kind bug
/kind regression

What happened:
After #5375 it is not possible to disable the nginx-ingress addon successfully. Currently the SecretData is not passed to the component and the component updates the DNSRecord Secret with empty data on Destroy.

What you expected to happen:
To successfully disable nginx-ingress addon.

How to reproduce it (as minimally and precisely as possible):

  1. Create a Shoot with spec.addons.nginxIngress.enabled=true.

  2. Set spec.addons.nginxIngress.enabled=false.

  3. Make sure that the ingress DNSRecord fails to be destroyed with

  lastError:
    description: 'Error deleting dnsrecord: could not get AWS credentials: secret
      does not contain any data'
    lastUpdateTime: "2022-02-23T10:38:01Z"

Environment:

  • Gardener version: v1.40.2
  • Kubernetes version (use kubectl version):
  • Cloud provider or hardware configuration:
  • Others:
@ialidzhikov ialidzhikov added the kind/bug Bug label Feb 23, 2022
@ialidzhikov ialidzhikov self-assigned this Feb 23, 2022
@gardener-robot gardener-robot added area/quality Output qualification (tests, checks, scans, automation in general, etc.) related kind/regression Bug that hit us already in the past and that is reappearing/requires a proper solution labels Feb 23, 2022
@ialidzhikov ialidzhikov mentioned this issue Feb 23, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ialidzhikov ialidzhikov

Labels
area/quality Output qualification (tests, checks, scans, automation in general, etc.) related kind/bug Bug kind/regression Bug that hit us already in the past and that is reappearing/requires a proper solution
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Set ingress DNSRecord component values even if the addon is disabled ialidzhikov/gardener
2 participants
@ialidzhikov @gardener-robot