feat: OTEL traces integration
alternative to #2434
Taking into account some of the reviews on #2434
I tried to integrate traces first on gRPC server.
This already works on a basic form but we still need to identify important places were to put the traces
Added more traces direclty on rpc handlers
๐ PR Checks Summary
All Automated Checks passed. โ
Manual Checks (for Reviewers):
- [ ] IGNORE the bot requirements for this PR (force green CI check)
Read More
๐ค This bot helps streamline PR reviews by verifying automated checks and providing guidance for contributors and reviewers.
โ Automated Checks (for Contributors):
๐ข Maintainers must be able to edit this pull request (more info) ๐ข Pending initial approval by a review team member, or review from tech-staff
โ๏ธ Contributor Actions:
- Fix any issues flagged by automated checks.
- Follow the Contributor Checklist to ensure your PR is ready for review.
- Add new tests, or document why they are unnecessary.
- Provide clear examples/screenshots, if necessary.
- Update documentation, if required.
- Ensure no breaking changes, or include
BREAKING CHANGEnotes. - Link related issues/PRs, where applicable.
โ๏ธ Reviewer Actions:
- Complete manual checks for the PR, including the guidelines and additional checks if applicable.
๐ Resources:
Debug
Automated Checks
Maintainers must be able to edit this pull request (more info)
If
๐ข Condition met โโโ ๐ข And โโโ ๐ข The base branch matches this pattern: ^master$ โโโ ๐ข The pull request was created from a fork (head branch repo: Villaquiranm/gno)Then
๐ข Requirement satisfied โโโ ๐ข Maintainer can modify this pull requestPending initial approval by a review team member, or review from tech-staff
If
๐ข Condition met โโโ ๐ข And โโโ ๐ข The base branch matches this pattern: ^master$ โโโ ๐ข Not (๐ด Pull request author is a member of the team: tech-staff)Then
๐ข Requirement satisfied โโโ ๐ข If โโโ ๐ข Condition โ โโโ ๐ข Or โ โโโ ๐ด At least one of these user(s) reviewed the pull request: [jefft0 leohhhn n0izn0iz notJoon omarsy x1unix] (with state "APPROVED") โ โโโ ๐ข At least 1 user(s) of the team tech-staff reviewed pull request โ โโโ ๐ด This pull request is a draft โโโ ๐ข Then โโโ ๐ข Not (๐ด This label is applied to pull request: review/triage-pending)Manual Checks
**IGNORE** the bot requirements for this PR (force green CI check)
If
๐ข Condition met โโโ ๐ข On every pull requestCan be checked by
- Any user with comment edit permission
Codecov Report
:x: Patch coverage is 17.69912% with 93 lines in your changes missing coverage. Please review.
:loudspeaker: Thoughts on this report? Let us know!
Looks good ๐ฏ
Please drop the gRPC mode for the telemetry entirely ๐
Thanks for the review Miloลก, I think I have answered all your comments
LGTM