Skip to content

refactor(coprocessor): enable multi threaded test execution in CI#2121

Merged
mergify[bot] merged 4 commits intomainfrom
panos/multi-thread-ci-testing
Mar 18, 2026
Merged

refactor(coprocessor): enable multi threaded test execution in CI#2121
mergify[bot] merged 4 commits intomainfrom
panos/multi-thread-ci-testing

Conversation

@PanGan21
Copy link
Copy Markdown
Contributor

@PanGan21 PanGan21 commented Mar 17, 2026 •
edited
Loading

Summary

  • Use dynamic port allocation health check tests instead of hardcoded ports, fixing port conflicts when tests run concurrently
  • Expose test containers Postgres port to fix intermittent PortNotExposed errors
  • Remove --test-threads=1 from CI tests

Closes: https://github.com/zama-ai/fhevm-internal/issues/1156
Addresses this comment: #2086 (comment)

@PanGan21 PanGan21 requested review from a team as code owners March 17, 2026 10:03
@cla-bot cla-bot bot added the cla-signed label Mar 17, 2026
@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 17, 2026

Changed Lines Coverage

Coverage of added/modified lines: 100.0%

Per-file breakdown

Diff Coverage

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

  • coprocessor/fhevm-engine/test-harness/src/instance.rs (100%)
  • coprocessor/fhevm-engine/test-harness/src/localstack.rs (100%)

Summary

  • Total: 3 lines
  • Missing: 0 lines
  • Coverage: 100%

@mergify
Copy link
Copy Markdown

mergify bot commented Mar 17, 2026 •
edited
Loading

🧪 CI Insights

Here's what we observed from your CI run for 62239b6.

🟢 All jobs passed!

But CI Insights is watching 👀

@PanGan21 PanGan21 changed the title refactor(coprocessor): enable parallel test execution in CI refactor(coprocessor): enable multi threaded test execution in CI Mar 17, 2026
rudy-6-4
rudy-6-4 previously approved these changes Mar 17, 2026
View reviewed changes
@PanGan21 PanGan21 requested a review from rudy-6-4 March 17, 2026 14:35
@PanGan21
Copy link
Copy Markdown
Contributor Author

PanGan21 commented Mar 17, 2026

@mergify queue

@mergify
Copy link
Copy Markdown

mergify bot commented Mar 17, 2026 •
edited
Loading

Merge Queue Status

  • 🟠 Waiting for queue conditions
  • ⏳ Enter queue
  • ⏳ Run checks
  • ⏳ Merge
Required conditions to enter a queue
  • -closed [📌 queue requirement]
  • -conflict [📌 queue requirement]
  • -draft [📌 queue requirement]
  • any of [📌 queue -> configuration change requirements]:
    • -mergify-configuration-changed
    • check-success = Configuration changed
  • any of [🔀 queue conditions]:
    • all of [📌 queue conditions of queue main]:
      • #approved-reviews-by >= 1 [🛡 GitHub branch protection]
      • #changes-requested-reviews-by = 0 [🛡 GitHub branch protection]
      • #review-threads-unresolved = 0 [🛡 GitHub branch protection]
      • base = main
      • branch-protection-review-decision = APPROVED [🛡 GitHub branch protection]
      • label!=do-not-merge
      • any of [🛡 GitHub branch protection]:
        • check-success = common-pull-request/lint (bpr)
        • check-neutral = common-pull-request/lint (bpr)
        • check-skipped = common-pull-request/lint (bpr)
      • any of [🛡 GitHub branch protection]:
        • check-success = coprocessor-cargo-test/cargo-tests (bpr)
        • check-neutral = coprocessor-cargo-test/cargo-tests (bpr)
        • check-skipped = coprocessor-cargo-test/cargo-tests (bpr)
      • any of [🛡 GitHub branch protection]:
        • check-success = coprocessor-dependency-analysis/dependencies-check (bpr)
        • check-neutral = coprocessor-dependency-analysis/dependencies-check (bpr)
        • check-skipped = coprocessor-dependency-analysis/dependencies-check (bpr)
      • any of [🛡 GitHub branch protection]:
        • check-skipped = gateway-contracts-deployment-tests/sc-deploy (bpr)
        • check-neutral = gateway-contracts-deployment-tests/sc-deploy (bpr)
        • check-success = gateway-contracts-deployment-tests/sc-deploy (bpr)
      • any of [🛡 GitHub branch protection]:
        • check-skipped = kms-connector-tests/test-connector (bpr)
        • check-neutral = kms-connector-tests/test-connector (bpr)
        • check-success = kms-connector-tests/test-connector (bpr)

@eudelins-zama
Copy link
Copy Markdown
Contributor

eudelins-zama commented Mar 18, 2026

@Mergifyio refresh

@mergify
Copy link
Copy Markdown

mergify bot commented Mar 18, 2026

refresh

✅ Pull request refreshed

@mergify mergify bot added the queued label Mar 18, 2026
@mergify
Copy link
Copy Markdown

mergify bot commented Mar 18, 2026 •
edited
Loading

Merge Queue Status

This pull request spent 2 hours 47 minutes 41 seconds in the queue, including 1 hour 46 minutes 34 seconds running CI.

Required conditions to merge
  • #approved-reviews-by >= 1 [🛡 GitHub branch protection]
  • #changes-requested-reviews-by = 0 [🛡 GitHub branch protection]
  • #review-threads-unresolved = 0 [🛡 GitHub branch protection]
  • branch-protection-review-decision = APPROVED [🛡 GitHub branch protection]
  • check-success = run-e2e-tests / fhevm-e2e-test
  • any of [🛡 GitHub branch protection]:
    • check-success = common-pull-request/lint (bpr)
    • check-neutral = common-pull-request/lint (bpr)
    • check-skipped = common-pull-request/lint (bpr)
  • any of [🛡 GitHub branch protection]:
    • check-success = coprocessor-cargo-test/cargo-tests (bpr)
    • check-neutral = coprocessor-cargo-test/cargo-tests (bpr)
    • check-skipped = coprocessor-cargo-test/cargo-tests (bpr)
  • any of [🛡 GitHub branch protection]:
    • check-success = coprocessor-dependency-analysis/dependencies-check (bpr)
    • check-neutral = coprocessor-dependency-analysis/dependencies-check (bpr)
    • check-skipped = coprocessor-dependency-analysis/dependencies-check (bpr)
  • any of [🛡 GitHub branch protection]:
    • check-skipped = gateway-contracts-deployment-tests/sc-deploy (bpr)
    • check-neutral = gateway-contracts-deployment-tests/sc-deploy (bpr)
    • check-success = gateway-contracts-deployment-tests/sc-deploy (bpr)
  • any of [🛡 GitHub branch protection]:
    • check-skipped = kms-connector-tests/test-connector (bpr)
    • check-neutral = kms-connector-tests/test-connector (bpr)
    • check-success = kms-connector-tests/test-connector (bpr)

@Eikix
Copy link
Copy Markdown
Contributor

Eikix commented Mar 18, 2026

@Mergifyio refresh

@mergify
Copy link
Copy Markdown

mergify bot commented Mar 18, 2026

refresh

✅ Pull request refreshed

mergify bot added a commit that referenced this pull request Mar 18, 2026
@mergify
Merge of #2121
3d3e082
@mergify mergify bot mentioned this pull request Mar 18, 2026
Closed
51 tasks
@mergify mergify bot merged commit 645baf1 into main Mar 18, 2026
63 checks passed
@mergify mergify bot deleted the panos/multi-thread-ci-testing branch March 18, 2026 12:25
@mergify mergify bot added the merge-queued label Mar 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rudy-6-4 rudy-6-4 rudy-6-4 approved these changes

Assignees

No one assigned

Labels

cla-signed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@PanGan21 @eudelins-zama @Eikix @rudy-6-4