Skip to content

fix: pulsar bookie member leave fail#2475

Merged
cjc7373 merged 1 commit intomainfrom
bugfix/pulsar-bookie-memberleave-fail
Mar 5, 2026
Merged

fix: pulsar bookie member leave fail#2475
cjc7373 merged 1 commit intomainfrom
bugfix/pulsar-bookie-memberleave-fail

Conversation

@cjc7373
Copy link
Contributor

@cjc7373 cjc7373 commented Mar 4, 2026

@cjc7373 cjc7373 requested review from a team and caiq1nyu as code owners March 4, 2026 09:45
@cjc7373 cjc7373 added pick-1.0 Auto cherry-pick to release-1.0 when PR merged pick-1.1 Auto cherry-pick to release-1.1 when PR merged labels Mar 4, 2026
@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 0% with 8 lines in your changes missing coverage. Please review.
✅ Project coverage is 0.00%. Comparing base (f118228) to head (4fed82d).

Files with missing lines Patch % Lines
...ulsar/scripts-ut-spec/bookies_member_leave_spec.sh 0.00% 8 Missing ⚠️
Additional details and impacted files
@@          Coverage Diff          @@
##            main   #2475   +/-   ##
=====================================
  Coverage   0.00%   0.00%           
=====================================
  Files         72      72           
  Lines       9197    9166   -31     
=====================================
+ Misses      9197    9166   -31     

☔ 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.

# TODO: consider using decommissionbookie? But decommissionbookie needs bookie to stop first, kb doesn't support a "postMemberLeave" hook.
echo "Formatting Bookie..."
export BOOKIE_CONF=/opt/pulsar/conf/bookkeeper.conf
bin/bookkeeper shell bookieformat -nonInteractive -force -deleteCookie
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个命令能指定 leave member ip,可以的话就比较简单了吗

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bookieformat 只能对 pod 所在的节点生效

@cjc7373
Copy link
Contributor Author

cjc7373 commented Mar 5, 2026

/approve

@cjc7373 cjc7373 merged commit 6d1e1ee into main Mar 5, 2026
32 checks passed
@cjc7373 cjc7373 deleted the bugfix/pulsar-bookie-memberleave-fail branch March 5, 2026 07:34
@apecloud-bot
Copy link
Collaborator

/cherry-pick release-1.0

@apecloud-bot
Copy link
Collaborator

/cherry-pick release-1.1

@apecloud-bot
Copy link
Collaborator

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

apecloud-bot pushed a commit that referenced this pull request Mar 5, 2026
@apecloud-bot
Copy link
Collaborator

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

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

Labels

pick-1.0 Auto cherry-pick to release-1.0 when PR merged pick-1.1 Auto cherry-pick to release-1.1 when PR merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants