Skip to content

Fix SKE cluster maintenance updated outside TF causing error#444

Merged
joaopalet merged 3 commits intomainfrom
jp/fix-ske-cluster-maintenance-updated-outside-tf
Jul 2, 2024
Merged

Fix SKE cluster maintenance updated outside TF causing error#444
joaopalet merged 3 commits intomainfrom
jp/fix-ske-cluster-maintenance-updated-outside-tf

Conversation

@joaopalet
Copy link
Copy Markdown
Contributor

@joaopalet joaopalet commented Jul 1, 2024

  • The API response always contains a mock date part in front of the time, so in case these are the same we just keep the current value in state (this part hasn't changed)
  • What I changed was that now if the API response is different (not only in format but also in the actual time), this takes precedent over the state value as usual (instead of erroring)

Comment thread stackit/internal/services/ske/cluster/resource.go Outdated
@joaopalet joaopalet merged commit 7e51a0a into main Jul 2, 2024
@joaopalet joaopalet deleted the jp/fix-ske-cluster-maintenance-updated-outside-tf branch July 2, 2024 08:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants