feat(meta/ci): allows to group ci evals #630
No reviewers
thubrecht
Labels
No labels
awaiting
awaiting-author
awaiting
awaiting-reviewer
bot
deployed
Silenced alert
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
No milestone
No project
No assignees
2 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
DGNum/infrastructure!630
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "ci-grouping"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
c9ed40674b657ceac206657ceac20607304b138307304b13837fa8d37d0a7fa8d37d0ae7ccf0ebfee7ccf0ebfe6683183f55453eec768e54cabf9048WIP: feat(meta/ci): allows to group ci evalsto feat(meta/ci): allows to group ci evals54cabf90488cbf292086il faudrait probablement groupé d'autres truc, mais t'as l'esprit
@ -269,2 +269,4 @@scripts = import ./scripts { inherit pkgs sources; };base-colmena = pkgs.callPackage "${sources.colmena}/package.nix" { };On peut l'appeler juste
colmenanon ? Ça clashe pas avec le wrapper qui est pas nommé et juste dans la liste des paquets du shell8cbf292086117910ffe2@ -0,0 +28,4 @@name = "Eval ${group}";run = nix-actions.lib.nix-shell {script = ''DRV=$(colmena eval -E 'import ./workflows/eval-nodes/drv-maker.nix ./bootstrap.nix "${group}"' --instantiate)en vrai, je pense qu'il y a moyen de faire mieux pour le ./workflows/eval-nodes/drv-maker.nix , en volant des trucs de nixopts-search.
Je vais faire ça
Tu peux pas faire une boucle for en bash sur la liste des noeuds associés à un groupe et appeler instantiate-node dessus ?
je parlais juste du chemin
La boucle for retire tout l'interet de la chose, à savoir d'instantiate les node en un unique call nix, et profiter du caching (tu remarquera que les 14*6sec d'eval time des switch de panthéon se sont transformé en 6 sec d'eval time de tout les switch de panthéon)
l'artifice avec le fichier que je fais, c'est juste pour avoir qu'une drv pour pas avoir a bouger la suite
Ah oui, en effet
mais en vrai pour le chemin c'est couteux en lisibilité pour pas grand chose, donc je ferai pas finalement
@ -0,0 +8,4 @@...}:letbootstrap = import bootstrap-file;On a vraiment besoin de pouvoir changer le fichier de bootstrap ?
J'ai fais ça pour éviter de mettre un chemin dans le fichier, je sais plus pourquoi
117910ffe2aaf137d3c7Merci !