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

New VMs: we need netavark 1.10.3 #327

Merged
merged 1 commit into from
Feb 15, 2024
Merged

Conversation

edsantiago
Copy link
Member

Signed-off-by: Ed Santiago [email protected]

Copy link

github-actions bot commented Feb 5, 2024

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

Stage Image Name IMAGE_SUFFIX
base debian do-not-use
base fedora do-not-use
base fedora-aws do-not-use
base fedora-aws-arm64 do-not-use
base image-builder do-not-use
base prior-fedora do-not-use
cache build-push c20240205t124622z-f39f38d13
cache debian c20240205t124622z-f39f38d13
cache fedora c20240205t124622z-f39f38d13
cache fedora-aws c20240205t124622z-f39f38d13
cache fedora-netavark c20240205t124622z-f39f38d13
cache fedora-netavark-aws-arm64 c20240205t124622z-f39f38d13
cache fedora-podman-aws-arm64 c20240205t124622z-f39f38d13
cache fedora-podman-py c20240205t124622z-f39f38d13
cache prior-fedora c20240205t124622z-f39f38d13
cache rawhide c20240205t124622z-f39f38d13
cache win-server-wsl c20240205t124622z-f39f38d13

@edsantiago
Copy link
Member Author

Sigh. Got .2 in rawhide, but not f39. Will try a respin later in the week.

debian prior-fedora fedora fedora-aws rawhide
base 13 38-1.6 39-1.5 ? 40-0
aardvark-dns 1.4.0-5 1.9.0-1 1.10.0-1 1.10.0-1 1.10.0-1
1.9.0-1 ⇑
buildah 1.33.3+ds1-2 1.34.0-1 1.34.0-1 1.34.0-1 1.34.0-3
1.33.2-1 ⇑
conmon 2.1.6+ds1-1 2:2.1.8-2 2:2.1.8-2 2:2.1.8-2 2:2.1.8-4
container-selinux ? 2:2.228.1-1 2:2.229.0-1 2:2.229.0-1 2:2.229.0-2
2:2.227.0-1 ⇑
containers-common ? 4:1-89 4:1-99 4:1-95 5:0.57.3-2
4:1-95 ⇑ 5:0.57.1-7 ⇑
criu 3.17.1-3 3.18-1 3.19-2 3.19-2 3.19-4
crun 1.14-1 1.12-1 1.14-1 1.14-1 1.14-1
golang 2:1.21~2 1.20.13-1 1.21.6-1 1.21.6-1 1.22~rc2-1
netavark 1.4.0-4 1.9.0-1 1.10.1-5 1.10.1-5 1.10.2-1
1.10.1-5 ⇑
passt 2023-12-30 2023-12-30 ? 2023-12-30 2023-12-30
podman 4.9.0+ds1-2 5:4.8.3-1 5:4.9.1-1 5:4.9.0-1 5:4.9.0-1
5:4.9.0-1 ⇑
runc 1.1.12+ds1-1 2:1.1.8-1 2:1.1.12-1 2:1.1.8-1 2:1.1.12-1
1.1.10+ds1-1 ⇑ 2:1.1.8-1 ⇑ 2:1.1.9-2 ⇑
skopeo 1.13.3+ds1-2 1:1.14.1-1 1:1.14.2-1 1:1.14.0-1 1:1.14.2-1
1:1.14.1-1 ⇑ 1:1.14.1-2 ⇑
slirp4netns 1.2.1-1 1.2.2-1 1.2.2-1 1.2.2-1 1.2.2-2
systemd-container 255.3-2 253.15-2 254.8-2 254.8-2 255.3-1
tar ? ? ? ? 2:1.35-3

@edsantiago
Copy link
Member Author

Ah - there is no .2 yet for f39. @lsm5, when convenient, could you tell packit to build netavark-1.10.2 for f39 please? We need this in CI; .1 is broken for some cases.

@lsm5
Copy link
Member

lsm5 commented Feb 6, 2024

Ah - there is no .2 yet for f39. @lsm5, when convenient, could you tell packit to build netavark-1.10.2 for f39 please? We need this in CI; .1 is broken for some cases.

bodhi has been submitted. If we need something right away, we can grab the koji build from https://koji.fedoraproject.org/koji/buildinfo?buildID=2396597 in the build.

