Skip to content

Update snapshots to 20260816 - #2599

Open
schutzbot wants to merge 2 commits into
osbuild:mainfrom
schutzbot:snapshots-20260816
Open

Update snapshots to 20260816#2599
schutzbot wants to merge 2 commits into
osbuild:mainfrom
schutzbot:snapshots-20260816

Conversation

@schutzbot

Copy link
Copy Markdown
Collaborator

Results of the snapshot jobs:
Job(s) succeeded: 243
Job(s) failed: 3

If these are false, rebuild the enumerate cache manually:
Enumerate cache job succeeded: true
Check snapshot succeeded:

Workflow run: https://github.com/osbuild/rpmrepo/actions/runs/31921654228


update_schutzfile.py output

@schutzbot
schutzbot requested a review from a team as a code owner August 16, 2026 06:36
@achilleas-k

Copy link
Copy Markdown
Member

The problem with the container resolver is that the podman run call is failing in github actions with

running registry.access.redhat.com/ubi9-micro:latest container failed: exit status 126
stderr:
[conmon:e]: Include journald in compilation path to log to systemd journal
Error: conmon failed: exit status 1

I can't reproduce the failure locally, neither on a Fedora host nor in an Ubuntu 24.04 container, which I used to try to replicate the CI environment.

@achilleas-k

Copy link
Copy Markdown
Member

Merged the container resolver tests into the unit tests that run in Fedora containers. They're no longer separate jobs so they wont be reported separately in the statuses.

@achilleas-k

Copy link
Copy Markdown
Member

The test explicitly refuses to run in a container and fails when forced. Let's see if I can change that.

@achilleas-k

Copy link
Copy Markdown
Member

I think I'm just going to remove these tests. They're covered by integration tests in GitLab. It's not worth the effort to get them fixed to run in GHA. It'd be a different story if they were quick unit tests that can easily be run locally, which would catch problems before people push and open PRs, but they're not.

Some change in the podman package in ubuntu, where this test runs in
GitHub Actions, is making this test fail because the error message is
logged differently (or not at all).  Running this test on Fedora in GHA
causes other issues, caused by it being run inside a container.

The test is not very important.  It's an error message matching test for
a container mount failure.  Checking that it fails as expected (while
working in the happy cases) is enough.

@croissanne croissanne left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

thanks!

@achilleas-k

Copy link
Copy Markdown
Member

Didn't drop the test. Instead changed it to be a simple error check without message matching.

@croissanne
croissanne enabled auto-merge August 17, 2026 13:04
@achilleas-k

Copy link
Copy Markdown
Member

The other test failed now (╯°□°)╯︵ ┻━┻

@achilleas-k

Copy link
Copy Markdown
Member

I'll check on it in a bit

@achilleas-k

Copy link
Copy Markdown
Member

The other test failed now (╯°□°)╯︵ ┻━┻

So the failure in this case was a timeout, which I foolishly assumed was a problem with our test, possibly related to the previous failure that I worked around. Given how GitHub is currently having a day again, I'm going to assume it's just regular old GH-borkage until proven otherwise.

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