Skip to content

fix(setup): fix gcloud taxonomy filter and resolve import conflicts using project-suffixed display name#1374

Merged
jeffonelson merged 3 commits into
GoogleCloudPlatform:mainfrom
jeffonelson:main
Jun 17, 2026
Merged

fix(setup): fix gcloud taxonomy filter and resolve import conflicts using project-suffixed display name#1374
jeffonelson merged 3 commits into
GoogleCloudPlatform:mainfrom
jeffonelson:main

Conversation

@jeffonelson

Copy link
Copy Markdown
Contributor

No description provided.

@jeffonelson jeffonelson requested a review from Lsubatin June 17, 2026 17:16

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the setup_lab.sh script to handle potential taxonomy name conflicts by appending the project ID to the taxonomy display name (LostCargoSecurity_${PROJECT_ID}) and queries the taxonomy using displayName instead of display_name. The review feedback identifies potential SIGPIPE (exit code 141) issues caused by using awk with exit or head -n 1 in pipelines when set -o pipefail is enabled. It suggests leveraging gcloud's built-in filtering capabilities and using awk 'NR==1' to safely extract the first match without closing the pipe early.

Comment thread codelabs/bigquery-graph-analytics/setup_lab.sh
Comment thread codelabs/bigquery-graph-analytics/setup_lab.sh Outdated
Comment thread codelabs/bigquery-graph-analytics/setup_lab.sh Outdated
@jeffonelson jeffonelson merged commit 814959e into GoogleCloudPlatform:main Jun 17, 2026
5 checks passed
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