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(Clone)]
+#[derive(Debug, Clone)]
 pub struct ContextLogger<C, L>
 where
     C: Clone + Default,

rs/monitoring/logger/src/replica_logger.rs

@@ -26,6 +26,7 @@ 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,6 +69,7 @@ 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(crate) fn get_ic_config(&self) -> Config {
+    pub 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::{Path, PathBuf};
+use std::path::PathBuf;
 
 pub type OrchestratorResult<T> = Result<T, OrchestratorError>;
 
@@ -67,10 +67,6 @@ 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())
     }