-
Notifications
You must be signed in to change notification settings - Fork 579
feat: ci-network-scenario label #17505
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.
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
feat: ci-network-scenario label #17505
Conversation
99620f8 to
b84378a
Compare
✅ Network Scenario Tests PASSEDTest Status: passed To re-run tests, add the |
b84378a to
430d20e
Compare
❌ Network Scenario Tests FAILEDTest Status: failed
|
430d20e to
37056c7
Compare
❌ Network Scenario Tests FAILEDTest Status: failed
|
37056c7 to
8cd6ff8
Compare
8cd6ff8 to
4886832
Compare
4886832 to
704a0e6
Compare
❌ Network Scenario Tests FAILEDTest Status: failed
|
704a0e6 to
78cf923
Compare
❌ Network Scenario Tests FAILEDTest Status: failed
|
78cf923 to
78794a5
Compare
| SALT=1757186002 | ||
| NAMESPACE=${{ inputs.namespace }} | ||
| AZTEC_DOCKER_IMAGE="${{ inputs.docker-image }}" | ||
| AZTEC_EPOCH_DURATION=8 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doesn't this network perpetually re-org? 8 slots isn't enough time to prove the epoch. The nightly runs are set to 32.
❌ Network Scenario Tests FAILEDTest Status: failed
|
6e7b6f7 to
f7b526b
Compare
❌ Network Scenario Tests FAILEDTest Status: failed
|
f7b526b to
1b97f02
Compare
✅ Network Scenario Tests PASSEDTest Status: passed To re-run tests, add the |
1b97f02 to
ad58f0a
Compare
❌ Network Scenario Tests FAILEDTest Status: failed
|
ad58f0a to
318defc
Compare
❌ Network Scenario Tests FAILEDTest Status: failed
|
|
|
||
| on: | ||
| pull_request: | ||
| types: [labeled] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think I'm of the camp that ci3.yml could handle everything here. We have already have a mix of machines running with diff CI settings on them for the merge queue path
318defc to
67cffac
Compare
❌ Network Scenario Tests FAILEDTest Status: failed
|
67cffac to
7bb68d7
Compare
✅ Network Scenario Tests PASSEDTest Status: passed Manual Cleanup RequiredIf cleanup failed, you may need to manually delete the namespace: kubectl delete namespace pr-17505-scenarioTo re-run tests, add the |
|
Closing because @ludamad did this better elsewhere 👍 |

Add GitHub Actions for Network Scenario Tests on PRs
This PR adds a new workflow that allows developers to trigger network scenario tests on pull requests by adding the
ci-network-scenariolabel. The workflow deploys a full Aztec network in a PR-specific Kubernetes namespace and runs scenario tests against it.Key features:
ci3-network-scenario.ymltriggered by theci-network-scenariolabelThe implementation extracts common functionality from the existing
test-network-scenarios.ymlworkflow into reusable GitHub Actions:setup-gcp-auth- Handles GCP service account authentication and kubectl setupsetup-network-env- Creates network environment configuration and SSH keysThis allows developers to run the same network scenario tests that are run on releases, but on their PRs before merging.