From d5d3f9cda75a652f5ca1515396af294446cdb5aa Mon Sep 17 00:00:00 2001
From: Archana Choudhary <archana1@microsoft.com>
Date: Sun, 21 Apr 2024 12:18:08 +0000
Subject: [PATCH] genpolicy: add default implementation for use_host_network

- Provide default implementation for use_host_network
- Remove default implementation from structs implementing the trait K8sResource

Fixes: #8960
Signed-off-by: Archana Choudhary <archana1@microsoft.com>
---
 src/tools/genpolicy/src/config_map.rs | 4 ----
 src/tools/genpolicy/src/list.rs       | 4 ----
 src/tools/genpolicy/src/no_policy.rs  | 4 ----
 src/tools/genpolicy/src/secret.rs     | 4 ----
 src/tools/genpolicy/src/yaml.rs       | 5 ++++-
 5 files changed, 4 insertions(+), 17 deletions(-)

diff --git a/src/tools/genpolicy/src/config_map.rs b/src/tools/genpolicy/src/config_map.rs
index 6f46d07753..34d822b451 100644
--- a/src/tools/genpolicy/src/config_map.rs
+++ b/src/tools/genpolicy/src/config_map.rs
@@ -99,10 +99,6 @@ impl yaml::K8sResource for ConfigMap {
         &self.metadata.annotations
     }
 
-    fn use_host_network(&self) -> bool {
-        panic!("Unsupported");
-    }
-
     fn use_sandbox_pidns(&self) -> bool {
         panic!("Unsupported");
     }
diff --git a/src/tools/genpolicy/src/list.rs b/src/tools/genpolicy/src/list.rs
index 834f8c334f..7b99b2b139 100644
--- a/src/tools/genpolicy/src/list.rs
+++ b/src/tools/genpolicy/src/list.rs
@@ -81,10 +81,6 @@ impl yaml::K8sResource for List {
         serde_yaml::to_string(&self).unwrap()
     }
 
-    fn use_host_network(&self) -> bool {
-        panic!("Unsupported");
-    }
-
     fn use_sandbox_pidns(&self) -> bool {
         panic!("Unsupported");
     }
diff --git a/src/tools/genpolicy/src/no_policy.rs b/src/tools/genpolicy/src/no_policy.rs
index e78bc17214..9face779e8 100644
--- a/src/tools/genpolicy/src/no_policy.rs
+++ b/src/tools/genpolicy/src/no_policy.rs
@@ -35,10 +35,6 @@ impl yaml::K8sResource for NoPolicyResource {
         self.yaml.clone()
     }
 
-    fn use_host_network(&self) -> bool {
-        panic!("Unsupported");
-    }
-
     fn use_sandbox_pidns(&self) -> bool {
         panic!("Unsupported");
     }
diff --git a/src/tools/genpolicy/src/secret.rs b/src/tools/genpolicy/src/secret.rs
index 12dcb1fd31..3ebb7e8ba1 100644
--- a/src/tools/genpolicy/src/secret.rs
+++ b/src/tools/genpolicy/src/secret.rs
@@ -84,10 +84,6 @@ impl yaml::K8sResource for Secret {
         serde_yaml::to_string(&self.doc_mapping).unwrap()
     }
 
-    fn use_host_network(&self) -> bool {
-        panic!("Unsupported");
-    }
-
     fn use_sandbox_pidns(&self) -> bool {
         panic!("Unsupported");
     }
diff --git a/src/tools/genpolicy/src/yaml.rs b/src/tools/genpolicy/src/yaml.rs
index def95ccaa1..a5b1034d66 100644
--- a/src/tools/genpolicy/src/yaml.rs
+++ b/src/tools/genpolicy/src/yaml.rs
@@ -78,7 +78,10 @@ pub trait K8sResource {
         panic!("Unsupported");
     }
 
-    fn use_host_network(&self) -> bool;
+    fn use_host_network(&self) -> bool {
+        panic!("Unsupported");
+    }
+
     fn use_sandbox_pidns(&self) -> bool;
 }