New - add support for profiling flag

[cleverchuk]

Summary

Adds support for a remote PROFILING flag in the settings response, allowing the backend to control whether profiling is active. The profiling span processor now checks this flag in addition to local config before profiling.

Details

The existing SAMPLE_BUCKET_ENABLED flag (bit 0x40) was unused. This change repurposes that bit position as the new PROFILING flag, which the backend can set to remotely enable or disable profiling.

The isProfilingEnabled() check in SolarwindsProfilingSpanProcessor is extended: after confirming local profiling config is enabled, it calls TraceDecisionUtil.isProfilingEnabledByFlags() to consult remote settings. The logic respects the OVERRIDE flag semantics — when override is set, the remote PROFILING flag is authoritative; when override is not set (or remote settings are unavailable), profiling defaults to enabled (assuming local config allows it).

Key changes:

• Renamed OBOE_SETTINGS_FLAG_SAMPLE_BUCKET_ENABLED to OBOE_SETTINGS_FLAG_PROFILING in the Settings constants
• Updated flag parsing in RpcSettings and JsonSettingWrapper to recognize the "PROFILING" token
• Added TraceConfig.hasProfilingFlag() for querying the flag with null-safety on the flags field
• Added TraceDecisionUtil.isProfilingEnabledByFlags() which encapsulates the override-aware logic
• Added null-safety to TraceConfig.hasSampleTriggerTraceFlag() (defensive fix while modifying the class)
• Updated test fixtures (solarwinds-apm-settings.json) to use PROFILING instead of SAMPLE_BUCKET_ENABLED
• Extended SettingsStub builder with a isProfiling parameter for test flexibility

Test services data

1. e-1712644058766987264
2. e-1712643928659124224
3. e-1742334541200846848
4. e-1777406072376840192

[Copilot]

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

[Copilot]

Pull request overview

Copilot reviewed 15 out of 15 changed files in this pull request and generated 3 comments.

[jerrytfleung]

LGTM.

[Copilot]

Pull request overview

Copilot reviewed 17 out of 17 changed files in this pull request and generated 3 comments.

[jerrytfleung]

LGTM