vendor: update containers/{image,storage}#24054
vendor: update containers/{image,storage}#24054openshift-merge-bot[bot] merged 2 commits intocontainers:mainfrom
Conversation
|
Ephemeral COPR build failed. @containers/packit-build please check. |
cb08fc6 to
7c3e2d1
Compare
|
centos-stream 9/10 don't have golang 1.22.6 so this is gonna fail on those envs. Do we need to ship this soon there? If not, then we can ignore centos failures. |
|
@giuseppe validate test is still failing |
7c3e2d1 to
05b35ed
Compare
go.mod
Outdated
There was a problem hiding this comment.
This is wrong, we do not want to force newer toolchains for no reason see the comment above.
There was a problem hiding this comment.
I reverted this part, pushed a new version let's see if it passes the validation...
There was a problem hiding this comment.
You cannot revert the go parts as this is hard enforced by go module now to be the minimum version used for builds. So if any dependency is using this we can either not bump the dep or have to live with that.
What we do control is the toolchain line and for this one we should never use it as the only purpose would be to force a newer version (compared to the go line) to compile with.
There was a problem hiding this comment.
unfortunately that doesn't solve the problem with centos-stream 9/10.
The updated go part is coming from containers/image. I either drop the c/image bump patch or we accept these failures for now.
There was a problem hiding this comment.
Well I brought that up when we did the update to go 1.22, I consider bumping to a z version as minimum fundamentally broken behavior that simply should not happen in a stable language but go decided to allow this so now we are forced to live with that if we want to consume dependencies that just force "random" go updates and yes this means we will be unable to build in slower updating envs. I don't know how much of a problem this will be in practise but I guess if we cannot build on RHEL or even fedora we have a rather big problem.
ref containers/image#2550 (comment)
and we decided in standup that we are ok to bump z versions.
6a41271 to
dba61ff
Compare
dba61ff to
71af6ea
Compare
|
Tests are failing |
c7922f7 to
c17c8f7
Compare
|
Cockpit tests failed for commit c7922f7d0e16189799ab7ef0926832d80a3c0301. @martinpitt, @jelly, @mvollmer please check. |
|
Ephemeral COPR build failed. @containers/packit-build please check. |
Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
894be15 to
7f29233
Compare
fixed now |
|
LGTM |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: giuseppe, Luap99 The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
/lgtm |
The go version there is only go 1.22.5 but we need go 1.22.6 as of containers#24054 It is not clear to me how to best monitor the repos there to see when they get the update. And then there is the fear that podman keeps updating faster then these envs which makes testing there immposible[1] [1] containers/image#2550 (comment) Signed-off-by: Paul Holzinger <pholzing@redhat.com>
Does this PR introduce a user-facing change?