chore(starknet_sequencer_node): cleanup explicit default config fns

commit-id:c320383b

crates/starknet_sequencer_node/src/config/component_config.rs

@@ -11,7 +11,7 @@ use crate::config::component_execution_config::{
 };
 
 /// The components configuration.
-#[derive(Clone, Debug, Serialize, Deserialize, Validate, PartialEq)]
+#[derive(Clone, Debug, Default, Serialize, Deserialize, Validate, PartialEq)]
 pub struct ComponentConfig {
     // Reactive component configs.
     #[validate]
@@ -34,23 +34,6 @@ pub struct ComponentConfig {
     pub monitoring_endpoint: ActiveComponentExecutionConfig,
 }
 
-impl Default for ComponentConfig {
-    fn default() -> Self {
-        Self {
-            // Reactive component configs.
-            batcher: ReactiveComponentExecutionConfig::batcher_default_config(),
-            gateway: ReactiveComponentExecutionConfig::gateway_default_config(),
-            mempool: ReactiveComponentExecutionConfig::mempool_default_config(),
-            mempool_p2p: ReactiveComponentExecutionConfig::mempool_p2p_default_config(),
-            state_sync: ReactiveComponentExecutionConfig::state_sync_default_config(),
-            // Active component configs.
-            consensus_manager: Default::default(),
-            http_server: Default::default(),
-            monitoring_endpoint: Default::default(),
-        }
-    }
-}
-
 impl SerializeConfig for ComponentConfig {
     fn dump(&self) -> BTreeMap<ParamPath, SerializedParam> {
         let sub_configs = vec![

crates/starknet_sequencer_node/src/config/component_execution_config.rs

@@ -98,75 +98,6 @@ impl Default for ActiveComponentExecutionConfig {
     }
 }
 
-/// Specific components default configurations.
-impl ReactiveComponentExecutionConfig {
-    pub fn gateway_default_config() -> Self {
-        Self {
-            execution_mode: ReactiveComponentExecutionMode::LocalExecutionWithRemoteDisabled,
-            local_server_config: Some(LocalServerConfig::default()),
-            remote_client_config: None,
-            remote_server_config: None,
-        }
-    }
-
-    // TODO(Tsabary/Lev): There's a bug here: the http server component does not need a local nor a
-    // remote config. However, the validation function requires that at least one of them is set. As
-    // a workaround I've set the local one, but this should be addressed.
-    pub fn http_server_default_config() -> Self {
-        Self {
-            execution_mode: ReactiveComponentExecutionMode::LocalExecutionWithRemoteEnabled,
-            local_server_config: Some(LocalServerConfig::default()),
-            remote_client_config: None,
-            remote_server_config: Some(RemoteServerConfig::default()),
-        }
-    }
-
-    pub fn mempool_default_config() -> Self {
-        Self {
-            execution_mode: ReactiveComponentExecutionMode::LocalExecutionWithRemoteDisabled,
-            local_server_config: Some(LocalServerConfig::default()),
-            remote_client_config: None,
-            remote_server_config: None,
-        }
-    }
-
-    pub fn batcher_default_config() -> Self {
-        Self {
-            execution_mode: ReactiveComponentExecutionMode::LocalExecutionWithRemoteDisabled,
-            local_server_config: Some(LocalServerConfig::default()),
-            remote_client_config: None,
-            remote_server_config: None,
-        }
-    }
-
-    pub fn consensus_manager_default_config() -> Self {
-        Self {
-            execution_mode: ReactiveComponentExecutionMode::LocalExecutionWithRemoteDisabled,
-            local_server_config: Some(LocalServerConfig::default()),
-            remote_client_config: None,
-            remote_server_config: None,
-        }
-    }
-
-    pub fn mempool_p2p_default_config() -> Self {
-        Self {
-            execution_mode: ReactiveComponentExecutionMode::LocalExecutionWithRemoteDisabled,
-            local_server_config: Some(LocalServerConfig::default()),
-            remote_client_config: None,
-            remote_server_config: None,
-        }
-    }
-
-    pub fn state_sync_default_config() -> Self {
-        Self {
-            execution_mode: ReactiveComponentExecutionMode::LocalExecutionWithRemoteDisabled,
-            local_server_config: Some(LocalServerConfig::default()),
-            remote_client_config: None,
-            remote_server_config: None,
-        }
-    }
-}
-
 impl ActiveComponentExecutionConfig {
     pub fn disabled() -> Self {
         Self { execution_mode: ActiveComponentExecutionMode::Disabled, remote_client_config: None }