-
Notifications
You must be signed in to change notification settings - Fork 160
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
[sync] fix: Prevent containers from running as root by removing unnecessary anyuid SCC bindings #1590
[sync] fix: Prevent containers from running as root by removing unnecessary anyuid SCC bindings #1590
Conversation
… and the security folder containing both, as they are no longer needed.
Skipping CI for Draft Pull Request. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## rhoai #1590 +/- ##
==========================================
- Coverage 20.33% 20.06% -0.28%
==========================================
Files 161 160 -1
Lines 11014 10953 -61
==========================================
- Hits 2240 2198 -42
+ Misses 8534 8520 -14
+ Partials 240 235 -5 ☔ View full report in Codecov by Sentry. |
Clean up RoleBinding from function doc
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: CFSNM, zdtsw The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
c4a2e3d
into
opendatahub-io:rhoai
Description
https://issues.redhat.com/browse/RHOAIENG-18030
This PR addresses the security concern of containers running with root privileges by removing the following ServiceAccounts from default
RoleBindings
granting theanyuid
SCC:These changes help ensure that containers run with non-root user privileges and comply with security policies. Additionally, this makes the default RoleBinding that assigns the
default
ServiceAccount to theanyuid
SCC unnecessary:How Has This Been Tested?
Screenshot or short clip
Merge criteria