Skip to content

Update Container User Id to react to changes in dotnet/dotnet-docker#4715#34128

Merged
baronfel merged 1 commit intodotnet:release/8.0.1xx-preview7from
baronfel:fix-new-dotnet-docker-user
Jul 20, 2023
Merged

Update Container User Id to react to changes in dotnet/dotnet-docker#4715#34128
baronfel merged 1 commit intodotnet:release/8.0.1xx-preview7from
baronfel:fix-new-dotnet-docker-user

Conversation

@baronfel
Copy link
Member

@baronfel baronfel commented Jul 20, 2023

dotnet/dotnet-docker#4715 changed the User Id for the Linux images, and we didn't update at the same time. A longer-term fix would detect and parse the user id from the specified container environment variable if it is present, but we haven't implemented that yet. That work is tracked in dotnet/sdk-container-builds#479.

This error has already been reported by a user in dotnet/dotnet-docker#4766.

Without this change, container images generated by our tooling from preview 6 onwards are broken unless the user

  • sets their own ContainerUser value (<ContainerUser>1654</ContainerUser>), or
  • sets the user for the container explicitly at container runtime (docker run -u XXXX)

@baronfel baronfel force-pushed the fix-new-dotnet-docker-user branch from 160d15e to acf76dd Compare July 20, 2023 15:49
@baronfel baronfel requested a review from a team as a code owner July 20, 2023 15:49
@baronfel baronfel added this to the 8.0.1xx milestone Jul 20, 2023
@baronfel baronfel enabled auto-merge July 20, 2023 17:18
@baronfel baronfel merged commit 8b67942 into dotnet:release/8.0.1xx-preview7 Jul 20, 2023
@baronfel baronfel deleted the fix-new-dotnet-docker-user branch July 20, 2023 18:01
@leecow leecow removed the untriaged Request triage from a team member label Nov 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants