Skip to content

fix: enhance resource creation with detailed process information #1978

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
Merged
ScharfViktor merged 1 commit into from
Dec 5, 2025
Merged

fix: enhance resource creation with detailed process information #1978

ScharfViktor merged 1 commit into from
Dec 5, 2025

Conversation

@micbar
Copy link
Contributor

@micbar micbar commented Dec 5, 2025

Description

Related Issue

Motivation and Context

ProcessOwner uses user.Current() which eventually calls lookup id, which returns an error when there no associated username for the ID.

How Has This Been Tested?

  • test environment:
  • test case 1:
  • test case 2:
  • ...

Screenshots (if appropriate):

image

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation added

@micbar micbar added the Type:Bug label Dec 5, 2025
@micbar micbar mentioned this pull request Dec 5, 2025
Closed
9 tasks
@micbar micbar requested a review from butonic December 5, 2025 11:11
@ScharfViktor
Copy link
Contributor

ScharfViktor commented Dec 5, 2025

update test between 4.0.0-rc.2 (OC_CONTAINER_UID_GID="1500:1500") and dev docker builded on your branch was successful. Instance is running, no errors error detecting resource: user: unknown userid 1500

@butonic
Copy link
Contributor

butonic commented Dec 5, 2025 •
edited
Loading

only flaky tests fail, can be merged anyway

@ScharfViktor ScharfViktor merged commit 64119b3 into main Dec 5, 2025
57 of 59 checks passed
@ScharfViktor ScharfViktor deleted the fix-tracing branch December 5, 2025 14:08
@openclouders openclouders mentioned this pull request Dec 5, 2025
Merged
1 task
micbar added a commit that referenced this pull request Dec 9, 2025
@micbar @stavros-k
fix: enhance resource creation with detailed process information (#1978)
188176a 
Co-authored-by: Stavros Kois <s.kois@outlook.com>
(cherry picked from commit 64119b3)
@micbar
Copy link
Contributor Author

micbar commented Dec 9, 2025

💚 All backports created successfully

Status Branch Result
✅ stable-4.0

Questions ?

Please refer to the Backport tool documentation

@micbar micbar mentioned this pull request Dec 9, 2025
Merged
ScharfViktor pushed a commit that referenced this pull request Dec 11, 2025
@micbar @stavros-k
fix: enhance resource creation with detailed process information (#1978…
26f5cd7 
…) (#2000)

(cherry picked from commit 64119b3)

Co-authored-by: Stavros Kois <s.kois@outlook.com>
@openclouders openclouders mentioned this pull request Dec 11, 2025
Merged
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@butonic butonic butonic approved these changes

Assignees

No one assigned

Labels

Type:Bug

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Updated from 4.0.0 RC2 to 4.0.0 and opencloud does not start anymore. user: unknown userid 1500

5 participants

@micbar @ScharfViktor @butonic @stavros-k