Merge pull request #834 from hzxuzhonghu/update-log

kmesh-bot · web-flow · commit b47fbcdbe673 · 2024-09-11T09:54:42.000+08:00
update confusing get workload logging
diff --git a/pkg/auth/rbac.go b/pkg/auth/rbac.go
@@ -506,7 +506,7 @@ func (r *Rbac) getIdentityByIp(ip []byte) Identity {
 	networkAddress.Address, _ = netip.AddrFromSlice(ip)
 	workload := r.workloadCache.GetWorkloadByAddr(networkAddress)
 	if workload == nil {
-		log.Warnf("get workload from ip %v FAILED", ip)
+		log.Debugf("cannot find workload %v", networkAddress.Address.String())
 		return Identity{}
 	}
 	return Identity{
diff --git a/pkg/controller/telemetry/metric.go b/pkg/controller/telemetry/metric.go
@@ -361,6 +361,7 @@ func (m *MetricController) getWorkloadByAddress(address []byte) (*workloadapi.Wo
 	networkAddr.Address, _ = netip.AddrFromSlice(address)
 	workload := m.workloadCache.GetWorkloadByAddr(networkAddr)
 	if workload == nil {
+		log.Debugf("cannot find workload %s", networkAddr.Address.String())
 		return nil, ""
 	}
 	return workload, networkAddr.Address.String()

Original file line number	Diff line number	Diff line change
`@@ -506,7 +506,7 @@ func (r *Rbac) getIdentityByIp(ip []byte) Identity {`
`506`	`506`	`networkAddress.Address, _ = netip.AddrFromSlice(ip)`
`507`	`507`	`workload := r.workloadCache.GetWorkloadByAddr(networkAddress)`
`508`	`508`	`if workload == nil {`
`509`		`- log.Warnf("get workload from ip %v FAILED", ip)`
	`509`	`+ log.Debugf("cannot find workload %v", networkAddress.Address.String())`
`510`	`510`	`return Identity{}`
`511`	`511`	`}`
`512`	`512`	`return Identity{`
Original file line number	Diff line number	Diff line change
`@@ -361,6 +361,7 @@ func (m MetricController) getWorkloadByAddress(address []byte) (workloadapi.Wo`
`361`	`361`	`networkAddr.Address, _ = netip.AddrFromSlice(address)`
`362`	`362`	`workload := m.workloadCache.GetWorkloadByAddr(networkAddr)`
`363`	`363`	`if workload == nil {`
	`364`	`+ log.Debugf("cannot find workload %s", networkAddr.Address.String())`
`364`	`365`	`return nil, ""`
`365`	`366`	`}`
`366`	`367`	`return workload, networkAddr.Address.String()`