summaryrefslogtreecommitdiff
path: root/pkg/specgen/generate/config_linux.go
diff options
context:
space:
mode:
authorOpenShift Merge Robot <openshift-merge-robot@users.noreply.github.com>2020-10-30 19:52:17 +0100
committerGitHub <noreply@github.com>2020-10-30 19:52:17 +0100
commit1fe79dd677c75af3915418d96f12b6f4a75ee568 (patch)
tree7eac63217e5cb71d03cab39120e8b1fc417134e1 /pkg/specgen/generate/config_linux.go
parent3653e57a2a589bebfdde99fb3c0334d04f350888 (diff)
parent831d7fb0d7ee007fc04b1b0ff24b77c7d5635f5e (diff)
downloadpodman-1fe79dd677c75af3915418d96f12b6f4a75ee568.tar.gz
podman-1fe79dd677c75af3915418d96f12b6f4a75ee568.tar.bz2
podman-1fe79dd677c75af3915418d96f12b6f4a75ee568.zip
Merge pull request #8177 from rhatdan/wrap
Stop excessive wrapping of errors
Diffstat (limited to 'pkg/specgen/generate/config_linux.go')
-rw-r--r--pkg/specgen/generate/config_linux.go9
1 files changed, 3 insertions, 6 deletions
diff --git a/pkg/specgen/generate/config_linux.go b/pkg/specgen/generate/config_linux.go
index fac02ad01..fcb7641d2 100644
--- a/pkg/specgen/generate/config_linux.go
+++ b/pkg/specgen/generate/config_linux.go
@@ -52,7 +52,7 @@ func addPrivilegedDevices(g *generate.Generator) error {
if err == unix.EPERM {
continue
}
- return errors.Wrapf(err, "stat %s", d.Path)
+ return err
}
// Skip devices that the user has not access to.
if st.Mode()&0007 == 0 {
@@ -90,7 +90,7 @@ func DevicesFromPath(g *generate.Generator, devicePath string) error {
}
st, err := os.Stat(resolvedDevicePath)
if err != nil {
- return errors.Wrapf(err, "cannot stat device path %s", devicePath)
+ return err
}
if st.IsDir() {
found := false
@@ -231,10 +231,7 @@ func addDevice(g *generate.Generator, device string) error {
}
if rootless.IsRootless() {
if _, err := os.Stat(src); err != nil {
- if os.IsNotExist(err) {
- return errors.Wrapf(err, "the specified device %s doesn't exist", src)
- }
- return errors.Wrapf(err, "stat device %s exist", src)
+ return err
}
perm := "ro"
if strings.Contains(permissions, "w") {