Bump cloud.google.com/go/storage from 1.36.0 to 1.63.0#98
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository: openshift/coderabbit/.coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
WalkthroughThis PR updates ChangesGo module dependency refresh
Estimated code review effort: 2 (Simple) | ~10 minutes Important Pre-merge checks failedPlease resolve all errors before merging. Addressing warnings is optional. ❌ Failed checks (1 error)
✅ Passed checks (14 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
Hi @dependabot[bot]. Thanks for your PR. I'm waiting for a openshift member to verify that this patch is reasonable to test. If it is, they should reply with Regular contributors should join the org to skip this step. Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: dependabot[bot] The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@go.mod`:
- Around line 3-68: The repo’s documented Go toolchain is outdated: update the
contributor guidance in CLAUDE.md from toolchain 1.22.1 to 1.25.8 so it matches
the current Go version declared in go.mod and the CI image already used. Locate
the toolchain reference in CLAUDE.md and change only that documented version to
keep the repo instructions consistent.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository: openshift/coderabbit/.coderabbit.yaml
Review profile: CHILL
Plan: Enterprise
Run ID: 5da8fc46-b912-4709-9acb-ade63fedd75e
⛔ Files ignored due to path filters (1)
go.sumis excluded by!**/*.sum
📒 Files selected for processing (1)
go.mod
| go 1.25.8 | ||
|
|
||
| require ( | ||
| cloud.google.com/go/storage v1.36.0 | ||
| cloud.google.com/go/storage v1.63.0 | ||
| github.com/google/go-github/v51 v51.0.0 | ||
| github.com/spf13/cobra v1.7.0 | ||
| google.golang.org/api v0.150.0 | ||
| google.golang.org/api v0.285.0 | ||
| ) | ||
|
|
||
| require ( | ||
| cloud.google.com/go v0.110.8 // indirect | ||
| cloud.google.com/go/compute v1.23.1 // indirect | ||
| cloud.google.com/go/compute/metadata v0.2.3 // indirect | ||
| cloud.google.com/go/iam v1.1.3 // indirect | ||
| cel.dev/expr v0.25.1 // indirect | ||
| cloud.google.com/go v0.123.0 // indirect | ||
| cloud.google.com/go/auth v0.20.0 // indirect | ||
| cloud.google.com/go/auth/oauth2adapt v0.2.8 // indirect | ||
| cloud.google.com/go/compute/metadata v0.9.0 // indirect | ||
| cloud.google.com/go/iam v1.11.0 // indirect | ||
| cloud.google.com/go/monitoring v1.29.0 // indirect | ||
| github.com/GoogleCloudPlatform/opentelemetry-operations-go/detectors/gcp v1.31.0 // indirect | ||
| github.com/GoogleCloudPlatform/opentelemetry-operations-go/exporter/metric v0.57.0 // indirect | ||
| github.com/GoogleCloudPlatform/opentelemetry-operations-go/internal/resourcemapping v0.57.0 // indirect | ||
| github.com/cespare/xxhash/v2 v2.3.0 // indirect | ||
| github.com/cli/safeexec v1.0.0 // indirect | ||
| github.com/golang/groupcache v0.0.0-20210331224755-41bb18bfe9da // indirect | ||
| github.com/google/s2a-go v0.1.7 // indirect | ||
| github.com/google/uuid v1.4.0 // indirect | ||
| github.com/googleapis/enterprise-certificate-proxy v0.3.2 // indirect | ||
| github.com/googleapis/gax-go/v2 v2.12.0 // indirect | ||
| github.com/kr/text v0.2.0 // indirect | ||
| go.opencensus.io v0.24.0 // indirect | ||
| golang.org/x/sync v0.5.0 // indirect | ||
| golang.org/x/text v0.13.0 // indirect | ||
| golang.org/x/time v0.3.0 // indirect | ||
| golang.org/x/xerrors v0.0.0-20220907171357-04be3eba64a2 // indirect | ||
| google.golang.org/genproto v0.0.0-20231016165738-49dd2c1f3d0b // indirect | ||
| google.golang.org/genproto/googleapis/api v0.0.0-20231016165738-49dd2c1f3d0b // indirect | ||
| google.golang.org/genproto/googleapis/rpc v0.0.0-20231030173426-d783a09b4405 // indirect | ||
| google.golang.org/grpc v1.59.0 // indirect | ||
| github.com/cncf/xds/go v0.0.0-20260202195803-dba9d589def2 // indirect | ||
| github.com/envoyproxy/go-control-plane/envoy v1.37.0 // indirect | ||
| github.com/envoyproxy/protoc-gen-validate v1.3.3 // indirect | ||
| github.com/felixge/httpsnoop v1.0.4 // indirect | ||
| github.com/go-jose/go-jose/v4 v4.1.4 // indirect | ||
| github.com/go-logr/logr v1.4.3 // indirect | ||
| github.com/go-logr/stdr v1.2.2 // indirect | ||
| github.com/google/s2a-go v0.1.9 // indirect | ||
| github.com/google/uuid v1.6.0 // indirect | ||
| github.com/googleapis/enterprise-certificate-proxy v0.3.16 // indirect | ||
| github.com/googleapis/gax-go/v2 v2.22.0 // indirect | ||
| github.com/planetscale/vtprotobuf v0.6.1-0.20240319094008-0393e58bdf10 // indirect | ||
| github.com/spiffe/go-spiffe/v2 v2.6.0 // indirect | ||
| go.opentelemetry.io/auto/sdk v1.2.1 // indirect | ||
| go.opentelemetry.io/contrib/detectors/gcp v1.42.0 // indirect | ||
| go.opentelemetry.io/contrib/instrumentation/google.golang.org/grpc/otelgrpc v0.68.0 // indirect | ||
| go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp v0.67.0 // indirect | ||
| go.opentelemetry.io/otel v1.44.0 // indirect | ||
| go.opentelemetry.io/otel/metric v1.44.0 // indirect | ||
| go.opentelemetry.io/otel/sdk v1.44.0 // indirect | ||
| go.opentelemetry.io/otel/sdk/metric v1.44.0 // indirect | ||
| go.opentelemetry.io/otel/trace v1.44.0 // indirect | ||
| golang.org/x/sync v0.21.0 // indirect | ||
| golang.org/x/text v0.38.0 // indirect | ||
| golang.org/x/time v0.15.0 // indirect | ||
| google.golang.org/genproto v0.0.0-20260519071638-aa98bba5eb94 // indirect | ||
| google.golang.org/genproto/googleapis/api v0.0.0-20260615183401-62b3387ff324 // indirect | ||
| google.golang.org/genproto/googleapis/rpc v0.0.0-20260615183401-62b3387ff324 // indirect | ||
| google.golang.org/grpc v1.81.1 // indirect | ||
| gopkg.in/yaml.v3 v3.0.1 // indirect | ||
| ) | ||
|
|
||
| require ( | ||
| github.com/ProtonMail/go-crypto v0.0.0-20230217124315-7d5c6f04bbb8 | ||
| github.com/cli/go-gh/v2 v2.4.0 | ||
| github.com/cloudflare/circl v1.1.0 // indirect | ||
| github.com/golang/protobuf v1.5.3 // indirect | ||
| github.com/google/go-querystring v1.1.0 // indirect | ||
| github.com/inconshreveable/mousetrap v1.1.0 // indirect | ||
| github.com/spf13/pflag v1.0.5 // indirect | ||
| golang.org/x/crypto v0.14.0 // indirect | ||
| golang.org/x/net v0.17.0 // indirect | ||
| golang.org/x/oauth2 v0.13.0 | ||
| golang.org/x/sys v0.13.0 // indirect | ||
| google.golang.org/appengine v1.6.7 // indirect | ||
| google.golang.org/protobuf v1.31.0 // indirect | ||
| golang.org/x/crypto v0.53.0 // indirect | ||
| golang.org/x/net v0.56.0 // indirect | ||
| golang.org/x/oauth2 v0.36.0 | ||
| golang.org/x/sys v0.46.0 // indirect | ||
| google.golang.org/protobuf v1.36.11 // indirect |
There was a problem hiding this comment.
🩺 Stability & Availability | 🟡 Minor | ⚡ Quick win
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
rg -n --hidden --glob '!**/vendor/**' 'go 1\.25\.8|toolchain|1\.22\.1|golang:1\.22|golang:1\.25' .Repository: openshift/backplane-tools
Length of output: 238
🏁 Script executed:
#!/bin/bash
set -euo pipefail
rg -n --hidden --glob '!**/vendor/**' 'go 1\.25\.8|toolchain|1\.22\.1|golang:1\.22|golang:1\.25|Go 1\.21\+|1\.22\.1|1\.25\.8' .Repository: openshift/backplane-tools
Length of output: 238
🏁 Script executed:
#!/bin/bash
set -euo pipefail
# Locate CI / release / build files that may pin a Go image or toolchain.
git ls-files | rg '(^|/)(\.ci-operator\.yaml|Dockerfile|dockerfile|Makefile|go\.mod|go\.work|CLAUDE\.md|README\.md|hack/|\.github/|\.prow/|\.tekton/|release|build|pipeline|workflow)'
# Search for explicit Go version pins in those files.
rg -n --hidden --glob '!**/vendor/**' 'go 1\.25\.8|toolchain|1\.22\.1|1\.25\.8|golang:1\.22|golang:1\.25|go-version|GOVERSION|GO_VERSION' \
.ci-operator.yaml .github .prow .tekton hack . 2>/dev/nullRepository: openshift/backplane-tools
Length of output: 375
🏁 Script executed:
#!/bin/bash
set -euo pipefail
printf '\n## .ci-operator.yaml\n'
sed -n '1,220p' .ci-operator.yaml
printf '\n## CLAUDE.md (relevant lines)\n'
sed -n '60,72p' CLAUDE.md
printf '\n## README.md (Go/toolchain references)\n'
rg -n 'Go 1\.|toolchain|go1\.' README.md Makefile hack/codecov.shRepository: openshift/backplane-tools
Length of output: 619
Sync the documented Go toolchain
CLAUDE.md still says toolchain 1.22.1; update it to 1.25.8 so contributor guidance matches the repo’s current Go requirement. .ci-operator.yaml already points at rhel-9-release-golang-1.25-openshift-4.22.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@go.mod` around lines 3 - 68, The repo’s documented Go toolchain is outdated:
update the contributor guidance in CLAUDE.md from toolchain 1.22.1 to 1.25.8 so
it matches the current Go version declared in go.mod and the CI image already
used. Locate the toolchain reference in CLAUDE.md and change only that
documented version to keep the repo instructions consistent.
Bumps [cloud.google.com/go/storage](https://github.com/googleapis/google-cloud-go) from 1.36.0 to 1.63.0. - [Release notes](https://github.com/googleapis/google-cloud-go/releases) - [Changelog](https://github.com/googleapis/google-cloud-go/blob/main/documentai/CHANGES.md) - [Commits](googleapis/google-cloud-go@dlp/v1.36.0...compute/v1.63.0) --- updated-dependencies: - dependency-name: cloud.google.com/go/storage dependency-version: 1.63.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
dbd92a1 to
64bc0b0
Compare
|
@dependabot[bot]: all tests passed! Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #98 +/- ##
=====================================
Coverage 0.00% 0.00%
=====================================
Files 34 34
Lines 1594 1594
=====================================
Misses 1594 1594 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Bumps cloud.google.com/go/storage from 1.36.0 to 1.63.0.
Release notes
Sourced from cloud.google.com/go/storage's releases.
... (truncated)
Changelog
Sourced from cloud.google.com/go/storage's changelog.
... (truncated)
Commits
033f4fechore: librarian release pull request: 20260514T191310Z (#14589)bf1c57achore(agentplatform): Organize the replay tests.d600fbbchore(agentplatform): Mock backend for unit testingec55a12test(spanner): retry query after database recreation in integration test (#14...66574dbfix(agentplatform): Set the agentplatform's dedicated user-agent in header.9072072fix(pubsub/v2/pstest): make ackIDs unique per delivery (#14554)9873268refactor(pubsub/v2): flatten implementation control logic for Ack and ModAck ...b1db39echore(storage): Fix corner cases in MRD (#14522)df96b2efeat: update API sources and regenerate (#14581)658faa8fix(bigquery): handle reset case for table clustering (#14579)Summary by CodeRabbit