-
Notifications
You must be signed in to change notification settings - Fork 9
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(snapshot): reset num_used_clusters_cache when snapshot created from thin provisioned lvol #30
fix(snapshot): reset num_used_clusters_cache when snapshot created from thin provisioned lvol #30
Conversation
Do we need to make this function part of public SPDK API ? Can't it just be part of internal logic when creating lvol snapshot ? |
Not needed for now. I will remove it. |
It seems, to operate on blob, the operations should be in blob lib and should be acccess through interface in lvol lib. |
I'd also recommend to perform a full BDD run in Mayastor against this commit before merging it, to make sure nothing unexpected is introduced by these SPDK changes. |
Full BDD executed and no issues observed with this commit. |
…om thin provisioned lvol Signed-off-by: Hrudaya <[email protected]>
No description provided.