Change the definition of `device.id` #1951

bencehornak · 2025-02-27T20:17:22Z

Warning

This PR must be merged together with #1897 as they are highly related, dear maintainers, please add them to the same milestone.

Changes

I have given a new definition to device.id, which now truly identifies the device. It has been marked as opt-in though, as its usage is generally discouraged due to privacy implications, and should only be enabled in special cases, when it does not violate the privacy of the user.

The old definition of device.id is more-or-less covered by the proposed app.installation.id in #1897, which is recommended for general use instead of this, as it provides more privacy to the users.

Merge requirement checklist

CONTRIBUTING.md guidelines followed.
Change log entry added, according to the guidelines in When to add a changelog entry.
- If your PR does not need a change log, start the PR title with [chore]
schema-next.yaml updated with changes to existing conventions.

docs/resource/device.md

trask · 2025-03-12T20:48:35Z

cc @open-telemetry/semconv-mobile-approvers

trask · 2025-03-12T20:52:44Z

cc @open-telemetry/semconv-mobile-approvers

hm, it looks like CODEOWNERS is set up so that @open-telemetry/semconv-mobile-approvers should have been added as reviewers on this PR 🤔

UPDATE oh, my bad, they were: #1951 (comment)

I guess I didn't realize (forgot?) that the group name is replaced once someone from that group reviews it

model/device/registry.yaml

docs/resource/device.md

trask · 2025-03-18T19:41:23Z

it would be great if we can get approvals from the iOS and browser perspectives as well (if this attribute is applicable there)

docs/attributes-registry/device.md

docs/resource/device.md

bidetofevil

The new description sufficiently neuters device.id and gives the appropriate warnings for commercial apps and SDKs to NOT user it, so this works for me.

joaopgrassi

Looks good to me, but I wonder if we should be more clear in guiding users to use the new attribute now instead of this one. By reading the description now I get the feeling that it is a bit vague still.

model/device/registry.yaml

trask · 2025-03-26T14:45:41Z

This PR must be merged together with #1897 as they are highly related, dear maintainers, please add them to the same milestone.

will get this merged soon 👍

Co-authored-by: Trask Stalnaker <[email protected]>

Co-authored-by: Joao Grassi <[email protected]> Co-authored-by: Liudmila Molkova <[email protected]> Co-authored-by: Trask Stalnaker <[email protected]>

