fix: when PR raised from fork, skip the attach-sovereign-rollup workf…

…low (#462) * fix: when PR raised from fork, skip the attach-sovereign-rollup workflow Signed-off-by: Ji Hwan <[email protected]> * chore: fix ci Signed-off-by: Ji Hwan <[email protected]> --------- Signed-off-by: Ji Hwan <[email protected]>
0xPolygon · Feb 4, 2025 · 85f0dc2 · 85f0dc2
1 parent 5acb297
commit 85f0dc2
Showing 1 changed file with 12 additions and 4 deletions.
diff --git a/.github/workflows/deploy.yml b/.github/workflows/deploy.yml
@@ -149,7 +149,7 @@ jobs:
       - name: Monitor CDK chain verified batches (Central RPC)
         run: |
           result=$(yq --raw-output '.args.consensus_contract_type' ${{ matrix.file_name }})
-          if [[ "$result" == "true" ]]; then
+          if [[ "$result" == "pessimistic" ]]; then
             echo "Skipping block verification as consensus is pessimistic."
           else
             sequencer_type=$(yq --raw-output '.args.sequencer_type' ${{ matrix.file_name }})
@@ -173,7 +173,7 @@ jobs:
       - name: Monitor CDK chain verified batches (zkEVM Permissionless RPC)
         run: |
           result=$(yq --raw-output '.args.consensus_contract_type' ${{ matrix.file_name }})
-          if [[ "$result" == "true" ]]; then
+          if [[ "$result" == "pessimistic" ]]; then
             echo "Skipping block verification as consensus is pessimistic."
           else
             result=$(yq --raw-output '.args.additional_services // [] | contains(["pless_zkevm_node"])' ${{ matrix.file_name }})
@@ -648,6 +648,7 @@ jobs:
           path: ./dump
 
   attach-sovereign-rollup:
+    # This workflow depends on the existence of a valid secrets.SP1_PRIVATE_KEY, and will only run if this dependency is met.
     runs-on: ubuntu-latest
     timeout-minutes: 30
     steps:
@@ -697,6 +698,9 @@ jobs:
 
       - name: Inspect enclave
         run: kurtosis enclave inspect ${{ env.ENCLAVE_NAME }}
+        if: ${{ env.agglayer_prover_sp1_key && env.agglayer_prover_sp1_key != '' }}
+        env :
+          agglayer_prover_sp1_key: ${{ secrets.SP1_PRIVATE_KEY }}
 
       - name: Send bridge transactions from L1 to OP Sovereign L2
         if: ${{ env.agglayer_prover_sp1_key || env.agglayer_prover_sp1_key != '' }}
@@ -826,12 +830,16 @@ jobs:
             agglayer_prover_sp1_key: ${{ secrets.SP1_PRIVATE_KEY }}
 
       - name: Dump enclave
-        if: ${{ !cancelled() }}
+        if: ${{ !cancelled() && env.agglayer_prover_sp1_key && env.agglayer_prover_sp1_key != ''}}
         run: kurtosis enclave dump ${{ env.ENCLAVE_NAME }} ./dump
+        env :
+          agglayer_prover_sp1_key: ${{ secrets.SP1_PRIVATE_KEY }}
 
       - name: Upload enclave dump
-        if: ${{ !cancelled() }}
+        if: ${{ !cancelled() && env.agglayer_prover_sp1_key && env.agglayer_prover_sp1_key != ''}}
         uses: actions/upload-artifact@v4
         with:
           name: dump_attach_sovereign_${{ github.run_id }}
           path: ./dump
+        env :
+          agglayer_prover_sp1_key: ${{ secrets.SP1_PRIVATE_KEY }}