Merge pull request #208 from Nordix/reconciler-race

ip-reconciler: fix for duplicate IP allocation due
k8snetworkplumbingwg · May 19, 2022 · 1c2cd40 · 1c2cd40
2 parents 06d3340 + 47e5b17
commit 1c2cd40
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/pkg/reconciler/iploop.go b/pkg/reconciler/iploop.go
@@ -48,14 +48,14 @@ func NewReconcileLooper(ctx context.Context, timeout int) (*ReconcileLooper, err
 }
 
 func newReconcileLooper(ctx context.Context, k8sClient *kubernetes.Client, timeout int) (*ReconcileLooper, error) {
-	pods, err := k8sClient.ListPods(ctx)
+	ipPools, err := k8sClient.ListIPPools(ctx)
 	if err != nil {
-		return nil, err
+		return nil, logging.Errorf("failed to retrieve all IP pools: %v", err)
 	}
 
-	ipPools, err := k8sClient.ListIPPools(ctx)
+	pods, err := k8sClient.ListPods(ctx)
 	if err != nil {
-		return nil, logging.Errorf("failed to retrieve all IP pools: %v", err)
+		return nil, err
 	}
 
 	whereaboutsPodRefs := getPodRefsServedByWhereabouts(ipPools)