fix: increase default timeout to 20 minutes

Author: kangasta

.web-docs/components/builder/upcloud/README.md

@@ -55,7 +55,7 @@ The upcloud builder is used to generate storage templates on UpCloud.
 - `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`.
+- `state_timeout_duration` (duration string | ex: "1h5m2s") - The amount of time to wait for resource state changes. Defaults to `20m`.
 
 - `boot_wait` (duration string | ex: "1h5m2s") - The amount of time to wait after booting the server. Defaults to '0s'
 

CHANGELOG.md

@@ -5,6 +5,10 @@ See updating [Changelog example here](https://keepachangelog.com/en/1.0.0/)
 
 ## [Unreleased]
 
+### Changed
+
+- Increase default timeout to `20m`.
+
 ## [1.9.1] - 2025-08-29
 
 ### Fixed

builder/upcloud/config.go

@@ -26,7 +26,7 @@ const (
 	DefaultSSHUsername                    = "root"
 	DefaultCommunicator                   = "ssh"
 	DefaultStorageSize                    = 25
-	DefaultTimeout                        = 5 * time.Minute
+	DefaultTimeout                        = 20 * time.Minute
 	DefaultStorageTier                    = upcloud.StorageTierMaxIOPS
 	InterfaceTypePublic     InterfaceType = upcloud.IPAddressAccessPublic
 	InterfaceTypeUtility    InterfaceType = upcloud.IPAddressAccessUtility
@@ -103,7 +103,7 @@ type Config struct {
 	// 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`.
+	// The amount of time to wait for resource state changes. Defaults to `20m`.
 	Timeout time.Duration `mapstructure:"state_timeout_duration"`
 
 	// The amount of time to wait after booting the server. Defaults to '0s'

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

@@ -25,7 +25,7 @@
 - `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`.
+- `state_timeout_duration` (duration string | ex: "1h5m2s") - The amount of time to wait for resource state changes. Defaults to `20m`.
 
 - `boot_wait` (duration string | ex: "1h5m2s") - The amount of time to wait after booting the server. Defaults to '0s'
 

post-processor/upcloud-import/post-processor_acc_test.go

@@ -43,6 +43,7 @@ func TestPostProcessorAcc_raw(t *testing.T) {
 	imageFile, err := os.CreateTemp(os.TempDir(), fmt.Sprintf("%s-*.raw", testName))
 	require.NoError(t, err)
 	defer func() {
+		imageFile.Close()
 		if err := os.Remove(imageFile.Name()); err != nil {
 			t.Logf("Warning: failed to remove temp file: %v", err)
 		}