feat(scripts): Add a script to easily launch a vm with port forwarding #7

Merged
thubrecht merged 2 commits from launch_vm_script into main 2023-10-27 14:36:48 +02:00
Member
No description provided.
Author
Member

Bon l'UI c'est pas encore ça mais j'ai pas le courage d'y retoucher ce soir.

Sentez vous libre de pousser des modifications sur cette branche si vous avez des améliorations à proposer

Bon l'UI c'est pas encore ça mais j'ai pas le courage d'y retoucher ce soir. Sentez vous libre de pousser des modifications sur cette branche si vous avez des améliorations à proposer
mdebray force-pushed launch_vm_script from 2dc5831249 to 2b0e80b207 2023-10-03 11:21:19 +02:00 Compare
mdebray force-pushed launch_vm_script from 2b0e80b207 to 4d5d81393f 2023-10-05 16:27:43 +02:00 Compare
thubrecht reviewed 2023-10-05 16:44:25 +02:00
@ -0,0 +23,4 @@
if [ -z "$MACHINE" ]; then echo "-o option needed"; exit 1; fi
PATH=$(nix-store -r "$(@colmena@/bin/colmena eval -E "{nodes, ...}: nodes.$MACHINE.config.system.build.vm.drvPath" | jq -r)")
Owner

Cette ligne plante si on n'a pas jq d'installé, il faut remplacer par @jq@/bin/jq et rajouter jq dans les substitutions de scripts/default.ni

Cette ligne plante si on n'a pas `jq` d'installé, il faut remplacer par `@jq@/bin/jq` et rajouter `jq` dans les substitutions de `scripts/default.ni`
mdebray marked this conversation as resolved
mdebray force-pushed launch_vm_script from 4d5d81393f to e24d5ea4fb 2023-10-15 17:59:02 +02:00 Compare
Author
Member
@thubrecht
mdebray force-pushed launch_vm_script from e24d5ea4fb to 91b525fa0b 2023-10-20 19:40:23 +02:00 Compare
mdebray force-pushed launch_vm_script from 1a85309a60 to a587302e14 2023-10-22 19:40:34 +02:00 Compare
thubrecht requested changes 2023-10-22 19:48:31 +02:00
@ -0,0 +1,5 @@
{
virtualisation.vmVariant = {
Owner

Pourquoi c'est seulement pour web-01 ? On risque pas d'avoir le même problème si on fait une vm pour storage01 ou compute01 ?
Si c'est valable pour toutes les machines je mettrais bien ça dans hive.nix à la place

Pourquoi c'est seulement pour web-01 ? On risque pas d'avoir le même problème si on fait une vm pour storage01 ou compute01 ? Si c'est valable pour toutes les machines je mettrais bien ça dans `hive.nix` à la place
Author
Member

Oui après ma logique était que ce fichier contenait des sortes de bugfixs spécifiques à certains services mais effectivement il y en a qui sont commun à tous

Oui après ma logique était que ce fichier contenait des sortes de bugfixs spécifiques à certains services mais effectivement il y en a qui sont commun à tous
Author
Member

Bon j'ai créé un module et si on a des trucs spécifiques on pourra filtrer dans le module le nom de la machine je pense.

Bon j'ai créé un module et si on a des trucs spécifiques on pourra filtrer dans le module le nom de la machine je pense.
mdebray marked this conversation as resolved
@ -0,0 +31,4 @@
DRV_PATH=$(@colmena@/bin/colmena eval -E "{nodes, ...}: nodes.$MACHINE.config.system.build.vm.drvPath" | @jq@/bin/jq -r)
echo "Realising $DRV_PATH"
PATH=$(nix-store -r "$DRV_PATH")
Owner

Je sais pas si c'est une bonne idée de modifier la variable PATH, dans le doute je mettrais plutôt RESULT

Je sais pas si c'est une bonne idée de modifier la variable `PATH`, dans le doute je mettrais plutôt `RESULT`
Author
Member

J'ai pas fait gaffe mais oui

J'ai pas fait gaffe mais oui
mdebray marked this conversation as resolved
mdebray force-pushed launch_vm_script from a587302e14 to c5a8660c47 2023-10-23 11:23:25 +02:00 Compare
mdebray force-pushed launch_vm_script from c5a8660c47 to 644a03b3cd 2023-10-25 14:36:08 +02:00 Compare
mdebray force-pushed launch_vm_script from 644a03b3cd to 1287ab9e45 2023-10-25 15:00:27 +02:00 Compare
thubrecht force-pushed launch_vm_script from 1287ab9e45 to c30d8961b9 2023-10-27 14:36:20 +02:00 Compare
thubrecht merged commit c30d8961b9 into main 2023-10-27 14:36:48 +02:00
thubrecht deleted branch launch_vm_script 2023-10-27 14:36:48 +02:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
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#7
No description provided.