Azure-cli: acr module should use their mgmt SDK instead of including code in module

[derekbekoe]

Whilst https://github.com/Azure/azure-cli/pull/859 has been merged, it includes mgmt_acr in the module code.

Instead, it should depend on an mgmt SDK.
When that mgmt SDK has been packaged and published, the acr module should be updated accordingly.

This requires the swagger spec to be merged first https://github.com/Azure/azure-rest-api-specs/pull/549.

FYI: @djyou, @SajayAntony, @mayurid

[derekbekoe]

@JasonRShaver can we create an 'ACR' service label so we can label this issue?

[mayurid]

Marking this for Connect Milestone as Swagger is in review and believe this will happen for that. Let me know if there are concerns.

[mayurid]

@jasonrshaver and @derekbekoe : the label is created as well.

[sajayantony]

@amarzavery can you please light up the SDK.

[mayurid]

@SajayAntony : I believe you are looking for the Python SDK to be published. You will need to work with @lmazuel who owns the https://github.com/Azure/azure-sdk-for-python repo

[lmazuel]

If the swagger is merged I can do that :). Is it?

[lmazuel]

@SajayAntony Building the SDK, I have:

msrestazure.azure_exceptions.CloudError: Azure Error: InvalidResourceNamespace
Message: The resource namespace 'Microsoft.ContainerRegistry' is invalid.

Do you need my subscription id for me to be able to test this?

[sajayantony]

@lmazuel yes you need an approved subscription.

[lmazuel]

@derekbekoe @SajayAntony Released as a Python SDK:
https://pypi.python.org/pypi/azure-mgmt-containerregistry/0.1.0

[derekbekoe]

PR merged. Closing issue.