Skip to content

[skip-ci] test containers-common rpm via packit#28601

Closed
Luap99 wants to merge 1 commit intocontainers:mainfrom
Luap99:test-config-rpm
Closed

[skip-ci] test containers-common rpm via packit#28601
Luap99 wants to merge 1 commit intocontainers:mainfrom
Luap99:test-config-rpm

Conversation

@Luap99
Copy link
Copy Markdown
Member

@Luap99 Luap99 commented Apr 28, 2026

@lsm5
Copy link
Copy Markdown
Member

lsm5 commented Apr 28, 2026

Hmm, the fedora-42 testing-farm job doesn't seem to have installed the rpm from the container-libs PR containers-common-0.68.0~dev-1.20260428163917133560.main.718.ge3b0c9aa78.fc42.noarch . Looking ..

@lsm5
Copy link
Copy Markdown
Member

lsm5 commented Apr 28, 2026

I suspect it's because containers-common has the ridiculously high Epoch for podman-next builds that overwrites one from the container-libs PR. Suggest retrying with this change:

diff --git a/contrib/packit-tmt/update-deps.sh b/contrib/packit-tmt/update-deps.sh
index dc2d14dc95..d094cb871c 100644
--- a/contrib/packit-tmt/update-deps.sh
+++ b/contrib/packit-tmt/update-deps.sh
@@ -4,4 +4,4 @@ set -exo pipefail

 # This should work even when podman-next isn't installed. It'll fetch the
 # highest versions available across all repos.
-dnf -y upgrade --allowerasing --disable-repo=testing-farm-tag-repository --exclude=podman*
+dnf -y upgrade --allowerasing --disable-repo=testing-farm-tag-repository --exclude=podman* --exclude=containers-common*

That shouldn't upgrade containers-common after setting up podman-next.

@Luap99
Copy link
Copy Markdown
Member Author

Luap99 commented Apr 28, 2026

AFAIK last time I did this it worked fine, I think it may have not liked the comment in the PR description, i.e. I see no packit eyes which it normally does

let me try again

@Luap99
Copy link
Copy Markdown
Member Author

Luap99 commented Apr 28, 2026

/packit test containers/container-libs#795

@packit-as-a-service
Copy link
Copy Markdown

Cockpit tests failed for commit a191d81. @martinpitt, @jelly, @mvollmer please check.

@Luap99
Copy link
Copy Markdown
Member Author

Luap99 commented Apr 28, 2026

looks like this did the trick https://artifacts.dev.testing-farm.io/4cb39ae3-f4d7-4e8e-8dcd-244a1ac06abc/

containers-common-0.68.0~dev-1.20260428143457469173.pr795.718.g676611e43e.fc45.noarch

@Luap99
Copy link
Copy Markdown
Member Author

Luap99 commented Apr 29, 2026

/packit test containers/container-libs#795

@packit-as-a-service
Copy link
Copy Markdown

[NON-BLOCKING] Packit jobs failed. @containers/packit-build please check. Everyone else, feel free to ignore.

/packit test containers/container-libs#795

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

Luap99 commented Apr 30, 2026

/packit test containers/container-libs#795
/packit test cockpit-project/cockpit-podman#2525

@lsm5
Copy link
Copy Markdown
Member

lsm5 commented Apr 30, 2026

/packit test containers/container-libs#795 /packit test cockpit-project/cockpit-podman#2525

Latest run shows containers-common from podman-next https://artifacts.dev.testing-farm.io/3197963d-fbd1-4d52-8282-fb9e250df719/ . Not sure offhand why that's messing up if an earlier run went fine, maybe try only one slash-packit test in one comment.

@Luap99
Copy link
Copy Markdown
Member Author

Luap99 commented Apr 30, 2026

/packit test containers/container-libs#795 /packit test cockpit-project/cockpit-podman#2525

Latest run shows containers-common from podman-next https://artifacts.dev.testing-farm.io/3197963d-fbd1-4d52-8282-fb9e250df719/ . Not sure offhand why that's messing up if an earlier run went fine, maybe try only one slash-packit test in one comment.

I don't think it works in multiple comments, a force push seems to unset the previous comments and if you add a new comment it just triggers new test runs using only the latest repo. Not sure it supports using two different repos.

But sure I can try

@Luap99
Copy link
Copy Markdown
Member Author

Luap99 commented Apr 30, 2026

/packit test cockpit-project/cockpit-podman#2525

@Luap99
Copy link
Copy Markdown
Member Author

Luap99 commented Apr 30, 2026

/packit test containers/container-libs#795

@packit-as-a-service
Copy link
Copy Markdown

Cockpit tests failed for commit 5f46322. @martinpitt, @jelly, @mvollmer please check.

@lsm5
Copy link
Copy Markdown
Member

lsm5 commented Apr 30, 2026

so the podman system test jobs correctly fetch the build from container-libs PR now.

RE: cockpit jobs, packit upstream said neither of 2 commands in single comment nor 2 separate commands in 2 separate comments will currently work.

There's an open PR to support multiple arguments for a single /packit test ... command, but for now, we'll have to add the cockpit copr either in packit or TMT config to fetch the right builds.

@Luap99
Copy link
Copy Markdown
Member Author

Luap99 commented Apr 30, 2026

@lsm5 Thanks, yeah I do not think this is worth the trouble. My cockpit PR is safe enough without having to go through all this trouble IMO. We now based on the podman tests the rpm works.

@Luap99 Luap99 closed this May 4, 2026
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