Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: add support for the new Cluster State enum that contains INVALID_CREDENTIALS #203

Merged
merged 2 commits into from
Oct 3, 2023

Conversation

pggb25
Copy link
Collaborator

@pggb25 pggb25 commented Oct 3, 2023

No description provided.

@pggb25 pggb25 added the enhancement New feature or request label Oct 3, 2023
@pggb25 pggb25 changed the title fix: add support for the new Cluster State enum fix: add support for the new Cluster State enum that contains INVALID_CREDENTIALS Oct 3, 2023
@@ -1402,6 +1424,13 @@ func IsTerminalState(state qovery.StateEnum) bool {
strings.HasSuffix(string(state), "ERROR")
}

func IsTerminalClusterState(state qovery.ClusterStateEnum) bool {
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe having invalid credentials should be considered as a terminal state ?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes I didn't see the strings.HasSuffix(string(state), "ERROR")

@pggb25 pggb25 merged commit e0f590f into main Oct 3, 2023
5 checks passed
@pggb25 pggb25 deleted the feat/remoce-aut-deploy-from-deployment-rule branch October 3, 2023 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants