Skip to content
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.

Sign up for GitHub

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

Jump to bottom

fix: use copy uid gid when running toi applications #1224

Closed
wants to merge 1 commit into from
Closed

fix: use copy uid gid when running toi applications #1224

wants to merge 1 commit into from

Conversation

Skarlso
Copy link
Contributor

@Skarlso Skarlso commented Jan 6, 2025

What this PR does / why we need it

Which issue(s) this PR fixes

Fixes open-component-model/ocm-project#359

@Skarlso
fix: use copy uid gid when running toi applications
9dfbf7c 
Signed-off-by: Gergely Brautigam <182850+Skarlso@users.noreply.github.com>
@Skarlso Skarlso requested a review from a team as a code owner January 6, 2025 11:21
@github-actions github-actions bot added the kind/bugfix Bug label Jan 6, 2025
@Skarlso Skarlso mentioned this pull request Jan 6, 2025
Closed
@github-actions github-actions bot added the size/xs Extra small label Jan 6, 2025
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jan 6, 2025

Mend Scan Summary: ❌

Repository: open-component-model/ocm

VIOLATION DESCRIPTION NUMBER OF VIOLATIONS
HIGH/CRITICAL SECURITY VULNERABILITIES 5
MAJOR UPDATES AVAILABLE 0
LICENSE REQUIRES REVIEW 2
HIGH RISK LICENSES 9
RESTRICTIED LICENSE FOR ON-PREMISE DELIVERY 0

Detailed Logs: mend-scan-> Generate Report
Mend UI

@hilmarf hilmarf added this to the 2025-Q1 milestone Jan 6, 2025
@Skarlso
Copy link
Contributor Author

Skarlso commented Jan 6, 2025

This is actually making the command fail in a different manner:

Error: error copying to / in container: Error response from daemon: getent unable to find entry "1001" in passwd database

@Skarlso Skarlso closed this Jan 6, 2025
@hilmarf hilmarf deleted the fix-toi-permissions branch January 9, 2025 09:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hilmarf hilmarf hilmarf approved these changes

Assignees
No one assigned
Labels
kind/bugfix Bug size/xs Extra small
Projects
OCM Backlog Board
Status: 🔒Closed
Milestone
2025-Q1
Development

Successfully merging this pull request may close these issues.

executor image for ocm toi must run as root
2 participants
@Skarlso @hilmarf