[monitor-trace-association] Add preview extension for Microsoft.Monitor/traceAssociations (draft) - #10167
[monitor-trace-association] Add preview extension for Microsoft.Monitor/traceAssociations (draft)#10167bragi92 wants to merge 3 commits into
Conversation
…or/traceAssociations (draft) Adds `az monitor trace-association` create/update/show/delete/list for Microsoft.Monitor/traceAssociations, API version 2026-01-01-preview. DRAFT: command bodies call ARM directly (send_raw_request) as a functional prototype; to be replaced by aaz-dev-generated code once the spec is published to the public Azure/azure-rest-api-specs repo (currently only in azure-rest-api-specs-pr #27737). Do not merge until the public spec lands. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: fe50b7ed-a3b8-4f18-85e2-a20194432da9
|
Azure Pipelines: There may be pipelines that require an authorized user to comment /azp run to run. |
|
/azp run |
|
Azure Pipelines: Successfully started running 2 pipeline(s). |
…n update The azdev linter rule no_parameter_defaults_for_update_commands (HIGH) failed because 'name' was registered with default='default' in the shared 'monitor trace-association' argument context, which propagated to the update command. Drop the explicit default from the shared context and let each command supply its own default via its function signature. update_trace_association now takes name=None and resolves the singleton name internally, so behaviour is unchanged. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 534f4033-4217-4c50-880f-9b8c44d46964
|
/azp run |
|
Azure Pipelines: Successfully started running 2 pipeline(s). |
…e, list The azdev linter rule missing_command_example (HIGH) requires every modified command to carry at least one example. Only 'create' had one. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 534f4033-4217-4c50-880f-9b8c44d46964
|
/azp run |
|
Azure Pipelines: Successfully started running 2 pipeline(s). |
Live test results —
|
There was a problem hiding this comment.
Automated Review Summary — PR #10167
CI status: All 28 completed CI checks passed (0 failed, 0 pending).
Live test (Tester): ❌ Failed — azdev test exit code 2 for the changed test file:
src/monitor-trace-association/azext_monitor_trace_association/tests/latest/test_monitor_trace_association.py
The test run did not execute (RAN_TESTS=false), which indicates a collection/setup
error rather than an assertion failure — likely a missing recording, import error, or
fixture issue for this brand-new test file. See the workflow run for details:
https://github.com/Azure/issue-sentinel/actions/runs/32347009155
A detailed run comment (with the last 80 lines of azdev output) was already posted to
this PR by the live-test workflow. Please review that output and address the failure.
Draft — do not merge (blocked on public REST API spec)
Adds a new preview extension
monitor-trace-associationforMicrosoft.Monitor/traceAssociations(API version2026-01-01-preview).az monitor trace-associationmaps a scope (Application Insights component, resource group, or subscription) to an Azure Monitor Workspace for trace routing. Singletondefault; multi-homing via scope inheritance.Commands
create— PUT (CreateOrUpdate)update— GET + merge + PUTshow— GETdelete— DELETElist— GET (ListByScope)Args:
--resource-uri/--scope(required),--azure-monitor-workspace-resource-id/--amw-id(required on create),--name/-n(defaultdefault).Status / why this is a draft
azure-rest-api-specs-pr(PR #27737); it is not yet in publicAzure/azure-rest-api-specs. A 120-day ARM lease is holding the API version (expires ~Sept 2026).send_raw_request. They will be replaced by aaz-dev-generated code (with the matchingAzure/aazcommand-model PR) once the swagger is published publicly.isPreview: true.Validation done locally
python -m py_compilepasses for all modules.az extension add, and verified-hfor the group and every command: command tree, required args, defaults, and examples all render.TODO before ready-for-review
azure-rest-api-specsPR merged (port from-pr#27737).Azure/aazcommand-model PR.aaz/code.Co-authored with GitHub Copilot CLI.