Skip to content

model/profile: add location information#1

Closed
florianl wants to merge 2 commits intoprofiles-symb-levelfrom
profiles-location-folded
Closed

model/profile: add location information#1
florianl wants to merge 2 commits intoprofiles-symb-levelfrom
profiles-location-folded

Conversation

@florianl
Copy link
Copy Markdown
Owner

@florianl florianl commented Jul 30, 2025

Changes

Add Profiling specific attributes for pprof location.

This change is based on top of open-telemetry#2522 to not conflict with registry.profile.pprof.

FYI: @open-telemetry/profiling-approvers

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)

Signed-off-by: Florian Lehner <florian.lehner@elastic.co>
florianl added a commit to florianl/opentelemetry-proto that referenced this pull request Jul 30, 2025
Similar to the boolean attributes `Mapping.has_*` (open-telemetry#595 and open-telemetry/semantic-conventions#2522) also drop `Location.is_folded`.


The complementary PR for the OTel SemConv, that builds on top of open-telemetry/semantic-conventions#2522 is florianl/semantic-conventions#1
Signed-off-by: Florian Lehner <florian.lehner@elastic.co>
@florianl
Copy link
Copy Markdown
Owner Author

florianl commented Aug 7, 2025

Cherry-picked this PR into open-telemetry#2522.

@florianl florianl closed this Aug 7, 2025
tigrannajaryan added a commit to open-telemetry/opentelemetry-proto that referenced this pull request Aug 11, 2025
Similar to the boolean attributes `Mapping.has_*` (#595 and open-telemetry/semantic-conventions#2522) also drop `Location.is_folded`.


The complementary PR for the OTel SemConv, that builds on top of open-telemetry/semantic-conventions#2522 is florianl/semantic-conventions#1

Co-authored-by: Tigran Najaryan <4194920+tigrannajaryan@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant