feat: add cidrlist parameter to loadbalancer rule

Author: chrxmvtik

cloudstack/resource_cloudstack_loadbalancer_rule.go

@@ -97,6 +97,15 @@ func resourceCloudStackLoadBalancerRule() *schema.Resource {
 				Set:      schema.HashString,
 			},
 
+			"cidrlist": {
+				Type:     schema.TypeSet,
+				Optional: true,
+				Computed: true,
+				ForceNew: true,
+				Elem:     &schema.Schema{Type: schema.TypeString},
+				Set:      schema.HashString,
+			},
+
 			"project": {
 				Type:     schema.TypeString,
 				Optional: true,
@@ -142,6 +151,13 @@ func resourceCloudStackLoadBalancerRuleCreate(d *schema.ResourceData, meta inter
 	if protocol, ok := d.GetOk("protocol"); ok {
 		p.SetProtocol(protocol.(string))
 	}
+	//TEST
+	var cidrList []string
+	for _, id := range d.Get("cidrlist").(*schema.Set).List() {
+		cidrList = append(cidrList, id.(string))
+	}
+
+	p.SetCidrlist(cidrList)
 
 	// Set the ipaddress id
 	p.SetPublicipid(d.Get("ip_address_id").(string))
@@ -216,6 +232,11 @@ func resourceCloudStackLoadBalancerRuleRead(d *schema.ResourceData, meta interfa
 	d.Set("private_port", private_port)
 	d.Set("protocol", lb.Protocol)
 
+	// Only set cidr if user specified it to avoid spurious diffs
+	if _, ok := d.GetOk("cidrlist"); ok {
+		d.Set("cidrlist", strings.Split(lb.Cidrlist, ","))
+	}
+
 	// Only set network if user specified it to avoid spurious diffs
 	if _, ok := d.GetOk("network_id"); ok {
 		d.Set("network_id", lb.Networkid)

cloudstack/resource_cloudstack_loadbalancer_rule_test.go

@@ -112,6 +112,8 @@ func TestAccCloudStackLoadBalancerRule_forceNew(t *testing.T) {
 						"cloudstack_loadbalancer_rule.foo", "public_port", "80"),
 					resource.TestCheckResourceAttr(
 						"cloudstack_loadbalancer_rule.foo", "private_port", "80"),
+					resource.TestCheckResourceAttr(
+						"cloudstack_loadbalancer_rule.foo", "cidrlist.0", "10.0.0.0/8"),
 				),
 			},
 
@@ -129,6 +131,8 @@ func TestAccCloudStackLoadBalancerRule_forceNew(t *testing.T) {
 						"cloudstack_loadbalancer_rule.foo", "private_port", "443"),
 					resource.TestCheckResourceAttr(
 						"cloudstack_loadbalancer_rule.foo", "protocol", "tcp-proxy"),
+					resource.TestCheckResourceAttr(
+						"cloudstack_loadbalancer_rule.foo", "cidrlist.0", "20.0.0.0/8"),
 				),
 			},
 		},
@@ -177,6 +181,8 @@ func TestAccCloudStackLoadBalancerRule_vpcUpdate(t *testing.T) {
 						"cloudstack_loadbalancer_rule.foo", "public_port", "80"),
 					resource.TestCheckResourceAttr(
 						"cloudstack_loadbalancer_rule.foo", "private_port", "80"),
+					resource.TestCheckResourceAttr(
+						"cloudstack_loadbalancer_rule.foo", "cidrlist.0", "10.0.0.0/8"),
 				),
 			},
 
@@ -192,6 +198,10 @@ func TestAccCloudStackLoadBalancerRule_vpcUpdate(t *testing.T) {
 						"cloudstack_loadbalancer_rule.foo", "public_port", "443"),
 					resource.TestCheckResourceAttr(
 						"cloudstack_loadbalancer_rule.foo", "private_port", "443"),
+					resource.TestCheckResourceAttr(
+						"cloudstack_loadbalancer_rule.foo", "cidrlist.0", "20.0.0.0/8"),
+					resource.TestCheckResourceAttr(
+						"cloudstack_loadbalancer_rule.foo", "cidrlist.1", "30.0.0.0/8"),
 				),
 			},
 		},
@@ -290,6 +300,7 @@ resource "cloudstack_loadbalancer_rule" "foo" {
   public_port = 80
   private_port = 80
   member_ids = [cloudstack_instance.foobar1.id]
+  cidrlist = ["10.0.0.0/8"]
 }`
 
 const testAccCloudStackLoadBalancerRule_update = `
@@ -323,6 +334,7 @@ resource "cloudstack_loadbalancer_rule" "foo" {
   public_port = 80
   private_port = 80
   member_ids = [cloudstack_instance.foobar1.id]
+  cidrlist = ["10.0.0.0/8"]
 }`
 
 const testAccCloudStackLoadBalancerRule_forcenew = `
@@ -357,6 +369,7 @@ resource "cloudstack_loadbalancer_rule" "foo" {
   private_port = 443
   protocol = "tcp-proxy"
   member_ids = [cloudstack_instance.foobar1.id]
+  cidrlist = ["20.0.0.0/8"]
 }`
 
 const testAccCloudStackLoadBalancerRule_vpc = `
@@ -399,6 +412,7 @@ resource "cloudstack_loadbalancer_rule" "foo" {
   public_port = 80
   private_port = 80
   member_ids = [cloudstack_instance.foobar1.id]
+  cidrlist = ["10.0.0.0/8"]
 }`
 
 const testAccCloudStackLoadBalancerRule_vpc_update = `
@@ -451,4 +465,5 @@ resource "cloudstack_loadbalancer_rule" "foo" {
   public_port = 443
   private_port = 443
   member_ids = [cloudstack_instance.foobar1.id, cloudstack_instance.foobar2.id]
+  cidrlist = ["20.0.0.0/8"]
 }`