Skip to content

hydra-evaluator: Rewrite in Rust#1605

Closed
Ericson2314 wants to merge 1 commit intoNixOS:masterfrom
obsidiansystems:no-c-plus-plus
Closed

hydra-evaluator: Rewrite in Rust#1605
Ericson2314 wants to merge 1 commit intoNixOS:masterfrom
obsidiansystems:no-c-plus-plus

Conversation

@Ericson2314
Copy link
Copy Markdown
Member

Port hydra-evaluator from C++ to Rust, removing the last C++ executable from the project. The new implementation uses sqlx and tokio, matching the async patterns already established by hydra-queue-runner and hydra-builder.

Build/packaging changes:

  • Remove all C++ dependencies from subprojects/hydra/meson.build
  • Move Rust packaging from hydra-queue-runner/package.nix to subprojects/rust-package.nix with named outputs (queue_runner, builder, evaluator)
  • Add evaluatorExecutable option to the NixOS web-app module, matching how the queue runner and builder are discovered via their own modules
  • Clean up stale references in hydra-tests/meson.build and dev-shell.nix

Port hydra-evaluator from C++ to Rust, removing the last C++ executable
from the project. The new implementation uses sqlx and tokio, matching
the async patterns already established by hydra-queue-runner and
hydra-builder.

Build/packaging changes:
- Remove all C++ dependencies from `subprojects/hydra/meson.build`
- Move Rust packaging from `hydra-queue-runner/package.nix` to
  `subprojects/rust-package.nix` with named outputs (`queue_runner`,
  `builder`, `evaluator`)
- Add `evaluatorExecutable` option to the NixOS web-app module, matching
  how the queue runner and builder are discovered via their own modules
- Clean up stale references in `hydra-tests/meson.build` and
  `dev-shell.nix`

Co-authored-by: Claude <noreply@anthropic.com>
@Ericson2314
Copy link
Copy Markdown
Member Author

This is now #1608, I just changed the base branch so @Mic92 and others can push to it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant