merge fix

otterize · Sep 1, 2024 · 3087014 · 3087014
1 parent 8bdec72
commit 3087014
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/src/node-agent/pkg/container/containerinfo.go b/src/node-agent/pkg/container/containerinfo.go
@@ -3,10 +3,11 @@ package container
 import "github.com/otterize/network-mapper/src/bintools/bininfo"
 
 type ContainerInfo struct {
-	Id    string
-	Pid   int    `json:"pid"`
-	PodId string `json:"sandboxId"`
-	PodIP string
+	Id             string
+	Pid            int    `json:"pid"`
+	PodId          string `json:"sandboxId"`
+	PodIP          string
+	ExecutableInfo ExecutableInfo
 }
 
 type ExecutableInfo struct {

diff --git a/src/node-agent/pkg/container/manager.go b/src/node-agent/pkg/container/manager.go
@@ -60,9 +60,9 @@ func (m *ContainerManager) GetContainerInfo(ctx context.Context, pod v1.Pod, con
 	info.PodIP = pod.Status.PodIP
 
 	// Get the executable info
-	execInfo, err := m.InspectContainerExec(info.GetPID())
+	execInfo, err := m.InspectContainerExec(info.Pid)
 	if err != nil {
-		return nil, errors.Wrap(err)
+		return ContainerInfo{}, errors.Wrap(err)
 	}
 	info.ExecutableInfo = execInfo