Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BUG: Python test #24 failure due to munmap() #335

Closed

Conversation

drakenclimber
Copy link
Member

The python live test, 24-live-arg_allow.py, started failing on
Fedora 34 with kernel 5.13.9-200.fc34.x86_64. To fix this, allow
the munmap() syscall in the test's seccomp filter.

Signed-off-by: Tom Hromatka tom.hromatka@oracle.com

The python live test, 24-live-arg_allow.py, started failing on
Fedora 34 with kernel 5.13.9-200.fc34.x86_64.  To fix this, allow
the munmap() syscall in the test's seccomp filter.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>
@drakenclimber drakenclimber added this to the v2.5.2 milestone Aug 31, 2021
@drakenclimber
Copy link
Member Author

@pcmoore - I ran into this test failure on step #14 of the release process.

I'm not sure yet if it's an issue on this particular machine, but I was able to reproduce it both in the git sandbox and the unpacked dist-gzip tarball.

Thoughts?

Oh, and I'll obviously fix it in main first, then cherry-pick the fix into the release-2.5 branch. Just wanted to get this out there.

@drakenclimber drakenclimber changed the title tests: Allow munmap() syscall in Python test #24 BUG: Python test #24 failure due to munmap() Aug 31, 2021
@drakenclimber
Copy link
Member Author

Without this change:

$ ./regression -m python -b 24-live-arg_allow -T live
=============== Tue Aug 31 04:10:59 PM MDT 2021 ===============
Regression Test Report ("regression -m python -b 24-live-arg_allow -T live")
 batch name: 24-live-arg_allow
 test mode:  python
 test type:  live
live
Test 24-live-arg_allow%%001-00001 result:   FAILURE 24-live-arg_allow 1 ALLOW rc=159
Regression Test Summary
 tests run: 1
 tests skipped: 0
 tests passed: 0
 tests failed: 1
 tests errored: 0
============================================================

With this change:

$ ./regression -m python -b 24-live-arg_allow -T live
=============== Tue Aug 31 04:11:31 PM MDT 2021 ===============
Regression Test Report ("regression -m python -b 24-live-arg_allow -T live")
 batch name: 24-live-arg_allow
 test mode:  python
 test type:  live
live
Test 24-live-arg_allow%%001-00001 result:   SUCCESS
Regression Test Summary
 tests run: 1
 tests skipped: 0
 tests passed: 1
 tests failed: 0
 tests errored: 0
============================================================

@coveralls
Copy link

Coverage Status

Coverage remained the same at 89.487% when pulling 33cfbec on drakenclimber:issues/fix-test24 into 01a7e98 on seccomp:release-2.5.

@drakenclimber
Copy link
Member Author

It looks like python 3.9 invokes munmap() on quit.

python 3.8.10 works on Ubuntu 20.10:

$ python --version
Python 3.8.10

$ ./regression -T live -m python -b 24-li
ve-arg_allow
=============== Wed Sep  1 15:25:03 UTC 2021 ===============
Regression Test Report ("regression -T live -m python -b 24-live-arg_allow")
 batch name: 24-live-arg_allow
 test mode:  python
 test type:  live
live
Test 24-live-arg_allow%%001-00001 result:   SUCCESS
Regression Test Summary
 tests run: 1
 tests skipped: 0
 tests passed: 1
 tests failed: 0
 tests errored: 0
============================================================

But Python 3.9.7 fails. No other changes made to the system:

$ python --version
Python 3.9.7

$ ./regression -T live -m python -b 24-live-arg_allow
=============== Wed Sep  1 15:25:57 UTC 2021 ===============
Regression Test Report ("regression -T live -m python -b 24-live-arg_allow")
 batch name: 24-live-arg_allow
 test mode:  python
 test type:  live
live
Test 24-live-arg_allow%%001-00001 result:   FAILURE 24-live-arg_allow 1 ALLOW rc=1
Regression Test Summary
 tests run: 1
 tests skipped: 0
 tests passed: 0
 tests failed: 1
 tests errored: 0
============================================================

Copy link
Member

@pcmoore pcmoore left a comment

Choose a reason for hiding this comment

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

Not sure why I do not see this in my nightly testing on Rawhide ... ? Regardless, it looks reasonable to me, thanks for catching this!

Acked-by: Paul Moore <paul@paul-moore.com>

@drakenclimber
Copy link
Member Author

Not sure why I do not see this in my nightly testing on Rawhide ... ? Regardless, it looks reasonable to me, thanks for catching this!

That's really strange. As I said above, I was able to reproduce it on Fedora 34 with Python 3.9.6 and Ubuntu 20.10 with Python 3.9.7.

I would love to add Fedora (or something similar) to the Github Actions CI, but their supported distro list is rather scant. There is a way to add self-hosted runners (and we've had success in libcgroup using them), but it does put a small sysadmin burden on us.

@drakenclimber
Copy link
Member Author

Thanks, @pcmoore

Merged into main in commit 4f34c6e
Merged into release-2.5 in commit 9b07575

@drakenclimber drakenclimber deleted the issues/fix-test24 branch September 13, 2021 19:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants