serviceoffering: add params for custom offering, storage tags,

encryptroot

Fixes #182

Adds params for Custom offerings. Constrained custom offerings allow
min/max values for CPU and memory. Also adds params for encrypt root and
storage tags.

New params added,
- customized
- min_cpu_number
- max_cpu_number
- min_memory
- max_memory
- encrypt_root
- storage_tags

Signed-off-by: Abhishek Kumar <[email protected]>

Author: shwstppr

cloudstack/resource_cloudstack_service_offering.go

@@ -22,6 +22,7 @@ package cloudstack
 import (
 	"fmt"
 	"log"
+	"strconv"
 
 	"github.com/apache/cloudstack-go/v2/cloudstack"
 	"github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema"
@@ -97,6 +98,49 @@ func resourceCloudStackServiceOffering() *schema.Resource {
 					return
 				},
 			},
+			"customized": {
+				Description: "Whether service offering allows custom CPU/memory or not",
+				Type:        schema.TypeBool,
+				Optional:    true,
+				ForceNew:    true,
+				Default:     false,
+			},
+			"min_cpu_number": {
+				Description: "Minimum number of CPU cores allowed",
+				Type:        schema.TypeInt,
+				Optional:    true,
+				ForceNew:    true,
+			},
+			"max_cpu_number": {
+				Description: "Maximum number of CPU cores allowed",
+				Type:        schema.TypeInt,
+				Optional:    true,
+				ForceNew:    true,
+			},
+			"min_memory": {
+				Description: "Minimum memory allowed (MB)",
+				Type:        schema.TypeInt,
+				Optional:    true,
+				ForceNew:    true,
+			},
+			"max_memory": {
+				Description: "Maximum memory allowed (MB)",
+				Type:        schema.TypeInt,
+				Optional:    true,
+				ForceNew:    true,
+			},
+			"encrypt_root": {
+				Description: "Encrypt the root disk for VMs using this service offering",
+				Type:        schema.TypeBool,
+				Optional:    true,
+				ForceNew:    true,
+			},
+			"storage_tags": {
+				Description: "Storage tags to associate with the service offering",
+				Type:        schema.TypeString,
+				Optional:    true,
+				ForceNew:    true,
+			},
 		},
 	}
 }
@@ -136,6 +180,34 @@ func resourceCloudStackServiceOfferingCreate(d *schema.ResourceData, meta interf
 		p.SetStoragetype(v.(string))
 	}
 
+	if v, ok := d.GetOk("customized"); ok {
+		p.SetCustomized(v.(bool))
+	}
+
+	if v, ok := d.GetOk("min_cpu_number"); ok {
+		p.SetMincpunumber(v.(int))
+	}
+
+	if v, ok := d.GetOk("max_cpu_number"); ok {
+		p.SetMaxcpunumber(v.(int))
+	}
+
+	if v, ok := d.GetOk("min_memory"); ok {
+		p.SetMinmemory(v.(int))
+	}
+
+	if v, ok := d.GetOk("max_memory"); ok {
+		p.SetMaxmemory(v.(int))
+	}
+
+	if v, ok := d.GetOk("encrypt_root"); ok {
+		p.SetEncryptroot(v.(bool))
+	}
+
+	if v, ok := d.GetOk("storage_tags"); ok {
+		p.SetTags(v.(string))
+	}
+
 	log.Printf("[DEBUG] Creating Service Offering %s", name)
 	s, err := cs.ServiceOffering.CreateServiceOffering(p)
 
@@ -177,6 +249,53 @@ func resourceCloudStackServiceOfferingRead(d *schema.ResourceData, meta interfac
 		"memory":        s.Memory,
 		"offer_ha":      s.Offerha,
 		"storage_type":  s.Storagetype,
+		"customized":    s.Iscustomized,
+		"min_cpu_number": func() interface{} {
+			if s.Serviceofferingdetails == nil {
+				return nil
+			}
+			if v, ok := s.Serviceofferingdetails["mincpunumber"]; ok {
+				if i, err := strconv.Atoi(v); err == nil {
+					return i
+				}
+			}
+			return nil
+		}(),
+		"max_cpu_number": func() interface{} {
+			if s.Serviceofferingdetails == nil {
+				return nil
+			}
+			if v, ok := s.Serviceofferingdetails["maxcpunumber"]; ok {
+				if i, err := strconv.Atoi(v); err == nil {
+					return i
+				}
+			}
+			return nil
+		}(),
+		"min_memory": func() interface{} {
+			if s.Serviceofferingdetails == nil {
+				return nil
+			}
+			if v, ok := s.Serviceofferingdetails["minmemory"]; ok {
+				if i, err := strconv.Atoi(v); err == nil {
+					return i
+				}
+			}
+			return nil
+		}(),
+		"max_memory": func() interface{} {
+			if s.Serviceofferingdetails == nil {
+				return nil
+			}
+			if v, ok := s.Serviceofferingdetails["maxmemory"]; ok {
+				if i, err := strconv.Atoi(v); err == nil {
+					return i
+				}
+			}
+			return nil
+		}(),
+		"encrypt_root": s.Encryptroot,
+		"storage_tags": s.Storagetags,
 	}
 
 	for k, v := range fields {
@@ -247,6 +366,24 @@ func resourceCloudStackServiceOfferingUpdate(d *schema.ResourceData, meta interf
 
 	}
 
+	if d.HasChange("tags") {
+		log.Printf("[DEBUG] Tags changed for %s, starting update", name)
+
+		// Create a new parameter struct
+		p := cs.ServiceOffering.NewUpdateServiceOfferingParams(d.Id())
+
+		// Set the new tags
+		p.SetStoragetags(d.Get("tags").(string))
+
+		// Update the host tags
+		_, err := cs.ServiceOffering.UpdateServiceOffering(p)
+		if err != nil {
+			return fmt.Errorf(
+				"Error updating the storage tags for service offering %s: %s", name, err)
+		}
+
+	}
+
 	return resourceCloudStackServiceOfferingRead(d, meta)
 }
 

