Remove harcoded VNI from nexthop when announcing/deleting a route (#57)

damyan · web-flow · commit 0d1d4308208b · 2025-07-16T17:32:38.000+02:00
diff --git a/internal/metalbond-speaker/service_controller.go b/internal/metalbond-speaker/service_controller.go
@@ -67,7 +67,7 @@ func (r *ServiceReconciler) delete(ctx context.Context, log logr.Logger, service
 	}
 	nextHop := metalbond.NextHop{
 		TargetAddress: netip.MustParseAddr(r.NodeAddress),
-		TargetVNI:     50,
+		TargetVNI:     uint32(r.VNI),
 		Type:          pb.NextHopType_STANDARD,
 	}
 	if err := r.MetalBond.WithdrawRoute(metalbond.VNI(r.VNI), dest, nextHop); err != nil {
@@ -102,7 +102,7 @@ func (r *ServiceReconciler) reconcile(ctx context.Context, log logr.Logger, serv
 	}
 	nextHop := metalbond.NextHop{
 		TargetAddress: netip.MustParseAddr(r.NodeAddress),
-		TargetVNI:     50,
+		TargetVNI:     uint32(r.VNI),
 		Type:          pb.NextHopType_STANDARD,
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -67,7 +67,7 @@ func (r *ServiceReconciler) delete(ctx context.Context, log logr.Logger, service`
`67`	`67`	`}`
`68`	`68`	`nextHop := metalbond.NextHop{`
`69`	`69`	`TargetAddress: netip.MustParseAddr(r.NodeAddress),`
`70`		`- TargetVNI: 50,`
	`70`	`+ TargetVNI: uint32(r.VNI),`
`71`	`71`	`Type: pb.NextHopType_STANDARD,`
`72`	`72`	`}`
`73`	`73`	`if err := r.MetalBond.WithdrawRoute(metalbond.VNI(r.VNI), dest, nextHop); err != nil {`
`@@ -102,7 +102,7 @@ func (r *ServiceReconciler) reconcile(ctx context.Context, log logr.Logger, serv`
`102`	`102`	`}`
`103`	`103`	`nextHop := metalbond.NextHop{`
`104`	`104`	`TargetAddress: netip.MustParseAddr(r.NodeAddress),`
`105`		`- TargetVNI: 50,`
	`105`	`+ TargetVNI: uint32(r.VNI),`
`106`	`106`	`Type: pb.NextHopType_STANDARD,`
`107`	`107`	`}`
`108`	`108`