Skip to content

fix(e2e): use high UID range to avoid host user conflicts#978

Open
derekwaynecarr wants to merge 1 commit intoNVIDIA:mainfrom
derekwaynecarr:fix/e2e-custom-image-uid-conflict
Open

fix(e2e): use high UID range to avoid host user conflicts#978
derekwaynecarr wants to merge 1 commit intoNVIDIA:mainfrom
derekwaynecarr:fix/e2e-custom-image-uid-conflict

Conversation

@derekwaynecarr
Copy link
Copy Markdown
Contributor

Summary

Change sandbox user UID from 1000 to 1000660000 in custom image examples and E2E tests.

Using a high UID range (1000000000+) prevents conflicts with host users when running without user namespace remapping, where container UIDs map directly to host UIDs.

This resolves fork failures caused by RLIMIT_NPROC enforcement when the host user already has many threads running.

Related Issue

Changes

Testing

  • [x ] mise run pre-commit passes
  • Unit tests added/updated
  • [x ] E2E tests added/updated (if applicable)

Checklist

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

Change sandbox user UID from 1000 to 1000660000 in custom image examples and E2E tests.
Using a high UID range (1000000000+) prevents conflicts with host users when running
without user namespace remapping, where container UIDs map directly to host UIDs.

This resolves fork failures caused by RLIMIT_NPROC enforcement when the host user
already has many threads running.

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

copy-pr-bot Bot commented Apr 25, 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.

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.

1 participant