Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What does this PR do / Why do we need it:
This fixes a problem during seed cleanup. In #7558 we made it so that the KubermaticConfiguration is also synced onto each seed cluster. But because the changed code ( https://github.com/kubermatic/kubermatic/pull/7558/files#diff-de0ded63a6c09d74715c1b4f850442460cb0bf652d4ec20adc5035f7204526f6R181-R183 ) overwrites
err
with something new,err == nil
doesn't represent "seed exists", but "config exists" now. And this causes the actual deletion on the seed to never happen.This problem should affect KKP 2.18+, judging from the PR that broke it.
Does this PR close any issues?:
Fixes #9542
Does this PR introduce a user-facing change?: