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
OCP:Fix rule kubelet_enable_streaming_connections #8841
Conversation
/retest |
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.
Please ensure that your files end with a single newline. It looks like there might be some trailing spaces on some of the files in this PR.
9ac89ce
to
45cf53c
Compare
Thanks for the Review, just fixed that |
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.
Thanks for the quick respone.
Looks like there are still two files that need the newline at the end.
"shutdownGracePeriod": "0s", | ||
"shutdownGracePeriodCriticalPods": "0s" | ||
} | ||
EOF |
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.
Still need a newline after the EOF.
"shutdownGracePeriod": "5m", | ||
"shutdownGracePeriodCriticalPods": "0s" | ||
} | ||
EOF |
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.
Still need a newline after the EOF.
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.
could you help me to review it one more time? thanks @Mab879
/retest |
05bf9c9
to
09e25fe
Compare
This PR fix the rule kubelet_enable_streaming_connections, so that it will check the configuration based on variable dynamically.
Code Climate has analyzed commit d6414ff and detected 0 issues on this pull request. The test coverage on the diff in this pull request is 100.0% (50% is the threshold). This pull request will bring the total coverage in the repository to 26.1% (0.0% change). View more on Code Climate. |
/retest |
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
I think there was a BZ about this right? If yes, just please remember to mark it as MODIFIED when merging.
@jhrozek Why didn't you merge it? What is this waiting for? |
This PR fix the rule kubelet_enable_streaming_connections, so that it will check the configuration based on variable dynamically.