stub: allow overriding plugin name with Option

marquiz · marquiz · commit 5d0a25f06f51 · 2025-08-11T11:03:33.000+03:00
Use the given name instead of erroring out. The command line flag takes
precedence over the NRI_PLUGIN_NAME environment variable.

Signed-off-by: Markus Lehtonen &lt;markus.lehtonen@intel.com&gt;
diff --git a/pkg/stub/stub.go b/pkg/stub/stub.go
@@ -217,7 +217,7 @@ func WithOnClose(onClose func()) Option {
 func WithPluginName(name string) Option {
 	return func(s *stub) error {
 		if s.name != "" {
-			return fmt.Errorf("plugin name already set (%q)", s.name)
+			log.Infof(noCtx, "Plugin name overridden: %q (previously %q)", name, s.name)
 		}
 		s.name = name
 		return nil

Original file line number	Diff line number	Diff line change
`@@ -217,7 +217,7 @@ func WithOnClose(onClose func()) Option {`
`217`	`217`	`func WithPluginName(name string) Option {`
`218`	`218`	`return func(s *stub) error {`
`219`	`219`	`if s.name != "" {`
`220`		`- return fmt.Errorf("plugin name already set (%q)", s.name)`
	`220`	`+ log.Infof(noCtx, "Plugin name overridden: %q (previously %q)", name, s.name)`
`221`	`221`	`}`
`222`	`222`	`s.name = name`
`223`	`223`	`return nil`