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

[BUG] - QHub destroy command is too strict relax destroy to best effort #1108

Closed
costrouc opened this issue Feb 23, 2022 · 2 comments · Fixed by #1109
Closed

[BUG] - QHub destroy command is too strict relax destroy to best effort #1108

costrouc opened this issue Feb 23, 2022 · 2 comments · Fixed by #1109
Labels
type: bug 🐛 Something isn't working

Comments

@costrouc
Copy link
Member

OS system and architecture in which you are running QHub

Linux

Expected behavior

Currently QHub destroy fails to continue deletion if is fails deleting a later stage. For example Kubernetes resource deletion fails but due to this failure it stops and doesn't attempt to delete the infrastructure. Changing this behavior to make qhub destroy continue on and attempt to delete as much as it can.

Actual behavior

Stop at first error when destroying.

How to Reproduce the problem?

Run the qhub-integration-tests on aws

Command output

No response

Versions and dependencies used.

No response

Compute environment

No response

Integrations

No response

Anything else?

No response

@costrouc
Copy link
Member Author

There are plenty of examples of this:

Since there are 8 stages we attempt to delete 8, 7, 6, 5, 4, 3, 2, 1 in order. If a single stage fails to delete continue and delete the next one. In the case of AWS and all the cloud providers we really only need 4, 2, 1 to succeed. For a local deployment ideally 8-1 should all succeed.

@costrouc
Copy link
Member Author

When we have an extension mechanism in place this should also allow us to more resiliently delete stages.

costrouc added a commit that referenced this issue Feb 23, 2022
* Relax qhub destroy to attempt to continue destroying resources

Closes #1108

* Black formatting
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug 🐛 Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant