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
osd/OSDMonitor: misc. fixes #10491
osd/OSDMonitor: misc. fixes #10491
Conversation
…grained If the specified osd is not down or doesn't exist, we shall return error instead of success. So caller can abort(if does not exist) or mark osd as down and retry(if not down). Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
@tchaikov Can you please help to confirm this one too? Thanks! |
@@ -628,7 +628,7 @@ COMMAND("osd erasure-code-profile ls", \ | |||
"list all erasure code profiles", \ | |||
"osd", "r", "cli,rest") | |||
COMMAND("osd set " \ | |||
"name=key,type=CephChoices,strings=full|pause|noup|nodown|noout|noin|nobackfill|norebalance|norecover|noscrub|nodeep-scrub|notieragent|sortbitwise|require_jewel_osds", \ | |||
"name=key,type=CephChoices,strings=full|pause|noup|nodown|noout|noin|nobackfill|norebalance|norecover|noscrub|nodeep-scrub|notieragent|sortbitwise|require_jewel_osds|require_kraken_osds", \ |
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.
probably you can reference this line: https://github.com/ceph/ceph/blob/9bef6cf/src/mon/OSDMonitor.cc#L6355, in the commit message.
i am not able to identify the piece of code referenced by #9248 (comment) anymore.
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.
Sorry, I am trying to add a reference for why this change is necessary.
Will remove that later.
For newly created cluster the CEPH_OSDMAP_REQUIRE_KRAKEN will be automatically set, while for existing clusters it will not. This change add "require_jewel_osds" to white list, so user can access it by the "ceph osd set *" command family. Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
We shall check CEPH_FEATURE_SERVER_JEWEL carried by osd_features of MOSDBoot message, as it will be properly filled by every OSD during OSD::_send_boot() process. Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
b0cbcaf
to
2ead847
Compare
lgtm. |
retest this please |
No description provided.