Skip to content

fix(e2e): add /dev/urandom to provider test sandbox policy#948

Merged
johntmyers merged 1 commit intoNVIDIA:mainfrom
derekwaynecarr:fix-python-tests
Apr 24, 2026
Merged

fix(e2e): add /dev/urandom to provider test sandbox policy#948
johntmyers merged 1 commit intoNVIDIA:mainfrom
derekwaynecarr:fix-python-tests

Conversation

@derekwaynecarr
Copy link
Copy Markdown
Collaborator

Summary

Python runtime requires /dev/urandom access during initialization to seed the hash randomizer. The _default_policy() in provider tests was missing this path, causing exec_python tests to fail with: 'Fatal Python error: _Py_HashRandomization_Init: failed to get random numbers to initialize Python'.

Add /dev/urandom to read_only paths to match the policy used in test_sandbox_policy.py, allowing Python to initialize successfully.

Related Issue

Changes

Testing

  • mise run e2e:python

Checklist

  • [x ] Follows Conventional Commits
  • [x ] Commits are signed off (DCO)
  • Architecture docs updated (if applicable)

Python runtime requires /dev/urandom access during initialization
to seed the hash randomizer. The _default_policy() in provider tests
was missing this path, causing exec_python tests to fail with:
'Fatal Python error: _Py_HashRandomization_Init: failed to get
random numbers to initialize Python'.

Add /dev/urandom to read_only paths to match the policy used in
test_sandbox_policy.py, allowing Python to initialize successfully.

Signed-off-by: Derek Carr <decarr@redhat.com>
@derekwaynecarr derekwaynecarr requested a review from a team as a code owner April 23, 2026 22:40
@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented Apr 23, 2026

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@johntmyers johntmyers merged commit 87f50f5 into NVIDIA:main Apr 24, 2026
10 checks passed
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.

2 participants