@edsantiago edsantiago changed the title New VMs: we need netavark 1.10.2-1.fc40 New VMs: we need netavark 1.10.2 Feb 6, 2024
Copy link

github-actions bot commented Feb 7, 2024

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

Stage Image Name IMAGE_SUFFIX
base debian do-not-use
base fedora do-not-use
base fedora-aws do-not-use
base fedora-aws-arm64 do-not-use
base image-builder do-not-use
base prior-fedora do-not-use
cache build-push c20240207t013159z-f39f38d13
cache debian c20240207t013159z-f39f38d13
cache fedora c20240207t013159z-f39f38d13
cache fedora-aws c20240207t013159z-f39f38d13
cache fedora-netavark c20240207t013159z-f39f38d13
cache fedora-netavark-aws-arm64 c20240207t013159z-f39f38d13
cache fedora-podman-aws-arm64 c20240207t013159z-f39f38d13
cache fedora-podman-py c20240207t013159z-f39f38d13
cache prior-fedora c20240207t013159z-f39f38d13
cache rawhide c20240207t013159z-f39f38d13
cache win-server-wsl c20240207t013159z-f39f38d13

@edsantiago
Copy link
Member Author

Yay, finally have new netavark. Testing in 17831.

debian prior-fedora fedora fedora-aws rawhide
base 13 38-1.6 39-1.5 ? 40-0
aardvark-dns 1.4.0-5 1.9.0-1 1.10.0-1 1.10.0-1 1.10.0-1
buildah 1.33.3+ds1-2 1.34.0-1 1.34.0-1 1.34.0-1 1.34.0-3
conmon 2.1.6+ds1-1 2:2.1.8-2 2:2.1.8-2 2:2.1.8-2 2:2.1.8-4
container-selinux ? 2:2.228.1-1 2:2.229.0-1 2:2.229.0-1 2:2.229.0-2
containers-common ? 4:1-89 4:1-99 4:1-95 5:0.57.3-2
criu 3.17.1-3 3.18-1 3.19-2 3.19-2 3.19-4
crun 1.14-1 1.12-1 1.14-1 1.14-1 1.14-1
golang 2:1.21~2 1.20.13-1 1.21.6-1 1.21.6-1 1.22~rc2-1
netavark 1.4.0-4 1.10.2-1 1.10.2-1 1.10.2-1 1.10.2-1
1.9.0-1 ⇑ 1.10.1-5 ⇑ 1.10.1-5 ⇑
passt 2023-12-30 2023-12-30 ? 2023-12-30 2023-12-30
podman 4.9.0+ds1-2 5:4.8.3-1 5:4.9.2-1 5:4.9.0-1 5:4.9.0-1
5:4.9.1-1 ⇑
runc 1.1.12+ds1-1 2:1.1.8-1 2:1.1.12-1 2:1.1.12-1 2:1.1.12-1
2:1.1.8-1 ⇑
skopeo 1.13.3+ds1-2 1:1.14.1-1 1:1.14.2-1 1:1.14.0-1 1:1.14.2-1
slirp4netns 1.2.1-1 1.2.2-1 1.2.2-1 1.2.2-1 1.2.2-2
systemd-container 255.3-2 253.15-2 254.8-2 254.8-2 255.3-1
tar ? ? ? ? 2:1.35-3

@edsantiago
Copy link
Member Author

@containers/podman-maintainers @containers/netavark-maintainers I'm really confused here. Every single fedora e2e test failed, and every single one of them in netavark tests: https://cirrus-ci.com/build/4795190137847808

(Debian failed too, but that's because of the new runc, and I'm not even going to bother looking at that today).

This is a diff against the packages in the current-podman-CI VMs. I have to conclude that netavark-1.10.2 is broken:

debian prior-fedora fedora fedora-aws rawhide
base 13 38-1.6 39-1.5 ? 40-0
kernel 6.6.13-1 6.7.3-100 6.7.3-200 6.7.3-200 6.8.0-0.rc0.20240112git70d201a40823.5
aardvark-dns 1.4.0-5 1.9.0-1 1.10.0-1 1.10.0-1 1.10.0-1
1.9.0-1 ⇑
netavark 1.4.0-4 1.10.2-1 1.10.2-1 1.10.2-1 1.10.2-1
1.9.0-1 ⇑ 1.10.1-5 ⇑ 1.10.1-5 ⇑ 1.10.1-5 ⇑
buildah 1.33.3+ds1-2 1.34.0-1 1.34.0-1 1.34.0-1 1.34.0-3
1.33.2-1 ⇑
conmon 2.1.6+ds1-1 2.1.8-2 2.1.8-2 2.1.8-2 2.1.8-4
container-selinux ? 2.228.1-1 2.229.0-1 2.229.0-1 2.229.0-2
2.227.0-1 ⇑
containers-common ? 1-89 1-99 1-95 0.57.3-2
1-95 ⇑ 0.57.1-7 ⇑
criu 3.17.1-3 3.18-1 3.19-2 3.19-2 3.19-4
crun 1.14-1 1.12-1 1.14-1 1.14-1 1.14-1
golang 2:1.21~2 1.20.13-1 1.21.6-1 1.21.6-1 1.22~rc2-1
passt 2023-12-30 2023-12-30 2023-12-30 2023-12-30 2023-12-30
podman 4.9.0+ds1-2 4.8.3-1 4.9.2-1 4.9.0-1 4.9.0-1
4.9.0-1 ⇑
runc 1.1.12+ds1-1 1.1.8-1 1.1.12-1 1.1.12-1 1.1.12-1
1.1.10+ds1-1 ⇑ 1.1.8-1 ⇑ 1.1.8-1 ⇑ 1.1.9-2 ⇑
skopeo 1.13.3+ds1-2 1.14.1-1 1.14.2-1 1.14.0-1 1.14.2-1
1.14.1-1 ⇑ 1.14.1-2 ⇑
slirp4netns 1.2.1-1 1.2.2-1 1.2.2-1 1.2.2-1 1.2.2-2
systemd-container 255.3-2 253.15-2 254.8-2 254.8-2 255.3-1
tar 1.34+dfsg-1.2 1.34-8 1.35-2 1.35-2 1.35-3

@Luap99
Copy link
Member

Luap99 commented Feb 7, 2024

I have to conclude that netavark-1.10.2 is broken

That seems possible although I would not understand why it broke as the only fix was the network update issue and I see failing tests do not not call update at all. All failures seem related to dns so maybe the bug is on the aardvark-dns side. I will see if I can reproduce locally and given they failed on all tests that shouldn't be to hard.

@edsantiago
Copy link
Member Author

But aardvark-dns didn't change! (Okay, AWS, but nowhere else)

@Luap99
Copy link
Member

Luap99 commented Feb 7, 2024

yes it is netavark, the patch containers/netavark#912 is buggy

@edsantiago
Copy link
Member Author

Thank you for looking into it. I'm really sorry to have dumped it on you.

@edsantiago edsantiago changed the title New VMs: we need netavark 1.10.2 New VMs: we need netavark 1.10.3 Feb 8, 2024
Also, add "rpm -qa" (fedora) and "dpkg -l" (debian) so Ed's
package-version script can get better data. It would be nice
if we could save those to an artifact file, but we can't.

Signed-off-by: Ed Santiago <[email protected]>
Copy link

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

Stage Image Name IMAGE_SUFFIX
base debian do-not-use
base fedora do-not-use
base fedora-aws do-not-use
base fedora-aws-arm64 do-not-use
base image-builder do-not-use
base prior-fedora do-not-use
cache build-push c20240212t122113z-f39f38d13
cache debian c20240212t122113z-f39f38d13
cache fedora c20240212t122113z-f39f38d13
cache fedora-aws c20240212t122113z-f39f38d13
cache fedora-netavark c20240212t122113z-f39f38d13
cache fedora-netavark-aws-arm64 c20240212t122113z-f39f38d13
cache fedora-podman-aws-arm64 c20240212t122113z-f39f38d13
cache fedora-podman-py c20240212t122113z-f39f38d13
cache prior-fedora c20240212t122113z-f39f38d13
cache rawhide c20240212t122113z-f39f38d13
cache win-server-wsl c20240212t122113z-f39f38d13

@edsantiago
Copy link
Member Author

debian prior-fedora fedora fedora-aws rawhide
base 13 38-1.6 39-1.5 ? 40-0
kernel 6.6.13-1 6.7.4-100 6.7.4-200 6.7.4-200 6.8.0-0.rc3.20240209git1f719a2f3fa6.31
aardvark-dns 1.4.0-5 1.10.0-1 1.10.0-1 1.10.0-1 1.10.0-1
1.9.0-1 ⇑ 1.9.0-1 ⇑
netavark 1.4.0-4 1.10.3-1 1.10.3-1 1.10.3-1 1.10.3-1
1.9.0-1 ⇑ 1.10.1-5 ⇑ 1.10.1-5 ⇑ 1.10.1-5 ⇑
buildah 1.33.5+ds1-4 1.34.0-1 1.34.0-1 1.34.0-1 1.34.0-3
1.33.3+ds1-2 ⇑ 1.33.2-1 ⇑
conmon 2.1.6+ds1-1 2.1.8-2 2.1.8-2 2.1.8-2 2.1.8-4
container-selinux ? 2.228.1-1 2.229.0-1 2.229.0-1 2.229.0-2
2.227.0-1 ⇑
containers-common ? 1-89 1-99 1-99 0.57.3-3
1-95 ⇑ 1-95 ⇑ 0.57.1-7 ⇑
criu 3.17.1-3 3.18-1 3.19-2 3.19-2 3.19-4
crun 1.14-1 1.14-1 1.14.1-1 1.14-1 1.14.1-1
1.12-1 ⇑ 1.14-1 ⇑ 1.14-1 ⇑
golang 2:1.21~2 1.20.13-1 1.21.7-1 1.21.6-1 1.22.0-1
1.21.6-1 ⇑ 1.22~rc2-1 ⇑
passt 2023-12-30 2023-12-30 2023-12-30 2023-12-30 2023-12-30
podman 4.9.2+ds1-2 4.8.3-1 4.9.2-1 4.9.0-1 5.0.0~rc1-2
4.9.0+ds1-2 ⇑ 4.9.0-1 ⇑ 4.9.0-1 ⇑
runc 1.1.12+ds1-1 1.1.12-1 1.1.12-1 1.1.12-1 1.1.12-1
1.1.10+ds1-1 ⇑ 1.1.8-1 ⇑ 1.1.8-1 ⇑ 1.1.8-1 ⇑ 1.1.9-2 ⇑
skopeo 1.13.3+ds1-2 1.14.1-1 1.14.2-1 1.14.0-1 1.14.2-1
1.14.1-1 ⇑ 1.14.1-2 ⇑
slirp4netns 1.2.1-1 1.2.2-1 1.2.2-1 1.2.2-1 1.2.2-2
systemd-container 255.3-2 253.15-2 254.9-1 254.9-1 255.3-1
254.8-2 ⇑ 254.8-2 ⇑
tar 1.34+dfsg-1.2 1.34-8 1.35-2 1.35-2 1.35-3

Copy link
Member Author

@edsantiago edsantiago left a comment

Choose a reason for hiding this comment

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

@cevich PTAL. VMs are already in use for podman: containers/podman#21618 so this is an FYI review, just so you're aware of the stuff that's going on here.

I am deliberately not tagging this build because there's a nasty rawhide bug that I really don't want to have to track in more repos.

lib.sh Show resolved Hide resolved
lib.sh Show resolved Hide resolved
cache_images/fedora_packaging.sh Show resolved Hide resolved
Copy link
Member

@cevich cevich left a comment

Choose a reason for hiding this comment

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

LGTM. Ed, there's a pair of minor things not worth rebuilding/testing for. Would you consider doing them in a [CI:DOCS] followup PR?

@edsantiago
Copy link
Member Author

Since these VMs are already in use in podman, I would hope that this can be merged.

The issues you raised (adding comments re: netavark bodhi; using some variable instead of getting it myself) are timebombed and will be obsolete within 24-48 hours, so I choose not to submit a new CI:DOCS PR to address them.

@cevich
Copy link
Member

cevich commented Feb 15, 2024

will be obsolete within 24-48 hours

Oh sorry I didn't realize they were that quick. Np then.

@cevich cevich merged commit 5af77ad into containers:main Feb 15, 2024
39 checks passed
@edsantiago edsantiago deleted the new_netavark branch February 15, 2024 18:07
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.

4 participants