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: --max_part and --nbds_max options for nbd map #12379
Conversation
@trociny , as we talked, I create a new pr for rbd nbd map, please take a look. |
@@ -1102,6 +1103,8 @@ | |||
--read-only mount read-only | |||
--exclusive forbid other clients write | |||
--device arg specify nbd device | |||
--nbds_max arg Override for module param nbds_max | |||
--max_part arg Override for module param max_part | |||
|
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.
The description for other options starts in lower case, please, be consistent.
Also it looks it would look more consistent without "for": "override module param nbds_max"...
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.
@trociny I agree. I have modified, please take a look. Thanks!
("device", po::value<std::string>(), "specify nbd device"); | ||
("device", po::value<std::string>(), "specify nbd device") | ||
("nbds_max", po::value<std::string>(), "Override for module param nbds_max") | ||
("max_part", po::value<std::string>(), "Override for module param max_part"); |
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.
@liupan1111 It looks you forgot to update here too
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.
@trociny yeah... modified. Sorry for the trouble.
Fixes: http://tracker.ceph.com/issues/18186 Signed-off-by: Pan Liu <pan.liu@istuary.com>
rbd: --max_part and --nbds_max options for nbd map #12379
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.
LGTM
…ep consistent with rbd_nbd map
Fixes: http://tracker.ceph.com/issues/18186
Signed-off-by: Pan Liu pan.liu@istuary.com