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

Ensure backup entry name is generated only once using non-empty strings #4454

Merged
merged 1 commit into from
Jul 30, 2021

Conversation

stoyanr
Copy link
Contributor

@stoyanr stoyanr commented Jul 29, 2021

How to categorize this PR?

/area control-plane
/kind bug

What this PR does / why we need it:

  • Ensures that the backup entry name is generated only once during the shoot reconciliation.
  • Ensures that attempting to generate the backup entry name with empty shoot technical ID or UID is treated as an error.

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:
This is intended as a quick fix to a recently observed issue that is caused by generating a backup entry name -- when deploying Etcd resources. It does not fix the root cause of the issue (likely some kind of data race), just aims to prevent it from happening by ensuring the backup entry name is generated correctly.
See gardener/gardener-extension-provider-azure#328 (comment).

Release note:

Ensured that the backup entry name is generated only once using non-empty strings to prevent issues with backup entry names generated as `--`.

@stoyanr stoyanr requested a review from a team as a code owner July 29, 2021 16:04
@gardener-robot gardener-robot added area/control-plane Control plane related kind/bug Bug needs/review size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Jul 29, 2021
@stoyanr stoyanr changed the title Ensure backup entry name is generated only once using correct data Ensure backup entry name is generated only once using non-empty strings Jul 29, 2021
@stoyanr
Copy link
Contributor Author

stoyanr commented Jul 29, 2021

/invite @timuthy @plkokanov @ialidzhikov

amshuman-kr
amshuman-kr previously approved these changes Jul 30, 2021
Copy link

@amshuman-kr amshuman-kr left a comment

Choose a reason for hiding this comment

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

Thanks @stoyanr for the changes! LGTM

timuthy
timuthy previously approved these changes Jul 30, 2021
Copy link
Member

@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
Thank you very much @stoyanr

@timuthy
Copy link
Member

timuthy commented Jul 30, 2021

/needs cherry-pick

Copy link
Member

@vpnachev vpnachev left a comment

Choose a reason for hiding this comment

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

Thanks!

Just some suggestions from my side.
And please, add a release note.

pkg/operation/shoot/types.go Show resolved Hide resolved
pkg/utils/gardener/backupentry.go Outdated Show resolved Hide resolved
Copy link
Contributor

@plkokanov plkokanov left a comment

Choose a reason for hiding this comment

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

Thanks alot for the PR!
/lgtm

@timuthy timuthy merged commit 4ceccd4 into gardener:master Jul 30, 2021
@stoyanr stoyanr deleted the fix-generate-backup-entry-name branch July 30, 2021 10:36
timuthy added a commit that referenced this pull request Jul 30, 2021
…upstream-release-v1.26

[release-v1.26] Automated cherry pick of #4454: Ensure backup entry name is generated only once using non-empty strings
stoyanr added a commit that referenced this pull request Jul 30, 2021
…upstream-release-v1.28

[release-v1.28] Automated cherry pick of #4454: Ensure backup entry name is generated only once using non-empty strings
stoyanr added a commit that referenced this pull request Jul 30, 2021
…upstream-release-v1.27

[release-v1.27] Automated cherry pick of #4454: Ensure backup entry name is generated only once using non-empty strings
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/control-plane Control plane related kind/bug Bug size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

10 participants