| 5 Jul 2024 |
Mic92 | * And it was missing the member permission before hand. | 16:11:59 |
Matt Sturgeon | Makes sense! I think it'd be nice if the button was disabled whenever perms are missing, rather than being enabled but having no effect. Showing an error message would also be good if a stop/rebuild/whatever fails š | 16:12:14 |
Matt Sturgeon | Thanks for your help | 16:13:08 |
Mic92 | Usually it would print some error message | 16:14:09 |
Mic92 | i.e. that you are not in the admin group | 16:14:18 |
Mic92 | Disabling the button would require some changes in buildbot itself. | 16:14:42 |
Matt Sturgeon | Even if I logout, I don't get any errors when I try to use the buttons | 16:15:51 |
| Artur Manuel changed their display name from Ć
smund Balkesen to Artur Manuel. | 17:26:22 |
Matt Sturgeon | Mic92 Our last few PR pushes don't seem to be triggering buildbot to run. Did something change roughly an hour ago? | 17:30:53 |
Mic92 | there was one 500er error for webhooks: https://github.com/nix-community/nixvim/settings/hooks/452738934?tab=deliveries | 17:32:54 |
Mic92 | here i can see buildbot: https://github.com/nix-community/nixvim/commits/main/ | 17:33:32 |
Matt Sturgeon | In reply to @joerg:thalheim.io there was one 500er error for webhooks: https://github.com/nix-community/nixvim/settings/hooks/452738934?tab=deliveries GaƩtan Lepage I can't access webhook settings, could you take a look? | 17:33:52 |
Mic92 | nothing here: https://github.com/nix-community/nixvim/pull/1814 | 17:34:00 |
GaƩtan Lepage |  Download clipboard.png | 17:35:00 |
Mic92 | Looks like a github problem | 17:36:38 |
Mic92 | there is no event for my PR: https://github.com/nix-community/nixvim/pull/1815 | 17:36:46 |
Mic92 | Matt Sturgeon: https://github.com/nix-community/nixvim/pull/1815 | 17:38:53 |
Mic92 | * Matt Sturgeon: https://status.github.com | 17:39:08 |
Mic92 | looks like a global outage for webhooks | 17:39:19 |
Mic92 | The probably come back later. | 17:39:34 |
Mic92 | * They probably come back later. | 17:40:08 |
Matt Sturgeon | Ah ok, sorry to ping you then. Thanks for investigating | 17:40:20 |
GaƩtan Lepage | Mic92 is it possible to enforce allow-import-from-derivation=false in build-bot ? | 20:14:11 |
| 6 Jul 2024 |
| @nuko:shimeji.cafe left the room. | 01:55:54 |
Mic92 | GaƩtan Lepage: it should work, we did not disallow it if I remember correctly: https://github.com/nix-community/nix-eval-jobs/blob/9bed665aa71907dcacc8ae6dba1a3055a9ff2de7/src/nix-eval-jobs.cc#L370 | 03:38:26 |
Mic92 | however you should avoid building large derivations in there because it will than do builds in the evaluation phase, which is not so great. | 03:39:04 |
Mic92 | ah you mean the opposite. | 03:39:37 |
Mic92 | puh. We don't have per project settings. However at some point we should add them. | 03:40:27 |
GaƩtan Lepage | Yes, we were relying on IFD until... yesterday and we were finally able to get rid of it.
We would then want to strictly disable it in the CI to avoid introducing it by mistake in the future.
Not a blocker though. | 09:20:15 |
Matt Sturgeon | In an ideal world we'd have an (e.g.) .github/builtbot.yml config file where we can define a nix-config object used when running jobs. IDK how easy that is to support on your end though š«¤
# e.g.
build-eval:
nix-config:
allow-import-from-derivation: false
| 11:25:35 |