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

Add wireguard-tools and passt packages #235

Merged
merged 1 commit into from
Nov 30, 2022
Merged

Conversation

cevich
Copy link
Member

@cevich cevich commented Nov 14, 2022

Fixes: #234
Fixes: #232

@cevich cevich requested a review from Luap99 November 14, 2022 17:59
Copy link
Member

@Luap99 Luap99 left a comment

Choose a reason for hiding this comment

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

LGTM

@cevich
Copy link
Member Author

cevich commented Nov 14, 2022

There ya go @Luap99, just blast that c5518085466095616 image-suffix value into whatever repo's CI needs the new pacakges (remember to update the Distro names to F37 & F36 too).

@Luap99
Copy link
Member

Luap99 commented Nov 16, 2022

The passt version in fedora repo is to old, it does not have the fixes required to pass podman CI.
It needs 2022_11_04 or newer. @cevich can we use the copr https://copr.fedorainfracloud.org/coprs/sbrivio/passt/ instead?

@cevich
Copy link
Member Author

cevich commented Nov 21, 2022

can we use the copr https://copr.fedorainfracloud.org/coprs/sbrivio/passt/ instead?

Sure, no problem.

@Luap99
Copy link
Member

Luap99 commented Nov 21, 2022

@cevich updates-testing from the main repos should also work

@Luap99
Copy link
Member

Luap99 commented Nov 21, 2022

CI is red. Given https://bodhi.fedoraproject.org/updates/FEDORA-2022-754a5818f2, we could also wait a couple of days until this lands in stable.

@cevich
Copy link
Member Author

cevich commented Nov 21, 2022

Let's try just enabling it for the latest podman CI VMs...

@cevich
Copy link
Member Author

cevich commented Nov 21, 2022

Context: CI was red simply because the copr command doesn't exist by default in the F36 image. The F37 image is already setup to use updates-testing, however using copr for new feature testing (assuming this is) might be more flexible. Let's see if just having this in F37 can work...

@Luap99
Copy link
Member

Luap99 commented Nov 21, 2022

SGTM

@cevich
Copy link
Member Author

cevich commented Nov 22, 2022

Once again for the third time.

@containers containers deleted a comment from github-actions bot Nov 22, 2022
@cevich
Copy link
Member Author

cevich commented Nov 22, 2022

E-mailed cirrus-support. There's a bug (likely on the github-side) causing a failure-to-clone.

@cevich
Copy link
Member Author

cevich commented Nov 29, 2022

Looks like the clouds are having trouble. Retrying the failed builds.

@containers containers deleted a comment from github-actions bot Nov 29, 2022
@containers containers deleted a comment from github-actions bot Nov 29, 2022
@github-actions
Copy link

Cirrus CI build successful. Found built image names and IDs:

Stage Image Name IMAGE_SUFFIX
base fedora b4683285087911936
base fedora-aws b4683285087911936
base fedora-aws-arm64 b4683285087911936
base image-builder b4683285087911936
base prior-fedora b4683285087911936
base ubuntu b4683285087911936
cache build-push c4683285087911936
cache fedora c4683285087911936
cache fedora-aws c4683285087911936
cache fedora-netavark c4683285087911936
cache fedora-netavark-aws-arm64 c4683285087911936
cache fedora-podman-aws-arm64 c4683285087911936
cache fedora-podman-py c4683285087911936
cache prior-fedora c4683285087911936
cache ubuntu c4683285087911936
cache win-server-wsl c4683285087911936

@cevich
Copy link
Member Author

cevich commented Nov 29, 2022

@Luap99 there we go, got the build to pass. Please use the image suffix referenced above.

@Luap99
Copy link
Member

Luap99 commented Nov 29, 2022

@Luap99 there we go, got the build to pass. Please use the image suffix referenced above.

Yeah thanks, already pushed in containers/podman#16525

@cevich
Copy link
Member Author

cevich commented Nov 29, 2022

K, good deal. I'm going to merge this then. But if any more image changes/updates are needed I've got another PR coming to this repo. shortly (to use the release-F37 cloud-images instead of devel).

@cevich cevich merged commit 2467d9c into containers:main Nov 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

add wireguard-tools package for netavark and Podman Add passt (pasta) package for podman
2 participants