exclude IN_DELETION volumes from lvol list and /connect APIs#1058
exclude IN_DELETION volumes from lvol list and /connect APIs#1058boddumanohar wants to merge 2 commits into
Conversation
mxsrc
left a comment
There was a problem hiding this comment.
Good fix, we should really add a helper with a default include_deleted=False flag for this. The API needs a query argument for this as well. Neither is urgent though.
|
We cannot just hide them from the listing. There are tools that need to see them, like the control center. We should have a way to adjust filters. Maybe something like |
noctarius
left a comment
There was a problem hiding this comment.
Please see the comment in slack!
|
I just realised that this could cause inconsistency during volume create with the same name (as the one in_deletion). And an other issue as Chris mentioned that this could also break API contract. So we need to discuss more deeply around this situation needs to be handled given that lvol delete is an asynchronous operation. |
No description provided.