Skip to content

fix: prohibit stop when post-provision is not completed#9910

Merged
kizuna-lek merged 3 commits intomainfrom
bugfix/has-no-pod-for-actions
Dec 4, 2025
Merged

fix: prohibit stop when post-provision is not completed#9910
kizuna-lek merged 3 commits intomainfrom
bugfix/has-no-pod-for-actions

Conversation

@kizuna-lek
Copy link
Collaborator

When post-provision is not completed but the stop field of the component is set to true, ITS will set the replica to 0. At this point, post-provision will continuously fail due to the lack of available pods for execution, preventing the component from ever transitioning to the stopped state. The result is that there are no pods under the component, but the component still shows as running, and the cluster also displays as running.

@kizuna-lek kizuna-lek requested a review from a team as a code owner December 4, 2025 06:59
@kizuna-lek kizuna-lek added the pick-1.0 Auto cherry-pick to release-1.0 when PR merged label Dec 4, 2025
@apecloud-bot
Copy link
Collaborator

Auto Cherry-pick Instructions

Usage:
  - /nopick: Not auto cherry-pick when PR merged.
  - /pick: release-x.x [release-x.x]: Auto cherry-pick to the specified branch when PR merged.

Example:
  - /nopick
  - /pick release-1.0

@github-actions github-actions bot added the size/XS Denotes a PR that changes 0-9 lines. label Dec 4, 2025
@codecov
Copy link

codecov bot commented Dec 4, 2025

Codecov Report

❌ Patch coverage is 75.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 51.09%. Comparing base (e7d2dc5) to head (706869a).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
...ers/apps/component/transformer_component_status.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #9910      +/-   ##
==========================================
+ Coverage   50.93%   51.09%   +0.16%     
==========================================
  Files         541      541              
  Lines       58289    58289              
==========================================
+ Hits        29690    29785      +95     
+ Misses      25668    25570      -98     
- Partials     2931     2934       +3     
Flag Coverage Δ
unittests 51.09% <75.00%> (+0.16%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@kizuna-lek kizuna-lek changed the title fix: skip post-provision when component is stop fix: prohibit stop when post-provision is not completed Dec 4, 2025
@github-actions github-actions bot added size/S Denotes a PR that changes 10-29 lines. and removed size/XS Denotes a PR that changes 0-9 lines. labels Dec 4, 2025
@apecloud-bot apecloud-bot added the approved PR Approved Test label Dec 4, 2025
@kizuna-lek
Copy link
Collaborator Author

/approve

@kizuna-lek kizuna-lek merged commit 84bf428 into main Dec 4, 2025
51 of 53 checks passed
@kizuna-lek kizuna-lek deleted the bugfix/has-no-pod-for-actions branch December 4, 2025 09:05
@github-actions github-actions bot added this to the Release 1.1.0 milestone Dec 4, 2025
@apecloud-bot
Copy link
Collaborator

/cherry-pick release-1.0

@apecloud-bot
Copy link
Collaborator

🤖 says: cherry pick action finished successfully 🎉!
See: https://github.com/apecloud/kubeblocks/actions/runs/19923447626

apecloud-bot pushed a commit that referenced this pull request Dec 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved PR Approved Test pick-1.0 Auto cherry-pick to release-1.0 when PR merged size/S Denotes a PR that changes 10-29 lines.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants