Skip to content

ci: accelerate hydra ci#2851

Merged
Alenar merged 2 commits into
mainfrom
djo/2822/nix/accelerate-hydra-ci
Dec 10, 2025
Merged

ci: accelerate hydra ci#2851
Alenar merged 2 commits into
mainfrom
djo/2822/nix/accelerate-hydra-ci

Conversation

@Alenar

@Alenar Alenar commented Dec 10, 2025 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Content

This PR includes adjustments to the nix based hydra ci to address the frequent timeout encountered:

  • build test in release mode to allow reuse of built artifact from the build phase (this revert 0e92798)
  • remove unneeded projects (mithrildemo and mithril-persistence) to keep only released binaries and libraries + tools eventually useful to third party (only the end to end at that time)

Here is a successful hydra evaluation based on the modification: https://ci.iog.io/eval/94135

We can see that the check phase reuse artifacts, speeding up the process (example from mithril-aggregator linux x86_64):

Running phase: checkPhase
+++ command cargo test --profile release -p mithril-aggregator --no-default-features
       Fresh unicode-ident v1.0.19
       Fresh smallvec v1.15.1
       [...]
    Finished `release` profile [optimized] target(s) in 1m 45s

For comparison, a previous run on the same job without the changes took 2m 39s and everything was recompiled in the check phase.

Pre-submit checklist

  • Branch
    • Commit sequence broadly makes sense
    • Key commits have useful messages
  • PR
    • All check jobs of the CI have succeeded
    • Self-reviewed the diff
    • Useful pull request description
    • Reviewer requested
  • Documentation
    • No new TODOs introduced

Issue(s)

Closes #2822

@Alenar
ci(hydra): remove unneeded packages
f291606 
Only public binaries and libraries should be built, with the addition of
eventual tools that may be usefull to third parties (only the e2e at the
moment).
@Alenar
ci(hydra): build test in release mode for faster runtime
6c29b0c 
This allow test to re-use artifacts built in the build phase.

This should reduce the number of timeout we see in the hydra ci.
@Alenar
Alenar requested a review from jpraynaud December 10, 2025 09:44
@Alenar Alenar self-assigned this Dec 10, 2025
@github-actions

github-actions Bot commented Dec 10, 2025

Copy link
Copy Markdown

Test Results

    4 files  ±0    172 suites  ±0   23m 34s ⏱️ +2s
2 271 tests ±0  2 271 ✅ ±0  0 💤 ±0  0 ❌ ±0 
7 125 runs  ±0  7 125 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 6c29b0c. ± Comparison against base commit 3afd04e.

@Alenar
Alenar temporarily deployed to testing-preview December 10, 2025 10:06 — with GitHub Actions Inactive
@Alenar
Alenar marked this pull request as ready for review December 10, 2025 10:18
jpraynaud
jpraynaud approved these changes Dec 10, 2025
View reviewed changes

@jpraynaud jpraynaud left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@Alenar
Alenar merged commit 4503402 into main Dec 10, 2025
106 of 107 checks passed
@Alenar
Alenar deleted the djo/2822/nix/accelerate-hydra-ci branch December 10, 2025 11:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jpraynaud jpraynaud jpraynaud approved these changes

Assignees

@Alenar Alenar

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Accelerate Hydra CI builds

2 participants

@Alenar @jpraynaud