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

Fix tunnel secret handling on Shoot deletion #2562

Merged
merged 1 commit into from
Jul 7, 2020

Conversation

timebertt
Copy link
Member

How to categorize this PR?

/area quality
/kind bug
/priority normal

What this PR does / why we need it:
Fix tunnel secret handling: only cleanup the tunnel secrets, when the Shoot is not yet marked for deletion.

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:
/cc @timuthy
Thanks for catching!

Release note:

Fixed a bug, that caused the tunnel secrets not to be deleted, in case the used tunnel has changed.

@timebertt timebertt requested a review from a team as a code owner July 7, 2020 07:24
@gardener-robot gardener-robot added area/quality Output qualification (tests, checks, scans, automation in general, etc.) related kind/bug Bug priority/normal labels Jul 7, 2020
Copy link
Contributor

@timuthy timuthy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm
Thanks for fixing!

@timuthy timuthy merged commit 6a2fab2 into gardener:master Jul 7, 2020
@timebertt timebertt deleted the fix/tunnel-secrets branch July 8, 2020 12:21
@gardener-robot gardener-robot added priority/3 Priority (lower number equals higher priority) and removed priority/3 Priority (lower number equals higher priority) labels Mar 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/quality Output qualification (tests, checks, scans, automation in general, etc.) related kind/bug Bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants