Skip to content

Conversation

@MoralCode
Copy link
Contributor

Description
This PR adds a copy of the end to end CI job that uses podman instead of docker to ensure that any changes are compatible with both

Signed commits

  • Yes, I signed my commits.

@MoralCode MoralCode marked this pull request as ready for review July 7, 2025 19:27
@MoralCode
Copy link
Contributor Author

ok so the podman job is failing for expected reasons (i.e. #3195)

Signed-off-by: Adrian Edwards <adredwar@redhat.com>
sgoggins
sgoggins previously approved these changes Jul 7, 2025
Copy link
Member

@sgoggins sgoggins left a comment

Choose a reason for hiding this comment

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

LGTM, and I don't really have a way to test this without merging it, so here we go! :D

Update: Ah. I can see the PR has the podman test on it and its running right now. Will wait of course.

@MoralCode
Copy link
Contributor Author

@sgoggins note I'm expecting this is likely to fail due to #3212 as well - I don't currently have a good workaround for that lifecycle hook though (at least one that won't break the openshift usecase based on my knowledge of it).

@sgoggins
Copy link
Member

sgoggins commented Jul 7, 2025

@MoralCode : Waiting to see what happens in the test, but looking at that issue I wonder if it is simply an issue of those two files not existing in the container yet, and maybe it could be solved with touch ? This could be all wrong, but I have seen failures like this before solved by simply making the file exist.

@JohnStrunk
Copy link
Member

buildah is using too much disk space:

Error: committing container for step {Env:[PATH=/augur/.venv/bin:/usr/bin/:/usr/local/bin:/usr/lib:/usr/local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin LANG=C.UTF-8 GPG_KEY=A035C8C19219BA821ECEA86B64E628F8D684696D PYTHON_VERSION=3.11.13 PYTHON_SHA256=8fb5f9fbc7609fa822cb31549884575db7fd9657cbffb89510b5d7975963a83a DEBIAN_FRONTEND=noninteractive UV_COMPILE_BYTECODE=1 UV_LINK_MODE=copy UV_LOCKED=1 SCC_DIR=/scc SCORECARD_DIR=/scorecard] Command:cmd Args:[/init.sh] Flags:[] Attrs:map[json:true] Message:CMD /init.sh Original:CMD ["/init.sh"]}: copying layers and metadata for container "729ca85e50830fe3342a9a986119b6d26c3d62bbefa92c478ca33a070fb40cce": writing blob: adding layer with blob "sha256:eafd5cd469df775bd8d1792a90d4b7aaae47dbdb93bd80a8adecc5e506b65b69": processing tar file(write /scorecard/scorecard: no space left on device): exit status 1
Error: Error: buildah exited with code 1

Signed-off-by: Adrian Edwards <adredwar@redhat.com>
@MoralCode MoralCode marked this pull request as draft July 7, 2025 20:59
Signed-off-by: Adrian Edwards <adredwar@redhat.com>
@MoralCode MoralCode marked this pull request as ready for review July 8, 2025 19:17
@MoralCode MoralCode requested a review from sgoggins July 8, 2025 19:17
Copy link
Member

@sgoggins sgoggins left a comment

Choose a reason for hiding this comment

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

All tests have passed! LGTM!

@sgoggins sgoggins requested a review from Ulincsys July 8, 2025 23:16
@sgoggins sgoggins added deployed version Live problems with deployed versions containers Related to augur in containers, container images, or the compose file, either in podman or in docker devops Development Operations labels Jul 8, 2025
Copy link
Contributor

@Ulincsys Ulincsys left a comment

Choose a reason for hiding this comment

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

LGTM

@sgoggins sgoggins merged commit cf66b77 into chaoss:main Jul 8, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

containers Related to augur in containers, container images, or the compose file, either in podman or in docker deployed version Live problems with deployed versions devops Development Operations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants