jclouds-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [jclouds] danielestevez edited a comment on issue #21: Adds methods listAll and listByLocation to VirtualMachineAPI
Date Wed, 10 Apr 2019 15:22:27 GMT
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

Mime
View raw message