fix: Gas Upgrades and Beta Fixes (#1055)

Co-authored-by: Andy Golay <[email protected]>
Co-authored-by: musitdev <[email protected]>
Co-authored-by: Mikhail Zabaluev <[email protected]>
Co-authored-by: Richard Melkonian <[email protected]>
Co-authored-by: Icarus131 <[email protected]>
Co-authored-by: primata <[email protected]>
Co-authored-by: Icarus131 <[email protected]>
Co-authored-by: Richard Melkonian <[email protected]>
Co-authored-by: Radu Popa <[email protected]>

Author: 8 people

.github/workflows/build-push-containers-all.yml

@@ -76,7 +76,11 @@ jobs:
 
       - name: Read the commit SHA
         id: vars
-        run: echo "CONTAINER_REV=${{ github.ref }}" >> .env
+        run: |
+          commit_hash=$(git rev-parse HEAD | cut -c1-7)
+          echo CONTAINER_REV=${commit_hash}
+          echo "CONTAINER_REV=${commit_hash}" >> .env
+
 
       - name: Display .env file
         run: cat .env

.github/workflows/checks-all.yml

@@ -65,7 +65,7 @@ jobs:
             -p move-rocks \
             -p movement-types \
             -p movement-config \
-            -p movement-celestia-da-util \
+            -p movement-da-util \
             -p movement-signer-test
 
   movement-full-node-local:

Cargo.lock

@@ -45,6 +45,7 @@ members = [
   "util/signing/integrations/aptos",
   "util/signing/providers/aws-kms",
   "util/signing/providers/hashicorp-vault",
+  "util/signing/testing",
   "demo/hsm",
   "protocol-units/execution/maptos/framework/releases/*",
   "protocol-units/execution/maptos/framework/migrations/*",
@@ -361,9 +362,9 @@ http-body-util = "0.1"
 tap = "1.0.1"
 
 # trying to pin diesel
-diesel = { version = "2.2.4", features = ["postgres", "numeric", "r2d2"] }
+diesel = { version = "2.2.7", features = ["postgres", "numeric", "r2d2"] }
 diesel_migrations = { version = "2.2.0" }
-bigdecimal = "0.4.0"
+bigdecimal = "0.3"
 num_cpus = "=1.16.0"
 ahash = "=0.8.11"
 

Cargo.toml

@@ -17,6 +17,9 @@ RUN rust_binary="./target/release/movement-full-node"; dest_dir="/tmp/runtime";
 
 FROM alpine:latest
 
+# Install dependencies: git
+RUN apk add --no-cache git
+
 # Copy the build artifact from the builder stage
 COPY --from=builder /tmp/build/target/release/movement-full-node /app/movement-full-node
 COPY --from=builder /tmp/runtime/nix/store /nix/store

docker/build/movement-full-node/Dockerfile

@@ -0,0 +1,16 @@
+version: "3"
+
+processes:
+
+  setup:
+    environment:
+      - "KNOWN_FRAMEWORK_RELEASE=elsa"
+
+  test-migrate-elsa-to-biarritz-rc1:
+    command: |
+      cargo run --bin movement-full-node admin bring-up elsa-to-biarritz-rc1 upgrade 0x1 0x2 0x3
+    depends_on:
+      movement-full-node:
+        condition: process_healthy
+      movement-faucet:
+        condition: process_healthy

process-compose/movement-full-node/process-compose.test-bring-up-elsa-to-biarritz-rc1.yml

@@ -4,7 +4,7 @@ processes:
 
   setup:
     environment:
-      - "KNOWN_FRAMEWORK_RELEASE=biarritz-rc1"
+      - "KNOWN_FRAMEWORK_RELEASE=elsa"
 
   test-migrate-elsa-to-biarritz-rc1:
     command: |

process-compose/movement-full-node/process-compose.test-migrate-elsa-to-biarritz-rc1.yml

@@ -10,12 +10,14 @@ publish = { workspace = true }
 rust-version = { workspace = true }
 
 [dependencies]
-diesel = { workspace = true, features = ["postgres", "numeric", "r2d2", "chrono"] }
-diesel_migrations = { workspace = true }
-bigdecimal = { workspace = true }
+# Use specific version different from Aptos core one.
+# Could use workplace version when Aptos core will have upgrade for a more recent version.
+diesel = { version = "2.2.4", features = ["postgres", "numeric", "r2d2", "chrono"] }
+diesel_migrations = { version = "2.2.0" }
+bigdecimal = "0.4.0"
+
 serde = { workspace = true }
 bridge-util = { workspace = true }
-bridge-config = { workspace = true }
 anyhow = { workspace = true }
 hex = { workspace = true }
 chrono = { workspace = true }

protocol-units/bridge/indexer-db/Cargo.toml

@@ -1,7 +1,6 @@
 use crate::migrations::run_migrations;
 use crate::models::*;
 use crate::schema::*;
-use bridge_config::Config;
 use bridge_util::chains::bridge_contracts::BridgeContractEvent;
 use bridge_util::types::BridgeTransferId;
 use bridge_util::TransferActionType;
@@ -23,8 +22,8 @@ impl Client {
 		Self { conn }
 	}
 
-	pub fn from_bridge_config(config: &Config) -> Result<Self, anyhow::Error> {
-		let conn = PgConnection::establish(&config.indexer.indexer_url)
+	pub fn build_from_db_url(db_url: &str) -> Result<Self, anyhow::Error> {
+		let conn = PgConnection::establish(&db_url)
 			.map_err(|e| anyhow::anyhow!("Failed to connect to postgresql instance: {}", e))?;
 		Ok(Self::new(conn))
 	}

protocol-units/bridge/indexer-db/src/client.rs

@@ -1,5 +1,4 @@
 use crate::client::Client;
-use bridge_config::Config;
 use bridge_util::chains::bridge_contracts::BridgeContractMonitoring;
 use bridge_util::types::BridgeTransferId;
 use bridge_util::TransferActionType;
@@ -16,7 +15,7 @@ pub async fn run_indexer_client<
 	SOURCE: Send + TryFrom<Vec<u8>> + std::clone::Clone + 'static + std::fmt::Debug,
 	TARGET: Send + TryFrom<Vec<u8>> + std::clone::Clone + 'static + std::fmt::Debug,
 >(
-	config: Config,
+	db_url: String,
 	mut stream_source: impl BridgeContractMonitoring<Address = SOURCE>,
 	mut stream_target: impl BridgeContractMonitoring<Address = TARGET>,
 	_relayer_actions: Option<mpsc::Sender<(BridgeTransferId, TransferActionType)>>,
@@ -25,7 +24,7 @@ where
 	Vec<u8>: From<SOURCE>,
 	Vec<u8>: From<TARGET>,
 {
-	let mut indexer_db_client = match Client::from_bridge_config(&config) {
+	let mut indexer_db_client = match Client::build_from_db_url(&db_url) {
 		Ok(mut client) => {
 			client.run_migrations()?;
 			client