OTEL_JAVAAGENT_DEBUG enables okhttp debug

[anuraaga]

I'm surprised by this since I thought we only enable debug logging for io.opentelemetry, but I noticed that when enabling debug logging there is a lot of spam from okhttp like this

app_1        | [otel.javaagent 2021-12-07 04:39:19:400 +0000] [OkHttp http://otel:4317/...] DEBUG okhttp3.internal.concurrent.TaskRunner - Q10002 scheduled after   0 µs: OkHttp ConnectionPool
app_1        | [otel.javaagent 2021-12-07 04:39:19:400 +0000] [OkHttp TaskRunner] DEBUG okhttp3.internal.concurrent.TaskRunner - Q10002 starting              : OkHttp ConnectionPool
app_1        | [otel.javaagent 2021-12-07 04:39:19:400 +0000] [OkHttp ConnectionPool] DEBUG okhttp3.internal.concurrent.TaskRunner - Q10002 run again after 300 s : OkHttp ConnectionPool

[laurit]

I though that it was done deliberately, enabling debug also enables debug for all bundled libraries. For loggers that are too chatty set log level to info https://github.com/open-telemetry/opentelemetry-java-instrumentation/blob/main/javaagent-tooling/src/main/java/io/opentelemetry/javaagent/tooling/LoggingConfigurer.java

[trask]

For loggers that are too chatty set log level to info https://github.com/open-telemetry/opentelemetry-java-instrumentation/blob/main/javaagent-tooling/src/main/java/io/opentelemetry/javaagent/tooling/LoggingConfigurer.java

Here's the updated location now:

opentelemetry-java-instrumentation/javaagent-internal-logging-simple/src/main/java/io/opentelemetry/javaagent/logging/simple/Slf4jSimpleLoggingCustomizer.java

Lines 39 to 42 in 2ebf413

	if (earlyConfig.getBoolean("otel.javaagent.debug", false)) {
	setSystemPropertyDefault(SIMPLE_LOGGER_DEFAULT_LOG_LEVEL_PROPERTY, "DEBUG");
	setSystemPropertyDefault(SIMPLE_LOGGER_PREFIX + "okhttp3.internal.http2", "INFO");
	}

[trask]

Closing, can revisit later if anyone has renewed interest.

[laurit]

okhttp3.internal.concurrent.TaskRunner was excluded from debug logs in #9814