[GitHub] [jclouds] danielestevez edited a comment on issue #21: Adds methods listAll and listByLocation to VirtualMachineAPI

[GitBox <gi...@apache.org>]

danielestevez edited a comment on issue #21: Adds methods listAll and listByLocation to VirtualMachineAPI
URL: https://github.com/apache/jclouds/pull/21#issuecomment-481736540
 
 
   Just a quick update on the state of this PR:
   The changes to the adapter were minor but i found one of those nasty ARM API behaviour with case-sensitivity.
   In short, these new methods return the resourcegroup part of the resource id in capital letters randomly and this causes some of the LiveTests to fail, so it's not safe to add them to the adapter yet
   The goal to make all the provider case-insensitive (as Microsoft [recommends](https://stackoverflow.com/a/50082864/2369157) in integrations) is too wide for this PR but i found a way to fix this problem just for the resourcegroups.
   Running all the LiveTests with this fix i only found a problem still in `AzureComputeServiceLiveTest#testListNodesByIds` that i will try to fix too

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services