Skip to content

Rename resource group type to entity.#714

Merged
jsuereth merged 5 commits intoopen-telemetry:mainfrom
jsuereth:wip-rename-resource-to-entity
Apr 30, 2025
Merged

Rename resource group type to entity.#714
jsuereth merged 5 commits intoopen-telemetry:mainfrom
jsuereth:wip-rename-resource-to-entity

Conversation

@jsuereth
Copy link
Contributor

This is part of the move to Entity as a grouping of attributes on Resource. An Entity represents a named set of attributes with its own versioning defined by weaver / semantic conventions. Combined with the previous entity_associations work, this unlocks validating Resource in live_check.

Note: This is breaking for all templates that directly use group.type.
While JQ helpers have been updated to support the chagne, direct
group.type usage will see entity in the future.

This is part of the move to Entity as a grouping of attributes on
Resource. An Entity represents a named set of attributes with its
own versioning defined by weaver / semantic conventions. Combined
with the previous `entity_associations` work, this unlocks
validating Resource in `live_check`.

Note: This is breaking for all tempaltes that directly use `group.type`.
      While JQ helpers have been updated to support the chagne, direct
      group.type usage will see `entity` in the future.
@jsuereth jsuereth marked this pull request as ready for review April 28, 2025 12:40
@jsuereth jsuereth requested a review from a team as a code owner April 28, 2025 12:40
@jsuereth
Copy link
Contributor Author

Added a fix to prevent semconv from breaking with this: open-telemetry/semantic-conventions#2192

Copy link
Contributor

@lquerel lquerel left a comment

Choose a reason for hiding this comment

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

LGTM

jsuereth and others added 2 commits April 28, 2025 19:57
Co-authored-by: Liudmila Molkova <limolkova@microsoft.com>
@lmolkova lmolkova changed the title Rename resource group type to `entity. Rename resource group type to entity. Apr 30, 2025
@jsuereth jsuereth moved this to To consider for the next release in OTel Weaver Project Apr 30, 2025
@lmolkova lmolkova moved this from To consider for the next release to Next Release in OTel Weaver Project Apr 30, 2025
@codecov
Copy link

codecov bot commented Apr 30, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.6%. Comparing base (3ce6a09) to head (e8d789b).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##            main    #714     +/-   ##
=======================================
- Coverage   74.7%   74.6%   -0.1%     
=======================================
  Files         67      67             
  Lines       5093    5093             
=======================================
- Hits        3805    3804      -1     
- Misses      1288    1289      +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jsuereth jsuereth merged commit 53a58c7 into open-telemetry:main Apr 30, 2025
23 checks passed
@github-project-automation github-project-automation bot moved this from Next Release to Done in OTel Weaver Project Apr 30, 2025
@jsuereth jsuereth deleted the wip-rename-resource-to-entity branch April 30, 2025 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants