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
librbd: extract group module from librbd/internal #11070
librbd: extract group module from librbd/internal #11070
Conversation
75e6836
to
21b56f2
Compare
@@ -560,6 +560,7 @@ $DAEMONOPTS | |||
filestore wbthrottle btrfs ios start flusher = 10 | |||
filestore wbthrottle btrfs ios hard limit = 20 | |||
filestore wbthrottle btrfs inodes hard limit = 30 | |||
journal dio = false |
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.
Minor: appears to be an unrelated change
@@ -17,6 +17,7 @@ set(librbd_internal_srcs | |||
ImageCtx.cc | |||
ImageState.cc | |||
ImageWatcher.cc | |||
Group.cc |
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.
Minor: move between ExclusiveLock and ImageCtx to keep alphabetical
@@ -0,0 +1,382 @@ | |||
|
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.
Minor: extra blank line
|
||
#define dout_subsys ceph_subsys_rbd | ||
#undef dout_prefix | ||
#define dout_prefix *_dout << "librbd: " |
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.
Minor: librbd::Group:
namespace librbd { | ||
// Consistency groups functions | ||
|
||
int group_create(librados::IoCtx& io_ctx, const char *group_name) |
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.
Minor: no indentation required within a namespace
std::vector<group_image_status_t>& images); | ||
int image_get_group(ImageCtx *ictx, group_spec_t *group_spec); | ||
} | ||
#endif |
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.
Minor: "#endif // CEPH_LIBRBD_GROUP_H"
21b56f2
to
7cc97fc
Compare
|
||
return 0; | ||
} | ||
} |
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.
Minor: "} // namespace librbd"
#define CEPH_LIBRBD_GROUP_H | ||
|
||
namespace librbd { | ||
// Consistency groups functions |
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.
Minor: no indentation required for namespace -- just leave a blank line between the namespace block and its contents for clarity.
Signed-off-by: Victor Denisov <denisovenator@gmail.com>
7cc97fc
to
36d2d8a
Compare
lgtm |
No description provided.