Skip to content

Commit 2b10a74

Browse files
committed
chore: remove logging check
1 parent 6471dcd commit 2b10a74

1 file changed

Lines changed: 0 additions & 33 deletions

File tree

  • rust/operator-binary/src/controller/build/properties/product_logging

rust/operator-binary/src/controller/build/properties/product_logging/mod.rs

Lines changed: 0 additions & 33 deletions
Original file line numberDiff line numberDiff line change
@@ -134,36 +134,3 @@ fn create_superset_config(log_config: &AutomaticContainerLogConfig, log_dir: &st
134134
.to_python_expression(),
135135
)
136136
}
137-
138-
#[cfg(test)]
139-
mod tests {
140-
use super::*;
141-
142-
/// The vendored `vector.yaml` keeps only the sources Superset produces and drops the ones it
143-
/// does not (log4j/log4j2/airlift/opa/tracing). Guards against accidental drift.
144-
#[test]
145-
fn test_vector_config_file_content() {
146-
let content = vector_config_file_content();
147-
assert!(!content.is_empty());
148-
// Superset logs JSON to `superset.py.json`, so the Python-JSON source must be present.
149-
assert!(content.contains("files_py"));
150-
assert!(content.contains("*.py.json"));
151-
// Sources Superset does not emit must have been trimmed out.
152-
for dropped in [
153-
"files_log4j",
154-
"files_log4j2",
155-
"files_airlift",
156-
"files_opa_json",
157-
"files_tracing_rs",
158-
] {
159-
assert!(
160-
!content.contains(dropped),
161-
"vendored vector.yaml should not contain the dropped source {dropped}"
162-
);
163-
}
164-
// The config is env-var-parameterized (resolved at runtime by the Vector container), not
165-
// baked, so the role-group identity must appear as placeholders.
166-
assert!(content.contains("${ROLE_NAME}"));
167-
assert!(content.contains("${VECTOR_AGGREGATOR_ADDRESS}"));
168-
}
169-
}

0 commit comments

Comments
 (0)