Skip to content

Commit 0d591b4

Browse files
authored
Merge pull request #676 from ovh/dev/aamstutz/fix-dedicated-install-bug
fix: resource DedicatedServerInstallTask: don't retry task if creation failed
2 parents d4dffe4 + fe0c426 commit 0d591b4

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

ovh/resource_dedicated_server_install_task.go

+5
Original file line numberDiff line numberDiff line change
@@ -189,6 +189,11 @@ func resourceDedicatedServerInstallTaskCreate(d *schema.ResourceData, meta inter
189189
task := &DedicatedServerTask{}
190190

191191
if err := config.OVHClient.Post(endpoint, opts, task); err != nil {
192+
// If task was not created because of an error, return it immediately.
193+
if task != nil && task.Id == 0 {
194+
return fmt.Errorf("failed to create install task: %w", err)
195+
}
196+
192197
// POST on install tasks can fail randomly so in order to avoid issues, let's allow
193198
// a retry via waitForDedicatedServerTask
194199
log.Printf("[WARN] Ignored error when calling POST %s: %v", endpoint, err)

0 commit comments

Comments
 (0)