Skip to content

Conversation

@codeboten
Copy link
Contributor

@codeboten codeboten commented Nov 20, 2025

This is an attempt at producing a separate otelconf/x package to support experimental features in the configuration schema. This is doing the bare minimum of using a sed patch to hide all experimental structs from the generated model code before moving it to the otelconf package. This has the upside of being pretty straight forward to do, but ends up with a lot of duplicate code.

Part of #8203

Signed-off-by: alex boten <[email protected]>
Signed-off-by: alex boten <[email protected]>
Signed-off-by: alex boten <[email protected]>
Signed-off-by: alex boten <[email protected]>
Signed-off-by: alex boten <[email protected]>
* added validation for various JSON unmarshallers
* updated tests to ensure expected model matches the test file
* add custom unmarshalers to support nil values for certain config options

Signed-off-by: alex boten <[email protected]>
Signed-off-by: alex boten <[email protected]>
Signed-off-by: alex boten <[email protected]>
Signed-off-by: alex boten <[email protected]>
Signed-off-by: alex boten <[email protected]>
Signed-off-by: alex boten <[email protected]>
Signed-off-by: alex boten <[email protected]>
Signed-off-by: alex boten <[email protected]>
Signed-off-by: alex boten <[email protected]>
Signed-off-by: alex boten <[email protected]>
Signed-off-by: alex boten <[email protected]>
Signed-off-by: alex boten <[email protected]>
codeboten and others added 18 commits November 6, 2025 08:39
Signed-off-by: alex boten <[email protected]>
Signed-off-by: alex boten <[email protected]>
Signed-off-by: alex boten <[email protected]>
Signed-off-by: alex boten <[email protected]>
Signed-off-by: alex boten <[email protected]>
This is an attempt at producing a separate `otelconf/x` package to support
experimental features in the configuration schema. This is doing the bare minimum
of using a `sed` patch to hide all experimental structs from the generated model
code before moving it to the `otelconf` package. This has the upside of being
pretty straight forward to do, but ends up with a lot of duplicate code.

Part of open-telemetry#8203

Signed-off-by: alex boten <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants