Skip to content

rpm/buildah.spec tests: require xz and /usr/bin/selinuxenabled#6849

Merged
lsm5 merged 2 commits into
containers:mainfrom
nalind:rpm-tests-deps
May 14, 2026
Merged

rpm/buildah.spec tests: require xz and /usr/bin/selinuxenabled#6849
lsm5 merged 2 commits into
containers:mainfrom
nalind:rpm-tests-deps

Conversation

@nalind
Copy link
Copy Markdown
Member

@nalind nalind commented May 13, 2026

What type of PR is this?

/kind failing-test

What this PR does / why we need it:

Tests use selinuxenabled to check if SELinux is enabled and relabeling is required when using volumes.
Some tarballs downloaded by tests are compressed using xz, as some are created using bzip2, which is called by tar, so make sure it's there.

How to verify it

Testing farm Raw Hide tests should pass.

Which issue(s) this PR fixes:

Taking a guess at the cause of the errors seen at #6800 (comment).

Special notes for your reviewer:

Does this PR introduce a user-facing change?

None

@dosubot dosubot Bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label May 13, 2026
nalind added 2 commits May 13, 2026 13:37
Tests use selinuxenabled to check if SELinux is enabled and relabeling
is required when using volumes.
Some tarballs downloaded by tests are compressed using xz, as some are
created using bzip2, which is called by tar, so make sure it's there.

Signed-off-by: Nalin Dahyabhai <nalin@redhat.com>
Signed-off-by: Nalin Dahyabhai <nalin@redhat.com>
Copy link
Copy Markdown
Member

@Luap99 Luap99 left a comment

Choose a reason for hiding this comment

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

LGTM
cc @lsm5

Copy link
Copy Markdown
Member

@lsm5 lsm5 left a comment

Choose a reason for hiding this comment

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

LGTM

Comment thread rpm/buildah.spec
@lsm5 lsm5 merged commit f3dc2ef into containers:main May 14, 2026
40 checks passed
@nalind nalind deleted the rpm-tests-deps branch May 14, 2026 13:23
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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants