-
Notifications
You must be signed in to change notification settings - Fork 24
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
Replace kubectl with Kubernetes Python client #3
Comments
Seeking feedback on the approach to resolve issue helm#3
/assign |
I wonder if such a change would actually cause the code to be harder to maintain?
@jdolitsky do you still see this issue as a valuable improvement? |
@marckhouzam I think for commands which require counting number of results etc. this would simply alot For example this function: https://github.com/helm/acceptance-testing/blob/master/lib/Kubectl.py#L17 |
Agreed |
Remove kind deps
Instead of shelling out to kubectl, should just import and use the Python library: https://github.com/kubernetes-client/python
The text was updated successfully, but these errors were encountered: