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: use node externalIP if no internalIP defined #8653
Conversation
53bf52d
to
2cf5b77
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.
@JrCs See the unit test that needs to be updated:
=== RUN TestGetNodeInfoFromNode
node_test.go:226:
Error Trace: node_test.go:226
Error: Expected value not to be nil.
Test: TestGetNodeInfoFromNode
--- FAIL: TestGetNodeInfoFromNode (0.00s)
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.
I'm ok with the patch. Trying internalIP and then fallback to externalIP if possible is probably better than adding a new CRD setting. Please fix the unit tests and add new ones too. Merci.
In some cases node internalIP is not defined. Then use externalIP if it exists. Signed-off-by: JrCs <90z7oey02@sneakemail.com>
2cf5b77
to
c606f4c
Compare
@leseb tests added |
This pull request has merge conflicts that must be resolved before it can be merged. @JrCs please rebase it. https://rook.io/docs/rook/master/development-flow.html#updating-your-fork |
ceph: use node externalIP if no internalIP defined (backport #8653)
Description of your changes:
Use node externalIP if no internalIP defined
Which issue is resolved by this Pull Request:
Resolves #7866