Skip to content

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

Open
copybara-service[bot] wants to merge 1 commit into
mainfrom
copybara_1010226096
Open

copybara-service[bot] wants to merge 1 commit into
mainfrom
copybara_1010226096

Conversation

@copybara-service

Copy link
Copy Markdown
Contributor

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

AdkApp converts a project number into a project ID with a Cloud Resource
Manager call during set_up(), but tolerated only PermissionDenied and
Unauthenticated. Any other failure -- including google-cloud-aiplatform
not being installed, which surfaced as UnboundLocalError -- stopped the
agent from starting. The lookup is cosmetic, so it now falls back to the
project number on any error and logs a warning.

FUTURE_COPYBARA_INTEGRATE_REVIEW=#7235 from googleapis:release-please--branches--main fe718d1

@copybara-service
copybara-service Bot requested a review from a team as a code owner October 30, 2026 05:20
@product-auto-label product-auto-label Bot added size: m Pull request size is medium. api: vertex-ai Issues related to the googleapis/python-aiplatform API. labels Oct 30, 2026
…work.

`AdkApp` converts a project number into a project ID with a Cloud Resource
Manager call during `set_up()`, but tolerated only `PermissionDenied` and
`Unauthenticated`. Any other failure -- including `google-cloud-aiplatform`
not being installed, which surfaced as `UnboundLocalError` -- stopped the
agent from starting. The lookup is cosmetic, so it now falls back to the
project number on any error and logs a warning.

FUTURE_COPYBARA_INTEGRATE_REVIEW=#7235 from googleapis:release-please--branches--main fe718d1
PiperOrigin-RevId: 1010226096
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