Description

The UnifiedConfig.OTelLoggingSupported method requires the Merged Unified Config (the result of combining the user set config and the built in config ) to determine correctly if "OTel Logging" supports the presented logging pipelines.

This breaks the Feature Tracking design of only analyzing the user configuration, but the alternatives (e.g. refactor Unified Config) would need refactoring in other packages or present other issues (e.g. apps and confgenerator dependency cycle). The mergedUc is only used for getOTelLoggingSupportedConfig.

The use for otel_logging_supported_config will be removed when Otel Logging Support is fully released (see b/399354366). Added TODO to track this.

This is a followup the PR's :

• Create otel_logging_supported_config feature tracking metric. #1861
• confgenerator : Fix otel logging config supported detection by setting ReceiverMixin as property #1897

Explanation

The merged config is required to be able to match exactly the OTel Logging Support feedback a user would get when trying to enable experimental_otel_logging: true from the congenerator config validation. The config validation uses the merged config with the built in config pipelines. When using only the user config, there are edge cases (see goldens example ) where a logging processors might not detected as not supported (e.g. parse_regex in the example).

Related issue

• b/390671495
• b/399354366

How has this been tested?

Unit tests were modified.

Checklist:

• Unit tests
  • Unit tests do not apply.
  • Unit tests have been added/modified and passed for this PR.
• Integration tests
  • Integration tests do not apply.
  • [] Integration tests have been added/modified and passed for this PR.
• Documentation
  • This PR introduces no user visible changes.
  • This PR introduces user visible changes and the corresponding documentation change has been made.
• Minor version bump
  • This PR introduces no new features.
  • This PR introduces new features, and there is a separate PR to bump the minor version since the last release already.
  • This PR bumps the version.