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

docs: (IAC-1244) jmespath requirement not listed #15

Closed
raphaelpoumarede opened this issue Jul 8, 2022 · 2 comments · Fixed by #106
Closed

docs: (IAC-1244) jmespath requirement not listed #15

raphaelpoumarede opened this issue Jul 8, 2022 · 2 comments · Fixed by #106
Assignees
Labels
documentation Improvements or additions to documentation enhancement New feature or request

Comments

@raphaelpoumarede
Copy link

I'm using the tool for a bare metal deployment using the ./oss-k8s.sh script (not the docker container) and I get the following failure because jmespath is not installed.

TASK [kubernetes/sas-iac-buildinfo : Create the sas-iac-buildinfo ConfigMap manifest file] *************************************************************
Friday 08 July 2022  07:27:35 +0000 (0:00:00.372)       0:00:00.982 ***********
An exception occurred during task execution. To see the full traceback, use -vvv. The error was: ansible.errors.AnsibleError: You need to install "jmespath" prior to running json_query filter
fatal: [localhost]: FAILED! => {"changed": false, "msg": "AnsibleError: You need to install \"jmespath\" prior to running json_query filter"}`

I see it is part of the requirements.txt file and installed when building the docker image, but it should be part of the documented requirements when not using the docker image.

@thpang thpang added documentation Improvements or additions to documentation enhancement New feature or request labels Jul 8, 2022
@thpang
Copy link
Member

thpang commented Jul 8, 2022

Yeah that one is also tricky as it uses terraform so the login around adding in the iac-buildinfo into the cluster needs to be revisited. We'll look at adding the required list of tools here as well. Somewhat related to #12

@jarpat jarpat self-assigned this Dec 5, 2023
@jarpat
Copy link
Contributor

jarpat commented Dec 5, 2023

Internal ticket created for this doc update.

@jarpat jarpat changed the title jmespath requirement not listed docs: (IAC-1244) jmespath requirement not listed Dec 5, 2023
@jarpat jarpat linked a pull request Jan 26, 2024 that will close this issue
@jarpat jarpat closed this as completed in 41beecb Feb 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants