Skip to content

remove any explicit userIds in Dockerfile steps#5820

Closed
arkodg wants to merge 1 commit intoenvoyproxy:mainfrom
arkodg:fix-docker-perms
Closed

remove any explicit userIds in Dockerfile steps#5820
arkodg wants to merge 1 commit intoenvoyproxy:mainfrom
arkodg:fix-docker-perms

Conversation

@arkodg
Copy link
Contributor

@arkodg arkodg commented Apr 25, 2025

it will allow a user in k8s to override the USER
directive with runAsUser if they'd like to

it will allow a user in k8s to override the USER
directive with `runAsUser` if they'd like to

Signed-off-by: Arko Dasgupta <arko@tetrate.io>
@arkodg arkodg requested a review from a team as a code owner April 25, 2025 18:39
@arkodg arkodg requested a review from zhaohuabing April 25, 2025 18:39
@codecov
Copy link

codecov bot commented Apr 25, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 65.09%. Comparing base (096cb8d) to head (0d335a5).
Report is 88 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5820      +/-   ##
==========================================
- Coverage   65.19%   65.09%   -0.11%     
==========================================
  Files         214      219       +5     
  Lines       34321    34857     +536     
==========================================
+ Hits        22377    22691     +314     
- Misses      10591    10780     +189     
- Partials     1353     1386      +33     

☔ 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.

@arkodg arkodg closed this May 2, 2025
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