8000 OpenStack: Fix VM deletion by Akrog · Pull Request #810 · karmab/kcli · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

OpenStack: Fix VM deletion #810

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

Merged
merged 1 commit into from
May 13, 2025
Merged

OpenStack: Fix VM deletion #810

merged 1 commit into from
May 13, 2025

Conversation

Akrog
Copy link
Contributor
@Akrog Akrog commented May 12, 2025

When deleting an OpenStack instance, for example the bootstrap node when deploying OCP or OCP itself on cluster cleanup, we are likely to encounter an error similar to this one:

cinderclient.exceptions.ClientException: ConflictNovaUsingAttachment: Detach volume from instance 0b54b909-43d8-4968-a183-c82004ad1e2e using the Compute API (HTTP 409) (Request-ID: req-2f8babd2-3aac-491e-83fa-032810da8880)

Which is caused by kcli being too fast to delete the volumes and requesting their deletion before the VM has actually stopped using them.

This patch fixes this by making sure we don't try to delete volumes until the nova instance has disappeared, at which point its resources would have been freed.

When deleting an OpenStack instance, for example the bootstrap node when
deploying OCP or OCP itself on cluster cleanup, we are likely to
encounter an error similar to this one:

```
cinderclient.exceptions.ClientException: ConflictNovaUsingAttachment: Detach volume from instance 0b54b909-43d8-4968-a183-c82004ad1e2e using the Compute API (HTTP 409) (Request-ID: req-2f8babd2-3aac-491e-83fa-032810da8880)
```

Which is caused by kcli being too fast to delete the volumes and
requesting their deletion before the VM has actually stopped using them.

This patch fixes this by making sure we don't try to delete volumes
until the nova instance has disappeared, at which point its resources
would have been freed.
@karmab karmab merged commit 5c72127 into karmab:main May 13, 2025
1 check passed
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
0