-
Notifications
You must be signed in to change notification settings - Fork 36
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
Adds parameters for MicroCeph bootstrap #252
Adds parameters for MicroCeph bootstrap #252
Conversation
a7ad624
to
33e7443
Compare
33e7443
to
a1d6966
Compare
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
Are you folks going to implement |
Doesn't it make more sense if we say the new option as ref: #197 |
The |
Makes sense. Thanks for the clarification. |
40c9c33
to
300f666
Compare
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.
Hey Utkarsh,
Many thanks for that -- I've added some comments.
Would it be possible to add some testing for the multinode test case as well?
6ac8769
to
daaa4e4
Compare
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.
Hi Utkarsh, many thanks -- think the approach with separating out bootstrapping will serve well for future feature additions
Some comments inline.
Two things will be needed in addition:
- Functional testing
- Updateing the
microceph init
command to work with the new approach
313ef0e
to
f8c3fd1
Compare
f8c3fd1
to
386966b
Compare
668a703
to
762d7ec
Compare
Signed-off-by: Utkarsh Bhatt <[email protected]>
Signed-off-by: Utkarsh Bhatt <[email protected]>
Signed-off-by: Utkarsh Bhatt <[email protected]>
762d7ec
to
b2f10fa
Compare
Signed-off-by: Utkarsh Bhatt <[email protected]>
b2f10fa
to
b826f87
Compare
Recreated PR#:#266 |
As a way of providing a custom public network for the mon cluster, a new bootstrap parameter is added to cli.
Desired monitor IP can be provided to the
cluster bootstrap
command as--mon-ip
parameter.Manual Testing ✅: