prohibit usage of vni zero

telekom · Mar 15, 2024 · bfdc62a · bfdc62a
1 parent e000db3
commit bfdc62a
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 0 deletions.
diff --git a/api/v1alpha1/layer2networkconfiguration_types.go b/api/v1alpha1/layer2networkconfiguration_types.go
@@ -37,6 +37,7 @@ type Layer2NetworkConfigurationSpec struct {
 	MTU int `json:"mtu"`
 
 	// +kubebuilder:validation:Required
+	// +kubebuilder:validation:Minimum=1
 	// VXLAN VNI Id for the layer 2 network
 	VNI int `json:"vni"`
 

diff --git a/config/crd/bases/network.schiff.telekom.de_layer2networkconfigurations.yaml b/config/crd/bases/network.schiff.telekom.de_layer2networkconfigurations.yaml
@@ -126,6 +126,7 @@ spec:
                 x-kubernetes-map-type: atomic
               vni:
                 description: VXLAN VNI Id for the layer 2 network
+                minimum: 1
                 type: integer
               vrf:
                 description: VRF to attach Layer2 network to, default if not set

diff --git a/pkg/reconciler/layer3.go b/pkg/reconciler/layer3.go
@@ -165,6 +165,12 @@ func (r *reconcile) createVrfConfigMap(l3vnis []networkv1alpha1.VRFRouteConfigur
 			continue
 		}
 
+		if vni == 0 {
+			err := fmt.Errorf("VNI can not be set to 0")
+			r.Logger.Error(err, "VNI can not be set to 0, ignoring", "vrf", spec.VRF, "name", l3vnis[i].ObjectMeta.Name, "namespace", l3vnis[i].ObjectMeta.Namespace)
+			continue
+		}
+
 		cfg, err := createVrfConfig(vrfConfigMap, &spec, vni, rt)
 		if err != nil {
 			return nil, err