Don't include non-running pods in node capacity check (#2583)

metalbear-co · Jul 10, 2024 · b0f04a1 · b0f04a1
1 parent d2d4281
commit b0f04a1
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/changelog.d/2582.fixed.md b/changelog.d/2582.fixed.md
@@ -0,0 +1 @@
+Don't include non-running pods in node capacity check
diff --git a/mirrord/kube/src/api/runtime.rs b/mirrord/kube/src/api/runtime.rs
@@ -184,7 +184,10 @@ impl RuntimeData {
 
         let mut pod_count = 0;
         let mut list_params = ListParams {
-            field_selector: Some(format!("spec.nodeName={}", self.node_name)),
+            field_selector: Some(format!(
+                "status.phase=Running,spec.nodeName={}",
+                self.node_name
+            )),
             ..Default::default()
         };
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		Don't include non-running pods in node capacity check