feat: make storage tier configurable (#66)

Users can now choose between 'maxiops', 'standard', and 'archive' storage
tiers in both builder and post-processor.

Signed-off-by: Ville Vesilehto <ville.vesilehto@upcloud.com>

Author: thevilledev

CHANGELOG.md

@@ -5,6 +5,10 @@ See updating [Changelog example here](https://keepachangelog.com/en/1.0.0/)
 
 ## [Unreleased]
 
+### Added
+
+- Enable storage tier customisation through `storage_tier` parameter
+
 ### Fixed
 
 - Update Go version to 1.24 and fix security vulnerabilities

builder/upcloud/builder_acc_test.go

@@ -31,6 +31,9 @@ var testBuilderStorageName string
 //go:embed test-fixtures/json/networking.json
 var testBuilderNetworking string
 
+//go:embed test-fixtures/json/basic_standard_tier.json
+var testBuilderBasicStandardTier string
+
 func TestBuilderAcc_default(t *testing.T) {
 	testAccPreCheck(t)
 
@@ -65,6 +68,17 @@ func TestBuilderAcc_storageName(t *testing.T) {
 	acctest.TestPlugin(t, testCase)
 }
 
+func TestBuilderAcc_standardTier(t *testing.T) {
+	testAccPreCheck(t)
+	testCase := &acctest.PluginTestCase{
+		Name:     t.Name(),
+		Template: testBuilderBasicStandardTier,
+		Check:    checkTestResult(),
+		Teardown: teardown(t.Name()),
+	}
+	acctest.TestPlugin(t, testCase)
+}
+
 func TestBuilderAcc_networking(t *testing.T) {
 	testAccPreCheck(t)
 	testCase := &acctest.PluginTestCase{

builder/upcloud/config.go

@@ -24,6 +24,7 @@ const (
 	DefaultCommunicator                 = "ssh"
 	DefaultStorageSize                  = 25
 	DefaultTimeout                      = 5 * time.Minute
+	DefaultStorageTier                  = upcloud.StorageTierMaxIOPS
 	InterfaceTypePublic   InterfaceType = upcloud.IPAddressAccessPublic
 	InterfaceTypeUtility  InterfaceType = upcloud.IPAddressAccessUtility
 	InterfaceTypePrivate  InterfaceType = upcloud.IPAddressAccessPrivate
@@ -103,6 +104,10 @@ type Config struct {
 	// The operating system disk can also be later extended if needed. Note that Windows templates require large storage size, than default 25 Gb.
 	StorageSize int `mapstructure:"storage_size"`
 
+	// The storage tier to use. Available options are `maxiops`, `archive`, and `standard`. Defaults to `maxiops`.
+	// For most production workloads, MaxIOPS is recommended for best performance.
+	StorageTier string `mapstructure:"storage_tier"`
+
 	// The amount of time to wait for resource state changes. Defaults to `5m`.
 	Timeout time.Duration `mapstructure:"state_timeout_duration"`
 
@@ -158,6 +163,10 @@ func (c *Config) Prepare(raws ...interface{}) ([]string, error) {
 		c.StorageSize = DefaultStorageSize
 	}
 
+	if c.StorageTier == "" {
+		c.StorageTier = DefaultStorageTier
+	}
+
 	if c.Timeout == 0 {
 		c.Timeout = DefaultTimeout
 	}

builder/upcloud/config.hcl2spec.go

@@ -48,6 +48,7 @@ func (s *StepCreateServer) Run(ctx context.Context, state multistep.StateBag) mu
 		Zone:         s.Config.Zone,
 		SshPublicKey: sshKeyPublic,
 		Networking:   networking,
+		StorageTier:  s.Config.StorageTier,
 	})
 	if err != nil {
 		return stepHaltWithError(state, err)

builder/upcloud/step_create_server.go

@@ -0,0 +1,9 @@
+source "upcloud" "standard-tier" {
+  storage_uuid = "01000000-0000-4000-8000-000150020100" # Rocky Linux 9
+  zone         = "nl-ams1"
+  storage_tier = "standard"
+}
+
+build {
+  sources = ["source.upcloud.standard-tier"]
+} 

builder/upcloud/test-fixtures/hcl2/basic_standard_tier.hcl

@@ -0,0 +1,8 @@
+{
+    "builders": [{
+        "type": "upcloud",
+        "zone": "nl-ams1",
+        "storage_uuid": "01000000-0000-4000-8000-000150020100",
+        "storage_tier": "standard"
+    }]
+}

builder/upcloud/test-fixtures/json/basic_standard_tier.json

@@ -16,6 +16,9 @@
   Changing this value is useful if you aim to build a template for larger server configurations where the preconfigured server disk is larger than 25 GB.
   The operating system disk can also be later extended if needed. Note that Windows templates require large storage size, than default 25 Gb.
 
+- `storage_tier` (string) - The storage tier to use. Available options are `maxiops`, `archive`, and `standard`. Defaults to `maxiops`.
+  For most production workloads, MaxIOPS is recommended for best performance.
+
 - `state_timeout_duration` (duration string | ex: "1h5m2s") - The amount of time to wait for resource state changes. Defaults to `5m`.
 
 - `boot_wait` (duration string | ex: "1h5m2s") - The amount of time to wait after booting the server. Defaults to '0s'

docs-partials/builder/upcloud/Config-not-required.mdx

@@ -32,6 +32,8 @@ source "upcloud" "test" {
   zone            = "nl-ams1"
   storage_name    = "ubuntu server 20.04"
   template_prefix = "ubuntu-server"
+  # uncomment to use standard tier storage
+  # storage_tier = "standard
 }
 
 build {

example/build.pkr.hcl

@@ -35,6 +35,7 @@ build {
       username         = "${var.username}"
       password         = "${var.password}"
       zones            = ["pl-waw1", "fi-hel2"]
+      storage_tier     = "maxiops"
     }
   }
 }