fix: leave implementers to define their Debug, so removed impl Debug for dyn NodeExporter

Cargo.lock

@@ -690,6 +690,24 @@ dependencies = [
  "tokio",
 ]
 
+[[package]]
+name = "brocade-snmp-server"
+version = "0.1.0"
+dependencies = [
+ "base64 0.22.1",
+ "brocade",
+ "env_logger",
+ "harmony",
+ "harmony_cli",
+ "harmony_macros",
+ "harmony_secret",
+ "harmony_types",
+ "log",
+ "serde",
+ "tokio",
+ "url",
+]
+
 [[package]]
 name = "brocade-switch"
 version = "0.1.0"
@@ -1885,6 +1903,25 @@ dependencies = [
  "url",
 ]
 
+[[package]]
+name = "example-opnsense-node-exporter"
+version = "0.1.0"
+dependencies = [
+ "async-trait",
+ "cidr",
+ "env_logger",
+ "harmony",
+ "harmony_cli",
+ "harmony_macros",
+ "harmony_secret",
+ "harmony_secret_derive",
+ "harmony_types",
+ "log",
+ "serde",
+ "tokio",
+ "url",
+]
+
 [[package]]
 name = "example-pxe"
 version = "0.1.0"
@@ -6095,21 +6132,6 @@ version = "0.5.1"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "8f50febec83f5ee1df3015341d8bd429f2d1cc62bcba7ea2076759d315084683"
 
-[[package]]
-name = "test-score"
-version = "0.1.0"
-dependencies = [
- "base64 0.22.1",
- "env_logger",
- "harmony",
- "harmony_cli",
- "harmony_macros",
- "harmony_types",
- "log",
- "tokio",
- "url",
-]
-
 [[package]]
 name = "thiserror"
 version = "1.0.69"

examples/opnsense_node_exporter/src/main.rs

@@ -18,6 +18,7 @@ use harmony::{
 };
 use harmony_macros::{ip, ipv4, mac_address};
 
+#[derive(Debug)]
 struct OpnSenseTopology {
     node_exporter: Arc<dyn NodeExporter>,
 }

harmony/src/domain/topology/node_exporter.rs

@@ -3,15 +3,15 @@ use async_trait::async_trait;
 use crate::executors::ExecutorError;
 
 #[async_trait]
-pub trait NodeExporter: Send + Sync {
+pub trait NodeExporter: Send + Sync + std::fmt::Debug {
     async fn ensure_initialized(&self) -> Result<(), ExecutorError>;
     async fn commit_config(&self) -> Result<(), ExecutorError>;
     async fn reload_restart(&self) -> Result<(), ExecutorError>;
 }
 
-//TODO complete this impl
+// //TODO complete this impl
-impl std::fmt::Debug for dyn NodeExporter {
+// impl std::fmt::Debug for dyn NodeExporter {
-    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
+//     fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
-        f.write_fmt(format_args!("NodeExporter ",))
+//         f.write_fmt(format_args!("NodeExporter ",))
-    }
+//     }
-}
+// }