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
osd: print warning message if no matching node found for osd #13046
Conversation
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
Hi @subhamkrai , I'm not really familiar with the tests. Do you think those 2 failures are related to this PR? |
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.
Can you confirm if you tested this manually? It's difficult to confirm in the automated tests when a logging change is made. The CI failure look like known intermittent issues.
No, I didn't test it manually since it's hard for me to build and deploy it locally. (It's only a homelab setup and I don't have much resource to do it). This is a simple change and passed other tests so maybe it's enough? Of course it will be great if someone else want to test it manually. |
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 ran a quick test with these changes, just one more suggestion, thanks.
As seen in rook#1988, it's a common mistake to configure rook nodes with names that don't match Kubernetes's node label. This PR prints more detailed message to help debugging problem. Signed-off-by: Bin Wang <bin.wang@mail.binwang.me> Add break according to review comment Co-authored-by: Travis Nielsen <tnielsen@redhat.com> Update log message according to review comment Co-authored-by: Travis Nielsen <tnielsen@redhat.com>
osd: print warning message if no matching node found for osd (backport #13046)
Description of your changes:
As seen in #1988, it's a common mistake to configure rook nodes with names that don't match Kubernetes's node label. This PR prints more detailed message to help debugging problem.
Which issue is resolved by this Pull Request:
Resolves #
Checklist: