fix(server): use plans default storage tier when creating server (#318)

Author: kangasta

CHANGELOG.md

@@ -17,6 +17,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - In all outputs of `server plans`, sort plans by CPU count, memory amount, and storage size.
 - In human readable output of `server plans`, group plans by type.
 
+### Fixed
+
+- Omit storage tier from server create payload to use plans default storage tier. This allows creating servers with developer plans that do not allow creating MaxIOPS storages with the server. Other plan types will continue to use MaxIOPS by default.
+
 ## [3.8.1] - 2024-05-24
 
 ### Changed

internal/commands/server/create.go

@@ -102,7 +102,6 @@ func (s *createParams) processParams(exec commands.Executor) error {
 			Storage: osStorage.UUID,
 			Title:   fmt.Sprintf("%s-OS", ui.TruncateText(s.Hostname, 64-7)),
 			Size:    size,
-			Tier:    upcloud.StorageTierMaxIOPS,
 			Type:    upcloud.StorageTypeDisk,
 		})
 	}

internal/commands/server/create_test.go

@@ -118,7 +118,6 @@ func TestCreateServer(t *testing.T) {
 					Storage: StorageDef.UUID,
 					Title:   "example.com-OS",
 					Size:    50,
-					Tier:    upcloud.StorageTierMaxIOPS,
 					Type:    upcloud.StorageTypeDisk,
 				}},
 			},
@@ -148,7 +147,6 @@ func TestCreateServer(t *testing.T) {
 					Storage: Storage1.UUID,
 					Title:   "example.com-OS",
 					Size:    100,
-					Tier:    upcloud.StorageTierMaxIOPS,
 					Type:    upcloud.StorageTypeDisk,
 				}},
 			},
@@ -207,7 +205,6 @@ func TestCreateServer(t *testing.T) {
 					Storage: StorageDef.UUID,
 					Title:   "example.com-OS",
 					Size:    10,
-					Tier:    upcloud.StorageTierMaxIOPS,
 					Type:    upcloud.StorageTypeDisk,
 				}},
 				Labels: &upcloud.LabelSlice{upcloud.Label{
@@ -242,7 +239,6 @@ func TestCreateServer(t *testing.T) {
 					Storage: StorageDef.UUID,
 					Title:   "example.com-OS",
 					Size:    50,
-					Tier:    upcloud.StorageTierMaxIOPS,
 					Type:    upcloud.StorageTypeDisk,
 				}},
 				Labels: &upcloud.LabelSlice{
@@ -289,7 +285,6 @@ func TestCreateServer(t *testing.T) {
 						Storage:   StorageDef.UUID,
 						Title:     "example.com-OS",
 						Size:      50,
-						Tier:      upcloud.StorageTierMaxIOPS,
 						Type:      upcloud.StorageTypeDisk,
 					},
 					request.CreateServerStorageDevice{
@@ -346,7 +341,6 @@ func TestCreateServer(t *testing.T) {
 					Storage: StorageDef.UUID,
 					Title:   "example.com-OS",
 					Size:    50,
-					Tier:    upcloud.StorageTierMaxIOPS,
 					Type:    upcloud.StorageTypeDisk,
 				}},
 				Networking: &request.CreateServerNetworking{Interfaces: request.CreateServerInterfaceSlice{