Revert "fix(guestos): initialize basic firewall upon ic-replica start."

rs/monitoring/context_logger/src/lib.rs

@@ -34,7 +34,7 @@ pub trait Logger<T>: Clone {
 ///
 /// Methods of this trait should not be used directly, instead use the macros
 /// found in macros.rs
-#[derive(Debug, Clone)]
+#[derive(Clone)]
 pub struct ContextLogger<C, L>
 where
     C: Clone + Default,

rs/monitoring/logger/src/replica_logger.rs

@@ -26,7 +26,6 @@ impl From<LogEntryLogger> for ReplicaLogger {
 }
 
 /// Logs `LogEntry`s using `slog`
-#[derive(Debug)]
 pub struct LogEntryLogger {
     pub root: slog::Logger,
     // Only logs at `level` or above

rs/orchestrator/BUILD.bazel

@@ -69,7 +69,6 @@ rust_binary(
     deps = [
         ":lib",
         "//rs/async_utils",
-        "//rs/monitoring/logger",
         "@crate_index//:clap",
         "@crate_index//:tokio",
     ],

rs/orchestrator/src/args.rs

@@ -85,7 +85,7 @@ impl OrchestratorArgs {
 
     /// Parse `self.replica_config_file` and persist in
     /// [`TempDir`][tempfile::TempDir] "ic_config"
-    pub fn get_ic_config(&self) -> Config {
+    pub(crate) fn get_ic_config(&self) -> Config {
         let tmpdir = tempfile::Builder::new()
             .prefix("ic_config")
             .tempdir()

rs/orchestrator/src/error.rs

@@ -5,7 +5,7 @@ use ic_types::{registry::RegistryClientError, NodeId, RegistryVersion, ReplicaVe
 use std::error::Error;
 use std::fmt;
 use std::io;
-use std::path::PathBuf;
+use std::path::{Path, PathBuf};
 
 pub type OrchestratorResult<T> = Result<T, OrchestratorError>;
 
@@ -67,6 +67,10 @@ pub enum OrchestratorError {
 }
 
 impl OrchestratorError {
+    pub(crate) fn file_write_error(file_path: &Path, e: io::Error) -> Self {
+        OrchestratorError::IoError(format!("Failed to write to file: {:?}", file_path), e)
+    }
+
     pub(crate) fn invalid_configuration_error(msg: impl ToString) -> Self {
         OrchestratorError::InvalidConfigurationError(msg.to_string())
     }