Skip to content

merge queue: embarking main (bfc89a5) and #2119 together#2175

Closed
mergify[bot] wants to merge 9 commits intomainfrom
mergify/merge-queue/c316354f8b
Closed

merge queue: embarking main (bfc89a5) and #2119 together#2175
mergify[bot] wants to merge 9 commits intomainfrom
mergify/merge-queue/c316354f8b

Conversation

@mergify
Copy link
Copy Markdown

@mergify mergify bot commented Mar 25, 2026 •
edited
Loading

🎉 This pull request has been checked successfully and will be merged soon. 🎉

Branch main (bfc89a5) and #2119 are embarked together for merge.

This pull request has been created by Mergify to speculatively check the mergeability of #2119.
You don't need to do anything. Mergify will close this pull request automatically when it is complete.

Required conditions of queue rule main for 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)

Required conditions to stay in the queue:

---
checking_base_sha: bfc89a5d06224095eb1ce006719765f59ec3a095
previous_failed_batches:
  - checked_pull_requests:
      - 2119
      - 2157
    draft_pr_number: 2171
pull_requests:
  - number: 2119
    scopes: []
scopes: []
...

@cla-bot cla-bot bot added the cla-signed label Mar 25, 2026
@mergify mergify bot mentioned this pull request Mar 25, 2026
Merged
@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 26, 2026

Changed Lines Coverage

Coverage of added/modified lines: 97.0%

Per-file breakdown

Diff Coverage

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

  • coprocessor/fhevm-engine/fhevm-engine-common/src/utils.rs (100%)
  • coprocessor/fhevm-engine/host-listener/src/cmd/mod.rs (100%)
  • coprocessor/fhevm-engine/host-listener/src/database/dependence_chains.rs (100%)
  • coprocessor/fhevm-engine/host-listener/src/database/ingest.rs (89.7%): Missing lines 495,513,523,543,547,551
  • coprocessor/fhevm-engine/host-listener/src/health_check.rs (100%)

Summary

  • Total: 259 lines
  • Missing: 6 lines
  • Coverage: 97%

coprocessor/fhevm-engine/host-listener/src/database/ingest.rs

  491         );
  492 
  493         assert!(
  494             slow.is_empty(),
! 495             "no chain should be slow at exactly the threshold"
  496         );
  497     }
  498 
  499     // Single chain with exactly max_per_chain ops is not slow.

  509             max,
  510         );
  511         assert!(
  512             at_boundary.is_empty(),
! 513             "exactly at threshold should be fast"
  514         );
  515 
  516         let over_boundary = classify_slow_by_split_dependency_closure(
  517             &chains,

  519             max,
  520         );
  521         assert!(
  522             over_boundary.contains(&chains[0].hash),
! 523             "one over threshold should be slow"
  524         );
  525     }
  526 
  527     // Non linear: A -> B, A -> C, B -> D, C -> D

  539 
  540         assert!(slow.contains(&chains[0].hash), "A should be slow");
  541         assert!(
  542             slow.contains(&chains[1].hash),
! 543             "B should be slow (depends on A)"
  544         );
  545         assert!(
  546             slow.contains(&chains[2].hash),
! 547             "C should be slow (depends on A)"
  548         );
  549         assert!(
  550             slow.contains(&chains[3].hash),
! 551             "D should be slow (depends on B and C)"
  552         );
  553     }
  554 }

@mergify mergify bot closed this Mar 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

cla-signed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@PanGan21