Kami edited a comment on issue #1318: added azure_arm ex_delete_public_ip
URL: https://github.com/apache/libcloud/pull/1318#issuecomment-513379049
 
 
   EDIT: I was wrong - about ``ex_resource_delete`` working for IP addresses.
   
   It looks like IP address functionality is part of the new API version 
(https://docs.microsoft.com/en-us/rest/api/virtualnetwork/publicipaddress(preview)/delete)
 and code change in c83b61d won't actually work.
   
   I'll revert my change in c83b61d and leave your original code change which 
uses newer API version in place.
   
   EDIT: Here we go - 52195a6e9d15a31d0668d90111542435d638b62b. Next time would 
also be good to add a code comment (like I did) to make it clear why we aren't 
directly calling ``ex_delete_resource`` method.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to