From d04a73f3368b5e0b632abab571294b501ab9821b Mon Sep 17 00:00:00 2001
From: Mathis <sveitser@gmail.com>
Date: Mon, 16 Sep 2024 16:22:32 +0200
Subject: [PATCH] Pin version of espresso-dev-node with legacy contract ABI
 (#53)

* CI: print docker logs on failure

* Pin version before LC ABI changes

In https://github.com/EspressoSystems/espresso-sequencer/pull/1939
the light client ABI changed and as a result we need to make changes in
multiple places.

The tracking issue for the changes in our nitro integration is

https://github.com/EspressoSystems/nitro-contracts/issues/20

* Pin the espresso-dev-node image
---
 .github/workflows/smoke-test.yml | 5 +++++
 docker-compose.yaml              | 3 ++-
 2 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/.github/workflows/smoke-test.yml b/.github/workflows/smoke-test.yml
index a3726a07..074d41e8 100644
--- a/.github/workflows/smoke-test.yml
+++ b/.github/workflows/smoke-test.yml
@@ -70,3 +70,8 @@ jobs:
         run: |
           ${{ matrix.test-script }}
         timeout-minutes: 30
+
+      - name: Dump docker logs on failure
+        if: failure()
+        run: |
+          docker compose logs
diff --git a/docker-compose.yaml b/docker-compose.yaml
index 5f9585af..93487079 100644
--- a/docker-compose.yaml
+++ b/docker-compose.yaml
@@ -390,7 +390,8 @@ services:
       - /var/run/docker.sock:/var/run/docker.sock
 
   espresso-dev-node:
-    image: ghcr.io/espressosystems/espresso-sequencer/espresso-dev-node:main
+    # TODO: revert to `:main` tag once we are compatible with the updated LC ABI
+    image: ghcr.io/espressosystems/espresso-sequencer/espresso-dev-node:20240906-lc-legacy-abi
     ports:
       - "$ESPRESSO_SEQUENCER_API_PORT:$ESPRESSO_SEQUENCER_API_PORT"
       - "$ESPRESSO_BUILDER_PORT:$ESPRESSO_BUILDER_PORT"