Fixed missing pods/deployments with more than 1 container when using …

…operator ls (#2470)
metalbear-co · May 29, 2024 · 65a8022 · 65a8022
1 parent 329f470
commit 65a8022
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/changelog.d/+operator-ls-sidecars.fixed.md b/changelog.d/+operator-ls-sidecars.fixed.md
@@ -0,0 +1 @@
+Fixed missing pods/deployments with more than 1 container when using operator ls
diff --git a/mirrord/cli/src/main.rs b/mirrord/cli/src/main.rs
@@ -520,7 +520,7 @@ async fn print_targets(args: &ListTargetArgs) -> Result<()> {
                         .filter_map(|target_crd| {
                             let target = target_crd.spec.target.as_ref()?;
                             if let Some(container) = target.container_name() {
-                                if !SKIP_NAMES.contains(container.as_str()) {
+                                if SKIP_NAMES.contains(container.as_str()) {
                                     return None;
                                 }
                             }
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		Fixed missing pods/deployments with more than 1 container when using operator ls