Skip to content

Commit

Permalink
resmgr: use module-global logger.
Browse files Browse the repository at this point in the history
Having loggers embedded in resmgr types as members was a bad idea.
Replace them with module global logger instances.

Signed-off-by: Krisztian Litkey <krisztian.litkey@intel.com>
  • Loading branch information
klihub authored and askervin committed Sep 20, 2024
1 parent daafb63 commit 22327fe
Show file tree
Hide file tree
Showing 3 changed files with 105 additions and 102 deletions.
4 changes: 2 additions & 2 deletions pkg/resmgr/flags.go
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ import (
"flag"
"time"

nri "github.com/containerd/nri/pkg/api"
"github.com/containerd/nri/pkg/api"
"github.com/containers/nri-plugins/pkg/pidfile"
)

Expand Down Expand Up @@ -54,7 +54,7 @@ func init() {
"NRI plugin name to register.")
flag.StringVar(&opt.NriPluginIdx, "nri-plugin-index", defaultPluginIndex,
"NRI plugin index to register.")
flag.StringVar(&opt.NriSocket, "nri-socket", nri.DefaultSocketPath,
flag.StringVar(&opt.NriSocket, "nri-socket", api.DefaultSocketPath,
"NRI unix domain socket path to connect to.")

flag.StringVar(&opt.PidFile, "pid-file", pidfile.GetPath(),
Expand Down
Loading

0 comments on commit 22327fe

Please sign in to comment.