unboundsoftware/schemas
6
0
Fork 0
Code Issues 1 Pull Requests 1 Actions Packages Projects Releases 32 Activity

refactor: remove Sentry integration and replace with OpenTelemetry #500

Merged
argoyle merged 1 commits from refactor-remove-sentry-replace-opentelemetry into main 2025-06-13 09:15:57 +00:00
Conversation 1 Commits 1 Files Changed 11
+306 -63
argoyle commented 2025-06-13 09:01:05 +00:00 (Migrated from gitlab.com)
Copy Link
Copy Source

Remove Sentry dependencies and configuration. Introduce monitoring
setup for OpenTelemetry. Update logging to include log format
options, and replace Sentry error handling middleware with
monitoring handlers for GraphQL playground. Adjust environment
variable handling to enhance configuration clarity and flexibility.

Remove Sentry dependencies and configuration. Introduce monitoring setup for OpenTelemetry. Update logging to include log format options, and replace Sentry error handling middleware with monitoring handlers for GraphQL playground. Adjust environment variable handling to enhance configuration clarity and flexibility.
argoyle (Migrated from gitlab.com) scheduled this pull request to auto merge when all checks succeed 2025-06-13 09:01:16 +00:00
argoyle commented 2025-06-13 09:08:04 +00:00 (Migrated from gitlab.com)
Copy Link
Copy Source

Codecov Report

Attention: Patch coverage is 0% with 131 lines in your changes missing coverage. Please review.

Project coverage is 1.00%. Comparing base (9539e6b) to head (e84df1d).

Files with missing lines Patch % Lines
monitoring/otel.go 0.00% 67 Missing ⚠️
monitoring/graphql.go 0.00% 25 Missing ⚠️
monitoring/span.go 0.00% 16 Missing ⚠️
logging/log.go 0.00% 13 Missing ⚠️
cmd/service/service.go 0.00% 10 Missing ⚠️
Additional details and impacted files 
@@           Coverage Diff            @@
##            main    #496      +/-   ##
========================================
- Coverage   1.04%   1.00%   -0.05%     
========================================
  Files         39      42       +3     
  Lines       1905    1996      +91     
========================================
  Hits          20      20              
- Misses      1885    1976      +91

