display a useful error message in the runtimes

ghubertpalo · ghubertpalo · commit e53e516eb43e · 2023-02-23T11:18:35.000+01:00
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/mithril-aggregator/Cargo.toml b/mithril-aggregator/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "mithril-aggregator"
-version = "0.2.22"
+version = "0.2.23"
 description = "A Mithril Aggregator server"
 authors = { workspace = true }
 edition = { workspace = true }
diff --git a/mithril-aggregator/src/runtime/error.rs b/mithril-aggregator/src/runtime/error.rs
@@ -60,6 +60,9 @@ pub enum RuntimeError {
     #[error("signer registration error: {0}")]
     SignerRegistration(#[from] SignerRegistrationError),
 
+    #[error("era not supported: {0}")]
+    EraNotSupported(String),
+
     #[error("general error: {0}")]
     General(Box<dyn StdError + Sync + Send>),
 }
diff --git a/mithril-aggregator/src/runtime/state_machine.rs b/mithril-aggregator/src/runtime/state_machine.rs
@@ -101,7 +101,7 @@ impl AggregatorRuntime {
 
         loop {
             if let Err(e) = self.cycle().await {
-                error!("STATE MACHINE: an error occurred: "; "error" => ?e);
+                error!("STATE MACHINE: an error occurred: {e}");
             }
 
             info!(
diff --git a/mithril-signer/Cargo.toml b/mithril-signer/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "mithril-signer"
-version = "0.2.14"
+version = "0.2.15"
 description = "A Mithril Signer"
 authors = { workspace = true }
 edition = { workspace = true }
diff --git a/mithril-signer/src/runtime/state_machine.rs b/mithril-signer/src/runtime/state_machine.rs
@@ -97,7 +97,7 @@ impl StateMachine {
 
         loop {
             if let Err(e) = self.cycle().await {
-                error!("STATE MACHINE: an error occured: "; "error" => ?e);
+                error!("STATE MACHINE: an error occured: {e}");
             }
 
             info!(

Original file line number	Diff line number	Diff line change
`@@ -60,6 +60,9 @@ pub enum RuntimeError {`
`60`	`60`	`#[error("signer registration error: {0}")]`
`61`	`61`	`SignerRegistration(#[from] SignerRegistrationError),`
`62`	`62`
	`63`	`+ #[error("era not supported: {0}")]`
	`64`	`+ EraNotSupported(String),`
	`65`	`+`
`63`	`66`	`#[error("general error: {0}")]`
`64`	`67`	`General(Box<dyn StdError + Sync + Send>),`
`65`	`68`	`}`
Original file line number	Diff line number	Diff line change
`@@ -101,7 +101,7 @@ impl AggregatorRuntime {`
`101`	`101`
`102`	`102`	`loop {`
`103`	`103`	`if let Err(e) = self.cycle().await {`
`104`		`- error!("STATE MACHINE: an error occurred: "; "error" => ?e);`
	`104`	`+ error!("STATE MACHINE: an error occurred: {e}");`
`105`	`105`	`}`
`106`	`106`
`107`	`107`	`info!(`
Original file line number	Diff line number	Diff line change
`@@ -97,7 +97,7 @@ impl StateMachine {`
`97`	`97`
`98`	`98`	`loop {`
`99`	`99`	`if let Err(e) = self.cycle().await {`
`100`		`- error!("STATE MACHINE: an error occured: "; "error" => ?e);`
	`100`	`+ error!("STATE MACHINE: an error occured: {e}");`
`101`	`101`	`}`
`102`	`102`
`103`	`103`	`info!(`