Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[receiver/gitproviderreceiver] rename metrics to match new conventions #34278

Merged
merged 13 commits into from
Aug 13, 2024

Conversation

adrielp
Copy link
Contributor

@adrielp adrielp commented Jul 28, 2024

Description:
Updated metrics and attributes to match newest semantic conventions for VCSs added in the registry.

Attribute Changes:
- branch.name is now ref.name
- Added a ref.type attribute
- pull_request.state is now change.state
- git.vendor.name is now vcs.vendor.name
Metric Changes:
- git.repository.count is now vcs.repository.count
- git.repository.contributor.count is now vcs.repository.contributor.count
- git.repository.branch.count is now vcs.repository.ref.count
- git.repository.branch.time is now vcs.repository.ref.time
- git.repository.branch.commit.aheadby.count is now vcs.repository.ref.revisions_ahead
- git.repository.branch.commit.behindby.count is now vcs.repository.ref.revisions_behind
- git.repository.branch.line.addition.count is now vcs.repository.ref.lines_added
- git.repository.branch.line.deletion.count is now vcs.repository.ref.lines_deleted
- git.repository.pull_request.time_open is now vcs.change.time_open
- git.repository.pull_request.time_to_merge is now vcs.change.time_to_merge
- git.repository.pull_request.time_to_approval is now vcs.change.time_to_approval
- git.repository.pull_request.count is now vcs.change.count

@adrielp adrielp requested a review from crobert-1 July 31, 2024 18:46
Copy link
Member

@andrzej-stencel andrzej-stencel left a comment

Choose a reason for hiding this comment

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

Awesome!

@andrzej-stencel andrzej-stencel merged commit bf30c79 into open-telemetry:main Aug 13, 2024
154 checks passed
@github-actions github-actions bot added this to the next release milestone Aug 13, 2024
f7o pushed a commit to f7o/opentelemetry-collector-contrib that referenced this pull request Sep 12, 2024
open-telemetry#34278)

**Description:**
Updated metrics and attributes to match newest semantic conventions for
[VCSs added in the
registry](https://github.com/open-telemetry/semantic-conventions/blob/main/docs/attributes-registry/vcs.md).

  Attribute Changes:
    - `branch.name` is now `ref.name`
    - Added a `ref.type` attribute
    - `pull_request.state` is now `change.state`
    - `git.vendor.name` is now `vcs.vendor.name`
  Metric Changes:
    - `git.repository.count` is now `vcs.repository.count`
- `git.repository.contributor.count` is now
`vcs.repository.contributor.count`
    - `git.repository.branch.count` is now `vcs.repository.ref.count`
    - `git.repository.branch.time` is now `vcs.repository.ref.time`
- `git.repository.branch.commit.aheadby.count` is now
`vcs.repository.ref.revisions_ahead`
- `git.repository.branch.commit.behindby.count` is now
`vcs.repository.ref.revisions_behind`
- `git.repository.branch.line.addition.count` is now
`vcs.repository.ref.lines_added`
- `git.repository.branch.line.deletion.count` is now
`vcs.repository.ref.lines_deleted`
- `git.repository.pull_request.time_open` is now `vcs.change.time_open`
- `git.repository.pull_request.time_to_merge` is now
`vcs.change.time_to_merge`
- `git.repository.pull_request.time_to_approval` is now
`vcs.change.time_to_approval`
    - `git.repository.pull_request.count` is now `vcs.change.count`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants