Infra: expose and use checks for vmOptions and nixosConfigurations #488

Merged
kiara merged 2 commits from Niols/Fediversity:flake-checks-for-vmOptions-and-nixosConfigurations into main 2025-07-31 15:41:04 +02:00
Owner

Following #478 (comment), here is a PR that plugs the infra's vmOptions and nixosConfigurations outputs into flake checks, instead of calling random Nix commands from the CI. There is still a bit of magic in the CI, but that's because we don't have yet a Nix-aware CI that exposes one job per flake check.

Following https://git.fediversity.eu/Fediversity/Fediversity/pulls/478#issuecomment-9163, here is a PR that plugs the infra's `vmOptions` and `nixosConfigurations` outputs into flake checks, instead of calling random Nix commands from the CI. There is still a bit of magic in the CI, but that's because we don't have yet a Nix-aware CI that exposes one job per flake check.
Niols added 2 commits 2025-07-31 00:41:29 +02:00
Expose and use checks for vmOptions and nixosConfigurations
All checks were successful
/ check-pre-commit (pull_request) Successful in 14s
/ check-data-model (pull_request) Successful in 34s
/ check-mastodon (pull_request) Successful in 22s
/ check-peertube (pull_request) Successful in 21s
/ check-panel (pull_request) Successful in 1m29s
/ check-deployment-basic (pull_request) Successful in 34s
/ check-deployment-cli (pull_request) Successful in 42s
/ check-deployment-panel (pull_request) Successful in 1m49s
/ check-resources (pull_request) Successful in 4m21s
d4fcf6a7e4
Niols force-pushed flake-checks-for-vmOptions-and-nixosConfigurations from d4fcf6a7e4 to 9d539d790d 2025-07-31 00:52:34 +02:00 Compare
kiara approved these changes 2025-07-31 11:14:38 +02:00
kiara merged commit 588bb77a94 into main 2025-07-31 15:41:04 +02:00
kiara deleted branch flake-checks-for-vmOptions-and-nixosConfigurations 2025-07-31 15:41:05 +02:00
fricklerhandwerk reviewed 2025-07-31 17:44:28 +02:00
@ -187,0 +200,4 @@
{ pkgs, ... }:
{
checks =
listToAttrs (

Why not mapAttrs'? (No need for a fix, this is readable enough as it is, just not very idiomatic IMO)

Why not `mapAttrs'`? (No need for a fix, this is readable enough as it is, just not very idiomatic IMO)
Author
Owner

...because I didn't know about it! I did think such a helper had to exist, but Noogle didn't give it to me. Easy PR!

...because I didn't know about it! I did think such a helper had to exist, but Noogle didn't give it to me. Easy PR!
Author
Owner

cf #489

cf https://git.fediversity.eu/Fediversity/Fediversity/pulls/489
Sign in to join this conversation.
No description provided.