search

Azure / azure-sdk-for-net

This repository is for active development of the Azure SDK for .NET. For consumers of the SDK we recommend visiting our public developer docs at https://learn.microsoft.com/dotnet/azure/ or our versioned developer docs at https://azure.github.io/azure-sdk-for-net.
MIT License
5.46k stars 4.8k forks source link

NetworkListResponse.VirtualNetworkSite needs a Location property #737

Closed GarrettDavis closed 9 years ago

GarrettDavis commented 10 years ago

During TechEd (May 12-15) or before, the concept of Regional Virtual Networks was announced. When a new Virtual Network is created in the Portal, we don't even have the option of specifying an Affinity Group for the location, only a Region. Unfortunately, the INetworkOperations.ListAsync() method's response (NetworkListResponse) does not provide the region (VirtualNetworkSite.Location) of the Virtual Network, only a null AffinityGroup. To work around this and obtain the region, I need to also make a call to INetworkOperations.GetConfigurationAsync() and parse the configuration XML. I know that the REST API response has VirtualNetworkSite.Location so the code for NetworkListResponse simply needs to be regenerated with the new schema.

BTW, where are the specifications for the generated files so when I make these requests in the future, I can simply fork a file and add a pull request?

https://github.com/Azure/azure-sdk-for-net/issues/737

bradygaster-zz commented 10 years ago

Kay, could you triage this in your team?

singhkays commented 10 years ago

Thanks @GarrettDavis, this is on our TO DO list.

GarrettDavis commented 10 years ago

Thanks. Do you have an idea when this will be in a released version? We're getting ready to release our "Cloud Migration" tool for migrating servers into Azure and I would like to be able to get the Location from this API rather than also getting and parsing the configuration.

singhkays commented 10 years ago

@GarrettDavis Unfortunatley, I don't have a date for you but we're trying to get this scheduled in one of the next few sprints. It's behind other high-pri large work items atm.

markcowl commented 9 years ago

Closing as obsolete. Please reopen if this issue recurs