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.

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

Change container image USER to UID #738

Merged
merged 3 commits into from
Feb 23, 2022
Merged

Conversation

kraman
Copy link
Contributor

@kraman kraman commented Feb 23, 2022

@kraman kraman requested review from a team and tigrannajaryan February 23, 2022 15:09
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Feb 23, 2022

CLA Signed

The committers are authorized under a signed CLA.

Dockerfile Outdated Show resolved Hide resolved
@pavolloffay
Copy link
Member

@kraman please sign the CLA

@pavolloffay pavolloffay merged commit 373fe17 into open-telemetry:main Feb 23, 2022
ItielOlenick pushed a commit to ItielOlenick/opentelemetry-operator that referenced this pull request May 1, 2024
* Change container image USER to UID

Kubernetes requires USER to be UID in order to verify running as non root.
ref https://github.com/kubernetes/kubernetes/blob/master/pkg/kubelet/kuberuntime/security_context_others.go#L49

* Use 65532 as nonroot UID
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