-
Notifications
You must be signed in to change notification settings - Fork 4k
Adds new ARM cmdlets #238
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
Adds new ARM cmdlets #238
Conversation
- Register-AzureProvider - Unregister-AzureProvider - Get-AzureProvider - Register-AzureProviderFeature - Get-AzureProviderFeature - Move-AzureResource
Hi @chadiel, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!
TTYL, AZPRBOT; |
@chadiel Shadi, it looks like what's happening is that a new API version in the resources client is not matching the APIVersion in the recorded mocks for the providers list call used by Get-AzureLocation. It is OK to do a search/replace on the mocks to change the API version, assuming the contracts haven't changed. |
I actually figured out what the problem – the API version didn’t change, instead there seems to be a bit of a behavior change where the .NET SDK doesn’t append the extra ‘&’ to the request URI (seems like it was a bug fix somewhere.) Anyway, it seems like the MockHttpServer you guys use does a simple match against the URI. This has 2 problems: 1) The mock server does a simple string lookup and is not intelligent enough to handle cosmetic changes to the query such as this one and more importantly 2) https traffic has it’s URIs Base64 encoded which means that you look for the encoded version of the strings – so it’s not as simple as just doing a find and replace. I’m going to only update the .NET SDK for the ARM library since I can (for the most part) re-record/patch those test cases to make them work but I cannot do it for all other recorded session. Will update the PR shortly. |
2.14.1 of the .NET SDK
RI from release to dev1-sync
No description provided.