cloudstack/resource_cloudstack_service_offering_test.go

@@ -83,3 +83,42 @@ func testAccCheckCloudStackServiceOfferingExists(n string, so *cloudstack.Servic
 		return nil
 	}
 }
+
+func TestAccCloudStackServiceOffering_customized(t *testing.T) {
+	var so cloudstack.ServiceOffering
+	resource.Test(t, resource.TestCase{
+		PreCheck:  func() { testAccPreCheck(t) },
+		Providers: testAccProviders,
+		Steps: []resource.TestStep{
+			{
+				Config: testAccCloudStackServiceOffering_customized,
+				Check: resource.ComposeTestCheckFunc(
+					testAccCheckCloudStackServiceOfferingExists("cloudstack_service_offering.custom", &so),
+					resource.TestCheckResourceAttr("cloudstack_service_offering.custom", "customized", "true"),
+					resource.TestCheckResourceAttr("cloudstack_service_offering.custom", "min_cpu_number", "1"),
+					resource.TestCheckResourceAttr("cloudstack_service_offering.custom", "max_cpu_number", "8"),
+					resource.TestCheckResourceAttr("cloudstack_service_offering.custom", "min_memory", "1024"),
+					resource.TestCheckResourceAttr("cloudstack_service_offering.custom", "max_memory", "16384"),
+					resource.TestCheckResourceAttr("cloudstack_service_offering.custom", "cpu_speed", "1000"),
+					resource.TestCheckResourceAttr("cloudstack_service_offering.custom", "encrypt_root", "true"),
+					resource.TestCheckResourceAttr("cloudstack_service_offering.custom", "storage_tags", "production,ssd"),
+				),
+			},
+		},
+	})
+}
+
+const testAccCloudStackServiceOffering_customized = `
+resource "cloudstack_service_offering" "custom" {
+  name             = "custom_service_offering"
+  display_text     = "Custom Test"
+  customized       = true
+  min_cpu_number   = 1
+  max_cpu_number   = 8
+  min_memory       = 1024
+  max_memory       = 16384
+  cpu_speed        = 1000
+  encrypt_root     = true
+  storage_tags     = "production,ssd"
+}
+`

website/docs/r/service_offering.html.markdown

@@ -49,6 +49,26 @@ The following arguments are supported:
 * `storage_type` - (Optional) The storage type of the service offering. Values are `local` and `shared`.
     Changing this forces a new resource to be created.
 
+* `customized` - (Optional) Whether the service offering allows custom CPU and memory values. Set to `true` to enable users to specify CPU/memory within the min/max constraints for constrained offerings and any value for unconstrained offerings.
+    Changing this forces a new resource to be created.
+
+* `min_cpu_number` - (Optional) Minimum number of CPU cores allowed for customized offerings.
+    Changing this forces a new resource to be created.
+
+* `max_cpu_number` - (Optional) Maximum number of CPU cores allowed for customized offerings.
+    Changing this forces a new resource to be created.
+
+* `min_memory` - (Optional) Minimum memory (in MB) allowed for customized offerings.
+    Changing this forces a new resource to be created.
+
+* `max_memory` - (Optional) Maximum memory (in MB) allowed for customized offerings.
+    Changing this forces a new resource to be created.
+
+* `encrypt_root` - (Optional) Whether to encrypt the root disk for VMs using this service offering.
+    Changing this forces a new resource to be created.
+
+* `storage_tags` - (Optional) Storage tags to associate with the service offering.
+
 ## Attributes Reference
 
 The following attributes are exported: