Skip to content

JM: Hide OpenTelemetry install lines from boot output - #2838

Open
maebeale wants to merge 1 commit into
mainfrom
maebeale/opentelemetry-instrumentation-hide
Open

maebeale wants to merge 1 commit into
mainfrom
maebeale/opentelemetry-instrumentation-hide

Conversation

@maebeale

@maebeale maebeale commented Oct 25, 2026 •

Copy link
Copy Markdown
Collaborator

🤖 suggested review level: 3 Read 📖 small logger filter in one initializer, prod-only

Make DO easier to use bc we don't get so much noise in opening/running console

Rails console and every web/job boot in production no longer print ~15 "Instrumentation … was successfully installed" lines.

What is the goal of this PR and why is this important?

  • The install lines bury the output you actually want when opening a prod console.
  • They only confirm setup; traces arriving in the backend prove the same thing.

How did you approach the change?

  • Custom OTel logger drops only messages starting Instrumentation: and containing was successfully installed.
  • OTel's other INFO lines, plus WARN/ERROR (exporter failures, install errors), still print.
  • Kept the log level at INFO on purpose — raising it to WARN would hide any future INFO messages too.

Anything else to add?

  • Initializer is skipped in dev/test (Rails.env.local?), so no spec runs it.
    • Checked by loading it in a standalone script: install line suppressed, other INFO and WARN lines printed.
  • If the gem rewords the message, the lines just come back. Nothing breaks.
Screenshot 2026-10-25 at 5 18 21 AM

🤖 Generated with Claude Code

Every console, web, and job boot printed ~15 INFO lines confirming each
instrumentation installed, burying the useful output. Filtering only that
message keeps OTel's other INFO, WARN, and ERROR lines visible.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@maebeale
maebeale marked this pull request as ready for review October 25, 2026 09:10
@maebeale
maebeale requested a review from jmilljr24 October 25, 2026 09:10
@maebeale maebeale changed the title Hide OpenTelemetry install lines from boot output JM: Hide OpenTelemetry install lines from boot output Oct 25, 2026
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.

1 participant