fix(controller): add finish logs for all gc functions

All gc functions only had "start to gc ..." logs without corresponding
completion logs, making it difficult to determine whether a gc cycle
completed successfully or hung. Add "finish to gc ..." logs before
each successful return to improve observability and debugging.

Signed-off-by: Mengxin Liu <liumengxinfly@gmail.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Mengxin Liu <liumengxinfly@gmail.com>

Author: oilbeater

pkg/controller/gc.go

@@ -81,6 +81,7 @@ func (c *Controller) gcLogicalRouterPort() error {
 		klog.Errorf("delete non-existent peer logical router port: %v", err)
 		return err
 	}
+	klog.Infof("finish to gc logical router port")
 	return nil
 }
 
@@ -124,6 +125,7 @@ func (c *Controller) gcVpcNatGateway() error {
 			}
 		}
 	}
+	klog.Infof("finish to gc vpc nat gateway")
 	return nil
 }
 
@@ -166,6 +168,7 @@ func (c *Controller) gcLogicalSwitch() error {
 			return err
 		}
 	}
+	klog.Infof("finish to gc logical switch")
 
 	klog.Infof("start to gc dhcp options")
 	dhcpOptions, err := c.OVNNbClient.ListDHCPOptions(c.config.EnableExternalVpc, nil)
@@ -186,6 +189,7 @@ func (c *Controller) gcLogicalSwitch() error {
 			return err
 		}
 	}
+	klog.Infof("finish to gc dhcp options")
 	return nil
 }
 
@@ -222,6 +226,7 @@ func (c *Controller) gcCustomLogicalRouter() error {
 			return err
 		}
 	}
+	klog.Infof("finish to gc logical router")
 	return nil
 }
 
@@ -280,6 +285,7 @@ func (c *Controller) gcNode() error {
 		}
 	}
 
+	klog.Infof("finish to gc nodes")
 	return nil
 }
 
@@ -308,13 +314,15 @@ func (c *Controller) gcVip() error {
 						klog.Errorf("failed to clean label from vip %s, %v", vip.Name, err)
 						return err
 					}
+					klog.Infof("finish to gc vips")
 					return nil
 				}
 				klog.Error(err)
 				return err
 			}
 		}
 	}
+	klog.Infof("finish to gc vips")
 	return nil
 }
 
@@ -416,6 +424,7 @@ func (c *Controller) gcIP() error {
 			}
 		}
 	}
+	klog.Infof("finish to gc ips")
 	return nil
 }
 
@@ -566,6 +575,7 @@ func (c *Controller) markAndCleanLSP() error {
 		return true
 	})
 
+	klog.V(4).Infof("finish to gc logical switch ports")
 	return nil
 }
 
@@ -640,6 +650,7 @@ func (c *Controller) gcLoadBalancer() error {
 			klog.Errorf("delete all load balancers: %v", err)
 			return err
 		}
+		klog.Infof("finish to gc load balancers")
 		return nil
 	}
 
@@ -770,6 +781,7 @@ func (c *Controller) gcLoadBalancer() error {
 		klog.Errorf("delete load balancers: %v", err)
 		return err
 	}
+	klog.Infof("finish to gc load balancers")
 	return nil
 }
 
@@ -804,6 +816,7 @@ func (c *Controller) gcAddressSet() error {
 		return err
 	}
 
+	klog.Infof("finish to gc address set")
 	return nil
 }
 
@@ -848,6 +861,7 @@ func (c *Controller) gcSecurityGroup() error {
 		klog.Errorf("failed to gc port group list,%v", err)
 		return err
 	}
+	klog.Infof("finish to gc security group residual port groups")
 	return nil
 }
 
@@ -932,6 +946,7 @@ func (c *Controller) gcNetworkPolicy() error {
 		return err
 	}
 
+	klog.Infof("finish to gc network policy")
 	return nil
 }
 
@@ -971,6 +986,7 @@ func (c *Controller) gcRoutePolicy() error {
 		}
 	}
 
+	klog.Infof("finish to gc route policy")
 	return nil
 }
 
@@ -1019,6 +1035,7 @@ func (c *Controller) gcStaticRoute() error {
 			}
 		}
 	}
+	klog.Infof("finish to gc static routes")
 	return nil
 }
 
@@ -1066,6 +1083,7 @@ func (c *Controller) gcChassis() error {
 		}
 	}
 
+	klog.Infof("finish to gc chassis")
 	return nil
 }
 
@@ -1186,6 +1204,7 @@ func (c *Controller) gcLbSvcPods() error {
 			}
 		}
 	}
+	klog.Infof("finish to gc lb svc pods")
 	return nil
 }
 
@@ -1253,6 +1272,7 @@ func (c *Controller) gcVPCDNS() error {
 			}
 		}
 	}
+	klog.Infof("finish to gc vpc dns")
 	return nil
 }