Skip to content
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

ceph.conf: fail if interface does not have any IP #2263

Closed
wants to merge 1 commit into from

Conversation

leseb
Copy link
Member

@leseb leseb commented Dec 14, 2017

We want to fail if the interface provided for either mon/rgw does not
have any IP.

Signed-off-by: Sébastien Han seb@redhat.com

@leseb leseb force-pushed the interface-not-empty branch 5 times, most recently from f7db8e8 to 98bced8 Compare December 15, 2017 10:33
We want to fail if the interface provided for either mon/rgw does not
have any IP.

Signed-off-by: Sébastien Han <seb@redhat.com>
@guits
Copy link
Collaborator

guits commented Apr 24, 2018

@leseb this feature could be addressed by the upcoming 'host var / pre check' validation module I think.
should we close this PR ?

@leseb
Copy link
Member Author

leseb commented Apr 24, 2018

I'm not sure, it's hard to tell. The use case here is: a user provides a network interface for a monitor and that interface doesn't have any IP. I'm not sure if the upcoming module will solve that in the first iteration.

I'll leave this one open just so we don't forget about it.
What do you think?

@guits
Copy link
Collaborator

guits commented Apr 24, 2018

if the upcoming module can access hostvars it should be able to handle this use case.
I'm ok to leave this open if we are not sure.

@guits
Copy link
Collaborator

guits commented Apr 24, 2018

@andrewschoen any thoughts on this?

@leseb
Copy link
Member Author

leseb commented Jul 23, 2018

Closing this in favor of #2915

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants