Skip to content

Bugfix: GCP multiple credentials specified error#741

Merged
Bobi-Wan merged 1 commit intogardener:mainfrom
Bobi-Wan:fix/bug-gcp-multiple-credentials
Jan 9, 2026
Merged

Bugfix: GCP multiple credentials specified error#741
Bobi-Wan merged 1 commit intogardener:mainfrom
Bobi-Wan:fix/bug-gcp-multiple-credentials

Conversation

@Bobi-Wan
Copy link
Copy Markdown
Contributor

@Bobi-Wan Bobi-Wan commented Jan 9, 2026

What this PR does / why we need it:
The GCP API storage client introduced a bug related to multiple credentials options being used.
googleapis/google-cloud-go#13531

This bug is triggered with the key_file flow for GCP authentication.
This PR bumps google.golang.org/api v0.259.0 (and indirectly googleapis/google-api-go-client) to v0.259.0, which contains the bugfix.

Bugfix:
googleapis/google-api-go-client@2c33732
the bugfix is in the following release:
https://github.com/googleapis/google-api-go-client/releases/tag/v0.259.0

Bumping our local dependency (google.golang.org/api v0.259.0) fixes the issue as expected.

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:

Release note:

GCP Storage client multiple credentials error bugfix

this fixes the multiple credentials bug
@Bobi-Wan Bobi-Wan requested a review from a team as a code owner January 9, 2026 13:52
@gardener-robot gardener-robot added needs/review Needs review size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Jan 9, 2026
Copy link
Copy Markdown
Member

@dnaeon dnaeon left a comment

Choose a reason for hiding this comment

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

/lgtm

@gardener-robot gardener-robot added reviewed/lgtm Has approval for merging and removed needs/review Needs review labels Jan 9, 2026
@Bobi-Wan Bobi-Wan merged commit 1373a0f into gardener:main Jan 9, 2026
5 checks passed
@gardener-robot gardener-robot added the status/closed Issue is closed (either delivered or triaged) label Jan 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

reviewed/lgtm Has approval for merging size/S Denotes a PR that changes 10-29 lines, ignoring generated files. status/closed Issue is closed (either delivered or triaged)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants