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 bpftrace for CI debugging #371

Merged
merged 2 commits into from
Aug 6, 2024
Merged

add bpftrace for CI debugging #371

merged 2 commits into from
Aug 6, 2024

Conversation

Luap99
Copy link
Member

@Luap99 Luap99 commented Aug 5, 2024

I like to run a bpftrace based program in CI to collect better logs for specific processes not observed in the normal testing such as the podman container cleanup command.

Given you need to have full privs to run ebpf and the package pulls in an entire toolchain which is almost 500MB in install size we do not add it the the container images to not bloat them without reason.

containers/podman#23487

I like to run a bpftrace based program in CI to collect better logs for
specific processes not observed in the normal testing such as the podman
container cleanup command.

Given you need to have full privs to run ebpf and the package pulls in
an entire toolchain which is almost 500MB in install size we do not add
it the the container images to not bloat them without reason.

containers/podman#23487

Signed-off-by: Paul Holzinger <pholzing@redhat.com>
@edsantiago
Copy link
Member

LGTM but you need to make IMG_SFX

@Luap99
Copy link
Member Author

Luap99 commented Aug 5, 2024

Right which greets me with

base_images/debian_base-setup.sh:52:timebomb 20240801 "prevent us from getting broken tar-1.35+dfsg-3"
cache_images/fedora_packaging.sh:220:    timebomb 20240710 "Temporary workaround for signed rpms (ima) in rawhide"

and I didn't really wanted to look at this right now. I guess I can just bump the time?

@edsantiago
Copy link
Member

Yes, just bump it.

Note that there WILL be problems. It's been a month since my last make here, and it will probably take 3-6 iterations before this is ready. I will try my best to help.

@Luap99
Copy link
Member Author

Luap99 commented Aug 5, 2024

Note that there WILL be problems

Yes I know that part... but I guess someone has to suffer through that no what what. Might as well be me.

Bump timebomb to Sep 1st, the podman issue is still not fixed and I
haven't looked at the debian bug but I assume it is also still not
fixed.

Signed-off-by: Paul Holzinger <pholzing@redhat.com>
Copy link

github-actions bot commented Aug 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 c20240805t173227z-f40f39d13
cache debian c20240805t173227z-f40f39d13
cache fedora c20240805t173227z-f40f39d13
cache fedora-aws c20240805t173227z-f40f39d13
cache fedora-netavark c20240805t173227z-f40f39d13
cache fedora-netavark-aws-arm64 c20240805t173227z-f40f39d13
cache fedora-podman-aws-arm64 c20240805t173227z-f40f39d13
cache fedora-podman-py c20240805t173227z-f40f39d13
cache prior-fedora c20240805t173227z-f40f39d13
cache rawhide c20240805t173227z-f40f39d13
cache win-server-wsl c20240805t173227z-f40f39d13

@edsantiago
Copy link
Member

