resmgr: exit when ttrpc connection goes down.

We do not have proper connection re-establishment and the policy- agnostic resource management layer will require changes for this to be possible. Until we get that done, exit the plugin when the ttrpc connection goes down instead of just sitting there pretend- ing everything is fine. Signed-off-by: Krisztian Litkey <[email protected]>
klihub · May 17, 2024 · 98e4d7d · 98e4d7d
1 parent bdddfce
commit 98e4d7d
Showing 1 changed file with 3 additions and 6 deletions.
diff --git a/pkg/resmgr/nri.go b/pkg/resmgr/nri.go
@@ -17,6 +17,7 @@ package resmgr
 import (
 	"context"
 	"fmt"
+	"os"
 
 	"github.com/containers/nri-plugins/pkg/instrumentation/tracing"
 	logger "github.com/containers/nri-plugins/pkg/log"
@@ -106,13 +107,9 @@ func (p *nriPlugin) stop() {
 	p.stub.Stop()
 }
 
-func (p *nriPlugin) restart() error {
-	return p.start()
-}
-
 func (p *nriPlugin) onClose() {
-	p.resmgr.Warn("connection to NRI/runtime lost, trying to reconnect...")
-	p.restart()
+	p.resmgr.Warn("connection to NRI/runtime lost, exiting...")
+	os.Exit(1)
 }
 
 func (p *nriPlugin) Configure(ctx context.Context, cfg, runtime, version string) (stub.EventMask, error) {