aboutsummaryrefslogtreecommitdiff
path: root/libpod/oci_conmon_linux.go
diff options
context:
space:
mode:
authorOpenShift Merge Robot <openshift-merge-robot@users.noreply.github.com>2022-03-22 15:14:38 +0100
committerGitHub <noreply@github.com>2022-03-22 15:14:38 +0100
commit901066acd113ee3bea709cc558d86c4c438e2f67 (patch)
tree018a8f7918b50e9d1bc58f7ce9a981f628c3d295 /libpod/oci_conmon_linux.go
parentfc963159b8078c7e3ca6571174fb3a82d1023b43 (diff)
parent06dd9136a253521cb74497a59f2e6894806a5b6d (diff)
downloadpodman-901066acd113ee3bea709cc558d86c4c438e2f67.tar.gz
podman-901066acd113ee3bea709cc558d86c4c438e2f67.tar.bz2
podman-901066acd113ee3bea709cc558d86c4c438e2f67.zip
Merge pull request #13580 from vrothberg/enable-linters
enable linters
Diffstat (limited to 'libpod/oci_conmon_linux.go')
-rw-r--r--libpod/oci_conmon_linux.go7
1 files changed, 4 insertions, 3 deletions
diff --git a/libpod/oci_conmon_linux.go b/libpod/oci_conmon_linux.go
index 0e8aed93a..ba4079bed 100644
--- a/libpod/oci_conmon_linux.go
+++ b/libpod/oci_conmon_linux.go
@@ -750,7 +750,7 @@ func openControlFile(ctr *Container, parentDir string) (*os.File, error) {
for i := 0; i < 600; i++ {
controlFile, err := os.OpenFile(controlPath, unix.O_WRONLY|unix.O_NONBLOCK, 0)
if err == nil {
- return controlFile, err
+ return controlFile, nil
}
if !isRetryable(err) {
return nil, errors.Wrapf(err, "could not open ctl file for terminal resize for container %s", ctr.ID())
@@ -1015,7 +1015,8 @@ func (r *ConmonOCIRuntime) getLogTag(ctr *Container) (string, error) {
}
data, err := ctr.inspectLocked(false)
if err != nil {
- return "", nil
+ // FIXME: this error should probably be returned
+ return "", nil // nolint: nilerr
}
tmpl, err := template.New("container").Parse(logTag)
if err != nil {
@@ -1596,7 +1597,7 @@ func readConmonPipeData(runtimeName string, pipe *os.File, ociLog string) (int,
ch <- syncStruct{si: si}
}()
- data := -1
+ data := -1 //nolint: wastedassign
select {
case ss := <-ch:
if ss.err != nil {