[monitor-query] Only works with Azure Commercial Cloud

[davemedvitz]

Consider the following code, using the AzureCLI auth method for local development.

    import { DefaultAzureCredential } from "@azure/identity";
    import { LogsQueryClient } from "@azure/monitor-query";

    const workspace = "7cxxxxx-xxxxx-xxxxx-xxxx-xxxxxxxxxxxx";

    const cred = new DefaultAzureCredential();
    const opts = {
        "endpoint": "https://api.loganalytics.us/v1"
    }

    const lac = new LogsQueryClient(cred,opts);
    lac.queryLogs(workspace,"Event","P30M");
    console.log(lac);

My expectation was that this would function, directing the client to the API endpoints in the Azure Gov Cloud. Instead, it errors on authentication.

As far as I can tell, there are two issues.

1. The defaultMonitorScope variable is hard coded to the Commercial cloud. If I change this locally the authentication works, but the queries still fail.
2. baseUri defaults to commercial and does not update to match endpoint (although $host does). Adjusting this locally appears to let the query work. (Not sure why this is an issue, as it appears to be set properly in the constructor for AzureLogAnalyticsContext )

I am using the 1.0.0-beta.1 version

Thank you.
Dave

[davemedvitz]

As a follow up to #2 above, it appears that the option endpoint is not a part of the options parameter in the call to the AzureLogAnalytics constuctor from the LogsQueryClient constructor.

[ramya-rao-a]

Thanks for reporting @davemedvitz

@richardpark-msft Can you take an initial look here and loop in Daniel as needed if this is an Identity issue?

[richardpark-msft]

@davemedvitz - I'll take a look at this today.

I believe I just need to plumb the proper endpoint through a little further, so hopefully not an identity issue.

[scottaddie]

@davemedvitz I'm very interested in talking to customers about their use cases and the beta 1 SDK that shipped last week. Please email me at scott dot addie at microsoft dot com if you're interested. Any feedback you can provide will help us build a better SDK for you and other customers.

[richardpark-msft]

@davemedvitz : we've now released 1.0.0-beta.2 based which fixes your issue:

https://www.npmjs.com/package/@azure/monitor-query/v/1.0.0-beta.2

Thank you very much for reporting this issue.