Skip to content

[release-1.40] tests: remove dependencies on online apt repositories#6835

Open
nalind wants to merge 1 commit into
podman-container-tools:release-1.40from
nalind:no-apt-1.40
Open

[release-1.40] tests: remove dependencies on online apt repositories#6835
nalind wants to merge 1 commit into
podman-container-tools:release-1.40from
nalind:no-apt-1.40

Conversation

@nalind

@nalind nalind commented May 7, 2026

Copy link
Copy Markdown
Contributor

What type of PR is this?

/kind failing-test

What this PR does / why we need it:

Rework places to avoid installing packages using apt as part of tests. Use ubi10 where we need to be able to call setcap, setfattr, and getfattr from the same image.

How to verify it

Updated integration tests!

Which issue(s) this PR fixes:

None

Special notes for your reviewer:

Manual cherry pick from #6834.

Does this PR introduce a user-facing change?

None

Rework places to avoid installing packages using apt as part of tests.
Use ubi10 where we need to be able to call setcap, setfattr, and
getfattr from the same image.

Signed-off-by: Nalin Dahyabhai <nalin@redhat.com>
@dosubot dosubot Bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label May 7, 2026
@nalind nalind changed the title tests: remove dependencies on online apt repositories [release-1.40] tests: remove dependencies on online apt repositories May 7, 2026

@lsm5 lsm5 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

TF jobs fail on not finding grpcnoop and passwd. Probably not worth blocking over until the move away from Cirrus.

@nalind

nalind commented May 7, 2026

Copy link
Copy Markdown
Contributor Author

TF jobs fail on not finding grpcnoop and passwd. Probably not worth blocking over until the move away from Cirrus.

Neither of those exist on this branch, and moving from Cirrus to something else isn't going to change that.

@nalind

nalind commented May 7, 2026

Copy link
Copy Markdown
Contributor Author

Am I reading https://dashboard.packit.dev/jobs/testing-farm/1535319 right? That it's building packages off of the PR for the Copr Build(s), and then installing packages built from the main branch and running the tests that are in that package using the binaries from that package, using the TMT configuration (tests/tmt directory) from this PR? If so, what is that supposed to be verifying?

@lsm5

lsm5 commented May 7, 2026

Copy link
Copy Markdown
Contributor

ugh, yes, the buildah rpm is fetched from podman-next, which is to blame. Thanks @nalind . And that's because of the super-high Epoch of podman-next packages.

So we can ignore all testing-farm jobs here. Should be fixed when the packit and TMT config is changed.

@nalind

nalind commented May 7, 2026

Copy link
Copy Markdown
Contributor Author

If it's known that they're going to fail, can we remove them from older branches until that's resolved?

@lsm5

lsm5 commented May 14, 2026

Copy link
Copy Markdown
Contributor

If it's known that they're going to fail, can we remove them from older branches until that's resolved?

sure thing. on it..

@github-actions

Copy link
Copy Markdown

A friendly reminder that this PR had no activity for 30 days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS This PR changes 0-9 lines, ignoring generated files. stale-pr

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants