| 11 Jan 2022 |
das_j | so it does look like we have different ideas how it's supposed to work :D | 14:29:51 |
@grahamc:nixos.org | it could be very expensive to find the most recent job that builds/evaluates, especially because does it not evaluate because it was removed, or does it not evaluate because it is broken -- I don't think we differentiate | 14:31:32 |
das_j | I can try a RunCommand for that particular jobset that sets the latest job result as "Keep" (if I can figure out how I do that) | 14:32:11 |
@grahamc:nixos.org | that is an interesting idea! | 14:33:56 |
@grahamc:nixos.org | you would keep stale GC roots around for jobsets that are deleted, but that is probably f ine | 14:42:21 |
ajs124 | doesn't that happen already? | 14:42:46 |
@grahamc:nixos.org | I don't think so | 14:43:14 |
@grahamc:nixos.org | but maybe? | 14:43:16 |
das_j | I don't think so, no | 14:43:24 |
das_j | it goes over all jobsets, marks the roots to keep, and then goes over all root and deletes unmarked ones | 14:44:09 |
das_j | also fyi:
# If the jobset has been hidden and disabled for more than one
# week, then don't keep its builds anymore.
if ($jobset->enabled == 0 && ($project->hidden == 1 || $jobset->hidden == 1) && (time() - ($jobset->lastcheckedtime || 0) > (7 * 24 * 3600))) {
print STDERR "*** skipping disabled jobset ", $project->name, ":", $jobset->name, "\n";
next;
}
| 14:46:01 |
@grahamc:nixos.org | oh that is a nice enough feature | 14:46:46 |
das_j | would be even nicer if anyone knew about it :D | 14:46:56 |
@grahamc:nixos.org | I am constantly amazed by features I find | 14:47:18 |
das_j | also:
# Don't delete roots that are less than a day old, to prevent
# a race where hydra-eval-jobs has added a root but
# hydra-evaluator hasn't added them to the database yet.
| 14:47:41 |
@grahamc:nixos.org | got it | 14:47:56 |
@grahamc:nixos.org | well, hydra-eval-jobset can't take more than 1h to complete, so that 1d is generously sufficient | 14:48:32 |
@grahamc:nixos.org | re: https://github.com/NixOS/hydra/blob/ff26ce0d06d222ac0ee2908b72130e179e779165/src/script/hydra-eval-jobset#L41 | 14:49:03 |
das_j | so many things I learn today :D | 14:49:23 |
@grahamc:nixos.org | you can pick a random file in Hydra, read it top to bottom, and learn a great deal | 14:49:38 |
@grahamc:nixos.org | writing this test was very educational: https://github.com/NixOS/hydra/pull/1101/files#diff-4136e9d36533256d985deeaf2febd84bf03160ae5ea5030b6e658bdc7efbfcbc | 14:50:00 |
das_j | oh finally a full list of events | 14:50:27 |
@grahamc:nixos.org | no this is not a full list | 14:50:33 |
@grahamc:nixos.org | just a full list from the evaluator | 14:50:39 |
das_j | oh finally a full list of events from the evaluator | 14:50:53 |
@grahamc:nixos.org | for example the order builds are inserted in to the builds table is randomized on purpose so the scheduler doesn't get bogged down in KDE: https://github.com/NixOS/hydra/commit/109cc35edf29dde584f7b4b7375c1ac639162cbe | 14:51:33 |
@grahamc:nixos.org | it is complicated having a bunch of experimental branches that I'm using all at the same time, sigh | 15:25:31 |
@grahamc:nixos.org | whew. Here we go, a cleaned up list of things brewing:
More predictable statuses for integrations like GitHub:
- https://github.com/NixOS/hydra/pull/1101
- https://github.com/DeterminateSystems/hydra/pull/5
- https://github.com/DeterminateSystems/hydra/pull/6
Dynamic RunCommand hooks:
- https://github.com/NixOS/hydra/pull/1103
- https://github.com/DeterminateSystems/hydra/pull/4
- https://github.com/DeterminateSystems/hydra/pull/4
Schema cleanup:
- https://github.com/NixOS/hydra/pull/1093
| 15:57:15 |
@grahamc:nixos.org | * whew. Here we go, a cleaned up list of things brewing:
More predictable statuses for integrations like GitHub:
- https://github.com/NixOS/hydra/pull/1101
- https://github.com/DeterminateSystems/hydra/pull/5
- https://github.com/DeterminateSystems/hydra/pull/6
Dynamic RunCommand hooks:
- https://github.com/NixOS/hydra/pull/1103
- https://github.com/DeterminateSystems/hydra/pull/4
Schema cleanup:
- https://github.com/NixOS/hydra/pull/1093
| 15:59:37 |
@ius:nltrix.net | grahamc (he/him): I reproduced it using some test data and dropped by analysis on the most relevant ticket I could find https://github.com/NixOS/hydra/issues/901#issuecomment-1010158760 | 16:51:56 |