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

improve delete nic #400

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

improve delete nic #400

wants to merge 1 commit into from

Conversation

outscale-mgo
Copy link
Contributor

up to this patch, if the API was returning an empty array on ReadNics after a DeleteNic, the delete_nic operation on terraform was waiting for the end of the timeout to validate the delet.
Not anymore.

Note that this bug did happen on ricochet, but was not hapenning on the API because the api was returning a nic either in deleting or deleted state. But it was still a bug nethertheless.

up to this patch, if the API was returning an empty array on ReadNics
after a DeleteNic, the delete_nic operation on terraform was waiting for
the end of the timeout to validate the delete.
Not anymore.

Note that this bug did happen on Ricochet, but was not hapenning on the API
because the API was returning a nic either in deleting or deleted state.
But it was still a bug nevertheless.

Signed-off-by: Matthias Gatto <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants