Skip to content

Conversation

@thompson-tomo
Copy link
Contributor

@thompson-tomo thompson-tomo commented Jul 29, 2025

Progresses #1107

Changes

This adds RavenDB as a db system, so that it can be used in telemetry.

Usage Example

  • Health Check of the connection to the db #2205 data about db & messaging checks Xabaril/AspNetCore.Diagnostics.HealthChecks#2228. The definition of a health check event is much like an exception/event and as mentioned in the semconv meeting, 28/7/25, if attributes are not listed they are implicitly opt-in. As such this is just like if you were to put db.system.name on an exception it is beneficial. In some health checks it is even possible to check the health of db's not just sever connection etc hence even more reason to leverage the db namespace to capture that data.

Note: if the PR is touching an area that is not listed in the existing areas, or the area does not have sufficient domain experts coverage, the PR might be tagged as experts needed and move slowly until experts are identified.

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
Copy link

This PR was marked stale due to lack of activity. It will be closed in 7 days.

@github-actions github-actions bot added the Stale label Aug 14, 2025
@thompson-tomo
Copy link
Contributor Author

Still Awaiting for review.

@joaopgrassi joaopgrassi removed the Stale label Aug 14, 2025
@joaopgrassi joaopgrassi moved this from Untriaged to Awaiting codeowners approval in Semantic Conventions Triage Aug 14, 2025
@joaopgrassi
Copy link
Member

@open-telemetry/semconv-db-approvers can you please take a look?

@github-actions
Copy link

This PR was marked stale due to lack of activity. It will be closed in 7 days.

@github-actions github-actions bot added the Stale label Aug 30, 2025
@thompson-tomo
Copy link
Contributor Author

awaiting feedback from @open-telemetry/semconv-db-approvers

@github-actions github-actions bot removed the Stale label Aug 31, 2025
@github-actions
Copy link

This PR was marked stale due to lack of activity. It will be closed in 7 days.

@github-actions github-actions bot added the Stale label Sep 15, 2025
@thompson-tomo thompson-tomo deleted the feature/#1107_AddRavenDB branch September 22, 2025 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:db enhancement New feature or request Stale

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants