fix not ready container detection

edrevo · Nov 20, 2019 · 7a1d31e · 7a1d31e
1 parent 0e8e595
commit 7a1d31e
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 7 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/Cargo.toml b/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "suspicious-pods"
-version = "0.5.0"
+version = "0.5.1"
 authors = ["edrevo <[email protected]>"]
 edition = "2018"
 keywords = ["k8s", "kubernetes"]
@@ -14,4 +14,4 @@ repository = "https://github.com/edrevo/suspicious-pods"
 [dependencies]
 clap = "2.33.0"
 failure = "0.1.6"
-suspicious-pods-lib = { version="0.3.0", path = "lib" }
+suspicious-pods-lib = { version="0.3.1", path = "lib" }
diff --git a/lib/Cargo.toml b/lib/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "suspicious-pods-lib"
-version = "0.3.0"
+version = "0.3.1"
 authors = ["edrevo <[email protected]>"]
 edition = "2018"
 keywords = ["k8s", "kubernetes"]

diff --git a/lib/src/lib.rs b/lib/src/lib.rs
@@ -83,7 +83,7 @@ fn is_suspicious_container(pod_name: &str, status: ContainerStatus) -> Option<Su
     Some(SuspiciousContainerReason::ContainerWaiting(msg))
   } else if state.terminated.is_some() && state.terminated.as_ref().unwrap().exit_code != 0 {
     Some(SuspiciousContainerReason::TerminatedWithError(state.terminated.unwrap().exit_code))
-  } else if !status.ready {
+  } else if state.running.is_some() && !status.ready {
     Some(SuspiciousContainerReason::NotReady)
   } else {
     None