Add support to the exporter for Availability Data telemetry - #48812
Conversation
|
Azure Pipelines: Successfully started running 1 pipeline(s). 9 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
Adds log-based Availability Data telemetry support to the Azure Monitor OpenTelemetry exporter.
Changes:
- Maps
microsoft.availability.*attributes toAvailabilityData. - Adds unit coverage and an availability sample.
- Updates documentation and changelog.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
_constants.py |
Defines availability attribute constants. |
_exporter.py |
Converts qualifying logs into availability telemetry. |
test_logs.py |
Tests conversion, validation, timestamps, and precedence. |
sample_availability_data.py |
Demonstrates availability logging. |
samples/logs/README.md |
Documents the new sample. |
CHANGELOG.md |
Records the feature. |
💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.
0769356 to
68aa937
Compare
68aa937 to
df6d6df
Compare
|
Azure Pipelines: Successfully started running 1 pipeline(s). 9 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
Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.
Suppressed comments (1)
sdk/monitor/azure-monitor-opentelemetry-exporter/azure/monitor/opentelemetry/exporter/export/logs/_exporter.py:263
- The AvailabilityData schema caps
messageat 8,192 characters. The explicit attribute is currently unbounded, while the body fallback can reach 32,768 via_map_body_to_message, so either path can produce an invalid availability envelope; truncate the final message to the schema limit.
message=(
availability_data["message"]
if "message" in availability_data
else _map_body_to_message(log_record.body)
),
There was a problem hiding this comment.
Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.
Note
This error may be related to your runner configuration. You can now configure runners for Copilot code review separately from Copilot cloud agent by creating a copilot-code-review.yml file with your setup steps. Read the docs for details.
All SDK Contribution checklist:
General Guidelines and Best Practices
Testing Guidelines