[improve][misc] Disable OTel by default when running the pulsar-perf tool

[Shawyeok]

Fixes #23475

Motivation

Since pulsar 3.3.0 (#22179), pulsar-perf constantly print warning message on the console if user haven't configured a OTel-compatible metric collector in their environment. To align with broker, proxy, and other components, OTel metric collection should be disabled in pulsar-perf by default. #22010

With this patch, the following log will appear before running the performance test:

Nov 10, 2024 3:36:19 PM io.opentelemetry.api.GlobalOpenTelemetry maybeAutoConfigureAndSetGlobal
INFO: AutoConfiguredOpenTelemetrySdk found on classpath but automatic configuration is disabled. To enable, run your JVM with -Dotel.java.global-autoconfigure.enabled=true

Verifying this change

• Make sure that the change passes the CI checks.

This change is a trivial rework / code cleanup without any test coverage.

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

• Dependencies (add or upgrade a dependency)
• The public API
• The schema
• The default values of configurations
• The threading model
• The binary protocol
• The REST endpoints
• The admin CLI options
• The metrics
• Anything that affects deployment

Documentation

• doc
• doc-required
• doc-not-needed
• doc-complete

Matching PR in forked repository

PR in forked repository:

[lhotari]

An alternative would be to have OTel configured, but disabled by default as it's done in Pulsar's OpenTelemetryService:

pulsar/pulsar-opentelemetry/src/main/java/org/apache/pulsar/opentelemetry/OpenTelemetryService.java

Lines 74 to 75 in dd1b579

	sdkBuilder.addPropertiesSupplier(() -> Map.of(
	OTEL_SDK_DISABLED_KEY, "true",

.

@dragosvictor Please review this PR and suggest a suitable approach.

[Shawyeok]

@lhotari
Thanks for pointed out, otel.sdk.disabled=true is documented in PIP-320, so I think it would be better have a consistent toggle option to enable OTel in pulsar.

[lhotari]

LGTM

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.23%. Comparing base (bbc6224) to head (55b8b36).
Report is 756 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff              @@
##             master   #23585      +/-   ##
============================================
+ Coverage     73.57%   74.23%   +0.66%     
+ Complexity    32624     3353   -29271     
============================================
  Files          1877     1945      +68     
  Lines        139502   149262    +9760     
  Branches      15299    16698    +1399     
============================================
+ Hits         102638   110809    +8171     
- Misses        28908    29876     +968     
- Partials       7956     8577     +621     

Flag	Coverage Δ
inttests	27.99% <ø> (+3.40%)	⬆️
systests	24.88% <ø> (+0.56%)	⬆️
unittests	73.76% <ø> (+0.92%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 670 files with indirect coverage changes

[lhotari]

Workaround for 4.0.0

export OTEL_SDK_DISABLED=true