Merge pull request vmware-tanzu#495 from TaoZou1/srpointer

Fix bug not using pointer
seanpang-vmware · Jan 22, 2024 · 5b2b440 · 5b2b440
2 parents e7fecda + 1349953
commit 5b2b440
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/pkg/controllers/staticroute/staticroute_controller.go b/pkg/controllers/staticroute/staticroute_controller.go
@@ -239,7 +239,7 @@ func (r *StaticRouteReconciler) GarbageCollector(cancel chan bool, timeout time.
 
 		for _, e := range nsxStaticRouteList {
 			elem := e
-			UID := r.Service.GetUID(&elem)
+			UID := r.Service.GetUID(elem)
 			if UID == nil {
 				continue
 			}

diff --git a/pkg/nsx/services/staticroute/staticroute.go b/pkg/nsx/services/staticroute/staticroute.go
@@ -137,11 +137,11 @@ func (service *StaticRouteService) DeleteStaticRoute(namespace string, uid strin
 	return service.DeleteStaticRouteByPath(path[2], path[4], *vpc[0].Id, uid)
 }
 
-func (service *StaticRouteService) ListStaticRoute() []model.StaticRoutes {
+func (service *StaticRouteService) ListStaticRoute() []*model.StaticRoutes {
 	staticRoutes := service.StaticRouteStore.List()
-	staticRouteSet := []model.StaticRoutes{}
+	staticRouteSet := []*model.StaticRoutes{}
 	for _, staticroute := range staticRoutes {
-		staticRouteSet = append(staticRouteSet, staticroute.(model.StaticRoutes))
+		staticRouteSet = append(staticRouteSet, staticroute.(*model.StaticRoutes))
 	}
 	return staticRouteSet
 }