Skip to content

feat(coprocessor): parallelize CI tests per service and reduce operator tests for prs#2164

Merged
mergify[bot] merged 5 commits intomainfrom
panos/parallelize-ci-test-per-service
Mar 27, 2026
Merged

feat(coprocessor): parallelize CI tests per service and reduce operator tests for prs#2164
mergify[bot] merged 5 commits intomainfrom
panos/parallelize-ci-test-per-service

Conversation

@PanGan21
Copy link
Copy Markdown
Contributor

@PanGan21 PanGan21 commented Mar 24, 2026

Summary

  • Parallelize coprocessor CI tests across 7 service-specific matrix jobs using per-package
  • Add dedicated coverage-report job that merges per-service LCOV artifacts with lcov
  • Add cargo-tests-status aggregator job to preserve the coprocessor-cargo-test/cargo-tests (bpr) branch protection check name
  • Runs the operator tests up to FHEUint64 on pr and everything on merge-queue
  • Runs the operator gpu tests only for FHEUint64 on pr and everything on merge-queue
  • Make test harness DB setup generic: extract DB name from URL instead of hardcoding "coprocessor", enabling isolated per-service test databases
  • Replace manual table truncation in transaction-sender tests with setup_test_db() for consistency
  • Removes unused postgres setup in docker from CI

Closes: https://github.com/zama-ai/fhevm-internal/issues/1186

@cla-bot cla-bot bot added the cla-signed label Mar 24, 2026
@PanGan21 PanGan21 force-pushed the panos/parallelize-ci-test-per-service branch 3 times, most recently from 48028b3 to 3f93d9a Compare March 24, 2026 12:35
@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 24, 2026

Changed Lines Coverage

Coverage of added/modified lines: 95.0%

Per-file breakdown

Diff Coverage

Diff: origin/main...HEAD, staged and unstaged changes

  • coprocessor/fhevm-engine/test-harness/src/instance.rs (95.2%): Missing lines 85

Summary

  • Total: 21 lines
  • Missing: 1 line
  • Coverage: 95%

coprocessor/fhevm-engine/test-harness/src/instance.rs

  81     let db_url = DatabaseURL::default();
  82 
  83     if with_reset {
  84         info!("Resetting local database at {db_url}");
! 85         let admin_db_url = admin_url_from(db_url.as_str());
  86         create_database(&admin_db_url, db_url.as_str(), mode).await?;
  87     }
  88 
  89     info!("Using existing local database at {db_url}");

@PanGan21 PanGan21 force-pushed the panos/parallelize-ci-test-per-service branch 3 times, most recently from c388eaf to 0374867 Compare March 24, 2026 13:11
@PanGan21 PanGan21 force-pushed the panos/parallelize-ci-test-per-service branch 3 times, most recently from d973b6c to 8768f0b Compare March 24, 2026 15:00
@PanGan21 PanGan21 marked this pull request as ready for review March 25, 2026 11:26
@PanGan21 PanGan21 requested review from a team as code owners March 25, 2026 11:26
@PanGan21 PanGan21 force-pushed the panos/parallelize-ci-test-per-service branch 9 times, most recently from 59d0b9c to 39b80b8 Compare March 25, 2026 15:28
@PanGan21 PanGan21 force-pushed the panos/parallelize-ci-test-per-service branch 2 times, most recently from db24fb0 to 48808a1 Compare March 26, 2026 09:37
@PanGan21 PanGan21 marked this pull request as draft March 26, 2026 10:03
@PanGan21 PanGan21 force-pushed the panos/parallelize-ci-test-per-service branch from 27d48df to e9f21fb Compare March 26, 2026 11:22
@PanGan21 PanGan21 marked this pull request as ready for review March 26, 2026 11:52
@antoniupop
Copy link
Copy Markdown
Collaborator

You may need to rename if not using nextest

antoniupop
antoniupop previously approved these changes Mar 26, 2026
@PanGan21 PanGan21 changed the title feat(coprocessor): parallelize CI tests across services using cargo-nextest feat(coprocessor): parallelize CI tests per service and reduce operator tests for prs Mar 26, 2026
Eikix
Eikix previously approved these changes Mar 26, 2026
Copy link
Copy Markdown
Contributor

@Eikix Eikix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@PanGan21 PanGan21 force-pushed the panos/parallelize-ci-test-per-service branch 2 times, most recently from ded89d0 to fdfae24 Compare March 26, 2026 20:38
@PanGan21 PanGan21 force-pushed the panos/parallelize-ci-test-per-service branch from fdfae24 to 14acc38 Compare March 26, 2026 20:39
@PanGan21 PanGan21 requested review from Eikix and antoniupop March 27, 2026 10:00
Copy link
Copy Markdown
Contributor

@Eikix Eikix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@PanGan21
Copy link
Copy Markdown
Contributor Author

@mergify queue

@mergify
Copy link
Copy Markdown

mergify bot commented Mar 27, 2026

Merge Queue Status

This pull request spent 1 hour 52 minutes 45 seconds in the queue, including 1 hour 14 minutes 55 seconds running CI.

Required conditions to merge

mergify bot added a commit that referenced this pull request Mar 27, 2026
@mergify mergify bot merged commit cd785d1 into main Mar 27, 2026
78 of 79 checks passed
@mergify mergify bot deleted the panos/parallelize-ci-test-per-service branch March 27, 2026 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants