Skip to content

Revert "Revert "Update to the latest version of the GCS library""#9941

Merged
vanja-p merged 3 commits into
masterfrom
revert-9938-revert-9926-vanja-gcs
Jul 25, 2025
Merged

Revert "Revert "Update to the latest version of the GCS library""#9941
vanja-p merged 3 commits into
masterfrom
revert-9938-revert-9926-vanja-gcs

Conversation

@vanja-p

@vanja-p vanja-p commented Jul 22, 2025

Copy link
Copy Markdown
Contributor

Reverts #9938

This upgrades the GCS dependency again, but this time fixing the error handling that was broken by googleapis/google-cloud-go#11519. I confirmed that the new error handling works for both HTTP and gRPC.

@tylerwilliams

Copy link
Copy Markdown
Member

(intentionally not approving this until we get a fix in / figure out what's up)

@vanja-p vanja-p marked this pull request as draft July 23, 2025 00:36
@vanja-p vanja-p marked this pull request as ready for review July 23, 2025 01:57
@vanja-p

vanja-p commented Jul 23, 2025

Copy link
Copy Markdown
Contributor Author

(intentionally not approving this until we get a fix in / figure out what's up)

Yeah, I meant to keep it as a draft until I figured it out. It's fixed now. I tested in dev with an invocation that was failing to load the build log.

@vanja-p vanja-p merged commit c8c342b into master Jul 25, 2025
13 of 15 checks passed
@vanja-p vanja-p deleted the revert-9938-revert-9926-vanja-gcs branch July 25, 2025 17:49
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.

3 participants