Commit graph

3705 commits

Author SHA1 Message Date
Graham Christensen 64a3e75c10 Event: init structure and parse existing messages 2021-08-13 16:51:29 -04:00
Graham Christensen a14c8ad5f8
Merge pull request #995 from DeterminateSystems/declarative-jobsets-plugin
Declarative jobsets: move event handling to a plugin
2021-08-12 15:56:13 -04:00
Graham Christensen 0f958f3425
Merge pull request #997 from DeterminateSystems/abstract-listener
Abstract over postgres' LISTEN/NOTIFY
2021-08-12 14:00:34 -04:00
Graham Christensen f30addb9d7
Merge pull request #996 from DeterminateSystems/cleanup-build-finished
hydra-notify: move buildFinished query in to the function impl
2021-08-12 13:57:13 -04:00
Graham Christensen 5027003285 Abstract over postgres' LISTEN/NOTIFY
This lets us test the event loop if we wanted, and lets us
test the listening behavior in isolation.
2021-08-12 13:54:05 -04:00
Graham Christensen 593af41808 Declarative jobsets: move event handling to a plugin
Declarative jobsets were sort of tucked in to the event hanlder
itself. It turned out that it could have been implemented as a
plugin without much trouble.
2021-08-12 12:48:18 -04:00
Graham Christensen 9c5f317453 hydra-notify: move buildFinished query in to the function impl
This is more consistent with the other event handlers, of dealing
with IDs and not objects.
2021-08-12 12:30:35 -04:00
Eelco Dolstra 9bce425c33
Merge pull request #994 from NixOS/update-nix
Update nix
2021-08-10 14:35:30 +02:00
Eelco Dolstra 5edb58b314 Fix build 2021-08-10 13:47:16 +02:00
Eelco Dolstra f1d5c254b5 flake.lock: Update
Flake input changes:

