cni: add more debug

Signed-off-by: Maryam Tahhan <[email protected]>
redhat-et · Oct 31, 2023 · 31c129c · 31c129c
1 parent d04fdc0
commit 31c129c
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/examples/nad_with_syncer.yaml b/examples/nad_with_syncer.yaml
@@ -14,7 +14,7 @@ spec:
       "logFile": "afxdp-cni.log",                            # CNI log file location (optional)
       "logLevel": "debug",                                   # CNI logging level (optional)
       "dpSyncer": true,                                      # Sync with Device Plugin over gRPC MUST BE SET for bpf map pinning
-      "skipUnloadBpf": true,
+      "skipUnloadBpf": true,                                 # Skip unloading BPF locally
       "ipam": {                                              # CNI IPAM plugin and associated config (optional)
         "type": "host-local",
         "subnet": "192.168.1.0/24",

diff --git a/internal/deviceplugin/poolManager.go b/internal/deviceplugin/poolManager.go
@@ -285,7 +285,6 @@ func (pm *PoolManager) Allocate(ctx context.Context,
 			}
 
 			if pm.BpfdClientEnable {
-				logging.Infof("BpfProg %s BpfSec %s", pm.BpfProg, pm.BpfSec)
 				err, xskmap := pm.BpfdClient.SubmitXdpProg(device.Name(), pm.Node, pm.DevicePrefix, pm.BpfProg, pm.BpfSec)
 				if err != nil {
 					logging.Errorf("Error SubmitXdpProg to bpfd %v", err)

diff --git a/internal/dpcnisyncerserver/server.go b/internal/dpcnisyncerserver/server.go
@@ -98,10 +98,11 @@ func (s *SyncerServer) DelNetDev(ctx context.Context, in *pb.DeleteNetDevReq) (*
 	if s.BpfdClientEnable && !s.BpfMapPinEnable {
 		err := s.BpfdClient.DeleteXdpProg(netDevName)
 		if err != nil {
-			logging.Errorf("Could NOT delete BPF Prog for %s", netDevName)
-			return &pb.DeleteNetDevResp{Ret: -1}, errors.Wrapf(err, "Could NOT delete BPF Prog for %s: %v", netDevName, err.Error())
+			logging.Errorf("BpfdClient could NOT delete BPF Prog for %s", netDevName)
+			return &pb.DeleteNetDevResp{Ret: -1}, errors.Wrapf(err, "BpfdClient could NOT delete BPF Prog for %s: %v", netDevName, err.Error())
 		}
 
+		logging.Infof("Network interface %s deleted", netDevName)
 		return &pb.DeleteNetDevResp{Ret: 0}, nil
 	}