Zimmergren/LogAnalytics.Client

Azure Sovereign Cloud support

Closed this issue · 6 comments

Hi,

Thanks for building and maintaining this client.

Would you be able to add Azure Sovereign Cloud support please, i.e.

Commercial = ods.opinsights.azure.com
Government = ods.opinsights.azure.us
China = ods.opinsights.azure.cn

https://docs.microsoft.com/en-us/azure/azure-government/compare-azure-government-global-azure

It would be great to add this as an optional setting during initialization.

Thanks, again,

Brett

Hi @brettmeyerowitz ,

Sure, we could add an optional configuration param for the various clouds.
However, I don't have the capability to try them out or run integration tests for those endpoints. Do you have any suggestions on how to achieve that?

Thanks,
Tobias

Thanks for your prompt response!

Unfortunately the only ones I have access to are our corporate ones here - which I cannot expose.

I can, of course, run local integration tests.

Regards,

Brett

Fair enough. I'll take a look when I have a few moments, and when it's ready you can run your local integration tests to verify they work. Meanwhile, the only real code change would be the endpoint, and as such, if they support the same APIs it shouldn't be any issue.

I'll ping here with the PR once done.

They definitely do support the same API :)

Please have a look at #39 , @brettmeyerowitz.

I've ran the integration tests locally for the default configuration (Azure Public cloud), and that looks good. I can't run any integration tests for non-public Azure clouds here. If you could verify/review, that'd be great.

Closed with #39