-
Notifications
You must be signed in to change notification settings - Fork 266
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
Add python-client #523
Add python-client #523
Conversation
Signed-off-by: Babak Ghadiri <bghadiri@hamravesh.com>
This is great. Thank you for contributing to Kuberhealthy and adding a nice example for Python programmers to get started. You may want to also add a check on the writing your own checks page. @jonnydawg is taking a look more deeply at this for feedback. |
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, thanks for doing this! Just some comments here and there I left for you
Signed-off-by: Babak Ghadiri <bghadiri@hamravesh.com>
Signed-off-by: Babak Ghadiri <bghadiri@hamravesh.com> Signed-off-by: Babak Ghadiri <bghadiri@hamravesh.com>
Signed-off-by: Babak Ghadiri <bghadiri@hamravesh.com> Signed-off-by: Babak Ghadiri <bghadiri@hamravesh.com>
Signed-off-by: Babak Ghadiri <bghadiri@hamravesh.com> Signed-off-by: Babak Ghadiri <bghadiri@hamravesh.com>
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! Thanks for your contributions
Python-client has been added by following JavaScript client codes.
It's better to make a PyPi library from it and publish that by your (kuberhealthy) credentials for making it more easier to use.
Signed-off-by: Babak Ghadiri bghadiri@hamravesh.com