Fix cni conf

Signed-off-by: Yakul Garg <2000yeshu@gmail.com>
neverinstall · Mar 28, 2024 · e4fec44 · e4fec44
1 parent 5101b94
commit e4fec44
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/cmd/ctr/commands/run/run.go b/cmd/ctr/commands/run/run.go
@@ -123,7 +123,7 @@ var Command = cli.Command{
 			Name:  "cni",
 			Usage: "Enable cni networking for the container",
 		},
-		cli.Uint64Flag{
+		cli.StringFlag{
 			Name:  "cni-bandwidth-conf",
 			Usage: "Cni conf for bandwidth plugin",
 		},
@@ -186,8 +186,14 @@ var Command = cli.Command{
 		}
 		var network gocni.CNI
 		if enableCNI {
-			if network, err = gocni.New(gocni.WithDefaultConf, gocni.WithConfFile(context.String("cni-bandwidth-conf"))); err != nil {
-				return err
+			if context.String("cni-bandwidth-conf") == "" {
+				if network, err = gocni.New(gocni.WithDefaultConf); err != nil {
+					return err
+				}
+			} else {
+				if network, err = gocni.New(gocni.WithDefaultConf, gocni.WithConfFile(context.String("cni-bandwidth-conf"))); err != nil {
+					return err
+				}
 			}
 		}