Skip to content

fix: skip postProvision for stopped components#35

Merged
wallyxjh merged 1 commit into
labring:fix/v0.9.3from
im0x0ing:fix-redis-stop-post-provision
Jun 9, 2026
Merged

fix: skip postProvision for stopped components#35
wallyxjh merged 1 commit into
labring:fix/v0.9.3from
im0x0ing:fix-redis-stop-post-provision

Conversation

@im0x0ing

@im0x0ing im0x0ing commented Jun 8, 2026

Copy link
Copy Markdown
Collaborator

Skip component postProvision reconciliation when the component is already stopped via spec.stop or the legacy replicas=0 stop path. This prevents RuntimeReady postProvision actions from blocking stop reconciliation after the workload has been scaled down to zero pods.

Do not mark postProvision as done when skipping, so the action can still run after the component is started again if the done annotation is absent. Add regression coverage for RuntimeReady, stop=true, and replicas=0 cases.

Skip component postProvision reconciliation when the component is already stopped via spec.stop or the legacy replicas=0 stop path. This prevents RuntimeReady postProvision actions from blocking stop reconciliation after the workload has been scaled down to zero pods.

Do not mark postProvision as done when skipping, so the action can still run after the component is started again if the done annotation is absent. Add regression coverage for RuntimeReady, stop=true, and replicas=0 cases.
@wallyxjh wallyxjh merged commit 3653ff9 into labring:fix/v0.9.3 Jun 9, 2026
3 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants