Skip to content
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

Wrong variable prevents from finding correct tools #9

Merged
merged 2 commits into from
Aug 6, 2019

Conversation

marckhouzam
Copy link
Member

My pip installation is for python2 so things didn't work until the PATH was properly set to point to the virtualenv bin directory which has pip installed for python3.

Also, I had forgotten to send stderr to /dev/null when checking for helm v3, so there was an error printout that might have been confusing.

Now make acceptance passes for me 😄

Signed-off-by: Marc Khouzam <marc.khouzam@gmail.com>
Signed-off-by: Marc Khouzam <marc.khouzam@gmail.com>
@marckhouzam
Copy link
Member Author

@jdolitsky Since you just merged my other PR, I thought maybe you missed this one. This one is important because of the typo in the name of the VENV_DIR variable.

Thanks!

@jdolitsky
Copy link
Collaborator

👍

@jdolitsky jdolitsky merged commit 24c38f7 into helm:master Aug 6, 2019
@marckhouzam marckhouzam deleted the fix/envvar branch August 6, 2019 10:32
pmacik added a commit to pmacik/acceptance-testing that referenced this pull request Nov 3, 2020
Signed-off-by: Pavel Macík <pavel.macik@gmail.com>
pratikjagrut pushed a commit to pratikjagrut/acceptance-testing that referenced this pull request Jan 11, 2021
Signed-off-by: Pavel Macík <pavel.macik@gmail.com>
pmacik added a commit to pmacik/acceptance-testing that referenced this pull request Aug 23, 2022
…m commits

-update pip to pip3

-run openshift tests

-Fix test data deployment version

-Install PV before test setup and fix deamon set tests

-Add helm dir in .acceptance

-Change to use current helm bin

-Disable test for version greater than 3.1

-Create blank.yml

-Enable tests related to Helm 3.2 bin. (helm#2)

-Add openshift cluster provider (helm#6)
* Run acceptance tests against provided k8s cluster
* Add openshift cluster provider

-Fix kubeVersion to support kube pre-releases (helm/helm#6190) (helm#9)

-Remove obsolete and redundand test cases from kubernetes_versions.robot (helm#12)

-Adjust completion test according to helm 3.4 flags (helm#11)

-Fixes the completion tests failure by adjusting tests according to new changes flags changes in helm 3.4.

Co-authored-by: Akash Shinde <akashshinde159@gmail.com>

Signed-off-by: Pavel Macík <pavel.macik@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants