{Monitor} Fix CloudEndpointNotSetException error - #10320
Conversation
|
Hi Julie Zhu (@yanzhudd), |
There was a problem hiding this comment.
🟢 Approval recommended
The change is a targeted string-key correction that aligns with existing Azure cloud endpoint metadata keys and has low risk.
Pull request overview
Fixes the Log Analytics AAZ data-plane client’s cloud endpoint metadata lookup by correcting the _CLOUD_HOST_METADATA_INDEX key so the client can resolve the proper Log Analytics endpoint from ctx.cli_ctx.cloud.endpoints and avoid CloudEndpointNotSetException.
Changes:
- Corrected the cloud endpoint metadata index from
logAnalyticslogAnalyticsResourceIdtologAnalyticsResourceIdin the Log Analytics AAZ client.
File summaries
| File | Description |
|---|---|
| src/log-analytics/azext_loganalytics/aaz/latest/_clients.py | Fixes the cloud endpoint metadata key used to resolve the Log Analytics data-plane base URL. |
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 0
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
Monitor |
Co-authored-by: Ethan Yang <necusjz@gmail.com>
Co-authored-by: Ethan Yang <necusjz@gmail.com>
|
[Release] Update index.json for extension [ log-analytics-1.0.0b2 ] : https://dev.azure.com/msazure/One/_build/results?buildId=180508096&view=results |
🤖 PR Validation — ️✔️ All clear
This checklist is used to make sure that common guidelines for a pull request are followed.
Related command
General Guidelines
azdev style <YOUR_EXT>locally? (pip install azdevrequired)python scripts/ci/test_index.py -qlocally? (pip install azdevrequired)For new extensions:
About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update
src/index.jsonautomatically.You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify
src/index.json.