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

chore: Use jq to extract field #8834

Merged
merged 1 commit into from
Nov 22, 2022
Merged

chore: Use jq to extract field #8834

merged 1 commit into from
Nov 22, 2022

Conversation

lqiu96
Copy link
Contributor

@lqiu96 lqiu96 commented Nov 22, 2022

Use JQ to extract field from JSON. This will help use remove the the python3 dependency. I'll need to install JQ on all the Java and GraalVM images.

Update the docs at cl/490322376

This is to fix the issue in GraalVM:

./helpers/common.sh: line 43: python3: command not found
cleanup

@lqiu96
Copy link
Contributor Author

lqiu96 commented Nov 22, 2022

The latest GraalVM and Java images have JQ installed now.

@lqiu96 lqiu96 marked this pull request as ready for review November 22, 2022 21:02
@lqiu96 lqiu96 added the owlbot:run Add this label to trigger the Owlbot post processor. label Nov 22, 2022
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Nov 22, 2022
@lqiu96 lqiu96 merged commit 02eba7b into main Nov 22, 2022
@lqiu96 lqiu96 deleted the main-terraform_ITs_add_jq branch November 22, 2022 22:18
alicejli pushed a commit that referenced this pull request Nov 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants