Supported 'order' attribute to gslb binding resources

Signed-off-by: Rohit Myali <[email protected]>

citrixadc/resource_citrixadc_gslbservicegroup_gslbservicegroupmember_binding.go

@@ -82,6 +82,12 @@ func resourceCitrixAdcGslbservicegroup_gslbservicegroupmember_binding() *schema.
 				Computed: true,
 				ForceNew: true,
 			},
+			"order": {
+				Type:     schema.TypeInt,
+				Optional: true,
+				Computed: true,
+				ForceNew: true,
+			},
 		},
 	}
 }
@@ -121,6 +127,7 @@ func createGslbservicegroup_gslbservicegroupmember_bindingFunc(d *schema.Resourc
 		Siteprefix:       d.Get("siteprefix").(string),
 		State:            d.Get("state").(string),
 		Weight:           d.Get("weight").(int),
+		Order:            d.Get("order").(int),
 	}
 
 	_, err := client.AddResource("gslbservicegroup_gslbservicegroupmember_binding", bindingId, &gslbservicegroup_gslbservicegroupmember_binding)
@@ -209,7 +216,8 @@ func readGslbservicegroup_gslbservicegroupmember_bindingFunc(d *schema.ResourceD
 	d.Set("servicegroupname", data["servicegroupname"])
 	d.Set("siteprefix", data["siteprefix"])
 	d.Set("state", data["state"])
-	d.Set("weight", data["weight"])
+	setToInt("weight", d, data["weight"])
+	setToInt("order", d, data["order"])
 
 	return nil
 

citrixadc/resource_citrixadc_gslbvserver_gslbservice_binding.go

@@ -42,6 +42,12 @@ func resourceCitrixAdcGslbvserver_gslbservice_binding() *schema.Resource {
 				Computed: true,
 				ForceNew: true,
 			},
+			"order": {
+				Type:     schema.TypeInt,
+				Optional: true,
+				Computed: true,
+				ForceNew: true,
+			},
 		},
 	}
 }
@@ -58,6 +64,7 @@ func createGslbvserver_gslbservice_bindingFunc(d *schema.ResourceData, meta inte
 		Name:        d.Get("name").(string),
 		Servicename: d.Get("servicename").(string),
 		Weight:      d.Get("weight").(int),
+		Order:       d.Get("order").(int),
 	}
 
 	err := client.UpdateUnnamedResource(service.Gslbvserver_gslbservice_binding.Type(), &gslbvserver_gslbservice_binding)
@@ -131,6 +138,7 @@ func readGslbvserver_gslbservice_bindingFunc(d *schema.ResourceData, meta interf
 	d.Set("name", data["name"])
 	d.Set("servicename", data["servicename"])
 	setToInt("weight", d, data["weight"])
+	setToInt("order", d, data["order"])
 
 	return nil
 

citrixadc/resource_citrixadc_gslbvserver_gslbservicegroup_binding.go

@@ -30,6 +30,12 @@ func resourceCitrixAdcGslbvserver_gslbservicegroup_binding() *schema.Resource {
 				Required: true,
 				ForceNew: true,
 			},
+			"order": {
+				Type:     schema.TypeInt,
+				Optional: true,
+				Computed: true,
+				ForceNew: true,
+			},
 		},
 	}
 }
@@ -44,6 +50,7 @@ func createGslbvserver_gslbservicegroup_bindingFunc(d *schema.ResourceData, meta
 	gslbvserver_gslbservicegroup_binding := gslb.Gslbvservergslbservicegroupbinding{
 		Name:             d.Get("name").(string),
 		Servicegroupname: d.Get("servicegroupname").(string),
+		Order:            d.Get("order").(int),
 	}
 
 	err := client.UpdateUnnamedResource("gslbvserver_gslbservicegroup_binding", &gslbvserver_gslbservicegroup_binding)
@@ -115,6 +122,7 @@ func readGslbvserver_gslbservicegroup_bindingFunc(d *schema.ResourceData, meta i
 
 	d.Set("name", data["name"])
 	d.Set("servicegroupname", data["servicegroupname"])
+	setToInt("order", d, data["order"])
 
 	return nil
 

citrixadc/utils.go

@@ -53,16 +53,18 @@ func stringListToIntList(in []interface{}) []int {
 
 // Check if the attribute is int, if not convert to int and set the value
 func setToInt(attributeName string, d *schema.ResourceData, value interface{}) {
+	log.Printf("[DEBUG] netscaler-provider: In setToInt for attribute %s", attributeName)
+
 	var v int
 	var err error
 
 	switch valueTyped := value.(type) {
 	case int:
 		v = valueTyped
 	case string:
-		v, _ = strconv.Atoi(valueTyped)
+		v, err = strconv.Atoi(valueTyped)
 	case nil:
-		v = 0
+		return
 	default:
 		log.Printf("[DEBUG] got unexpected type %T for int", value)
 		return