Skip to content

packit: Don't attempt to build on f39#400

Merged
openshift-merge-bot[bot] merged 2 commits intocontainers:mainfrom
cfergeau:packit
Sep 23, 2024
Merged

packit: Don't attempt to build on f39#400
openshift-merge-bot[bot] merged 2 commits intocontainers:mainfrom
cfergeau:packit

Conversation

@cfergeau
Copy link
Copy Markdown
Collaborator

It only has go 1.21 while we switched to go 1.22, so gvisor-tap-vsock
fails to build on f39.
I don't expect significant changes to gvisor-tap-vsock during the
remaining f39 lifetime, at worse podman in f39 can keep using an older
gvproxy release.

Copy link
Copy Markdown
Member

@lsm5 lsm5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Once Fedora 41 is released, fedora-latest-stable will point to fedora-41 and -development will point only to rawhide. So, you might wanna include an explicit fedora-40 to account for F41 release.

It only has go 1.21 while we switched to go 1.22, so gvisor-tap-vsock
fails to build on f39.
I don't expect significant changes to gvisor-tap-vsock during the
remaining f39 lifetime, at worse podman in f39 can keep using an older
gvproxy release.

Signed-off-by: Christophe Fergeau <cfergeau@redhat.com>
@cfergeau
Copy link
Copy Markdown
Collaborator Author

I added f40, but we can revert this commit once f41 is released.

@praveenkumar
Copy link
Copy Markdown
Contributor

I added f40, but we can revert this commit once f41 is released.

But even f41 release, f-39 will be EOL but not f-40 so may be we should keep using it until we again update go version which is not built for f-40 :)

Since commit 8de6b91, builds with go 1.21 are no longer supported.
This commit updates the go versions used in GitHub Actions to 1.22 and
1.23.

Signed-off-by: Christophe Fergeau <cfergeau@redhat.com>
@cfergeau
Copy link
Copy Markdown
Collaborator Author

I added f40, but we can revert this commit once f41 is released.

But even f41 release, f-39 will be EOL but not f-40 so may be we should keep using it until we again update go version which is not built for f-40 :)

If we revert, we get back to fedora-all, which will be f40, f41 and rawhide (and f42 after branching)

@praveenkumar
Copy link
Copy Markdown
Contributor

/lgtm
/approve

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci bot commented Sep 23, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cfergeau, praveenkumar

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-bot openshift-merge-bot bot merged commit 8794a1f into containers:main Sep 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants