-
Notifications
You must be signed in to change notification settings - Fork 2.4k
chore(common): merge e2e tests workflows #1676
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
base: main
Are you sure you want to change the base?
Conversation
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.
Pull request overview
This PR consolidates the e2e test workflows by merging test-suite-e2e-tests-mq.yml into test-suite-e2e-tests.yml and updates variable naming conventions across the FHEVM stack. The main changes standardize environment variable names by adding prefixes to distinguish between coprocessor and connector services, and update component versions from v0.10.2 to v0.10.5.
- Standardized variable naming with
COPROCESSOR_prefix for coprocessor services (e.g.,DB_MIGRATION_VERSION→COPROCESSOR_DB_MIGRATION_VERSION) - Merged separate workflow files into a single unified e2e test workflow
- Updated all FHEVM component versions from v0.10.2 to v0.10.5
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
test-suite/fhevm/fhevm-cli |
Updated environment variable names with COPROCESSOR_ prefix and bumped versions to v0.10.5 |
test-suite/fhevm/scripts/deploy-fhevm-stack.sh |
Updated log output to reference new prefixed variable names |
test-suite/fhevm/docker-compose/coprocessor-docker-compose.yml |
Updated Docker image tag references to use new prefixed environment variables |
.github/workflows/test-suite-orchestrate-e2e-tests.yml |
Refactored outputs from arrays to individual named outputs for each component version |
.github/workflows/test-suite-e2e-tests.yml |
Consolidated workflow with expanded inputs and simplified environment variable mapping |
.github/workflows/test-suite-e2e-tests-mq.yml |
Removed redundant workflow file as part of consolidation effort |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
4312185 to
6285a72
Compare
6285a72 to
601379f
Compare
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.
Pull request overview
Copilot reviewed 6 out of 6 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Closes https://github.com/zama-ai/fhevm-internal/issues/821
Github now supports more than 10 inputs in reusable workflows (https://github.blog/changelog/2025-12-04-actions-workflow-dispatch-workflows-now-support-25-inputs/), so I updated the e2e tests workflow to add all services' versions as inputs
Run example: https://github.com/zama-ai/fhevm/actions/runs/20578396037/job/59100510651
BLOCKING: this issue is closed in the
actionlintrepo, but still not deployed in a proper release, so our CI won't pass until then