Skip to content

Conversation

h3adex
Copy link
Contributor

@h3adex h3adex commented Sep 23, 2025

Description

This change aligns with the corresponding Terraform PR stackitcloud/terraform-provider-stackit#1010. It updates the CreateOrUpdateClusterWaitHandler in the ske/wait package to exit early when the cluster reports structured errors and is in a failure state. This prevents cluster creation or update operations from remaining stuck in an error state for up to 45 minutes until the Terraform provider times out.

The related Terraform logic is in the PR linked above. Before merging this PR, someone should also review the Terraform changes. Test SKE configurations are attached for verification.

Result warning trying to create a ske cluster using terraform with a non existent flavor for machines
Screenshot 2025-09-26 at 12 52 53

Checklist

  • Issue was linked above
  • No generated code was adjusted manually (check comments in file header)
  • Changelogs
    • Changelog in the root directory was adjusted (see here)
    • Changelog(s) of the service(s) were adjusted (see e.g. here)
  • VERSION file(s) of the service(s) were adjusted
  • Code format was applied: make fmt
  • Examples were added / adjusted (see examples/ directory)
  • Unit tests got implemented or updated
  • Unit tests are passing: make test (will be checked by CI)
  • No linter issues: make lint (will be checked by CI)

@h3adex h3adex requested a review from a team as a code owner September 23, 2025 16:09
@h3adex h3adex marked this pull request as draft September 23, 2025 16:09
@h3adex h3adex force-pushed the feat/return-ske-errors branch 5 times, most recently from 29a578c to e43005e Compare September 26, 2025 12:36
Copy link

github-actions bot commented Oct 4, 2025

This PR was marked as stale after 7 days of inactivity and will be closed after another 7 days of further inactivity. If this PR should be kept open, just add a comment, remove the stale label or push new commits to it.

@github-actions github-actions bot added the Stale label Oct 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant