Skip to content

tests/integration: fix seccomp tests on big-endian architectures#5307

Merged
kolyshkin merged 1 commit into
opencontainers:mainfrom
ricardobranco777:seccomp
Jun 17, 2026
Merged

tests/integration: fix seccomp tests on big-endian architectures#5307
kolyshkin merged 1 commit into
opencontainers:mainfrom
ricardobranco777:seccomp

Conversation

@ricardobranco777

@ricardobranco777 ricardobranco777 commented May 28, 2026

Copy link
Copy Markdown
Contributor

The hardcoded architecture list was little-endian only, causing seccomp_arch_add() to fail with -EDOM on s390x.

Drop it. It's optional and libseccomp automatically adds the native architecture when the filter is created.

Fixes: #4835

@ricardobranco777

ricardobranco777 commented May 28, 2026

Copy link
Copy Markdown
Contributor Author

@cyphar @kolyshkin cypr?

@danishprakash danishprakash 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.

LGTM

Comment thread tests/integration/seccomp.bats Outdated

@rata rata 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.

@ricardobranco777 thanks for the PR! I'm curious if there are simpler ways to solve it. Left a comment on that :)

Comment thread tests/integration/seccomp.bats Outdated
The hardcoded architecture list was little-endian only, causing
seccomp_arch_add() to fail with -EDOM on s390x.

Drop it.  It's optional and libseccomp automatically adds the native
architecture when the filter is created.

Fixes: opencontainers#4835

Signed-off-by: Ricardo Branco <rbranco@suse.de>
@ricardobranco777

ricardobranco777 commented Jun 1, 2026

Copy link
Copy Markdown
Contributor Author

Verification runs in openQA:

The last is not visible but it works.

@rata rata 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.

LGTM, thanks!

I kicked the failing jobs again, as they are unrelated to this. Let's see if that fixes it

@kolyshkin kolyshkin 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.

LGTM

@cyphar PTAL

@kolyshkin

Copy link
Copy Markdown
Contributor

fedora-rawhide fails consistently with

Cannot find device "dummy0"

which is also the case with fedora (released version).

@aojea I remember you were looking into it; have you found anything?

@lifubang

lifubang commented Jun 3, 2026

Copy link
Copy Markdown
Member

Cannot find device "dummy0"

The "Cannot find" message comes from the teardown phase.
The actual failure is: [[ "$output" == *"ether $mac_address "* ]] did not match, indicating that the MAC address was unexpectedly changed on Fedora when moving a net device from host network namespace to the container network namespace.

@lifubang

lifubang commented Jun 3, 2026

Copy link
Copy Markdown
Member

the MAC address was unexpectedly changed

From "00:11:22:33:44:55" to "56:78:60:ee:6a:c3", or "4e:c3:84:5c:9d:24" ...

@lifubang

lifubang commented Jun 3, 2026

Copy link
Copy Markdown
Member

@aojea I’ve opened PR #5310 to address this issue. PTAL when you get a chance.

@kolyshkin kolyshkin merged commit d114cbd into opencontainers:main Jun 17, 2026
239 of 251 checks passed
@ricardobranco777 ricardobranco777 deleted the seccomp branch June 17, 2026 22:30
@aojea

aojea commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

@aojea I’ve opened PR #5310 to address this issue. PTAL when you get a chance.

so sorry, I missed this ping, let me check

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.

runc seccomp test fails on s390x (numerical argument out of domain)

7 participants