Skip to content

CV2-6699: Disable Honeycomb#2423

Merged
melsawy merged 5 commits into
developfrom
CV2-6699-disable-honeycomb
Apr 15, 2026
Merged

CV2-6699: Disable Honeycomb#2423
melsawy merged 5 commits into
developfrom
CV2-6699-disable-honeycomb

Conversation

@melsawy
Copy link
Copy Markdown
Contributor

@melsawy melsawy commented Apr 14, 2026

Description

Remove Honeycomb code and keep only the part related to CheckStatistics

Reference commit

References: CV2-6699

How to test?

Re-run automated tests

Checklist

  • I have performed a self-review of my code and ensured that it is safe and runnable, that code coverage has not decreased, and that there are no new Code Climate issues. I have also followed Meedan's internal coding guidelines.

Comment thread config/initializers/zz_open_telemetry.rb
@melsawy melsawy changed the title CV2-6699: Remove Telemetry CV2-6699: Disable Honeycomb Apr 14, 2026
@melsawy melsawy marked this pull request as ready for review April 14, 2026 19:51
Copy link
Copy Markdown
Contributor

@caiosba caiosba left a comment

Choose a reason for hiding this comment

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

LGTM, but please wait for @vasconsaurus confirmation since she worked on that.

Copy link
Copy Markdown
Contributor

@vasconsaurus vasconsaurus left a comment

Choose a reason for hiding this comment

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

I worked mostly on implementing app metrics in Pender, so I'm not super familiar with the Check-API side, and not familiar with how it intersects with CheckStatistics.

But, I looked into teh reference commit, and if we are moving away from Honeycomb and fully discard tracing with Otel, this looks good to me! I left just one question.

Comment thread config/initializers/zz_open_telemetry.rb
@melsawy melsawy merged commit 2ef7de4 into develop Apr 15, 2026
10 checks passed
@melsawy melsawy deleted the CV2-6699-disable-honeycomb branch April 15, 2026 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants