go1.24 #1963
go1.24 #1963
Conversation
| @@ -36,7 +36,7 @@ var nameRegexp = regexp.MustCompile(`([[:graph:]]+)-([[:digit:]][\-.[:alnum:]]*( | |||
| // checkName returns the extracted package name from the above regexp. | |||
| func checkName(name string) string { | |||
| m := nameRegexp.FindStringSubmatch(name) | |||
| if m == nil || len(m) < 2 { | |||
| if len(m) < 2 { | |||
There was a problem hiding this comment.
This was called out after the linter update
| @@ -1,5 +1,5 @@ | |||
| # Compiling scanner binaries and staging repo2cpe and genesis manifests | |||
| FROM brew.registry.redhat.io/rh-osbs/openshift-golang-builder:rhel_8_1.23@sha256:0a070e4a8f2698b6aba3630a49eb995ff1b0a182d0c5fa264888acf9d535f384 AS builder | |||
| FROM brew.registry.redhat.io/rh-osbs/openshift-golang-builder:rhel_8_1.24@sha256:021ab8e6e4af3d1683b52ec985996f9421edb2111dc697b5781dee3c7595ae00 AS builder | |||
There was a problem hiding this comment.
I added konflux-build label because the builds don't trigger in PRs without it (we can revisit).
Please push any/empty commit to this PR to trigger Konflux CI.
Let's make sure openshift-golang-builder is also happy with the change.
There was a problem hiding this comment.
Thanks! I forgot the Konflux builds didn't run by default
There was a problem hiding this comment.
It looks like they changed the name of the tag, so I had to update that, too. I chose the option I figured would be best
RTann
force-pushed
the
go1.24
branch
3 times, most recently
from
9b10d05 to
2af62d4
Compare
|
|
||
| toolchain go1.23.6 | ||
| toolchain go1.24.3 |
There was a problem hiding this comment.
I learned we MUST have a toolchain version set to some version available downstream (Konflux kept failing at the prefetch stage until I made this change).
I originally removed the toolchain line and just set go 1.24 (to match the stackrox repo).
1.24 doesn't exist there, but 1.24.3 does. We could just update go 1.24 to go 1.24.3 and remove the toolchain; however, I prefer not requiring anyone who may use this repo as a library be forced to use some specific minor version of Go unless we absolutely require it. I opted to keep go 1.24 and just update the the toolchain to toolchain go1.24.3
Just a heads up we'll probably have to do this for the stackrox repo @janisz @msugakov
|
@RTann: The following tests failed, say
Full PR test history. Your PR dashboard. Instructions 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. |
See stackrox/rox-ci-image@0.4.8...0.4.9 for the updated Go version.
Also updated the linters, as they support go1.24 now