Skip to content

Conversation

@DiamondJoseph
Copy link
Contributor

@DiamondJoseph DiamondJoseph commented Nov 10, 2025

Configures a default 1000 user for container to run as. This allows a username to be found when running without the LDAP sidecar or when running as a user not registered in LDAP, e.g. the default 1000 user in the Helm chart/Kyverno policy.

As we remove the need for hostNetwork and continue to push for a non-dls_sw scratch area, the container does not require running as a specific user and can run as an arbitrary user with fewer permissions.

This fixes problems of attaching a devcontainer for debugging when running as a non-LDAP user.

@DiamondJoseph DiamondJoseph changed the title Run Dockerfile as uid 1000 fix(helm): Run Dockerfile as uid 1000 Nov 10, 2025
@codecov
Copy link

codecov bot commented Nov 10, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.76%. Comparing base (ab3be3d) to head (e9ab7f9).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1274   +/-   ##
=======================================
  Coverage   94.76%   94.76%           
=======================================
  Files          41       41           
  Lines        2674     2674           
=======================================
  Hits         2534     2534           
  Misses        140      140           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@DiamondJoseph DiamondJoseph marked this pull request as ready for review November 10, 2025 12:12
@DiamondJoseph DiamondJoseph requested a review from a team as a code owner November 10, 2025 12:12
@DiamondJoseph DiamondJoseph merged commit 1ab51e8 into main Nov 12, 2025
19 checks passed
@DiamondJoseph DiamondJoseph deleted the custom-user branch November 12, 2025 16:20
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.

3 participants