Introduce option to select storage tiers

ppc64le-cloud · Feb 3, 2025 · 813b312 · 813b312
1 parent cc48223
commit 813b312
Show file tree

Hide file tree

Showing 5 changed files with 11 additions and 2 deletions.
diff --git a/data/powervs/instance/variables.tf b/data/powervs/instance/variables.tf
@@ -51,8 +51,7 @@ variable "system_type" {
 }
 
 variable "storage_tier" {
-    description = "I/O operation per second (IOPS) based storage on requirement - tier1, tier3 "
-    default     = "tier1"
+    description = "I/O operation per second (IOPS) based storage on requirement - tier0, tier1, tier3 or tier5k"
 }
 
 variable "image_name" {

diff --git a/data/powervs/main.tf b/data/powervs/main.tf
@@ -17,6 +17,7 @@ module "master" {
 
   ibmcloud_api_key = var.powervs_api_key
   image_name = var.powervs_image_name
+  storage_tier = var.powervs_storage_tier
   memory = var.controlplane_powervs_memory
   network = var.powervs_network_name == "" ? ibm_pi_network.public_network[0].network_id : data.ibm_pi_network.existing_net[0].id
   powervs_service_instance_id = var.powervs_service_id
@@ -33,6 +34,7 @@ module "workers" {
 
   ibmcloud_api_key = var.powervs_api_key
   image_name = var.powervs_image_name
+  storage_tier = var.powervs_storage_tier
   memory = var.powervs_memory
   network = var.powervs_network_name == "" ? ibm_pi_network.public_network[0].network_id : data.ibm_pi_network.existing_net[0].id
   powervs_service_instance_id = var.powervs_service_id

diff --git a/data/powervs/variables-powervs.tf b/data/powervs/variables-powervs.tf
@@ -19,6 +19,10 @@ variable "powervs_image_name" {
   description = "PowerVS image name to be used for the deployment"
 }
 
+variable "powervs_storage_tier" {
+  description = "PowerVS storage-tier to be used for volumes"
+}
+
 # By design, only the worker node's CPU/Memory can be sized through the flags passed as arguments.
 variable "powervs_memory" {
   description = "Worker node's PowerVS memory in GB"

diff --git a/pkg/providers/powervs/powervs.go b/pkg/providers/powervs/powervs.go
@@ -55,6 +55,9 @@ func (p *Provider) BindFlags(flags *pflag.FlagSet) {
 	flags.StringVar(
 		&p.ImageName, "powervs-image-name", "", "Image ID(command: ibmcloud pi imgs)",
 	)
+	flags.StringVar(
+		&p.StorageTier, "powervs-storage-tier", "tier1", "Storage tier for backing volume[tier0, tier1, tier3 or tier5k]",
+	)
 	flags.Float64Var(
 		&p.Memory, "powervs-memory", 8, "Memory in GBs",
 	)

diff --git a/pkg/tfvars/powervs/powervs.go b/pkg/tfvars/powervs/powervs.go
@@ -10,6 +10,7 @@ type TFVars struct {
 	ServiceID     string  `json:"powervs_service_id"`
 	NetworkName   string  `json:"powervs_network_name"`
 	ImageName     string  `json:"powervs_image_name"`
+	StorageTier   string  `json:"powervs_storage_tier"`
 	Memory        float64 `json:"powervs_memory"`
 	Processors    float64 `json:"powervs_processors"`
 	SSHKey        string  `json:"powervs_ssh_key"`