New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
pybind/mgr/cephadm/upgrade: allow upgrades without reducing max_mds #46534
Conversation
9f4c45b
to
1b774fe
Compare
Commits have been kept like this just for the sake of clarity on what I've been doing around the PR. Will squash them once the reviews are done. |
8c36700
to
3579952
Compare
jenkins test make check |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks about right. This now needs QA tests in fs:upgrade
.
jenkins test make check |
This pull request can no longer be automatically merged: a rebase is needed and changes have to be manually resolved |
a532f80
to
85ec299
Compare
@dparmar18 You should remove the draft status by marking 'Ready for review' if it is so. |
Yeah, I'm waiting for something to pass, I'll definitely mark it as 'Ready for review' soon. |
Still a |
9464b3f
to
4a13932
Compare
Fixes: https://tracker.ceph.com/issues/55715 Signed-off-by: Dhairya Parmar <dparmar@redhat.com>
Signed-off-by: Dhairya Parmar <dparmar@redhat.com>
…ng cephadm Signed-off-by: Dhairya Parmar <dparmar@redhat.com>
… to 1. Signed-off-by: Dhairya Parmar <dparmar@redhat.com>
…ceeded Signed-off-by: Dhairya Parmar <dparmar@redhat.com>
…ing the fail_fs option Signed-off-by: Dhairya Parmar <dparmar@redhat.com>
I was experimenting with some workarounds and thus used multiple branches for the purpose. Rebasing this PR with those changes might bring some conflicts that need resolution therefore decided to open a new PR - #47092 and close this one. |
Contribution Guidelines
To sign and title your commits, please refer to Submitting Patches to Ceph.
If you are submitting a fix for a stable branch (e.g. "pacific"), please refer to Submitting Patches to Ceph - Backports for the proper workflow.
Checklist
Show available Jenkins commands
jenkins retest this please
jenkins test classic perf
jenkins test crimson perf
jenkins test signed
jenkins test make check
jenkins test make check arm64
jenkins test submodules
jenkins test dashboard
jenkins test dashboard cephadm
jenkins test api
jenkins test docs
jenkins render docs
jenkins test ceph-volume all
jenkins test ceph-volume tox
jenkins test windows