Skip to content

Fix default userns=host behavior to be opt-in#2386

Merged
jjbustamante merged 5 commits intomainfrom
jjbustamante/fix-pack-2369
May 31, 2025
Merged

Fix default userns=host behavior to be opt-in#2386
jjbustamante merged 5 commits intomainfrom
jjbustamante/fix-pack-2369

Conversation

@jjbustamante
Copy link
Copy Markdown
Member

Summary

The flag --userns-host is now required to enable the user namespace isolation for build containers. By default, this is disabled, which resolves the incompatibility with Docker plugins that don't support "userns as host" functionality.

This reverts the default behavior from pack 0.35.0 where user namespace isolation was enabled by default, while still allowing users who want this feature to opt in.

Output

Before

After

Documentation

  • Should this change be documented?
    • Yes, see #___
    • No

Related

Resolves #2369

  The flag `--userns-host` is now required to enable the user namespace
  isolation for build containers. By default, this is disabled, which
  resolves the incompatibility with Docker plugins that don't support
  "userns as host" functionality.

  This reverts the default behavior from pack 0.35.0 where user namespace
  isolation was enabled by default, while still allowing users who want
  this feature to opt in.

  Fixes #2369

Signed-off-by: Juan Bustamante <bustamantejj@gmail.com>
@jjbustamante jjbustamante requested review from a team as code owners May 21, 2025 02:04
@github-actions github-actions Bot added this to the 0.38.0 milestone May 21, 2025
@github-actions github-actions Bot added the type/enhancement Issue that requests a new feature or improvement. label May 21, 2025
Signed-off-by: Juan Bustamante <bustamantejj@gmail.com>
Signed-off-by: Juan Bustamante <bustamantejj@gmail.com>
@jjbustamante
Copy link
Copy Markdown
Member Author

@jkutner could you give me a review here? I got confirmation that is working as expected

@jjbustamante jjbustamante enabled auto-merge (squash) May 31, 2025 16:46
@jjbustamante jjbustamante disabled auto-merge May 31, 2025 16:46
@jjbustamante jjbustamante merged commit 8c7e57a into main May 31, 2025
15 checks passed
@jjbustamante jjbustamante deleted the jjbustamante/fix-pack-2369 branch May 31, 2025 16:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type/enhancement Issue that requests a new feature or improvement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add option to turn off userns=host configuration.

1 participant