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

install jmespath py library as root #814

Merged
merged 1 commit into from
Sep 3, 2024

Conversation

jmazzitelli
Copy link
Contributor

We are seeing issues on OpenShift not finding this jmespath library. I don't know if this fixes the problem, but installing that py library as root might be a solution. Need to test this.

@hhovsepy
Copy link
Contributor

hhovsepy commented Sep 3, 2024

works fine on jenkins pipelines on OCP cluster

@jmazzitelli jmazzitelli marked this pull request as ready for review September 3, 2024 15:23
@jmazzitelli jmazzitelli self-assigned this Sep 3, 2024
@jmazzitelli
Copy link
Contributor Author

OK. @hhovsepy you can approve and merge this then. Let's get this in now.

@hhovsepy hhovsepy merged commit 0907e2a into kiali:master Sep 3, 2024
1 check passed
@jmazzitelli jmazzitelli deleted the fix-jmespath-on-ocp branch September 3, 2024 15:43
jmazzitelli added a commit to jmazzitelli/kiali-operator that referenced this pull request Sep 3, 2024
hhovsepy pushed a commit that referenced this pull request Sep 3, 2024
@jshaughn jshaughn added the test: n/a PR does not need test additions or updates label Sep 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test: n/a PR does not need test additions or updates
Projects
Development

Successfully merging this pull request may close these issues.

3 participants