fix(kubernetes): ignore first 404 responses when waiting cluster state (#267)

Author: kangasta

CHANGELOG.md

@@ -5,6 +5,9 @@ See updating [Changelog example here](https://keepachangelog.com/en/1.0.0/)
 
 ## [Unreleased]
 
+### Changed
+- kubernetes: `WaitForKubernetesClusterState` ignores two first 404 responses to avoid errors caused by possible false _not found_ responses right after cluster has been created.
+
 ## [6.8.0]
 - Managed Object Storage support
 

upcloud/service/kubernetes.go

@@ -2,7 +2,10 @@ package service
 
 import (
 	"context"
+	"errors"
 	"fmt"
+	"log"
+	"net/http"
 	"time"
 
 	"github.com/UpCloudLtd/upcloud-go-api/v6/upcloud"
@@ -83,7 +86,13 @@ func (s *Service) WaitForKubernetesClusterState(ctx context.Context, r *request.
 			UUID: r.UUID,
 		})
 		if err != nil {
-			return nil, err
+			// Ignore first two 404 responses to avoid errors caused by possible false NOT_FOUND responses right after cluster has been created.
+			var ucErr *upcloud.Problem
+			if errors.As(err, &ucErr) && ucErr.Status == http.StatusNotFound && attempts < 3 {
+				log.Printf("ERROR: %+v", err)
+			} else {
+				return nil, err
+			}
 		}
 
 		if details.State == r.DesiredState {