debian prior-fedora fedora fedora-aws rawhide
base 13.5 39-1.5 Generic ? 41-0
13.3 ⇑
kernel 6.9.12-1 6.9.12-100 6.9.12-200 6.9.12-200 6.8.5-301
6.9.8-1 ⇑ 6.9.7-100 ⇑ 6.9.8-200 ⇑ 6.9.7-200 ⇑
grub2-common 2.12-5 2.06-121 2.06-123 2.06-123 2.06-124
2.12-2 ⇑
aardvark-dns 1.6.0-3 1.11.0-1 1.12.1-1 1.11.0-3 1.12.1-1
1.4.0-6 ⇑ 1.11.0-1 ⇑ 1.11.0-1 ⇑ 1.11.0-1 ⇑
netavark 1.6.0-2.1 1.11.0-1 1.12.1-1 1.11.0-3 1.11.0-3
1.4.0-4.1 ⇑ 1.11.0-1 ⇑ 1.11.0-1 ⇑ 1.11.0-1 ⇑
buildah 1.35.3+ds1-3 1.36.0-1 1.37.0-1 1.36.0-1 1.37.0-1
1.33.7+ds1-1 ⇑ 1.36.0-1 ⇑ 1.36.0-1 ⇑
conmon 2.1.10+ds1-1+b1 2.1.12-1 2.1.12-1 2.1.10-1 2.1.12-2
2.1.10-1 ⇑
container-selinux ? 2.232.1-1 2.232.1-1 2.232.1-1 2.232.1-2
2.232.1-1 ⇑
containers-common ? 1-99 0.60.0-1 0.60.0-1 0.60.0-1
0.59.1-2 ⇑ 0.59.1-2 ⇑ 0.59.1-1 ⇑
criu 3.17.1-3 3.19-2 3.19-6 3.19-4 3.19-7
3.19-4 ⇑ 3.19-5 ⇑
crun 1.15-1 1.15-1 1.15-1 1.15-1 1.15-2
1.15-1 ⇑
docker-ce 5:27.1.1-1~debian.12~bookworm ? ? ? ?
5:27.0.3-1~debian.12~bookworm ⇑
golang 2:1.22~3 1.21.12-1 1.22.5-1 1.22.5-1 1.23~rc2-1
1.21.11-1 ⇑ 1.22.4-1 ⇑ 1.22.4-1 ⇑ 1.23~rc1-2 ⇑
nmap-ncat 7.94+git20230807.3be01efb1+dfsg-4 7.92-2 7.92-2 7.92-2 7.92-3
7.95-1 ⇑ 7.95-1 ⇑ 7.95-1 ⇑ 7.95-1 ⇑
passt 2024-07-26 2024-06-24 2024-07-26 2024-06-24 2024-07-26
2024-06-24 ⇑ 2024-06-07 ⇑ 2024-06-24 ⇑ 2024-06-24 ⇑
podman 5.0.3+ds1-5 4.9.4-1 5.2.0-1 5.1.2-1 5.2.0~rc2-1
4.9.5+ds1-1 ⇑ 5.1.1-1 ⇑ 5.1.1-1 ⇑ 5.0.3-1 ⇑
runc 1.1.12+ds1-2 1.1.12-1 1.1.12-3 1.1.12-3 1.1.12-4
1.1.12-3 ⇑
skopeo 1.13.3+ds1-2+b1 1.15.2-1 1.16.0-1 1.15.2-1 1.16.0-1
1.15.1-1 ⇑ 1.15.1-1 ⇑ 1.15.1-1 ⇑ 1.15.1-1 ⇑
slirp4netns 1.2.1-1+b1 1.2.2-1 1.2.2-2 1.2.2-2 1.2.2-3
1.2.2-2 ⇑
systemd 256.4-2 254.16-1 255.10-1 255.10-1 256.4-1
256.2-1 ⇑ 254.14-1 ⇑ 255.8-1 ⇑ 255.8-1 ⇑ 256.1-8 ⇑
tar 1.34+dfsg-1.2+deb12u1 1.35-2 1.35-3 1.35-3 1.35-4
1.35-3 ⇑

edsantiago added a commit to edsantiago/libpod that referenced this pull request Aug 5, 2024
Source: containers/automation_images#371

Signed-off-by: Ed Santiago <santiago@redhat.com>
Luap99 pushed a commit to Luap99/libpod that referenced this pull request Aug 6, 2024
Source: containers/automation_images#371

Signed-off-by: Ed Santiago <santiago@redhat.com>
Signed-off-by: Paul Holzinger <pholzing@redhat.com>
@edsantiago
Copy link
Member

@Luap99 care to repush, to pick up new pasta? (containers/podman#23478 (comment))

@Luap99
Copy link
Member Author

Luap99 commented Aug 6, 2024

Shouldn't we merge this first as the images are now used in podman?

@Luap99
Copy link
Member Author

Luap99 commented Aug 6, 2024

Also do we actually get the packages from the testing repos here? I would assume we need to wait for it to hit stable, no?

@edsantiago edsantiago merged commit 83c9b16 into containers:main Aug 6, 2024
39 checks passed
@edsantiago
Copy link
Member

Eek I missed that. Okay, merged. @baude your turn, please rebase and rerun make IMG_SFX and let's see how that goes.

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.

2 participants