Skip to content

Commit

Permalink
Merge pull request #9788 from andrewshadura/rootless-friendly-error
Browse files Browse the repository at this point in the history
[CI:DOCS] rootless: Tell the user what was led to the error, not just what it is
  • Loading branch information
openshift-merge-robot authored Apr 27, 2021
2 parents 3148e01 + 09e640d commit abda796
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion pkg/rootless/rootless_linux.go
Original file line number Diff line number Diff line change
Expand Up @@ -116,7 +116,7 @@ func tryMappingTool(uid bool, pid int, hostID int, mappings []idtools.IDMap) err
}
path, err := exec.LookPath(tool)
if err != nil {
return errors.Wrapf(err, "cannot find %s", tool)
return errors.Wrapf(err, "command required for rootless mode with multiple IDs")
}

appendTriplet := func(l []string, a, b, c int) []string {
Expand Down
4 changes: 2 additions & 2 deletions troubleshooting.md
Original file line number Diff line number Diff line change
Expand Up @@ -203,12 +203,12 @@ Rootless Podman requires the newuidmap and newgidmap programs to be installed.

#### Symptom

If you are running Podman or buildah as a not root user, you get an error complaining about
If you are running Podman or Buildah as a rootless user, you get an error complaining about
a missing newuidmap executable.

```
podman run -ti fedora sh
cannot find newuidmap: exec: "newuidmap": executable file not found in $PATH
command required for rootless mode with multiple IDs: exec: "newuidmap": executable file not found in $PATH
```

#### Solution
Expand Down

0 comments on commit abda796

Please sign in to comment.