azurerm_netapp_snapshot waitLRO
#16973
Conversation
There was a problem hiding this comment.
hi @shu-ying789
Thanks for this PR - I've taken a look through and left a comment inline but if we can remove the WaitForState func then we should be able to take another look.
Thanks!
| return fmt.Errorf("deleting NetApp Snapshot %q (Resource Group %q): %+v", id.Name, id.ResourceGroup, err) | ||
| } | ||
| if err := future.WaitForCompletionRef(ctx, client.Client); err != nil { | ||
| return fmt.Errorf("waiting for deletion of %q: %+v", id, err) | ||
| } |
There was a problem hiding this comment.
does this mean we can remove the WaitForState func below?
There was a problem hiding this comment.
no, as per the service team maintainer it is still required
|
closing as this will be fixed by #17043 |
|
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions. |
test result:
linked to:
#15641