- Add the new metric API package structure prototyped in https://github.com/MrAlias/semconv-go Prototypes of new metric API use: - MrAlias/opentelemetry-go-contrib#6136 - MrAlias/opentelemetry-go-contrib#6135 - MrAlias/opentelemetry-go-contrib#6134 - Generate `semconv/v1.32.0` - Drop the `kestrel` metric namespace as this is a Java specific technology ## [`v1.32.0` semantic convention release notes](https://github.com/open-telemetry/semantic-conventions/releases/tag/v1.32.0): <div data-pjax="true" data-test-selector="body-content" data-view-component="true" class="markdown-body my-3"><p>📣 This release is the second release candidate for the Database Semantic Conventions, with <strong>db conventions stability planned to be declared in the subsequent release</strong>.</p> <h3>🛑 Breaking changes 🛑</h3> <ul> <li><code>device</code>: Change the definition of <code>device.id</code> and make it opt-in. (<a href="open-telemetry/semantic-conventions#1874" data-hovercard-type="issue" data-hovercard-url="/open-telemetry/semantic-conventions/issues/1874/hovercard">#1874</a>, <a href="open-telemetry/semantic-conventions#1951" data-hovercard-type="pull_request" data-hovercard-url="/open-telemetry/semantic-conventions/pull/1951/hovercard">#1951</a>)</li> <li><code>feature_flag</code>: Rename <code>evaluation</code> to <code>result</code> for feature flag evaluation result attributes (<a href="open-telemetry/semantic-conventions#1989" data-hovercard-type="pull_request" data-hovercard-url="/open-telemetry/semantic-conventions/pull/1989/hovercard">#1989</a>)</li> </ul> <h3>🚀 New components 🚀</h3> <ul> <li><code>app</code>: Create <code>app.installation.id</code> attribute (<a href="open-telemetry/semantic-conventions#1874" data-hovercard-type="issue" data-hovercard-url="/open-telemetry/semantic-conventions/issues/1874/hovercard">#1874</a>, <a href="open-telemetry/semantic-conventions#1897" data-hovercard-type="pull_request" data-hovercard-url="/open-telemetry/semantic-conventions/pull/1897/hovercard">#1897</a>)</li> <li><code>cpython</code>: Add CPython runtime garbage collector metrics (<a href="open-telemetry/semantic-conventions#1930" data-hovercard-type="issue" data-hovercard-url="/open-telemetry/semantic-conventions/issues/1930/hovercard">#1930</a>)</li> </ul> <h3>💡 Enhancements 💡</h3> <ul> <li><code>vcs</code>: Add owner and provider name to VCS attribute registry (<a href="open-telemetry/semantic-conventions#1452" data-hovercard-type="issue" data-hovercard-url="/open-telemetry/semantic-conventions/issues/1452/hovercard">#1452</a>)</li> <li><code>vcs</code>: Remove fallback value for VCS provider name attribute (<a href="open-telemetry/semantic-conventions#2020" data-hovercard-type="pull_request" data-hovercard-url="/open-telemetry/semantic-conventions/pull/2020/hovercard">#2020</a>)</li> <li><code>db</code>: Truncate <code>db.query.summary</code> to 255 characters if parsed from the query (<a href="open-telemetry/semantic-conventions#1978" data-hovercard-type="issue" data-hovercard-url="/open-telemetry/semantic-conventions/issues/1978/hovercard">#1978</a>)</li> <li><code>db</code>: Normalize spaces in <code>db.operation.name</code> (if any) (<a href="open-telemetry/semantic-conventions#2028" data-hovercard-type="issue" data-hovercard-url="/open-telemetry/semantic-conventions/issues/2028/hovercard">#2028</a>)</li> <li><code>db</code>: <code>db.operation.parameter.<key></code> should not be captured for batch operations (<a href="open-telemetry/semantic-conventions#2026" data-hovercard-type="issue" data-hovercard-url="/open-telemetry/semantic-conventions/issues/2026/hovercard">#2026</a>)</li> <li><code>db</code>: Add <code>db.stored_procedure.name</code> (<a href="open-telemetry/semantic-conventions#1491" data-hovercard-type="issue" data-hovercard-url="/open-telemetry/semantic-conventions/issues/1491/hovercard">#1491</a>)</li> <li><code>gcp</code>: Adds GCP AppHub labels for resource. (<a href="open-telemetry/semantic-conventions#2006" data-hovercard-type="pull_request" data-hovercard-url="/open-telemetry/semantic-conventions/pull/2006/hovercard">#2006</a>)</li> <li><code>error</code>: Add <code>error.message</code> property for human-readable error message on events. (<a href="open-telemetry/semantic-conventions#1992" data-hovercard-type="pull_request" data-hovercard-url="/open-telemetry/semantic-conventions/pull/1992/hovercard">#1992</a>)</li> <li><code>profile</code>: Extend the list of known frame types with a value for Go and Rust (<a href="open-telemetry/semantic-conventions#2003" data-hovercard-type="pull_request" data-hovercard-url="/open-telemetry/semantic-conventions/pull/2003/hovercard">#2003</a>)</li> <li><code>otel</code>: Adds SDK self-monitoring metrics for log processing (<a href="open-telemetry/semantic-conventions#1921" data-hovercard-type="pull_request" data-hovercard-url="/open-telemetry/semantic-conventions/pull/1921/hovercard">#1921</a>)</li> </ul>

Change the definition of device.id

6cd950c

bencehornak requested review from a team as code owners February 27, 2025 20:17

github-project-automation bot added this to Semantic Conventions Triage Feb 27, 2025

github-project-automation bot moved this to Untriaged in Semantic Conventions Triage Feb 27, 2025

bencehornak mentioned this pull request Feb 27, 2025

Create app.installation.id attribute #1897

Merged

3 tasks

Add changelog entry

b0ee7d3

github-actions bot added the breaking label Feb 27, 2025

bencehornak added 2 commits February 27, 2025 21:33

Fix reference to app.installation.id

f6f5fab

Resolve CI issues

fc5e3b2

bencehornak commented Feb 27, 2025

View reviewed changes

docs/resource/device.md Outdated Show resolved Hide resolved

LikeTheSalad approved these changes Mar 5, 2025

View reviewed changes

trask moved this from Untriaged to Needs More Approval in Semantic Conventions Triage Mar 12, 2025

bidetofevil reviewed Mar 14, 2025

View reviewed changes

model/device/registry.yaml Show resolved Hide resolved

bidetofevil reviewed Mar 14, 2025

View reviewed changes

docs/resource/device.md Outdated Show resolved Hide resolved

MSNev reviewed Mar 18, 2025

View reviewed changes

docs/attributes-registry/device.md Outdated Show resolved Hide resolved

MSNev reviewed Mar 18, 2025

View reviewed changes

docs/attributes-registry/device.md Show resolved Hide resolved

MSNev reviewed Mar 18, 2025

View reviewed changes

docs/resource/device.md Show resolved Hide resolved

MSNev approved these changes Mar 18, 2025

View reviewed changes

bencehornak added 2 commits March 18, 2025 21:40

Rephrase disclaimer

bd41e66

Merge branch 'main' into device-id

ce2040a

bidetofevil approved these changes Mar 18, 2025

View reviewed changes

joaopgrassi approved these changes Mar 24, 2025

View reviewed changes

joaopgrassi added area:device area:browser labels Mar 24, 2025

Merge branch 'main' into device-id

75ef5b5

lmolkova approved these changes Mar 25, 2025

View reviewed changes

lmolkova moved this from Needs More Approval to Ready to be Merged in Semantic Conventions Triage Mar 25, 2025

lmolkova modified the milestones: v1.31.0, 1.32.0 Mar 25, 2025

Liudmila Molkova and others added 3 commits March 25, 2025 09:24

Merge branch 'main' into device-id

b4c805d

Merge branch 'main' into device-id

da46615

Merge branch 'main' into device-id

2817c03

trask reviewed Mar 26, 2025

View reviewed changes

model/device/registry.yaml Outdated Show resolved Hide resolved

bencehornak force-pushed the device-id branch from b9ba18d to 31077bc Compare March 26, 2025 17:04

Change normative word to MAY

a730498

Co-authored-by: Trask Stalnaker <[email protected]>

bencehornak force-pushed the device-id branch from 31077bc to a730498 Compare March 26, 2025 17:05

lmolkova merged commit 8873c26 into open-telemetry:main Mar 26, 2025
15 checks passed

github-project-automation bot moved this from Ready to be Merged to Needs More Approval in Semantic Conventions Triage Mar 26, 2025

MrAlias mentioned this pull request May 15, 2025

Add semconv/v1.32.0 open-telemetry/opentelemetry-go#6782

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

Change the definition of `device.id` #1951

Change the definition of `device.id` #1951

Uh oh!

bencehornak commented Feb 27, 2025 •

edited

Loading

Uh oh!

Uh oh!

trask commented Mar 12, 2025

Uh oh!

trask commented Mar 12, 2025 •

edited

Loading

Uh oh!

Uh oh!

Uh oh!

trask commented Mar 18, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

bidetofevil left a comment

Uh oh!

joaopgrassi left a comment

Uh oh!

Uh oh!

trask commented Mar 26, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

Uh oh!

Change the definition of device.id #1951

Change the definition of device.id #1951

Uh oh!

Conversation

bencehornak commented Feb 27, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Changes

Merge requirement checklist

Uh oh!

Uh oh!

trask commented Mar 12, 2025

Uh oh!

trask commented Mar 12, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

trask commented Mar 18, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

bidetofevil left a comment

Choose a reason for hiding this comment

Uh oh!

joaopgrassi left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

trask commented Mar 26, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

Change the definition of `device.id` #1951

Change the definition of `device.id` #1951

bencehornak commented Feb 27, 2025 •

edited

Loading

trask commented Mar 12, 2025 •

edited

Loading