Remove the control for aggregation length v6 in native mode (#1220)

In the operator crds we default the value to 128, so running the
operator in native mode will fail when we have advertisements. We
still check for not having v6 addresses in native mode, so the
users will still know if something is not working.

Signed-off-by: Federico Paolinelli <fpaoline@redhat.com>

internal/config/validate.go

@@ -25,11 +25,6 @@ func DiscardFRROnly(c *configFile) error {
 		return errors.New("bfd profiles section set")
 	}
 	for _, p := range c.Pools {
-		for _, a := range p.BGPAdvertisements {
-			if a.AggregationLengthV6 != nil {
-				return fmt.Errorf("pool %s has aggregation-lenght-v6 set on native bgp mode", p.Name)
-			}
-		}
 		if p.Protocol == BGP {
 			for _, cidr := range p.Addresses {
 				nets, err := ParseCIDR(cidr)

internal/config/validate_test.go

@@ -63,22 +63,6 @@ func TestValidate(t *testing.T) {
 			},
 			mustFail: true,
 		},
-		{
-			desc: "aggregation v6",
-			config: &configFile{
-				Pools: []addressPool{
-					{
-						Name: "foo",
-						BGPAdvertisements: []bgpAdvertisement{
-							{
-								AggregationLengthV6: intPtr(47),
-							},
-						},
-					},
-				},
-			},
-			mustFail: true,
-		},
 		{
 			desc: "should pass",
 			config: &configFile{
@@ -243,7 +227,3 @@ func TestValidateFRR(t *testing.T) {
 		})
 	}
 }
-
-func intPtr(i int) *int {
-	return &i
-}