-
Notifications
You must be signed in to change notification settings - Fork 13
chore(deps): update google.golang.org/genproto/googleapis/rpc digest to 925bb5d #167
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,6 +1,6 @@ | ||
| module github.com/konflux-ci/qe-tools | ||
|
|
||
| go 1.23.0 | ||
| go 1.25.0 | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. [critical] Go version mismatch The go directive is bumped from 1.23.0 to 1.25.0, but CI workflows pin Go 1.23.x. The go.mod go directive acts as a minimum version gate: starting with Go 1.21, if the toolchain running the build is older than the version in go.mod, the build refuses to proceed. CI in .github/workflows/test.yml uses go-version: [1.23.x] (line 15) and go-version: 1.23 (line 53), .github/workflows/lint.yml uses go-version: 1.23 (line 19), and .github/workflows/pre-commit.yml uses go-version: 1.23 (line 17). All will fail when they encounter go 1.25 in go.mod. Additionally, the Dockerfile uses ubi9/go-toolset:9.8-1780373831, which ships Go 1.23 — that container build will also fail. Suggested fix: Either revert the go directive to go 1.23.0 (keeping the dependency bumps), or update all CI workflow Go versions and the Dockerfile base image to a Go 1.25-compatible toolchain. The CI files to update are .github/workflows/test.yml (lines 15 and 53), .github/workflows/lint.yml (line 19), .github/workflows/pre-commit.yml (line 17), plus Dockerfile (line 1). |
||
|
|
||
| require ( | ||
| cloud.google.com/go/storage v1.38.0 | ||
|
|
@@ -78,7 +78,7 @@ require ( | |
| github.com/gomodule/redigo v1.8.5 // indirect | ||
| github.com/google/btree v1.0.1 // indirect | ||
| github.com/google/gnostic v0.6.9 // indirect | ||
| github.com/google/go-cmp v0.6.0 // indirect | ||
| github.com/google/go-cmp v0.7.0 // indirect | ||
| github.com/google/go-querystring v1.1.0 // indirect | ||
| github.com/google/gofuzz v1.2.1-0.20221018181003-5dfa3b7056d4 // indirect | ||
| github.com/google/s2a-go v0.1.7 // indirect | ||
|
|
@@ -157,9 +157,9 @@ require ( | |
| google.golang.org/appengine v1.6.8 // indirect | ||
| google.golang.org/genproto v0.0.0-20240125205218-1f4bbc51befe // indirect | ||
| google.golang.org/genproto/googleapis/api v0.0.0-20240205150955-31a09d347014 // indirect | ||
| google.golang.org/genproto/googleapis/rpc v0.0.0-20240205150955-31a09d347014 // indirect | ||
| google.golang.org/genproto/googleapis/rpc v0.0.0-20260630182238-925bb5da69e7 // indirect | ||
| google.golang.org/grpc v1.61.0 // indirect | ||
| google.golang.org/protobuf v1.33.0 // indirect | ||
| google.golang.org/protobuf v1.36.11 // indirect | ||
| gopkg.in/inf.v0 v0.9.1 // indirect | ||
| gopkg.in/ini.v1 v1.67.2 // indirect | ||
| gopkg.in/yaml.v2 v2.4.0 // indirect | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[medium] stale-reference
The go directive is bumped from 1.22.0 to 1.25.0, but all CI workflows (.github/workflows/test.yml, lint.yml, pre-commit.yml, slack-message.yml) are pinned to Go 1.22.x, and the Dockerfile uses ubi9/go-toolset:9.8 which may not include Go 1.25. Since Go 1.21+, the go directive acts as a minimum version gate — CI builds will either fail or require GOTOOLCHAIN=auto to auto-download the correct toolchain.
Suggested fix: Either revert the Go directive to 1.22.0 and keep only the dependency updates, or update all CI workflows and the Dockerfile to use Go 1.25+ in a coordinated PR.