add support for set community additive

add option to enable setting community and extcommunity additive
instead of replacing the list.

Signed-off-by: Ivo Gosemann <ivo.gosemann@sap.com>

Author: IvoGoman

api/core/v1alpha1/routingpolicy_types.go

@@ -195,6 +195,18 @@ type SetASPathReplace struct {
 	Replacement intstr.IntOrString `json:"replacement"`
 }
 
+// CommunityOptions defines whether communities are added to the route's existing
+// set (ADD) or replace it (REPLACE).
+// +kubebuilder:validation:Enum=ADD;REPLACE
+type CommunityOptions string
+
+const (
+	// CommunityOptionsAdd appends the communities to the route's existing set.
+	CommunityOptionsAdd CommunityOptions = "ADD"
+	// CommunityOptionsReplace overwrites the route's communities with the configured set.
+	CommunityOptionsReplace CommunityOptions = "REPLACE"
+)
+
 // SetCommunityAction defines the action to set BGP standard communities.
 type SetCommunityAction struct {
 	// Communities is the list of BGP standard communities to set.
@@ -203,6 +215,12 @@ type SetCommunityAction struct {
 	// +required
 	// +kubebuilder:validation:MinItems=1
 	Communities []string `json:"communities"`
+
+	// Options controls whether the communities are added to the route's existing
+	// communities (ADD) or replace them (REPLACE).
+	// +optional
+	// +kubebuilder:default=REPLACE
+	Options CommunityOptions `json:"options,omitempty"`
 }
 
 // SetExtCommunityAction defines the action to set BGP extended communities.
@@ -213,6 +231,12 @@ type SetExtCommunityAction struct {
 	// +required
 	// +kubebuilder:validation:MinItems=1
 	Communities []string `json:"communities"`
+
+	// Options controls whether the communities are added to the route's existing
+	// communities (ADD) or replace them (REPLACE).
+	// +optional
+	// +kubebuilder:default=REPLACE
+	Options CommunityOptions `json:"options,omitempty"`
 }
 
 // RoutingPolicyStatus defines the observed state of RoutingPolicy.

charts/network-operator/templates/crd/routingpolicies.networking.metal.ironcore.dev.yaml

@@ -23,9 +23,11 @@ spec:
             communities:
               - "65137:100"
               - "65137:200"
+            options: ADD
           setExtCommunity:
             communities:
               - "65137:100"
+            options: ADD
     - sequence: 20
       conditions:
         matchPrefixSet:

config/crd/bases/networking.metal.ironcore.dev_routingpolicies.yaml

@@ -2821,12 +2821,12 @@ func (p *Provider) EnsureRoutingPolicy(ctx context.Context, req *provider.Ensure
 
 		if stmt.Actions.BgpActions != nil {
 			if stmt.Actions.BgpActions.SetCommunity != nil {
-				if err := e.SetCommunities(stmt.Actions.BgpActions.SetCommunity.Communities); err != nil {
+				if err := e.SetCommunities(stmt.Actions.BgpActions.SetCommunity.Communities, stmt.Actions.BgpActions.SetCommunity.Options); err != nil {
 					return err
 				}
 			}
 			if stmt.Actions.BgpActions.SetExtCommunity != nil {
-				if err := e.SetExtCommunities(stmt.Actions.BgpActions.SetExtCommunity.Communities); err != nil {
+				if err := e.SetExtCommunities(stmt.Actions.BgpActions.SetExtCommunity.Communities, stmt.Actions.BgpActions.SetExtCommunity.Options); err != nil {
 					return err
 				}
 			}

config/samples/v1alpha1_routingpolicy.yaml

@@ -27,11 +27,13 @@ type RouteMapEntry struct {
 	Action    Action `json:"action"`
 	Order     int32  `json:"order"`
 	SrttItems struct {
+		Additive  AdminSt `json:"additive,omitempty"`
 		ItemItems struct {
 			ItemList gnmiext.List[ExtCommItem, *ExtCommItem] `json:"Item-list,omitzero"`
 		} `json:"item-items,omitzero"`
 	} `json:"srtt-items,omitzero"`
 	SregcommItems struct {
+		Additive   AdminSt `json:"additive,omitempty"`
 		NoCommAttr AdminSt `json:"noCommAttr"`
 		ItemItems  struct {
 			ItemList gnmiext.List[string, *CommItem] `json:"Item-list,omitzero"`
@@ -73,7 +75,7 @@ type RouteMapEntry struct {
 
 func (e *RouteMapEntry) Key() int32 { return e.Order }
 
-func (e *RouteMapEntry) SetCommunities(communities []string) error {
+func (e *RouteMapEntry) SetCommunities(communities []string, options v1alpha1.CommunityOptions) error {
 	for _, comm := range communities {
 		c, err := Community(comm)
 		if err != nil {
@@ -82,17 +84,23 @@ func (e *RouteMapEntry) SetCommunities(communities []string) error {
 		e.SregcommItems.NoCommAttr = AdminStDisabled
 		e.SregcommItems.ItemItems.ItemList.Set(&CommItem{Community: c})
 	}
+	if options == v1alpha1.CommunityOptionsAdd {
+		e.SregcommItems.Additive = AdminStEnabled
+	}
 	return nil
 }
 
-func (e *RouteMapEntry) SetExtCommunities(communities []string) error {
+func (e *RouteMapEntry) SetExtCommunities(communities []string, options v1alpha1.CommunityOptions) error {
 	for _, comm := range communities {
 		c, err := RouteTarget(comm)
 		if err != nil {
 			return err
 		}
 		e.SrttItems.ItemItems.ItemList.Set(&ExtCommItem{Community: c, Scope: RtExtComScopeTransitive})
 	}
+	if options == v1alpha1.CommunityOptionsAdd {
+		e.SrttItems.Additive = AdminStEnabled
+	}
 	return nil
 }
 

internal/provider/cisco/nxos/provider.go

@@ -141,4 +141,28 @@ func init() {
 	comboRM.Name = "RM-COMBINED-MATCH"
 	comboRM.EntItems.EntryList.Set(comboEntry)
 	Register("route_map_combined_match", comboRM)
+
+	commAdditiveEntry := &RouteMapEntry{}
+	commAdditiveEntry.Order = 10
+	commAdditiveEntry.Action = ActionPermit
+	if err := commAdditiveEntry.SetCommunities([]string{"65000:10", "65000:20"}, v1alpha1.CommunityOptionsAdd); err != nil {
+		panic(err)
+	}
+
+	commAdditiveRM := &RouteMap{}
+	commAdditiveRM.Name = "RM-COMMUNITY-ADDITIVE"
+	commAdditiveRM.EntItems.EntryList.Set(commAdditiveEntry)
+	Register("route_map_community_additive", commAdditiveRM)
+
+	extCommAdditiveEntry := &RouteMapEntry{}
+	extCommAdditiveEntry.Order = 10
+	extCommAdditiveEntry.Action = ActionPermit
+	if err := extCommAdditiveEntry.SetExtCommunities([]string{"65000:10", "65000:20"}, v1alpha1.CommunityOptionsAdd); err != nil {
+		panic(err)
+	}
+
+	extCommAdditiveRM := &RouteMap{}
+	extCommAdditiveRM.Name = "RM-EXTCOMMUNITY-ADDITIVE"
+	extCommAdditiveRM.EntItems.EntryList.Set(extCommAdditiveEntry)
+	Register("route_map_extcommunity_additive", extCommAdditiveRM)
 }

internal/provider/cisco/nxos/routemap.go

@@ -0,0 +1,33 @@
+{
+  "rpm-items": {
+    "rtmap-items": {
+      "Rule-list": [
+        {
+          "name": "RM-COMMUNITY-ADDITIVE",
+          "ent-items": {
+            "Entry-list": [
+              {
+                "action": "permit",
+                "order": 10,
+                "sregcomm-items": {
+                  "additive": "enabled",
+                  "noCommAttr": "disabled",
+                  "item-items": {
+                    "Item-list": [
+                      {
+                        "community": "regular:as2-nn2:65000:10"
+                      },
+                      {
+                        "community": "regular:as2-nn2:65000:20"
+                      }
+                    ]
+                  }
+                }
+              }
+            ]
+          }
+        }
+      ]
+    }
+  }
+}

internal/provider/cisco/nxos/routemap_test.go

@@ -0,0 +1,2 @@
+route-map RM-COMMUNITY-ADDITIVE permit 10
+ set community 65000:10 65000:20 additive

internal/provider/cisco/nxos/testdata/route_map_community_additive.json

@@ -0,0 +1,34 @@
+{
+  "rpm-items": {
+    "rtmap-items": {
+      "Rule-list": [
+        {
+          "name": "RM-EXTCOMMUNITY-ADDITIVE",
+          "ent-items": {
+            "Entry-list": [
+              {
+                "action": "permit",
+                "order": 10,
+                "srtt-items": {
+                  "additive": "enabled",
+                  "item-items": {
+                    "Item-list": [
+                      {
+                        "community": "route-target:as2-nn2:65000:10",
+                        "scope": "transitive"
+                      },
+                      {
+                        "community": "route-target:as2-nn2:65000:20",
+                        "scope": "transitive"
+                      }
+                    ]
+                  }
+                }
+              }
+            ]
+          }
+        }
+      ]
+    }
+  }
+}