Skip to content

Commit

Permalink
Refactor pipelines builder, fix some issues (open-telemetry#5512)
Browse files Browse the repository at this point in the history
* Unconfigured receivers are not identified, this was not a real problem in final binaries since the validation of the config catch this.
* Allow configurations to contain "unused" receivers. Receivers that are configured but not used in any pipeline, this was possible already for exporters and processors.
* Remove the enforcement/check that Receiver factories create the same instance for the same config.

Signed-off-by: Bogdan Drutu <[email protected]>
  • Loading branch information
bogdandrutu authored Jun 21, 2022
1 parent 31b2f5c commit b05c0f3
Show file tree
Hide file tree
Showing 15 changed files with 611 additions and 1,594 deletions.
5 changes: 4 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,10 @@

- Update sum field of exponential histograms to make it optional (#5530)
- Remove redundant extension shutdown call (#5532)

- Refactor pipelines builder, fix some issues (#5512)
- Unconfigured receivers are not identified, this was not a real problem in final binaries since the validation of the config catch this.
- Allow configurations to contain "unused" receivers. Receivers that are configured but not used in any pipeline, this was possible already for exporters and processors.
- Remove the enforcement/check that Receiver factories create the same instance for the same config.
## v0.53.0 Beta

### 🛑 Breaking changes 🛑
Expand Down
188 changes: 0 additions & 188 deletions service/internal/builder/exporters_builder.go

This file was deleted.

114 changes: 0 additions & 114 deletions service/internal/builder/exporters_builder_test.go

This file was deleted.

77 changes: 0 additions & 77 deletions service/internal/builder/factories_test.go

This file was deleted.

Loading

0 comments on commit b05c0f3

Please sign in to comment.