Skip to content

App build id attribute#2591

Merged
jsuereth merged 8 commits intoopen-telemetry:mainfrom
LikeTheSalad:app-build-id
Aug 18, 2025
Merged

App build id attribute#2591
jsuereth merged 8 commits intoopen-telemetry:mainfrom
LikeTheSalad:app-build-id

Conversation

@LikeTheSalad
Copy link
Copy Markdown
Contributor

@LikeTheSalad LikeTheSalad commented Aug 4, 2025 •
edited
Loading

Changes

Adds an app.build_id attribute to identify app builds.

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]
  • Links to the prototypes or existing instrumentations (when adding or changing conventions)

@github-actions github-actions bot added enhancement New feature or request area:app labels Aug 4, 2025
@LikeTheSalad LikeTheSalad marked this pull request as ready for review August 4, 2025 08:13
@LikeTheSalad LikeTheSalad requested review from a team as code owners August 4, 2025 08:13
@trask trask moved this from Untriaged to Awaiting codeowners approval in Semantic Conventions Triage Aug 4, 2025
bidetofevil
bidetofevil approved these changes Aug 12, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@bidetofevil bidetofevil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. This is useful to distinguish between different pre-release app release that have the same public version build-specific information like deobfuscation mapping files or build configuration.

breedx-splk
breedx-splk reviewed Aug 13, 2025
View reviewed changes
lmolkova
lmolkova reviewed Aug 13, 2025
View reviewed changes
@github-project-automation github-project-automation bot moved this from Awaiting codeowners approval to Needs More Approval in Semantic Conventions Triage Aug 16, 2025
@lmolkova lmolkova moved this from Needs More Approval to Ready to be Merged in Semantic Conventions Triage Aug 16, 2025
@github-project-automation github-project-automation bot moved this from Ready to be Merged to Needs More Approval in Semantic Conventions Triage Aug 18, 2025
@jsuereth jsuereth enabled auto-merge August 18, 2025 12:52
@jsuereth jsuereth added this pull request to the merge queue Aug 18, 2025
Merged via the queue into open-telemetry:main with commit 72dedd2 Aug 18, 2025
15 checks passed
@MrAlias MrAlias mentioned this pull request Aug 26, 2025
Closed
ShipraJain01 pushed a commit to ShipraJain01/semantic-conventions that referenced this pull request Aug 29, 2025
@LikeTheSalad @breedx-splk @ShipraJain01
App build id attribute (open-telemetry#2591)
d0a8e43 
Co-authored-by: jason plumb <75337021+breedx-splk@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jsuereth jsuereth jsuereth approved these changes

@bidetofevil bidetofevil bidetofevil approved these changes

@lmolkova lmolkova lmolkova approved these changes

@breedx-splk breedx-splk breedx-splk approved these changes

+2 more reviewers

@thompson-tomo thompson-tomo thompson-tomo approved these changes

@MustafaHaddara MustafaHaddara MustafaHaddara approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

area:app enhancement New feature or request

Projects

Semantic Conventions Triage
Archived in project

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@LikeTheSalad @jsuereth @bidetofevil @lmolkova @MustafaHaddara @thompson-tomo @breedx-splk @trask