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

Ansible request for jq for test machines #3453

Closed
Tracked by #3711
sophia-guo opened this issue Mar 6, 2024 · 3 comments · Fixed by #3460
Closed
Tracked by #3711

Ansible request for jq for test machines #3453

sophia-guo opened this issue Mar 6, 2024 · 3 comments · Fixed by #3460
Labels

Comments

@sophia-guo
Copy link

jq is need to install for reproducible comparing tests with sbom json file.

For windows with cygwin, cygwin need install jq

adoptium/temurin-build#3687 (comment)

@sxa
Copy link
Member

sxa commented Mar 8, 2024

Based on discussions elsewhere I think the important thing will be to have this in the operating systems that we use on the Linux test machines that can run docker so that, plus Windows, should be the scope of this change.

Noting that jq does not exist in the repositories for CentOS7, so while there are CentOS74 and RHEL7 machines in that set, I am tempted to exclude them from this (maybe we should remove sw.tool.docker freom those hosts now?) since it would require installing from the repositories and since those OS versions are soon to be out of support I'm not sure there is benefit to doing that workaround. If anyone disagrees, please let me know.

@sxa
Copy link
Member

sxa commented Mar 8, 2024

Note: We may have to deploy jq under cygwin manually on the existing windows machines, as the change in the PR will only apply it to new installs of cygwin.

@sophia-guo
Copy link
Author

Thanks @steelhead31 to deploy it manully on jenkins windows machines!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants