Skip to content

Fix race condition in initial-maintenance job creation#614

Merged
github-actions[bot] merged 1 commit intodevelopfrom
fix/init-maint
Mar 12, 2026
Merged

Fix race condition in initial-maintenance job creation#614
github-actions[bot] merged 1 commit intodevelopfrom
fix/init-maint

Conversation

@mdnix
Copy link
Copy Markdown
Member

@mdnix mdnix commented Mar 12, 2026

Summary

On the first reconcile, the composite resource metadata may not be fully populated yet, causing a job named just initial-maintenance instead of {composite}-initial-maintenance with empty env vars that fails immediately. The orphaned error pods trigger KubePodNotReady alerts indefinitely.

Defer job creation until GetName() is non-empty. The next reconcile will have the full metadata.

Checklist

  • Update tests.
  • Link this PR to related issues.
  • Merge with /merge comment.

Component PR: vshn/component-appcat#1104

@mdnix mdnix self-assigned this Mar 12, 2026
@mdnix mdnix added bug Something isn't working patch labels Mar 12, 2026
@mdnix mdnix requested review from a team, Kidswiss, TheBigLee, mikeshootzz and zugao and removed request for a team March 12, 2026 14:10
@github-actions
Copy link
Copy Markdown
Contributor

@mdnix
Copy link
Copy Markdown
Member Author

mdnix commented Mar 12, 2026

/merge

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

@mdnix
Copy link
Copy Markdown
Member Author

mdnix commented Mar 12, 2026

/merge

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions github-actions bot merged commit 2db8de5 into develop Mar 12, 2026
9 checks passed
@github-actions github-actions bot deleted the fix/init-maint branch March 12, 2026 15:55
mikeshootzz pushed a commit to vshn/component-appcat that referenced this pull request Mar 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working patch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants