Commit 5b152c7
refactor(stackable-telemetry)!: Improve stackable-telemetry to make it easier to build the TraceGuard (#901)
* chore(stackable-telemetry): move tracing to a module directory
* feat(stackable-telemetry): Add a settings builder and implement it for console logging
* feat(stackable-telemetry): Add a settings builder for the otlp trace exporter
* feat(stackable-telemetry): Add a settings builder for the otlp log exporter
* chore(stackable-telemetry): Make settings available to the Tracing::init() function
* chore(stackable-telemetry): Introduce the CommonSettings trait for quicker access to common settings from subscriber specific settings
* refactor(stackable-telemetry): Rename env_var() for consistency, reorder fields
* chore(stackable-telemetry): Add SettingsDouble and SettingsTriple tuple and From impls as a config shortcut similar to before this PR
* chore(stackable-telemetry): Resolve clippy warnings
* refactor(stackable-telemetry): Prefix builder methods with `with_`.
I avoided prefixing `enabled()` because it sounds strange.
* refactor(stackable-telemetry): Revert SettingsDouble/SettingsTriple
Note: They will appear in a future PR
* docs(stackable-telemetry): Partially add docs
Note: More docs will appear in a future PR so that this PR isn't a blocker.
Tip: Enable `#![warn(missing_docs)]`
* refactor: Simplify From and Build trait usages
* refactor: Provide centralized default impls for CommonSettings trait
* docs: Fix doc tests
* docs: Fix doc comment references
* refactor: Use Deref instead of CommonSettings trait
* chore(stackable-telemetry): Apply suggestions from code review
Co-authored-by: Techassi <sascha.lautenschlaeger@stackable.tech>
* chore(stackable-telemetry): Update changelog
---------
Co-authored-by: Techassi <git@techassi.dev>
Co-authored-by: Techassi <sascha.lautenschlaeger@stackable.tech>1 parent 41c439c commit 5b152c7
File tree
6 files changed
+576
-100
lines changed- crates/stackable-telemetry
- src/tracing
- settings
6 files changed
+576
-100
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
7 | 18 | | |
8 | 19 | | |
9 | 20 | | |
| |||
0 commit comments