Skip to content

Commit

Permalink
Don't add cgroup manager param for podman-remote
Browse files Browse the repository at this point in the history
Since it doesn't work for either "run" or "start"

	Error: unknown flag: --cgroup-manager
  • Loading branch information
afbjorklund committed May 4, 2020
1 parent 29fb2b4 commit d5461bc
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions pkg/drivers/kic/oci/oci.go
Original file line number Diff line number Diff line change
Expand Up @@ -242,7 +242,7 @@ func createContainer(ociBin string, image string, opts ...createOpt) error {
args := []string{"run"}

// to run nested container from privileged container in podman https://bugzilla.redhat.com/show_bug.cgi?id=1687713
if ociBin == Podman {
if ociBin == Podman && runtime.GOOS == "linux" {
args = append(args, "--cgroup-manager", "cgroupfs")
}

Expand All @@ -263,7 +263,7 @@ func StartContainer(ociBin string, container string) error {
args := []string{"start"}

// to run nested container from privileged container in podman https://bugzilla.redhat.com/show_bug.cgi?id=1687713
if ociBin == Podman {
if ociBin == Podman && runtime.GOOS == "linux" {
args = append(args, "--cgroup-manager", "cgroupfs")
}

Expand Down

0 comments on commit d5461bc

Please sign in to comment.