Skip to content

test: Remove assertions for generation#798

Merged
NickLarsenNZ merged 1 commit intomainfrom
test/remove-tight-assertion
Mar 9, 2026
Merged

test: Remove assertions for generation#798
NickLarsenNZ merged 1 commit intomainfrom
test/remove-tight-assertion

Conversation

@NickLarsenNZ
Copy link
Member

@NickLarsenNZ NickLarsenNZ commented Mar 9, 2026

For some reason, restarts can happen. We don't know why, but the controller is changing something shortly after initially creating the STS, leading to a generation 2 of the pod template.
I think probably due to labels.

Note: It could also be the case that there are no restarts, but generation is >= 0

For some reason, restarts can happen
@NickLarsenNZ NickLarsenNZ requested a review from sbernauer March 9, 2026 09:43
@NickLarsenNZ NickLarsenNZ self-assigned this Mar 9, 2026
@NickLarsenNZ NickLarsenNZ moved this to Development: Waiting for Review in Stackable Engineering Mar 9, 2026
@sbernauer sbernauer moved this from Development: Waiting for Review to Development: In Review in Stackable Engineering Mar 9, 2026
@NickLarsenNZ NickLarsenNZ enabled auto-merge March 9, 2026 09:44
@NickLarsenNZ NickLarsenNZ moved this from Development: In Review to Development: Done in Stackable Engineering Mar 9, 2026
@NickLarsenNZ NickLarsenNZ added this pull request to the merge queue Mar 9, 2026
Merged via the queue into main with commit ddfe63b Mar 9, 2026
10 checks passed
@NickLarsenNZ NickLarsenNZ deleted the test/remove-tight-assertion branch March 9, 2026 09:56
@lfrancke lfrancke moved this from Development: Done to Done in Stackable Engineering Mar 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

2 participants