Skip to content

Commit

Permalink
fix: Daily Blockchain Reference Test Workflow Failures (#1588)
Browse files Browse the repository at this point in the history
  • Loading branch information
DavePearce authored Nov 29, 2024
1 parent cc8a35b commit 02b33e0
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 4 deletions.
13 changes: 11 additions & 2 deletions .github/workflows/gradle-ethereum-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -51,6 +51,15 @@ jobs:
reference-tests/build/reports/tests/**/*
tmp/local/*
- name: Failure Notification
if: ${{ failure() || cancelled() }}
uses: slackapi/[email protected]
with:
webhook: ${{ secrets.SLACK_WEBHOOK_URL }}
webhook-type: webhook-trigger
payload: |
name: "Daily Reference Tests (Rust Corset)"
status: "${{ job.status }}"
ethereum-tests-go-corset:
runs-on: ubuntu-latest-128
Expand All @@ -73,7 +82,7 @@ jobs:
run: cd ./linea-constraints; make zkevm_for_reference_tests.bin -B; cd ..

- name: Generate General State Reference Tests
run: ./gradlew generateGeneralStateReferenceTests -Dorg.gradle.parallel=true -Dorg.gradle.caching=true
run: ./gradlew generateGeneralStateReferenceTests -Dorg.gradle.caching=true
env:
JAVA_OPTS: -Dorg.gradle.daemon=false
CORSET_FLAGS: fields,expand,expand,expand
Expand Down Expand Up @@ -101,5 +110,5 @@ jobs:
webhook: ${{ secrets.SLACK_WEBHOOK_URL }}
webhook-type: webhook-trigger
payload: |
name: "Daily Reference Tests"
name: "Daily Reference Tests (Go Corset)"
status: "${{ job.status }}"
1 change: 0 additions & 1 deletion .github/workflows/reference-blockchain-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,6 @@ jobs:
- name: Run reference blockchain tests
run: ./gradlew referenceBlockchainTests -x spotlessCheck
timeout-minutes: 180
continue-on-error: true
env:
REFERENCE_TESTS_PARALLELISM: 7
JAVA_OPTS: -Dorg.gradle.daemon=false
Expand Down
3 changes: 2 additions & 1 deletion gradle/lint.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,8 @@ spotless {
}
groovyGradle {
target '*.gradle'
greclipse('4.21').configFile(rootProject.file('gradle/formatter.properties'))
// Temporarily disabled as was reporting XZFormatException
// greclipse('4.21').configFile(rootProject.file('gradle/formatter.properties'))
endWithNewline()
}
// Below this line are currently only license header tasks
Expand Down

0 comments on commit 02b33e0

Please sign in to comment.