| 11 Sep 2024 |
| nyanbinary left the room. | 15:29:02 |
| 16 Sep 2024 |
| silentlurker joined the room. | 20:00:40 |
| 17 Sep 2024 |
| Tom (deprecated) joined the room. | 21:10:15 |
| 18 Sep 2024 |
cleverca22 | John Ericson: https://github.com/NixOS/nix/commit/bc085022494fe90f733aef0832b6d7dcc34709cf has the fix to hydra been written yet? i updated my hydra recently, and now the cache is down | 03:42:13 |
John Ericson | In reply to @cleverca22:matrix.org John Ericson: https://github.com/NixOS/nix/commit/bc085022494fe90f733aef0832b6d7dcc34709cf has the fix to hydra been written yet? i updated my hydra recently, and now the cache is down which fix / which problem? | 03:58:20 |
John Ericson | there is some base64 thing that prevented updating cache.nixos.org yet' | 03:58:36 |
John Ericson | * there is some base64 thing that prevented updating cache.nixos.org yet | 03:58:39 |
John Ericson | https://github.com/NixOS/hydra/commits/hydra.nixos.org-nix-2.21 | 03:59:14 |
John Ericson | want to switch to that | 03:59:20 |
| subfractal changed their display name from gkze to subfractal. | 15:32:16 |
cleverca22 | In reply to @Ericson2314:matrix.org which fix / which problem? Caught exception in Hydra::Controller::Root->nar "Undefined subroutine &Hydra::Controller::Root::isValidPath called at /nix/store/hsizk3l5p792wim3xh4ddbmyyqrsw1fn-hydra-0-unstable-2024-09-15/libexec/hydra/lib/Hydra/Controller/Root.pm line 333." | 21:08:18 |
cleverca22 | the entire binary cache api is broken | 21:08:25 |
cleverca22 | narinfo files work, but nar's dont, so nothing can DL | 21:10:08 |
John Ericson | cleverca22: ah that looks like one we missed | 21:25:34 |
John Ericson | Make it $MACHINE_LOCAL_STORE->isValidPath(...) | 21:26:15 |
John Ericson | cleverca22: it would be nice if you could PR it to the nix-2.21 branch | 21:27:30 |
| 19 Sep 2024 |
| @max.hausch:helsinki-systems.de left the room. | 08:48:28 |
| 20 Sep 2024 |
cleverca22 | <div class="page-header">500 Internal Server Error </div> <div class="alert alert-error">Caught exception in Hydra::View::NARInfo->process "Can't locate object method "readFile" via package "/etc/nix/keys/secret-key-file" (perhaps you forgot to load "/etc/nix/keys/secret-key-file"?) at /nix/store/78qww3jbcnvbhw6xpfn7mc0bpph8ngx4-hydra-0.1.20240920.cc1b6d3/libexec/hydra/lib/Hydra/View/NARInfo.pm line 39."</div>
John Ericson applied that patch and upgraded hydra to 1 commit ahead of that branch, now the narinfo endpoint is broken instead
| 08:00:40 |
cleverca22 | --- a/src/lib/Hydra/View/NARInfo.pm
+++ b/src/lib/Hydra/View/NARInfo.pm
@@ -36,7 +36,7 @@ sub process {
# Optionally, sign the NAR info file we just created.
my $secretKeyFile = $c->config->{binary_cache_secret_key_file};
if (defined $secretKeyFile) {
- my $secretKey = readFile $secretKeyFile;
+ my $secretKey = readFile($secretKeyFile);
my $fingerprint = fingerprintPath($storePath, $narHash, $narSize, $refs);
my $sig = signString($secretKey, $fingerprint);
$info .= "Sig: $sig\n";
| 08:51:37 |
cleverca22 | i noticed, the function call here, is missing some parens | 08:51:44 |
das_j | In reply to @cleverca22:matrix.org i noticed, the function call here, is missing some parens That's valid in Perl 🤡 | 09:01:20 |
cleverca22 | In reply to @janne.hess:helsinki-systems.de That's valid in Perl 🤡 is it still valid in the latest version? has something changed? what does the error mean exactly? why is it treating the secret key as a module? | 09:17:48 |
das_j | readFile is only in the c++ codebase | 09:49:04 |
das_j | for reading a file, you would use:
use File::Slurper qw(read_text);
# …
read_text($secretKeyFile);
| 09:50:00 |
das_j | * for reading a file in the perl codebase, you would use:
use File::Slurper qw(read_text);
# …
read_text($secretKeyFile);
| 09:50:09 |
das_j | * for reading a file in the perl codebase, you would use:
use File::Slurper qw(read_text);
# …
my $secretKey = read_text($secretKeyFile);
| 09:50:31 |
das_j | I assume something went wrong here: https://github.com/NixOS/hydra/commit/b94a7b6d5c56362af9ea85d944f8454d861ec001 | 09:52:18 |
das_j | But there is no reason to use the nix function rather than the perl way of life | 09:52:29 |
Mic92 | cleverca22: https://github.com/NixOS/hydra/pull/1413 | 09:53:13 |
cleverca22 | In reply to @joerg:thalheim.io cleverca22: https://github.com/NixOS/hydra/pull/1413 CI tests fail, still deploying on my end | 10:04:02 |