Skip to content

Commit f8d1936

Browse files
authored
Merge branch 'main' into feature/dns-warning
2 parents c377dc4 + a7f9d9b commit f8d1936

File tree

2 files changed

+6
-1
lines changed

2 files changed

+6
-1
lines changed

CHANGELOG.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,10 @@ All notable changes to this project will be documented in this file.
77
### Changed
88

99
- Downgraded DNS errors to warnings ([#17]).
10+
- All output is now wrapped in a "containerdebug" span ([#18]).
1011

1112
[#17]: https://github.com/stackabletech/containerdebug/pull/17
13+
[#18]: https://github.com/stackabletech/containerdebug/pull/18
1214

1315
## [0.1.0] - 2024-12-09
1416

src/main.rs

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,10 @@ fn main() {
4242
APP_NAME,
4343
opts.tracing_target,
4444
);
45+
46+
// Wrap *all* output in a span, to separate it from main app output.
47+
let _span = tracing::error_span!("containerdebug").entered();
48+
4549
stackable_operator::utils::print_startup_string(
4650
crate_description!(),
4751
crate_version!(),
@@ -62,7 +66,6 @@ fn main() {
6266
let system_information = SystemInformation::collect();
6367

6468
let serialized = serde_json::to_string_pretty(&system_information).unwrap();
65-
// println!("{serialized}");
6669
if let Some(output_path) = &opts.output {
6770
std::fs::write(output_path, &serialized).unwrap();
6871
}

0 commit comments

Comments
 (0)