Merge pull request #9995 from rhatdan/debug

Fix message about runtime to show only the actual runtime
This commit is contained in:
OpenShift Merge Robot
2021-04-14 09:16:50 -04:00
committed by GitHub
2 changed files with 3 additions and 2 deletions

View File

@ -130,8 +130,8 @@ func newConmonOCIRuntime(name string, paths []string, conmonPath string, runtime
continue
}
foundPath = true
logrus.Tracef("found runtime %q", runtime.path)
runtime.path = path
logrus.Debugf("using runtime %q", path)
break
}

View File

@ -385,7 +385,7 @@ func makeRuntime(ctx context.Context, runtime *Runtime) (retErr error) {
// This will allow us to ship configs including optional
// runtimes that might not be installed (crun, kata).
// Only a infof so default configs don't spec errors.
logrus.Infof("Error initializing configured OCI runtime %s: %v", name, err)
logrus.Debugf("configured OCI runtime %s initialization failed: %v", name, err)
continue
}
@ -412,6 +412,7 @@ func makeRuntime(ctx context.Context, runtime *Runtime) (retErr error) {
runtime.defaultOCIRuntime = ociRuntime
}
}
logrus.Debugf("Using OCI runtime %q", runtime.defaultOCIRuntime.Path())
// Do we have at least one valid OCI runtime?
if len(runtime.ociRuntimes) == 0 {