continue on malformed IP, remove Default impl for TrafficPolicy

Signed-off-by: Zahari Dichev <[email protected]>
linkerd · Oct 9, 2024 · 67ae5c8 · 67ae5c8
1 parent fa5007c
commit 67ae5c8
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 9 deletions.
diff --git a/policy-controller/k8s/api/src/policy/unmeshed_network.rs b/policy-controller/k8s/api/src/policy/unmeshed_network.rs
@@ -13,17 +13,11 @@ use serde::{Deserialize, Serialize};
 #[serde(rename_all = "camelCase")]
 pub struct UnmeshedNetworkSpec {
     pub networks: Vec<Cidr>,
-    pub traffic_policy: DefaultPolicy,
+    pub traffic_policy: TrafficPolicy,
 }
 
 #[derive(Clone, Debug, PartialEq, Eq, Deserialize, Serialize, JsonSchema)]
-pub enum DefaultPolicy {
+pub enum TrafficPolicy {
     AllowUnknown,
     DenyUnknown,
 }
-
-impl Default for DefaultPolicy {
-    fn default() -> Self {
-        Self::AllowUnknown
-    }
-}
diff --git a/policy-controller/k8s/index/src/outbound/index.rs b/policy-controller/k8s/index/src/outbound/index.rs
@@ -275,7 +275,7 @@ impl kubert::index::IndexNamespacedResource<Pod> for Index {
                         Ok(addr) => addr,
                         Err(error) => {
                             tracing::error!(%error, "malformed pod IP: {ip}");
-                            return;
+                            continue;
                         }
                     };