Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: when PR raised from fork, skip the attach-sovereign-rollup workflow #462

Merged
merged 2 commits into from
Feb 4, 2025
Merged

fix: when PR raised from fork, skip the attach-sovereign-rollup workflow #462

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
b55a9b1
fix: when PR raised from fork, skip the attach-sovereign-rollup workflow
jhkimqd Feb 4, 2025
46b8367
chore: fix ci
jhkimqd Feb 4, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 12 additions & 4 deletions .github/workflows/deploy.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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 }})
Expand All @@ -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 }})
Expand Down Expand Up @@ -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:
Expand Down Expand Up @@ -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 != '' }}
Expand Down Expand Up @@ -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 }}
Loading