Skip to content

Commit

Permalink
Merge pull request #14344 from cdoern/podCreate
Browse files Browse the repository at this point in the history
podman pod create --uidmap patch
  • Loading branch information
openshift-merge-robot authored Jun 2, 2022
2 parents a208bfa + f728b7b commit e4e1006
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 1 deletion.
5 changes: 5 additions & 0 deletions pkg/specgen/podspecgen.go
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ import (
"net"

"github.com/containers/common/libnetwork/types"
storageTypes "github.com/containers/storage/types"
spec "github.com/opencontainers/runtime-spec/specs-go"
)

Expand Down Expand Up @@ -222,6 +223,10 @@ type PodResourceConfig struct {

type PodSecurityConfig struct {
SecurityOpt []string `json:"security_opt,omitempty"`
// IDMappings are UID and GID mappings that will be used by user
// namespaces.
// Required if UserNS is private.
IDMappings *storageTypes.IDMappingOptions `json:"idmappings,omitempty"`
}

// NewPodSpecGenerator creates a new pod spec
Expand Down
4 changes: 3 additions & 1 deletion test/system/170-run-userns.bats
Original file line number Diff line number Diff line change
Expand Up @@ -38,10 +38,12 @@ function _require_crun() {

@test "rootful pod with custom ID mapping" {
skip_if_rootless "does not work rootless - rootful feature"
skip_if_remote "remote --uidmap is broken (see #14233)"
random_pod_name=$(random_string 30)
run_podman pod create --uidmap 0:200000:5000 --name=$random_pod_name
run_podman pod start $random_pod_name
run_podman pod inspect --format '{{.InfraContainerID}}' $random_pod_name
run podman inspect --format '{{.HostConfig.IDMappings.UIDMap}}' $output
is "$output" ".*0:200000:5000" "UID Map Successful"

# Remove the pod and the pause image
run_podman pod rm $random_pod_name
Expand Down

0 comments on commit e4e1006

Please sign in to comment.