DGNum/infrastructure
10
5
Fork 3
Code Issues 34 Pull requests 26 Projects 1 Releases Packages Wiki Activity Actions

feat(meta/ci): allows to group ci evals #630

Merged
thubrecht merged 2 commits from ci-grouping into main 2025-09-11 21:59:18 +02:00
Conversation 5 Commits 2 Files changed 6 +343 -812
lbailly commented 2025-09-11 19:11:56 +02:00
Member
Copy link
No description provided.
lbailly changed title from WIP: feat(meta/ci): allows to group ci evals to feat(meta/ci): allows to group ci evals 2025-09-11 19:29:44 +02:00
requested review from thubrecht 2025-09-11 19:30:10 +02:00
lbailly commented 2025-09-11 19:30:39 +02:00
Author
Member
Copy link

il faudrait probablement groupé d'autres truc, mais t'as l'esprit

il faudrait probablement groupé d'autres truc, mais t'as l'esprit
thubrecht reviewed 2025-09-11 19:36:18 +02:00
default.nix Outdated
@ -269,2 +269,4 @@
scripts = import ./scripts { inherit pkgs sources; };
base-colmena = pkgs.callPackage "${sources.colmena}/package.nix" { };
thubrecht commented 2025-09-11 19:36:18 +02:00
Owner
Copy link

On peut l'appeler juste colmena non ? Ça clashe pas avec le wrapper qui est pas nommé et juste dans la liste des paquets du shell

On peut l'appeler juste `colmena` non ? Ça clashe pas avec le wrapper qui est pas nommé et juste dans la liste des paquets du shell
lbailly marked this conversation as resolved
lbailly reviewed 2025-09-11 19:48:52 +02:00
workflows/eval-nodes/default.nix Outdated
@ -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)
lbailly commented 2025-09-11 19:48:52 +02:00
Author
Member
Copy link

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

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
thubrecht commented 2025-09-11 20:55:10 +02:00
Owner
Copy link

Tu peux pas faire une boucle for en bash sur la liste des noeuds associés à un groupe et appeler instantiate-node dessus ?

Tu peux pas faire une boucle for en bash sur la liste des noeuds associés à un groupe et appeler instantiate-node dessus ?
lbailly commented 2025-09-11 20:59:41 +02:00
Author
Member
Copy link

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

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
thubrecht commented 2025-09-11 21:01:07 +02:00
Owner
Copy link

Ah oui, en effet

Ah oui, en effet
lbailly commented 2025-09-11 21:01:29 +02:00
Author
Member
Copy link

mais en vrai pour le chemin c'est couteux en lisibilité pour pas grand chose, donc je ferai pas finalement

mais en vrai pour le chemin c'est couteux en lisibilité pour pas grand chose, donc je ferai pas finalement
lbailly marked this conversation as resolved
thubrecht reviewed 2025-09-11 21:05:53 +02:00
workflows/eval-nodes/drv-maker.nix Outdated
@ -0,0 +8,4 @@
...
}:
let
bootstrap = import bootstrap-file;
thubrecht commented 2025-09-11 21:05:53 +02:00
Owner
Copy link

On a vraiment besoin de pouvoir changer le fichier de bootstrap ?

On a vraiment besoin de pouvoir changer le fichier de bootstrap ?
lbailly commented 2025-09-11 21:07:59 +02:00
Author
Member
Copy link

J'ai fais ça pour éviter de mettre un chemin dans le fichier, je sais plus pourquoi

J'ai fais ça pour éviter de mettre un chemin dans le fichier, je sais plus pourquoi
lbailly marked this conversation as resolved
thubrecht approved these changes 2025-09-11 21:59:14 +02:00
thubrecht left a comment
Owner
Copy link

Merci !

Merci !
thubrecht deleted branch ci-grouping 2025-09-11 21:59:18 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
thubrecht
Labels
Clear labels   
awaiting
awaiting-author
Awaiting for the author interaction (e.g. QA/confirmation/final merge)

  
awaiting
awaiting-reviewer
Awaiting for a privileged reviewer to approve this PR

  
bot

Pull Request opened by a bot

  
deployed

This PR is currently deployed on the infra

  
Silenced alert

This is a known and silenced alert.

  
bug

Something is not working

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
help wanted

Need some help

  
invalid

Something is wrong

  
question

More information is needed

  
wontfix

This won't be fixed

No labels
awaiting
awaiting-author
awaiting
awaiting-reviewer
bot
deployed
Silenced alert
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: DGNum/infrastructure#630
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?