Skip to content

Commit e66393c

Browse files
authoredDec 11, 2024··
[Monitor OpenTelemetry] Update Statsbeat Instrumentation Mapping (#32130)
### Packages impacted by this PR @azure/monitor-opentelemetry ### Describe the problem that is addressed by this PR We should have a mapping for 128 that is reserved for console instrumentation. This PR resolves that open instrumentation value. ### Checklists - [x] Added impacted package name to the issue description - [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_ - [x] Added a changelog (if necessary)
1 parent df390f9 commit e66393c

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed
 

‎sdk/monitor/monitor-opentelemetry/CHANGELOG.md

+1
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@
88

99
### Bugs Fixed
1010

11+
- Should not use reserved 7th bit for statsbeat instrumentations.
1112
- Fix incorrectly setting the cloud role name and role instance to undefined on standard metrics in AKS environments.
1213

1314
### Other Changes

‎sdk/monitor/monitor-opentelemetry/src/types.ts

+2-1
Original file line numberDiff line numberDiff line change
@@ -202,7 +202,7 @@ export enum StatsbeatInstrumentation {
202202
BUNYAN = 32,
203203
WINSTON = 64,
204204
/** OpenTelemetry Supported Instrumentations */
205-
AMQPLIB = 128,
205+
// Console instrumentation is not supported here - occupies 128
206206
CUCUMBER = 256,
207207
DATALOADER = 512,
208208
FS = 1024,
@@ -230,6 +230,7 @@ export enum StatsbeatInstrumentation {
230230
PINO = 4294967296,
231231
RESTIFY = 8589934592,
232232
ROUTER = 17179869184,
233+
AMQPLIB = 34359738368,
233234
}
234235

235236
/**

0 commit comments

Comments
 (0)
Please sign in to comment.