* Updated 'nix': 'github:NixOS/nix/d9864be4b757468d33bc49edddce5e4f04ef4b90' -> 'github:NixOS/nix/a6ba313a0aac3b6e2fef434cb42d190a0849238e'
* Added 'nix/lowdown-src': 'github:kristapsdz/lowdown/148f9b2f586c41b7e36e73009db43ea68c7a1a4d'
* Updated 'nix/nixpkgs': 'github:NixOS/nixpkgs/0e499fde7af3c28d63e9b13636716b86c3162b93' -> 'github:NixOS/nixpkgs/f77036342e2b690c61c97202bf48f2ce13acc022'
2021-08-10 13:41:04 +02:00
Graham Christensen 429be3ed44
Merge pull request #988 from DeterminateSystems/editorconfig
Initialize a basic editorconfig
2021-08-06 15:14:11 -04:00
Graham Christensen 3ceac80cdc Initialize a basic editorconfig
Based on NixOS/nixpkgs' editorconfig + some for Perl.
2021-08-06 14:59:40 -04:00
Graham Christensen ba1ea8d9c4
Merge pull request #989 from DeterminateSystems/docs/connect-local-db
docs: note how to connect to the database
2021-08-06 14:56:20 -04:00
Graham Christensen c7c322545d
Merge pull request #992 from DeterminateSystems/sql/fixup-comment
hydra.sql: Update comment on regeneration
2021-08-06 14:54:12 -04:00
Graham Christensen d1670b0526
Merge pull request #991 from DeterminateSystems/sql/fixup-generator-syntax
update-dbix.pl: correct indentation
2021-08-06 14:53:38 -04:00
Graham Christensen 512b0663e8
Merge pull request #990 from DeterminateSystems/docs/notifications
docs/notifications: fixup sending a notification
2021-08-06 14:52:55 -04:00
Graham Christensen 4169f22231 update-dbix.pl: correct indentation 2021-08-06 14:40:57 -04:00
Graham Christensen 5bd8dc171b hydra.sql: Update comment on regeneration
We no longer need to generate the hydra-postgres.sql document,
that is a relic from when sqlite was also supported.
2021-08-06 14:40:34 -04:00
Graham Christensen 5900e4d0b3 docs/notifications: fixup sending a notification 2021-08-06 14:35:38 -04:00
Graham Christensen 23ac4a665c docs: note how to connect to the database 2021-08-06 14:34:59 -04:00
Graham Christensen 4e94551602
Merge pull request #985 from kreisys/hydra-server-config-includes
Make hydra-server honor apache-style includes in hydra.conf like all the other components
2021-07-29 12:44:52 -04:00
Shay Bergmann d64caea870
Remove clause suggesting absolute paths don't work 2021-07-26 19:51:29 +00:00
Shay Bergmann f36bffa008
Use absolute path in example as it seems to be working now
might have something to do with `-IncludeRelative`...
2021-07-26 19:29:28 +00:00
Shay Bergmann 638f5a1ac9
Note in README how to run a single test 2021-07-26 17:16:41 +00:00
Shay Bergmann ba557972c7
Add test to verify includes work 2021-07-26 17:16:40 +00:00
Shay Bergmann 503b0e0b6f
Remove unnecessary comment 2021-07-26 17:16:40 +00:00
Shay Bergmann 173ecb5d92
Apply requested changes to Config.pm 2021-07-26 17:16:40 +00:00
Shay Bergmann 4f95774ca3
Factor out common options 2021-07-26 03:43:09 +00:00
Shay Bergmann 3c621ec386
... add some needed gitignores... 2021-07-26 03:42:08 +00:00
Shay Bergmann 45077a120d
... fix embarrassing typo... 2021-07-23 22:10:00 +00:00
Shay Bergmann e98bd0ec8d
Comment around dupe to avoid drift 2021-07-22 23:27:21 -04:00
Shay Bergmann 774194d681
Update docs 2021-07-22 23:12:25 -04:00
Shay Bergmann 54c8028ec0
Make hydra-server honor config file includes 2021-07-22 17:25:30 -04:00
Graham Christensen 2f0dc294af
Merge pull request #984 from DeterminateSystems/test-Jobset-constraints
Jobset: test `type`-related constraints
2021-07-20 13:45:33 -04:00
Cole Helbling e6a8412a12 Jobset: test type-related constraints
Prior to https://github.com/NixOS/hydra/pull/856, the `type`-related constraints
were a bit too loose, allowing some variations that should have been seen as
invalid. This commit adds a test (inspired by
https://github.com/NixOS/hydra/pull/856#issuecomment-772820929) to ensure these
constraints stay unviolated.
2021-07-20 10:38:22 -07:00
Graham Christensen 2bb1ba22d3
Merge pull request #983 from Taneb/libpqxx-deprecations
Fix build with libpqxx 7
2021-07-05 20:15:27 -04:00
Nathan van Doorn b63dfb7ad4 Rename version to version.txt
<version> is a standard header with C++20 which could cause issues if a library checks it exists then imports it

Because we have the root of this repo in the include path, it'd see that <version> exists (with, e.g., __has_include), and then try to include it as a header

But because it's just a file that says 0.1, this would fail

This happens with libpqxx 7
2021-07-05 19:47:58 +01:00
Nathan van Doorn 5c47343b40 Fix deprecation warnings with libpqxx usage 2021-07-05 19:45:25 +01:00
Graham Christensen e02f34722f
Merge pull request #887 from helsinki-systems/feat/eval-overview-duration
Show evaluation time in the evaluations overview
2021-06-29 10:59:02 -04:00
Janne Heß 94b88b34ae
Show evaluation time in the evaluations overview
Parts of this code have been authored by @grahamc
2021-06-29 16:53:10 +02:00
Jonathan Ringer 24959a3ca6 Allow for evaluator_pure_eval to be configured 2021-06-28 16:24:40 -07:00
Graham Christensen 6f9db20cd8
Merge pull request #976 from ztzg/x-16304/deep-clone-inputs
GitInput: Include deepClone option in the cache key
2021-06-28 15:38:38 -04:00
Graham Christensen 04fdb8e6e6
Merge pull request #980 from DeterminateSystems/fixup-drv-meta-maintainers
Docs: fixup `maintainers` derivation meta field example
2021-06-28 15:14:19 -04:00
Cole Helbling 5e72d42575
Docs: fixup maintainers derivation meta field example 2021-06-28 11:16:25 -07:00
Eelco Dolstra 082140bdb2
Merge pull request #979 from DeterminateSystems/hydra-timeout
Docs: document derivation attributes
2021-06-28 18:42:19 +02:00
Graham Christensen 59018323ae Docs: document derivation attributes 2021-06-28 10:00:05 -04:00
Damien Diederen df7dab1291 GitInput: Include deepClone option in the cache key
Without this commit, two jobsets using the same repository as input,
but different `deepClone` options, end up incorrectly sharing the same
"checkout" for a given (`uri`, `branch`, `revision`) tuple.  The
presence or absence of `.git` is determined by the jobset execution
order.

This patch adds the missing `isDeepClone` boolean to the cache key.

The database upgrade script empties the `CachedGitInputs` table, as we
don't know if existing checkouts are deep clones.  Unfortunately, this
generally forces rebuilds even for correct `deepClone` checkouts, as
the binary contents of `.git` are not deterministic.

Fixes #510
2021-06-19 17:37:40 +02:00
Graham Christensen b6921c2006
Merge pull request #974 from AmandaCameron/patch-1
flake: stdenv.lib -> lib
2021-06-17 17:11:51 -04:00
Amanda Cameron b0f8a207fe
flake: stdenv.lib -> lib
Nixpkgs on unstable has removed `stdenv.lib` as they've been warning for a while now. This removes the extra references to it in the flake.nix

I'm not entirely sure if this is right, but I figured it was trivial enough to give a quick try using the GH Editor while I was waiting for a job to finish
2021-06-17 16:50:38 -04:00
Graham Christensen ac1ef9361e
Merge pull request #913 from grahamc/notif-docs
Notifications: write up some docs
2021-06-17 13:27:21 -04:00