Skip to content

Commit

Permalink
[Monitor] Fix #4591: az monitor app-insights query application coul…
Browse files Browse the repository at this point in the history
…d not be found (#4635)

* fix ApplicationNotFoundError

* re-record affected tests

* update version and changelog
  • Loading branch information
necusjz authored Apr 12, 2022
1 parent 0986a8c commit 3bf9cde
Show file tree
Hide file tree
Showing 6 changed files with 384 additions and 358 deletions.
4 changes: 4 additions & 0 deletions src/application-insights/HISTORY.rst
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,10 @@
Release History
===============
0.1.16
++++++++++++++++++
* `az monitor app-insights query`: Fix application could not be found.

0.1.15
++++++++++++++++++
* `az monitor app-insights web-test`: Create/list/update/show/delete Application Insights Web Test.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,10 @@ def applicationinsights_data_plane_client(cli_ctx, _, subscription=None):
resource=cli_ctx.cloud.endpoints.app_insights_resource_id,
subscription_id=subscription
)
return ApplicationInsightsDataClient(cred)
return ApplicationInsightsDataClient(
cred,
base_url=f'{cli_ctx.cloud.endpoints.app_insights_resource_id}/v1'
)


def applicationinsights_mgmt_plane_client(cli_ctx, **kwargs):
Expand Down

Large diffs are not rendered by default.

Large diffs are not rendered by default.

Loading

0 comments on commit 3bf9cde

Please sign in to comment.