-
Notifications
You must be signed in to change notification settings - Fork 71
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
Fixes #24891 - return non-zero exit code when services are down #216
Conversation
Issues: #24891 |
c33a5f7
to
53c97fb
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.
Thank you @mbacovsky,
Please check a inline comment related to typo correction.
definitions/features/service.rb
Outdated
status | ||
end | ||
|
||
def alowed_action?(action) |
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.
Correct the typo it should be allowed_action?
53c97fb
to
19b955b
Compare
Sorting services also by name to make removing duplicites more efficient.
19b955b
to
e948b8c
Compare
@kgaikwad thanks for review, updated |
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.
Tested and works well. The info about --whitelist="service-status"
is a bit strange, but I agree this should be tacked in separate PR. @ntkathole do you want to give this a try as well before merge?
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.
Looks good! ACK
Thanks @mbacovsky |
No description provided.