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
Allow completion tests to pass on github #60
Conversation
fe4c860
to
cc3b27e
Compare
bf5e1ad
to
8528bc8
Compare
Note sure what is happening but the docker image is not able to run helm. In the last build we can see:
Note that everything works fine for completion outside of docker, but when we mount helm in docker, it give a No such file or directory error even though the file is there. |
f186cc2
to
5699554
Compare
09a7175
to
a02e37e
Compare
a02e37e
to
3bd284e
Compare
Signed-off-by: Marc Khouzam <marc.khouzam@montreal.ca>
3bd284e
to
61c95b1
Compare
The tests pass!!! |
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.
nice @marckhouzam !
* 'master' of github.com:helm/acceptance-testing: (41 commits) New helm global flags --kube-token and --kube-apiserver Add completion tests for release names Add completion tests for plugin completion Test kube-context completion without assuming kubectl Group related tests together Don't use a plugin named 'template' for completion Revert "Support if the help is either stdout or stderr (helm#64)" Update namespace flag tests for zsh Some fixes in preparation for plugin completion Make completion tests work with completion in Go Add completion test on centos Fix helm repo update not working Fix some of the KFAILs Fix the ZFAIL cases Support if the help is either stdout or stderr (helm#64) Print ERROR instead of FAIL Use linux image for docker (helm#60) fix: Helm 3 now supports the -c flag helm plugin 'remove' is now 'uninstall' maint: build helm 3 from master ...
Signed-off-by: Marc Khouzam marc.khouzam@montreal.ca