fix telemetry resource field metadata - #9693
Conversation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
Azure Pipelines: Successfully started running 1 pipeline(s). 20 pipeline(s) were filtered out due to trigger conditions. There may be pipelines that require an authorized user to comment /azp run to run. |
There was a problem hiding this comment.
Pull request overview
Consolidates telemetry service-name metadata while preserving emitted error keys.
Changes:
- Adds resource-field classification and purpose metadata.
- Reuses the canonical service-name key across error telemetry.
- Updates tests, schema documentation, and review guidance.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
docs/specs/metrics-audit/telemetry-schema.md |
Documents resource metadata. |
cli/azd/internal/tracing/fields/fields.go |
Consolidates and classifies field declarations. |
cli/azd/internal/cmd/errors.go |
Reuses the canonical service-name key. |
cli/azd/internal/cmd/errors_test.go |
Updates error-key expectations. |
cli/azd/cmd/telemetry_test.go |
Verifies resource-field metadata. |
.github/skills/azd-code-reviewer/reviewers.md |
Adds observability review checks. |
💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Azure Dev CLI Install InstructionsInstall scriptsMacOS/Linux
bash: pwsh: WindowsPowerShell install MSI install Standalone Binary
MSI
Documentationlearn.microsoft.com documentationtitle: Azure Developer CLI reference
|
Fixes #9692
Summary
service.namefield declarations on the canonicalServiceNameKeyerror.service.nameemission by reusing that key beforeMapErrorapplies theerror.prefixazd-code-reviewerguidanceNo runtime telemetry key names or values change.
Telemetry Change Checklist
New Fields
fields/fields.godocs/specs/metrics-audit/telemetry-schema.mdNew Events
Privacy
CustomerContentis emittedTesting
Downstream
Documentation