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
rbd: group misc cleanup and update rbd man page #20199
Conversation
Signed-off-by: songweibin <song.weibin@zte.com.cn>
Signed-off-by: songweibin <song.weibin@zte.com.cn>
Signed-off-by: songweibin <song.weibin@zte.com.cn>
Signed-off-by: songweibin <song.weibin@zte.com.cn>
Signed-off-by: songweibin <song.weibin@zte.com.cn>
Signed-off-by: songweibin <song.weibin@zte.com.cn>
src/librbd/api/Group.cc
Outdated
@@ -731,7 +731,7 @@ int Group<I>::snap_create(librados::IoCtx& group_ioctx, | |||
ldout(cct, 1) << "Failed to create io context for image" << dendl; | |||
} | |||
|
|||
ldout(cct, 20) << "Openning image with id" << image.spec.image_id << dendl; | |||
ldout(cct, 20) << "Openning image with id " << image.spec.image_id << dendl; |
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.
Nit: can you correct the spelling of “opening” while you are changing this line?
src/librbd/api/Group.cc
Outdated
break; | ||
} | ||
if (snap.name == new_snap_name) { | ||
return -EEXIST; |
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.
This is (supposed to be) handled by cls_client::group_snap_set
* Add some error message when returning a negative error code. * Validate dest name when renaming a group snap. Signed-off-by: songweibin <song.weibin@zte.com.cn>
30de8ce
to
0e69d4f
Compare
@dillaman Updated, thank you. |
No description provided.