Skip to content

Commit 1c64af9

Browse files
committed
Remove the mapping of app.kubernetes.io/instance to service.instance.id, because multiple containers
should never share the same `service.instance.id`
1 parent ef24a5f commit 1c64af9

File tree

1 file changed

+13
-3
lines changed

1 file changed

+13
-3
lines changed

.chloggen/service-instanc-id-mapping.yaml

+13-3
Original file line numberDiff line numberDiff line change
@@ -5,13 +5,23 @@ change_type: bug_fix
55
component: auto-instrumentation
66

77
# A brief description of the change. Surround your text with quotes ("") if it needs to start with a backtick (`).
8-
note: Remove the mapping of `app.kubernetes.io/instance` to `service.instance.id`, because multiple containers
9-
should never share the same `service.instance.id`
8+
note: Remove the mapping of `app.kubernetes.io/instance` to `service.instance.id`
109

1110
# One or more tracking issues related to the change
1211
issues: [3495]
1312

1413
# (Optional) One or more lines of additional information to render under the primary note.
1514
# These lines will be padded with 2 spaces and then inserted directly into the document.
1615
# Use pipe (|) for multiline entries.
17-
subtext:
16+
subtext: |
17+
Technically, this is a breaking change, but we regard it as a bug fix because the previous behavior was incorrect.
18+
19+
In you did have multiple containers and use `app.kubernetes.io/instance` to set the `service.instance.id`,
20+
you will now see multiple instances in the UI - which is the correct behavior.
21+
22+
You can still use the attribute `resource.opentelemetry.io/service.instance.id` to set the `service.instance.id`,
23+
which will be shared across all containers in the pod - but this is not recommended.
24+
25+
Refer to the [semantic conventions](https://opentelemetry.io/docs/specs/semconv/resource/#service-experimental)
26+
for more information.
27+

0 commit comments

Comments
 (0)