Release Hermes v1.7.4 (#3729)

* Improve `broadcast_errors` metric (#3719) * Use short description for 'broadcast_errors' metric * Add changelog entry * Fix hyperlink in doc comment Signed-off-by: Romain Ruetschi <github@romac.me> * Fix hyperlink in doc comment --------- Signed-off-by: Romain Ruetschi <romain@informal.systems> Co-authored-by: Romain Ruetschi <romain@informal.systems> * Fetch the light block at trusted_height + 1 when detecting misbehavior (#3727) * fetch the light block at trusted_height+1 when detecting misbehaviour * Add changelog entry * Fix typo --------- Co-authored-by: beer-1 <147697694+beer-1@users.noreply.github.com> * Initialise v1.7.4 release * Avoid retrieving a worker which is being removed by the idle worker clean-up process (#3725) * prevent returning stopping workers * clear only the specific worker * Add ibc-go v8 to integration tests (#3694) * Add simapp v8 to CI jobs * Add simapp v8 to nix flake * Handle CLI breaking changes of Cosmos SDK v0.50 in test bootstrap * Handle genesis config 'voting_perdiod' and CLI 'query txs' changes for Cosmos SDK v0.50.1 * Use 'MsgSubmitProposal' instead of deprecated 'UpgradeProposal' to initiate a chain upgrade * Update 'tx upgrade-chain' CLI template * Fix 'tx chain-upgrade' tests * Update chain upgrade for compatibility between different ibc-go versions * Improve assertion for client upgrade tests * Update ibc-proto-rs to v0.39.0 * Add changelog entry * Fix and improve guide section for client upgrade * Wait before querying client state for client upgrade tests * Apply suggestions from code review Co-authored-by: Romain Ruetschi <romain@informal.systems> Signed-off-by: Luca Joss <43531661+ljoss17@users.noreply.github.com> * Rename method 'ibc_version()' to 'version_specs' * Extract the verification of legacy version in a method * Implement 'FromStr' instead of 'TryFrom<String>' for 'ProposalStatus' * Fix cargo-doc warning * Add changelog entry for CLI * Change the '--gov-account' flag as optional but will fail if trying to upgrade a chain with ibc-go v8+ * Update guide * Move and update changelog entry for CLI * Return a 'Specs' struct in 'version_specs()' method * Fix clippy errors --------- Signed-off-by: Luca Joss <43531661+ljoss17@users.noreply.github.com> Co-authored-by: Romain Ruetschi <romain@informal.systems> * Update Gaia used in tests from v12 to v13 & v14 (#3700) * Update Gaia v12 to v13 and v14 in tests * Fix Celestia CI job * Replace Gaia v12 with v13 and v14 in multi-chain tests (#3701) * Add changelog entry --------- Signed-off-by: Luca Joss <43531661+ljoss17@users.noreply.github.com> Co-authored-by: beer-1 <147697694+beer-1@users.noreply.github.com> Co-authored-by: Romain Ruetschi <romain@informal.systems> * Add PR 3703 to patch release * Improve `backlog` metrics (#3722) * Update backlog metric when packet clearing is triggered * Fix update backlog and add unit tests * Uncomment conditional compilation from telemetry tests * Add changelog entry * Update guide section regarding 'backlog_*' metrics * Correctly update the backlog when querying commitments on chain * Update backlog_oldest_timestamp to backlog_latest_update_timestamp * Update changelog entry * Fix bug in backlog update * Guard tests module by test cfg Signed-off-by: Romain Ruetschi <github@romac.me> * Update from GitHub suggestions --------- Signed-off-by: Romain Ruetschi <github@romac.me> Co-authored-by: Romain Ruetschi <romain@informal.systems> * Update changelog * Update release summary --------- Signed-off-by: Romain Ruetschi <romain@informal.systems> Signed-off-by: Luca Joss <43531661+ljoss17@users.noreply.github.com> Co-authored-by: Romain Ruetschi <romain@informal.systems> Co-authored-by: beer-1 <147697694+beer-1@users.noreply.github.com>
informalsystems · Dec 20, 2023 · 41401bc · 41401bc
1 parent 61260cf
commit 41401bc
Show file tree

Hide file tree

Showing 20 changed files with 96 additions and 36 deletions.
diff --git a/.changelog/v1.7.4/bug-fixes/ibc-relayer-cli/3697-fix-evidence-report.md b/.changelog/v1.7.4/bug-fixes/ibc-relayer-cli/3697-fix-evidence-report.md
@@ -0,0 +1,4 @@
+- Fix a bug in the `evidence` command which would sometimes
+  prevent the detected misbehaviour evidence from being submitted,
+  instead erroring out with a validator set hash mismatch.
+  ([\#3697](https://github.com/informalsystems/hermes/pull/3697))
diff --git a/.changelog/v1.7.4/bug-fixes/ibc-relayer/3703-avoid-returning-stopped-worker.md b/.changelog/v1.7.4/bug-fixes/ibc-relayer/3703-avoid-returning-stopped-worker.md
@@ -0,0 +1,3 @@
+- Avoid retrieving a worker which is being removed by the idle worker clean-up
+  process.
+  process ([\#3703](https://github.com/informalsystems/hermes/issues/3703))
diff --git a/.changelog/v1.7.4/bug-fixes/ibc-telemetry/3720-fix-broadcasting-errors-metric.md b/.changelog/v1.7.4/bug-fixes/ibc-telemetry/3720-fix-broadcasting-errors-metric.md
@@ -0,0 +1,3 @@
+- Fix the issue where `broadcast_errors` metric would not correctly batch
+  the same errors together.
+  together ([\#3720](https://github.com/informalsystems/hermes/issues/3720))
diff --git a/.changelog/v1.7.4/bug-fixes/ibc-telemetry/3723-fix-backlog-metrics.md b/.changelog/v1.7.4/bug-fixes/ibc-telemetry/3723-fix-backlog-metrics.md
@@ -0,0 +1,4 @@
+- Update the values of `backlog` metrics when clearing packets.
+  Change the `backlog_oldest_timestamp` to `backlog_latest_update_timestamp`
+  which shows the last time the `backlog` metrics have been updated.
+  ([\#3723](https://github.com/informalsystems/hermes/issues/3723))
diff --git a/.changelog/v1.7.4/summary.md b/.changelog/v1.7.4/summary.md
@@ -0,0 +1,9 @@
+*December 15th, 2023*
+
+This release improves the monitoring of Hermes instances by fixing the `broadcast_errors` metric so
+that it correctly batches the same errors together. It also improves the metrics `backlog_*` by
+updating them whenever Hermes queries pending packets.
+
+This release also improves the reliability of the idle worker clean-up and 
+fixes a bug within the `evidence` command which would sometimes prevent
+the misbehaviour evidence from being reported.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,42 @@
 # CHANGELOG
 
+## v1.7.4
+
+*December 15th, 2023*
+
+Special thanks to Yun Yeo (@beer-1) for his contributions ([#3697] and [#3703]).
+
+This release improves the monitoring of Hermes instances by fixing the `broadcast_errors` metric so
+that it correctly batches the same errors together. It also improves the metrics `backlog_*` by
+updating them whenever Hermes queries pending packets.
+
+This release also improves the reliability of the idle worker clean-up and 
+fixes a bug within the `evidence` command which would sometimes prevent
+the misbehaviour evidence from being reported.
+
+### BUG FIXES
+
+- [Relayer Library](relayer)
+  - Avoid retrieving a worker which is being removed by the idle worker clean-up
+    process.
+    process ([\#3703](https://github.com/informalsystems/hermes/issues/3703))
+- [Relayer CLI](relayer-cli)
+  - Fix a bug in the `evidence` command which would sometimes
+    prevent the detected misbehaviour evidence from being submitted,
+    instead erroring out with a validator set hash mismatch.
+    ([\#3697](https://github.com/informalsystems/hermes/pull/3697))
+- [Telemetry & Metrics](telemetry)
+  - Fix the issue where `broadcast_errors` metric would not correctly batch
+    the same errors together.
+    together ([\#3720](https://github.com/informalsystems/hermes/issues/3720))
+  - Update the values of `backlog` metrics when clearing packets.
+    Change the `backlog_oldest_timestamp` to `backlog_latest_update_timestamp`
+    which shows the last time the `backlog` metrics have been updated.
+    ([\#3723](https://github.com/informalsystems/hermes/issues/3723))
+
+[#3697]: https://github.com/informalsystems/hermes/issues/3697
+[#3703]: https://github.com/informalsystems/hermes/issues/3703
+
 ## v1.7.3
 
 *November 29th, 2023*

diff --git a/Cargo.lock b/Cargo.lock
diff --git a/crates/chain-registry/Cargo.toml b/crates/chain-registry/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name         = "ibc-chain-registry"
-version      = "0.26.3"
+version      = "0.26.4"
 edition      = "2021"
 license      = "Apache-2.0"
 keywords     = ["cosmos", "ibc", "relayer", "chain", "registry"]
@@ -12,7 +12,7 @@ description  = """
 """
 
 [dependencies]
-ibc-relayer-types = { version = "0.26.3", path = "../relayer-types" }
+ibc-relayer-types = { version = "0.26.4", path = "../relayer-types" }
 ibc-proto         = { version = "0.39.0", features = ["serde"] }
 tendermint-rpc    = { version = "0.34.0", features = ["http-client", "websocket-client"] }
 

diff --git a/crates/relayer-cli/Cargo.toml b/crates/relayer-cli/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name       = "ibc-relayer-cli"
-version    = "1.7.3"
+version    = "1.7.4"
 edition    = "2021"
 license    = "Apache-2.0"
 readme     = "README.md"
@@ -25,11 +25,11 @@ telemetry   = ["ibc-relayer/telemetry", "ibc-telemetry"]
 rest-server = ["ibc-relayer-rest"]
 
 [dependencies]
-ibc-relayer-types  = { version = "0.26.3", path = "../relayer-types" }
-ibc-relayer        = { version = "0.26.3", path = "../relayer" }
-ibc-telemetry      = { version = "0.26.3", path = "../telemetry", optional = true }
-ibc-relayer-rest   = { version = "0.26.3", path = "../relayer-rest", optional = true }
-ibc-chain-registry = { version = "0.26.3" , path = "../chain-registry" }
+ibc-relayer-types  = { version = "0.26.4", path = "../relayer-types" }
+ibc-relayer        = { version = "0.26.4", path = "../relayer" }
+ibc-telemetry      = { version = "0.26.4", path = "../telemetry", optional = true }
+ibc-relayer-rest   = { version = "0.26.4", path = "../relayer-rest", optional = true }
+ibc-chain-registry = { version = "0.26.4" , path = "../chain-registry" }
 
 clap                     = { version = "3.2", features = ["cargo"] }
 clap_complete            = "3.2"

diff --git a/crates/relayer-rest/Cargo.toml b/crates/relayer-rest/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name         = "ibc-relayer-rest"
-version      = "0.26.3"
+version      = "0.26.4"
 authors      = ["Informal Systems <hello@informal.systems>"]
 edition      = "2021"
 license      = "Apache-2.0"
@@ -14,8 +14,8 @@ description  = """
 """
 
 [dependencies]
-ibc-relayer-types = { version = "0.26.3", path = "../relayer-types" }
-ibc-relayer       = { version = "0.26.3", path = "../relayer" }
+ibc-relayer-types = { version = "0.26.4", path = "../relayer-types" }
+ibc-relayer       = { version = "0.26.4", path = "../relayer" }
 
 crossbeam-channel = "0.5"
 serde             = "1.0"

diff --git a/crates/relayer-rest/tests/mock.rs b/crates/relayer-rest/tests/mock.rs
@@ -64,7 +64,7 @@ async fn version() {
 
     let rest_api_version = VersionInfo {
         name: "ibc-relayer-rest".to_string(),
-        version: "0.26.3".to_string(),
+        version: "0.26.4".to_string(),
     };
 
     let result: JsonResult<_, ()> = JsonResult::Success(vec![version.clone(), rest_api_version]);

diff --git a/crates/relayer-types/Cargo.toml b/crates/relayer-types/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name         = "ibc-relayer-types"
-version      = "0.26.3"
+version      = "0.26.4"
 edition      = "2021"
 license      = "Apache-2.0"
 readme       = "README.md"

diff --git a/crates/relayer/Cargo.toml b/crates/relayer/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name         = "ibc-relayer"
-version      = "0.26.3"
+version      = "0.26.4"
 edition      = "2021"
 license      = "Apache-2.0"
 readme       = "README.md"
@@ -21,8 +21,8 @@ telemetry = ["ibc-telemetry"]
 
 [dependencies]
 ibc-proto         = { version = "0.39.0", features = ["serde"] }
-ibc-telemetry     = { version = "0.26.3", path = "../telemetry", optional = true }
-ibc-relayer-types = { version = "0.26.3", path = "../relayer-types", features = ["mocks"] }
+ibc-telemetry     = { version = "0.26.4", path = "../telemetry", optional = true }
+ibc-relayer-types = { version = "0.26.4", path = "../relayer-types", features = ["mocks"] }
 
 subtle-encoding = "0.5"
 humantime-serde = "1.1.1"
@@ -109,7 +109,7 @@ version = "0.34.0"
 default-features = false
 
 [dev-dependencies]
-ibc-relayer-types = { version = "0.26.3", path = "../relayer-types", features = ["mocks"] }
+ibc-relayer-types = { version = "0.26.4", path = "../relayer-types", features = ["mocks"] }
 serial_test = "2.0.0"
 env_logger = "0.10.0"
 test-log = { version = "0.2.10", features = ["trace"] }

diff --git a/crates/relayer/src/lib.rs b/crates/relayer/src/lib.rs
@@ -16,7 +16,7 @@
 //!
 //! For the IBC relayer binary, please see [Hermes] (`ibc-relayer-cli` crate).
 //!
-//! [Hermes]: https://docs.rs/ibc-relayer-cli/1.7.2/
+//! [Hermes]: https://docs.rs/ibc-relayer-cli/1.7.4/
 
 extern crate alloc;
 

diff --git a/crates/telemetry/Cargo.toml b/crates/telemetry/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name         = "ibc-telemetry"
-version      = "0.26.3"
+version      = "0.26.4"
 edition      = "2021"
 license      = "Apache-2.0"
 readme       = "README.md"
@@ -13,7 +13,7 @@ description  = """
 """
 
 [dependencies]
-ibc-relayer-types = { version = "0.26.3", path = "../relayer-types" }
+ibc-relayer-types = { version = "0.26.4", path = "../relayer-types" }
 
 once_cell                = "1.17.0"
 opentelemetry            = { version = "0.19.0", features = ["metrics"] }

diff --git a/guide/README.md b/guide/README.md
@@ -10,7 +10,7 @@ mdBook is a utility to create modern online books from Markdown files.
 This guide should be permanently deployed at its latest stable version at
 [hermes.informal.systems](https://hermes.informal.systems).
 
-Current version: `v1.7.3`.
+Current version: `v1.7.4`.
 
 The version of this guide is aligned with the [versioning of the ibc crates](../README.md).
 

diff --git a/guide/src/SUMMARY.md b/guide/src/SUMMARY.md
@@ -1,6 +1,6 @@
 # Summary
 
-# Hermes v1.7.3
+# Hermes v1.7.4
 
 ---
 - [Introduction](./index.md)

diff --git a/guide/src/templates/hermes-version.md b/guide/src/templates/hermes-version.md
@@ -1 +1 @@
-v1.7.3
+v1.7.4
diff --git a/tools/integration-test/Cargo.toml b/tools/integration-test/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name         = "ibc-integration-test"
-version      = "0.26.3"
+version      = "0.26.4"
 edition      = "2021"
 rust-version = "1.71"
 license      = "Apache-2.0"

diff --git a/tools/test-framework/Cargo.toml b/tools/test-framework/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name       = "ibc-test-framework"
-version    = "0.26.3"
+version    = "0.26.4"
 edition    = "2021"
 license    = "Apache-2.0"
 readme     = "README.md"
@@ -14,9 +14,9 @@ description = """
 """
 
 [dependencies]
-ibc-relayer-types = { version = "=0.26.3",     path = "../../crates/relayer-types" }
-ibc-relayer       = { version = "=0.26.3",     path = "../../crates/relayer" }
-ibc-relayer-cli   = { version = "=1.7.3",      path = "../../crates/relayer-cli" }
+ibc-relayer-types = { version = "=0.26.4",     path = "../../crates/relayer-types" }
+ibc-relayer       = { version = "=0.26.4",     path = "../../crates/relayer" }
+ibc-relayer-cli   = { version = "=1.7.4",      path = "../../crates/relayer-cli" }
 ibc-proto         = { version = "0.39.0", features = ["serde"] }
 tendermint-rpc    = { version = "0.34.0", features = ["http-client", "websocket-client"] }