APIGOV-33169 - Replace selfLink comparison with metadata.id check

vivekschauhan · vivekschauhan · commit 8132890b528a · 2026-07-09T20:52:41.000+01:00
diff --git a/pkg/agent/handler/agentresource.go b/pkg/agent/handler/agentresource.go
@@ -88,16 +88,16 @@ func (h *agentResourceHandler) Handle(ctx context.Context, meta *proto.EventMeta
 		return nil
 	}
 	agentRes := h.agentResourceManager.GetAgentResource()
-	if agentRes == nil || agentRes.GetSelfLink() != resource.GetSelfLink() {
-		h.logger.WithField("selfLink", resource.GetSelfLink()).Trace("skipping handling agent resource")
+	if agentRes == nil || agentRes.Metadata.ID != resource.Metadata.ID {
+		h.logger.WithField("id", resource.Metadata.ID).
+			WithField("selfLink", resource.GetSelfLink()).Trace("skipping handling agent resource")
 		return nil
 	}
 	if action == proto.Event_SUBRESOURCEUPDATED && subres == definitions.XAgentDetails {
 		h.handleUpdateTrigger(resource)
 	}
-	handlerFunc(action, subres, resource)
 
-	return nil
+	return handlerFunc(action, subres, resource)
 }
 
 func (h *agentResourceHandler) handleUpdateTrigger(resource *v1.ResourceInstance) {

Original file line number	Diff line number	Diff line change
`@@ -88,16 +88,16 @@ func (h agentResourceHandler) Handle(ctx context.Context, meta proto.EventMeta`
`88`	`88`	`return nil`
`89`	`89`	`}`
`90`	`90`	`agentRes := h.agentResourceManager.GetAgentResource()`
`91`		`- if agentRes == nil \|\| agentRes.GetSelfLink() != resource.GetSelfLink() {`
`92`		`- h.logger.WithField("selfLink", resource.GetSelfLink()).Trace("skipping handling agent resource")`
	`91`	`+ if agentRes == nil \|\| agentRes.Metadata.ID != resource.Metadata.ID {`
	`92`	`+ h.logger.WithField("id", resource.Metadata.ID).`
	`93`	`+ WithField("selfLink", resource.GetSelfLink()).Trace("skipping handling agent resource")`
`93`	`94`	`return nil`
`94`	`95`	`}`
`95`	`96`	`if action == proto.Event_SUBRESOURCEUPDATED && subres == definitions.XAgentDetails {`
`96`	`97`	`h.handleUpdateTrigger(resource)`
`97`	`98`	`}`
`98`		`- handlerFunc(action, subres, resource)`
`99`	`99`
`100`		`- return nil`
	`100`	`+ return handlerFunc(action, subres, resource)`
`101`	`101`	`}`
`102`	`102`
`103`	`103`	`func (h agentResourceHandler) handleUpdateTrigger(resource v1.ResourceInstance) {`