feat(checkpoint-sync): add beacon_node_url input to action

This allows the user to specify the beacon node URL to use for checkpoint sync, and removes the need for the action to have knowledge of the different networks and their corresponding URLs. This makes the action more generic and reusable.
ethpandaops · Feb 21, 2025 · 6379fae · 6379fae
1 parent c7cf8df
commit 6379fae
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 19 deletions.
diff --git a/.github/actions/checkpoint-sync/action.yaml b/.github/actions/checkpoint-sync/action.yaml
@@ -1,4 +1,3 @@
-
 name: Run a checkpoint sync test
 description: Runs a consensus client and checkpoint syncs from the running checkpointz instance.
 
@@ -9,7 +8,9 @@ inputs:
   network:
     description: "The name of the network to run the test against (one of ropsten, sepolia, prater/goerli)."
     required: true
-
+  beacon_node_url:
+    description: "The beacon node API URL to use"
+    required: true
 
 runs:
   using: composite
@@ -27,21 +28,7 @@ runs:
         docker build . -t ethpandaops/checkpointz:local;
     - name: Configure checkpointz
       shell: bash
-      env:
-        MAINNET_BEACON_API_URL: "${{ secrets.MAINNET_BEACON_API_URL }}"
-        HOLESKY_BEACON_API_URL: "${{ secrets.HOLESKY_BEACON_API_URL }}"
-
       run: |
-        beacon_node=""
-        if [[ ${{ inputs.network }} == "mainnet" ]]; then
-          beacon_node="${{ env.MAINNET_BEACON_API_URL }}"
-        elif [[ ${{ inputs.network }} == "holesky" ]]; then
-          beacon_node="${{ env.HOLESKY_BEACON_API_URL }}"
-        else
-          echo "Unsupported network: ${{ inputs.network }}"
-          exit 1
-        fi
-
         cat <<EOF > checkpointz.yaml
         global:
           listenAddr: ":5555"
@@ -50,7 +37,7 @@ runs:
         beacon:
           upstreams:
           - name: state-provider
-            address: $beacon_node
+            address: ${{ inputs.beacon_node_url }}
             timeoutSeconds: 30
             dataProvider: true
         checkpointz:

diff --git a/.github/workflows/integration.yaml b/.github/workflows/integration.yaml
@@ -26,6 +26,7 @@ jobs:
         id: run-test
         uses: ./.github/actions/checkpoint-sync
         with:
-          ## Variables
           consensus: ${{ matrix.consensus }}
-          network: ${{ matrix.network }}
+          network: ${{ matrix.network }}
+          beacon_node_url: ${{ matrix.network == 'mainnet' && secrets.MAINNET_BEACON_API_URL || secrets.HOLESKY_BEACON_API_URL }}
+