☔ 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.
## [Codecov](https://app.codecov.io/gl/unboundsoftware/schemas/pull/496?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=gitlab&utm_content=comment&utm_campaign=pr+comments&utm_term=unboundsoftware) Report Attention: Patch coverage is `0%` with `131 lines` in your changes missing coverage. Please review. > Project coverage is 1.00%. Comparing base [(`9539e6b`)](https://app.codecov.io/gl/unboundsoftware/schemas/commit/9539e6bb1b9ff9c4ecc88f8cd5bfa1becd5e3614?dropdown=coverage&el=desc&utm_medium=referral&utm_source=gitlab&utm_content=comment&utm_campaign=pr+comments&utm_term=unboundsoftware) to head [(`e84df1d`)](https://app.codecov.io/gl/unboundsoftware/schemas/commit/e84df1db08f982b982f551a7bb70b7b5c05d4952?dropdown=coverage&el=desc&utm_medium=referral&utm_source=gitlab&utm_content=comment&utm_campaign=pr+comments&utm_term=unboundsoftware). | [Files with missing lines](https://app.codecov.io/gl/unboundsoftware/schemas/pull/496?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=gitlab&utm_content=comment&utm_campaign=pr+comments&utm_term=unboundsoftware) | Patch % | Lines | |---|---|---| | [monitoring/otel.go](https://app.codecov.io/gl/unboundsoftware/schemas/pull/496?src=pr&el=tree&filepath=monitoring%2Fotel.go&utm_medium=referral&utm_source=gitlab&utm_content=comment&utm_campaign=pr+comments&utm_term=unboundsoftware#diff-bW9uaXRvcmluZy9vdGVsLmdv) | 0.00% | [67 Missing :warning: ](https://app.codecov.io/gl/unboundsoftware/schemas/pull/496?src=pr&el=tree&utm_medium=referral&utm_source=gitlab&utm_content=comment&utm_campaign=pr+comments&utm_term=unboundsoftware) | | [monitoring/graphql.go](https://app.codecov.io/gl/unboundsoftware/schemas/pull/496?src=pr&el=tree&filepath=monitoring%2Fgraphql.go&utm_medium=referral&utm_source=gitlab&utm_content=comment&utm_campaign=pr+comments&utm_term=unboundsoftware#diff-bW9uaXRvcmluZy9ncmFwaHFsLmdv) | 0.00% | [25 Missing :warning: ](https://app.codecov.io/gl/unboundsoftware/schemas/pull/496?src=pr&el=tree&utm_medium=referral&utm_source=gitlab&utm_content=comment&utm_campaign=pr+comments&utm_term=unboundsoftware) | | [monitoring/span.go](https://app.codecov.io/gl/unboundsoftware/schemas/pull/496?src=pr&el=tree&filepath=monitoring%2Fspan.go&utm_medium=referral&utm_source=gitlab&utm_content=comment&utm_campaign=pr+comments&utm_term=unboundsoftware#diff-bW9uaXRvcmluZy9zcGFuLmdv) | 0.00% | [16 Missing :warning: ](https://app.codecov.io/gl/unboundsoftware/schemas/pull/496?src=pr&el=tree&utm_medium=referral&utm_source=gitlab&utm_content=comment&utm_campaign=pr+comments&utm_term=unboundsoftware) | | [logging/log.go](https://app.codecov.io/gl/unboundsoftware/schemas/pull/496?src=pr&el=tree&filepath=logging%2Flog.go&utm_medium=referral&utm_source=gitlab&utm_content=comment&utm_campaign=pr+comments&utm_term=unboundsoftware#diff-bG9nZ2luZy9sb2cuZ28=) | 0.00% | [13 Missing :warning: ](https://app.codecov.io/gl/unboundsoftware/schemas/pull/496?src=pr&el=tree&utm_medium=referral&utm_source=gitlab&utm_content=comment&utm_campaign=pr+comments&utm_term=unboundsoftware) | | [cmd/service/service.go](https://app.codecov.io/gl/unboundsoftware/schemas/pull/496?src=pr&el=tree&filepath=cmd%2Fservice%2Fservice.go&utm_medium=referral&utm_source=gitlab&utm_content=comment&utm_campaign=pr+comments&utm_term=unboundsoftware#diff-Y21kL3NlcnZpY2Uvc2VydmljZS5nbw==) | 0.00% | [10 Missing :warning: ](https://app.codecov.io/gl/unboundsoftware/schemas/pull/496?src=pr&el=tree&utm_medium=referral&utm_source=gitlab&utm_content=comment&utm_campaign=pr+comments&utm_term=unboundsoftware) | <details><summary>Additional details and impacted files</summary> ```diff @@ Coverage Diff @@ ## main #496 +/- ## ======================================== - Coverage 1.04% 1.00% -0.05% ======================================== Files 39 42 +3 Lines 1905 1996 +91 ======================================== Hits 20 20 - Misses 1885 1976 +91 ``` </details> [:umbrella: View full report in Codecov by Sentry](https://app.codecov.io/gl/unboundsoftware/schemas/pull/496?dropdown=coverage&src=pr&el=continue&utm_medium=referral&utm_source=gitlab&utm_content=comment&utm_campaign=pr+comments&utm_term=unboundsoftware). :loudspeaker: Have feedback on the report? [Share it here](https://gitlab.com/codecov-open-source/codecov-user-feedback/-/issues/4). <details><summary> :rocket: New features to boost your workflow: </summary> - :snowflake: [Test Analytics](https://docs.codecov.com/docs/test-analytics): Detect flaky tests, report on failures, and find test suite problems. </details>
argoyle (Migrated from gitlab.com) canceled auto merging this pull request when all checks succeed 2025-06-13 09:15:51 +00:00
argoyle (Migrated from gitlab.com) merged commit into main 2025-06-13 09:15:57 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
dependencies
Pull requests that update a dependency file
 docker
go
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
acctest (Shiny Acctest) argoyle (Joakim Olsson) buildtools (Buildtools) kubernetes (Kubernetes) releaser (Unbound Releaser) renovate (Renovate Bot)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: unboundsoftware/schemas#500
Delete Branch "refactor-remove-sentry-replace-opentelemetry"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?