Remove admin-cli for site-explorer and machine level overrides

Signed-off-by: aadvani <aadvani@nvidia.com>

Author: aadvani-nvidia

crates/admin-cli/src/machine/mod.rs

@@ -25,7 +25,6 @@ pub mod network;
 pub mod nvlink_info;
 pub mod positions;
 pub mod reboot;
-pub mod restart_ovs;
 pub mod show;
 
 #[cfg(test)]
@@ -79,9 +78,4 @@ pub enum Cmd {
     Positions(positions::Args),
     #[clap(subcommand, about = "Update/show NVLink info for an MNNVL machine")]
     NvlinkInfo(nvlink_info::Args),
-    #[clap(
-        name = "restart-ovs-on-use-admin-network-change",
-        about = "Set OVS restart on admin network change for a machine"
-    )]
-    RestartOvs(restart_ovs::Args),
 }

crates/admin-cli/src/machine/restart_ovs/args.rs

@@ -26,7 +26,6 @@ mod is_bmc_in_managed_host;
 mod re_explore;
 mod refresh_endpoint;
 mod remediation;
-mod restart_ovs;
 
 #[cfg(test)]
 mod tests;
@@ -64,6 +63,4 @@ pub enum Cmd {
     IsBmcInManagedHost(is_bmc_in_managed_host::Args),
     HaveCredentials(have_credentials::Args),
     CopyBfbToDpuRshim(copy_bfb_to_dpu_rshim::Args),
-    #[clap(about = "Enable or disable OVS restart when admin network configuration changes")]
-    RestartOvsOnUseAdminNetworkChange(restart_ovs::Args),
 }

crates/admin-cli/src/machine/restart_ovs/cmd.rs

@@ -59,7 +59,6 @@ use self::rpc::forge_server::Forge;
 use crate::cfg::file::CarbideConfig;
 use crate::dynamic_settings::DynamicSettings;
 use crate::ethernet_virtualization::EthVirtData;
-use crate::handlers::machine::set_restart_ovs_on_admin_network_change;
 use crate::logging::log_limiter::LogLimiter;
 use crate::scout_stream::ConnectionRegistry;
 use crate::{CarbideError, CarbideResult};
@@ -3085,13 +3084,6 @@ impl Forge for Api {
         crate::handlers::dpf::modify_dpf_state(self, request).await
     }
 
-    async fn set_restart_ovs_on_admin_network_change(
-        &self,
-        request: Request<rpc::SetRestartOvsOnAdminNetworkChangeRequest>,
-    ) -> Result<Response<()>, Status> {
-        set_restart_ovs_on_admin_network_change(self, request).await
-    }
-
     async fn get_dpf_state(
         &self,
         request: Request<rpc::GetDpfStateRequest>,

crates/admin-cli/src/machine/restart_ovs/mod.rs

@@ -865,35 +865,3 @@ pub(crate) async fn update_machine_nv_link_info(
 
     Ok(tonic::Response::new(()))
 }
-
-pub(crate) async fn set_restart_ovs_on_admin_network_change(
-    api: &Api,
-    request: Request<rpc::SetRestartOvsOnAdminNetworkChangeRequest>,
-) -> Result<Response<()>, Status> {
-    log_request_data(&request);
-    let request = request.get_ref();
-    let machine_id = convert_and_log_machine_id(request.machine_id.as_ref())?;
-    log_machine_id(&machine_id);
-
-    if !machine_id.machine_type().is_host() {
-        return Err(Status::invalid_argument(format!(
-            "machine_id '{}' must be a host machine ID",
-            machine_id
-        )));
-    }
-
-    let valid_values = ["enable", "none", "force_disable"];
-    if !valid_values.contains(&request.value.as_str()) {
-        return Err(Status::invalid_argument(format!(
-            "Invalid value '{}'. Must be one of: enable, none, force_disable",
-            request.value
-        )));
-    }
-
-    let mut txn = api.txn_begin().await?;
-    db::machine::set_restart_ovs_on_use_admin_network_change(&mut txn, &machine_id, &request.value)
-        .await?;
-    txn.commit().await?;
-
-    Ok(Response::new(()))
-}