Skip to content

fix: Fail open on any error resolving the project ID in the ADK templates. - #7149

Merged
copybara-service[bot] merged 1 commit into
mainfrom
copybara_981563579
Sep 15, 2026
Merged

copybara-service[bot] merged 1 commit into
mainfrom
copybara_981563579

Conversation

@copybara-service

Copy link
Copy Markdown
Contributor

fix: Fail open on any error resolving the project ID in the ADK templates.

AdkApp.project_id() converts a project number into a project ID with a Cloud
Resource Manager projects.get call, but tolerated only PermissionDenied and
Unauthenticated. Any other failure -- a gRPC Unknown when network
interception terminates the stream, a DeadlineExceeded from a slow lookup --
escaped set_up() and stopped the agent from starting. The lookup is cosmetic,
since every caller works with the project number, so it now falls back to the
project number on any error and logs a warning, matching what
aiplatform.initializer already does for the same call.

@product-auto-label product-auto-label Bot added the size: m Pull request size is medium. label Sep 15, 2026
@copybara-service
copybara-service Bot requested a review from a team as a code owner September 15, 2026 06:08
@product-auto-label product-auto-label Bot added the api: vertex-ai Issues related to the googleapis/python-aiplatform API. label Sep 15, 2026
@copybara-service
copybara-service Bot force-pushed the copybara_981563579 branch 3 times, most recently from 4b11506 to 047810a Compare September 15, 2026 18:02
…ates.

`AdkApp.project_id()` converts a project number into a project ID with a Cloud
Resource Manager `projects.get` call, but tolerated only `PermissionDenied` and
`Unauthenticated`. Any other failure -- a gRPC `Unknown` when network
interception terminates the stream, a `DeadlineExceeded` from a slow lookup --
escaped `set_up()` and stopped the agent from starting. The lookup is cosmetic,
since every caller works with the project number, so it now falls back to the
project number on any error and logs a warning, matching what
`aiplatform.initializer` already does for the same call.

PiperOrigin-RevId: 981933906
@copybara-service
copybara-service Bot merged commit 30bf665 into main Sep 15, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: vertex-ai Issues related to the googleapis/python-aiplatform API. google-contributor size: m Pull request size is medium.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant