Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Clarify hydra-notify eval_failed #1434

Open
Rowan-Mather opened this issue Jan 30, 2025 · 1 comment
Open

Clarify hydra-notify eval_failed #1434

Rowan-Mather opened this issue Jan 30, 2025 · 1 comment

Comments

@Rowan-Mather
Copy link

Is your feature request related to a problem? Please describe.
From the manual entry on hydra-notify, my understanding was that an event for eval_failed is triggered when you get an evaluation error such as the one below. But this does not seem to be the case.

Perhaps this is only triggered when hydra cannot perform an evaluation of any description - irrelevant of whether it results in an evaluation error or not?

Example evaluation error from the #tabs-errors page:

evaluation failed with exit code 255

Describe the solution you'd like
Please be specific in the manual.

@Rowan-Mather
Copy link
Author

Rowan-Mather commented Mar 10, 2025

From further experimentation, this is my new understanding:

  • eval_failed is triggered when no evaluation can occur at all upon the 'check' of the jobset.
  • lastChecked is the last time the evaluator checked this jobset, and should be triggered on input update, and then periodically.
  • Sometimes you might have a jobset which has some jobs that can be evaluated and some that cause evaluation errors - these would not trigger eval_failed.

So there doesn't seem to be any obvious way to determine whether a jobset has evaluation errors, save checking both eval_failed and every single completed evaluation for errors. NB that if you hook into eval_failed you will get repeat reports of the same errors until they're fixed, as hydra periodically checks the jobset.

You have two routes to do this:

  1. Integrate with hydra-notify, which is what is recommended in the manual, but this requires patching hydra with your desired functionality here: https://github.com/NixOS/hydra/blob/master/src/lib/Hydra/Plugin.pm
  2. Use the DB directly. Not recommended but doesn't require patching hydra and is simpler in my opinion.

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

No branches or pull requests

1 participant