rust: remove dead code

Signed-off-by: Piyush Jena <[email protected]>

Author: piyush-jena

sources/bloodhound/src/bin/kubernetes-cis-checks/checks.rs

@@ -522,53 +522,6 @@ impl Checker for K8S04020900Checker {
 
 // =>o.o<= =>o.o<= =>o.o<= =>o.o<= =>o.o<= =>o.o<= =>o.o<= =>o.o<= =>o.o<= =>o.o<=
 
-pub struct K8S04021000Checker {}
-
-// Not actually applicable for Bottlerocket, but leaving logic here in case we
-// make any changes in the future.
-impl Checker for K8S04021000Checker {
-    fn execute(&self, sac: &dyn SystemAccess) -> CheckerResult {
-        #[derive(Deserialize)]
-        struct KubeletConfig {
-            #[serde(rename = "rotateCertificates")]
-            rotate_certificates: bool,
-        }
-
-        let mut result = CheckerResult::default();
-
-        if let Ok(kubelet_file) = sac.open(KUBELET_CONF_FILE) {
-            if let Ok(config) = serde_yaml::from_reader::<_, KubeletConfig>(kubelet_file) {
-                if !config.rotate_certificates {
-                    result.error = "Kubelet rotateCertificates is disabled".to_string();
-                    result.status = CheckStatus::FAIL;
-                } else {
-                    result.status = CheckStatus::PASS;
-                }
-            } else {
-                // Default value is `false`, so it is a failure if this is not in the config file.
-                result.error = "Kubelet rotateCertificates is disabled".to_string();
-                result.status = CheckStatus::FAIL;
-            }
-        } else {
-            result.error = format!("unable to read '{KUBELET_CONF_FILE}'");
-        }
-
-        result
-    }
-
-    fn metadata(&self) -> CheckerMetadata {
-        CheckerMetadata {
-            title: "Ensure that the --rotate-certificates argument is not set to false".to_string(),
-            id: "4.2.10".to_string(),
-            level: 1,
-            name: "k8s04021000".to_string(),
-            mode: Mode::Automatic,
-        }
-    }
-}
-
-// =>o.o<= =>o.o<= =>o.o<= =>o.o<= =>o.o<= =>o.o<= =>o.o<= =>o.o<= =>o.o<= =>o.o<=
-
 pub struct K8S04021100Checker {}
 
 impl Checker for K8S04021100Checker {
@@ -1476,51 +1429,6 @@ tlsPrivateKeyFile: ""
         assert_eq!(result.status, CheckStatus::SKIP);
     }
 
-    // K8S04021000Checker tests - rotate certificates
-    #[test]
-    pub fn test_k8s04021000checker_pass() {
-        let mut sac = UnitTestSystemAccess::default();
-        let config = r#"
-rotateCertificates: true
-"#;
-        sac.register_file(KUBELET_CONF_FILE, config);
-        let checker = K8S04021000Checker {};
-        let result = checker.execute(&sac);
-        assert_eq!(result.status, CheckStatus::PASS);
-    }
-
-    #[test]
-    pub fn test_k8s04021000checker_fail_disabled() {
-        let mut sac = UnitTestSystemAccess::default();
-        let config = r#"
-rotateCertificates: false
-"#;
-        sac.register_file(KUBELET_CONF_FILE, config);
-        let checker = K8S04021000Checker {};
-        let result = checker.execute(&sac);
-        assert_eq!(result.status, CheckStatus::FAIL);
-    }
-
-    #[test]
-    pub fn test_k8s04021000checker_fail_not_present() {
-        let mut sac = UnitTestSystemAccess::default();
-        let config = r#"
-someOtherSetting: value
-"#;
-        sac.register_file(KUBELET_CONF_FILE, config);
-        let checker = K8S04021000Checker {};
-        let result = checker.execute(&sac);
-        assert_eq!(result.status, CheckStatus::FAIL);
-    }
-
-    #[test]
-    pub fn test_k8s04021000checker_file_missing() {
-        let sac = UnitTestSystemAccess::default();
-        let checker = K8S04021000Checker {};
-        let result = checker.execute(&sac);
-        assert_eq!(result.status, CheckStatus::SKIP);
-    }
-
     // K8S04021100Checker tests - rotate kubelet server certificate
     #[test]
     pub fn test_k8s04021100checker_pass_enabled() {

sources/netdog/src/cli/mod.rs

@@ -14,7 +14,7 @@ use crate::{
 pub(crate) use generate_hostname::GenerateHostnameArgs;
 pub(crate) use generate_net_config::GenerateNetConfigArgs;
 pub(crate) use node_ip::NodeIpArgs;
-use serde::{Deserialize, Serialize};
+use serde::Serialize;
 pub(crate) use set_hostname::SetHostnameArgs;
 use snafu::{ensure, OptionExt, ResultExt};
 use std::fmt::Write;
@@ -25,24 +25,6 @@ use std::{fs, io};
 pub(crate) use write_network_status::WriteNetworkStatusArgs;
 pub(crate) use write_resolv_conf::WriteResolvConfArgs;
 
-#[derive(Debug, PartialEq, Deserialize)]
-#[serde(rename_all = "kebab-case")]
-enum InterfaceType {
-    Dhcp,
-    Static,
-}
-
-#[derive(Debug, PartialEq, Deserialize)]
-#[serde(rename_all = "kebab-case")]
-enum InterfaceFamily {
-    Ipv4,
-    Ipv6,
-}
-
-// Implement `from_str()` so argh can attempt to deserialize args into their proper types
-derive_fromstr_from_deserialize!(InterfaceType);
-derive_fromstr_from_deserialize!(InterfaceFamily);
-
 /// Helper function that serializes the input to JSON and prints it
 fn print_json<S>(val: S) -> Result<()>
 where

sources/netdog/src/main.rs

@@ -26,7 +26,6 @@ supplementing any missing settings with DNS settings from the primary interface'
 is meant to be used as a restart command for DNS API settings.
 */
 
-#[macro_use]
 extern crate serde_plain;
 
 mod addressing;

sources/netdog/src/net_config/devices/mod.rs

@@ -42,15 +42,6 @@ impl Validate for NetworkDeviceV1 {
     }
 }
 
-#[derive(Debug, Deserialize)]
-#[serde(untagged)]
-pub(crate) enum DeviceType {
-    #[serde(rename = "bond")]
-    Bond,
-    #[serde(rename = "vlan")]
-    Vlan,
-}
-
 pub(crate) trait HasIpAddressing {
     fn has_static(&self) -> bool;