Skip to content

Add explicit token scope to ADC login command - #50

Merged
xianyuanjia merged 3 commits into
mainfrom
xianyuanjia-patch-1
May 11, 2026
Merged

Add explicit token scope to ADC login command#50
xianyuanjia merged 3 commits into
mainfrom
xianyuanjia-patch-1

Conversation

@xianyuanjia

Copy link
Copy Markdown
Collaborator

Resolves insufficient token scope errors arising from existing auth tokens on user host.

Resolves insufficient token scope errors arising from existing auth tokens on user host.
@xianyuanjia
xianyuanjia requested a review from ko1in1u May 11, 2026 19:29

@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 gcloud authentication process in src/results_uploader/gcloud_setup.py to include the cloud-platform scope. A review comment correctly identifies a syntax error in the added code, specifically a trailing backtick and unnecessary double quotes that would cause the gcloud command to fail.

Comment thread src/results_uploader/gcloud_setup.py Outdated
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@xianyuanjia
xianyuanjia merged commit a0bd3e8 into main May 11, 2026
2 checks passed
@xianyuanjia
xianyuanjia deleted the xianyuanjia-patch-1 branch May 11, 2026 20:28
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