Skip to content
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

Bump to Fedora 41 #3241

Merged
merged 3 commits into from
Dec 6, 2024
Merged

Bump to Fedora 41 #3241

merged 3 commits into from
Dec 6, 2024

Conversation

skitt
Copy link
Member

@skitt skitt commented Dec 4, 2024

See individual commits for details.

skitt added 2 commits December 4, 2024 13:26
This requires pulling in the host configuration for dnf.

dnf_install is fixed to fail if dnf fails (otherwise dnf errors don't
stop the build). The ulimit configuration is only applied if it
reduces the current limit (in setups where the ulimit is less than
1048576, dnf_install would attempt to raise it, which isn't allowed,
and now causes the whole script to fail).

Signed-off-by: Stephen Kitt <skitt@redhat.com>
This fixes a few buildx warnings:
* BASE_BRANCH needs a default value
* specifying ${TARGETPLATFORM} is redundant
* entrypoints should be specified as JSON arrays (this doesn't change
  anything in practice for Submariner, since the entrypoints are
  shell scripts which correctly set up signal handling)

Signed-off-by: Stephen Kitt <skitt@redhat.com>
@submariner-bot
Copy link
Contributor

🤖 Created branch: z_pr3241/skitt/fedora-41
🚀 Full E2E won't run until the "ready-to-test" label is applied. I will add it automatically once the PR has 2 approvals, or you can add it manually.

@submariner-bot submariner-bot added the ready-to-test When a PR is ready for full E2E testing label Dec 5, 2024
@tpantelis tpantelis enabled auto-merge (rebase) December 5, 2024 15:31
@tpantelis tpantelis merged commit 84d01fa into submariner-io:devel Dec 6, 2024
39 checks passed
@submariner-bot
Copy link
Contributor

🤖 Closed branches: [z_pr3241/skitt/fedora-41]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-test When a PR is ready for full E2E testing
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants