Skip to content
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Comments updated
  • Loading branch information
robsunday committed Dec 5, 2025
commit da19a8e151dfdfc2f68e1a17ccdd42b9941ee53e
Original file line number Diff line number Diff line change
Expand Up @@ -58,11 +58,10 @@ public static DeclarativeConfigProperties getDistributionConfig(
}

// TODO: This is temporary solution until distribution config support is implemented in the
// upstream.
// For now assume that distribution node is located under
// .instrumentation/development.java.distribution
// upstream. For now assume that distribution node is located under
// .instrumentation/development.java.distribution
// Replace this code with `return sdk.getConfigProvider().getDistributionConfig()` once is
// implemented
// implemented
DeclarativeConfigProperties instrumentationConfig = configProvider.getInstrumentationConfig();
if (instrumentationConfig == null) {
return null;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -231,6 +231,8 @@ void shouldThrowExceptionForInvalidProtocol() {
.isInstanceOf(ConfigurationException.class);
}

// TODO: Once ConfigProvider exposes ".distribution" config this method and YAMLs in tests must be
// updated to use valid location. For now it is temporary placed under instrumentation
private static DeclarativeConfigProperties getExporterConfig(
OpenTelemetryConfigurationModel model) {
Map<String, Object> properties =
Expand Down
Loading