feat(compute01/search): init search.infra #199

Merged
thubrecht merged 5 commits from search-infra into main 2024-12-28 22:49:42 +01:00
Member
No description provided.
default.nix Outdated
@ -117,0 +137,4 @@
"machines/nixos/compute01/search/static-data/images/favicon.ico"
"machines/nixos/compute01/search/static-data/images/favicon.png"
];
license = "Reserved"; # FIXME: doesn't exist
Author
Member

@thubrecht si tu sait ce qu'on peut mettre ici, on pourra merge

@thubrecht si tu sait ce qu'on peut mettre ici, on pourra merge
lbailly marked this conversation as resolved
requested review from thubrecht 2024-12-21 17:47:28 +01:00
@ -0,0 +1,75 @@
# SPDX-FileCopyrightText: 2024 Lubin Bailly <lubin.bailly@dgnum.eu>
Owner

Pareil dans /modules (cf ci-dessous)

Pareil dans `/modules` (cf ci-dessous)
lbailly marked this conversation as resolved
@ -0,0 +1,192 @@
# SPDX-FileCopyrightText: 2024 Lubin Bailly <lubin.bailly@dgnum.eu>
Owner

On veut pas mettre ce module dans /modules/nixos ?

On veut pas mettre ce module dans `/modules/nixos` ?
Author
Member

on peut si on veut, je sais pas trop comment on décide de si on le fait ou pas...
je remarque qu'on ne le fait pas pour :

ptet que à terme se serai pas mal de tout bouger

on peut si on veut, je sais pas trop comment on décide de si on le fait ou pas... je remarque qu'on ne le fait pas pour : - [compute01/ds-fr](https://git.dgnum.eu/DGNum/infrastructure/src/branch/main/machines/nixos/compute01/ds-fr) - [compute01/librenms](https://git.dgnum.eu/DGNum/infrastructure/src/branch/main/machines/nixos/compute01/librenms) - [compute01/satosa](https://git.dgnum.eu/DGNum/infrastructure/src/branch/main/machines/nixos/compute01/satosa) - [vault01/k-radius](https://git.dgnum.eu/DGNum/infrastructure/src/branch/main/machines/nixos/vault01/k-radius) - [web01/linkal](https://git.dgnum.eu/DGNum/infrastructure/src/branch/main/machines/nixos/web01/linkal) - [web01/wordpress](https://git.dgnum.eu/DGNum/infrastructure/src/branch/main/machines/nixos/web01/wordpress) ptet que à terme se serai pas mal de tout bouger
lbailly marked this conversation as resolved
Owner

Nitpick: tu peux renommer search/default.nix en search.nix éventuellement et refaire un peu ton historique si tu veux mais pour moi on est bon

Nitpick: tu peux renommer `search/default.nix` en `search.nix` éventuellement et refaire un peu ton historique si tu veux mais pour moi on est bon
mdebray approved these changes 2024-12-22 11:57:17 +01:00
Dismissed
lbailly force-pushed search-infra from 9e9c2a4fcf to a6b9a4488a 2024-12-22 12:11:09 +01:00 Compare
Author
Member

tu peux renommer search/default.nix en search.nix éventuellement

je suis moyen chaud, le dossier search contient aussi des fichier statique qui sont relatif à ça uniquement

> tu peux renommer `search/default.nix` en `search.nix` éventuellement je suis moyen chaud, le dossier search contient aussi des fichier statique qui sont relatif à ça uniquement
Owner

tu peux renommer search/default.nix en search.nix éventuellement

je suis moyen chaud, le dossier search contient aussi des fichier statique qui sont relatif à ça uniquement

Ah oui désolé. Oublie ducoup

> > tu peux renommer `search/default.nix` en `search.nix` éventuellement > > je suis moyen chaud, le dossier search contient aussi des fichier statique qui sont relatif à ça uniquement Ah oui désolé. Oublie ducoup
thubrecht requested changes 2024-12-22 17:01:56 +01:00
Dismissed
@ -0,0 +1 @@
All rights reserved, unless other license granted.
Owner

Juste all rights reserved

Juste all rights reserved
lbailly marked this conversation as resolved
@ -0,0 +21,4 @@
enable = true;
modules = {
"netconf" = {
paths = [
Owner

mkImports = root: builtins.map (mkImport root);

lib.extra.mkImports est mieux

https://git.dgnum.eu/DGNum/infrastructure/src/commit/383911d6198bfda2ed7d84a1a2c1483338ce2dbe/lib/nix-lib/default.nix#L135 lib.extra.mkImports est mieux
lbailly marked this conversation as resolved
@ -0,0 +28,4 @@
];
path-translations = [
{
base = "${hive-root}";
Owner

Pas besoin d'interpoler, hive-root suffit

Pas besoin d'interpoler, hive-root suffit
Author
Member

c'est nécessaire si les imports sont interpolé

c'est nécessaire si les imports sont interpolé
lbailly marked this conversation as resolved
@ -0,0 +33,4 @@
}
];
};
"infra DGNum" =
Owner

DGNum Infrastructure

DGNum Infrastructure
lbailly marked this conversation as resolved
@ -0,0 +9,4 @@
...
}:
let
inherit (lib)
Owner

Ce serait bien de trier la liste

Ce serait bien de trier la liste
lbailly marked this conversation as resolved
@ -0,0 +55,4 @@
translations = map (
{ base, url }:
{
url = "${url}${if hasSuffix "/" url then "" else "/"}";
Owner

optionalString

optionalString
lbailly marked this conversation as resolved
@ -0,0 +73,4 @@
++ [
(throw (
"${fullPath} is not in any base path of ${module-name}. Base paths are "
+ concatStringsSep "\n" (map ({ base, ... }: base) translations)
Owner

concatMapStringsSep

concatMapStringsSep
lbailly marked this conversation as resolved
@ -0,0 +182,4 @@
};
};
in
types.attrsOf (types.submodule module-mod);
Owner

Si module-mod est utilisé juste ici, ça sert pas à grand chose de le définir dans un let, et il vaut mieux l'inline. Pareil pour path-mod

Si module-mod est utilisé juste ici, ça sert pas à grand chose de le définir dans un let, et il vaut mieux l'inline. Pareil pour path-mod
Author
Member

j'ai l'habitude de le faire pour que se soit un peu plus lisible, mais osef ig

j'ai l'habitude de le faire pour que se soit un peu plus lisible, mais osef ig
lbailly marked this conversation as resolved
lbailly force-pushed search-infra from a6b9a4488a to 2fbb98de4b 2024-12-22 21:20:56 +01:00 Compare
lbailly dismissed mdebray's review 2024-12-22 21:20:56 +01:00
Reason:

New commits pushed, approval review dismissed automatically according to repository settings

requested review from thubrecht 2024-12-22 21:23:34 +01:00
thubrecht approved these changes 2024-12-28 22:25:21 +01:00
Dismissed
thubrecht dismissed thubrecht's review 2024-12-28 22:25:29 +01:00
Reason:

New commits pushed, approval review dismissed automatically according to repository settings

thubrecht scheduled this pull request to auto merge when all checks succeed 2024-12-28 22:25:40 +01:00
thubrecht canceled auto merging this pull request when all checks succeed 2024-12-28 22:26:56 +01:00
thubrecht scheduled this pull request to auto merge when all checks succeed 2024-12-28 22:44:30 +01:00
dgnum-chores referenced this pull request from a commit 2025-10-29 07:03:24 +01:00
Sign in to join this conversation.
No description provided.