[11:03:54]
<Guillaume Bouzige> salut,
j'essaie de suivre cette doc `https://github.com/YunoHost/ynh-dev` pour avoir un env de dev. j'ai bon si j'ai un yunohost installer dans un VM (ici libvirt/qemu) et que j'ai installe le script de deploiement ynh-dev ? ou il me faut absolument LXC/D dans ma VM ?
[11:33:30]
<kayou> ynh-dev est fortement lié à lxd
[11:33:55]
<kayou> mais un qemu peut suffire en fonction de ce que tu cherches à faire
[11:34:41]
<kayou> si c'est simplement pour installer des apps à tester, je pense que c'est ok, si tu veux modifier le code du core, ca va devenir plus complexe
[12:01:32]
<Aleks (he/him/il/lui)> Ou sinon ynh-dev peut aussi marcher en mode "remote developemnt path" où tu ajoute juste le script ynh-dev sur un VPS ou VM existante (mais dédié au dev, pas de la prod) et ça te fais les symlink entre lea git clone et le "vrai" code
[12:22:11]
<Guillaume Bouzige> c'est pour modifie le core, je pensais betement avoir juste besoin d'une machine virtuel de dev...et comme j'ai deja libvirt/qemu qui fonctionne la
[12:26:57]
<Aleks (he/him/il/lui)> oui tu peux aussi le faire avec juste une VM, il faut installer YunoHost dessus, et ensuite ynh-dev
[12:27:59]
<Aleks (he/him/il/lui)> https://github.com/YunoHost/ynh-dev#remote-development-environment (le texte parle de VPS mais une VM c'est juste un VPS en local ;))
[12:28:50]
<Guillaume Bouzige> voila du coup jai pas besoin de lxd...
[12:29:01]
<kayou> ok, je savais pas :)
[12:29:32]
<Guillaume Bouzige> jai pas besoin de CI perso aussi ?
[12:29:41]
<Aleks (he/him/il/lui)> hmm non pas spécialement
[12:30:20]
<Aleks (he/him/il/lui)> si tu veux vraiment faire tourner les tests en local tu pourra lancer `./ynh-dev test yunohost` mais bon c'est un peu long, ça dépends de si ta modif est vraiment complexe / impactante
[12:30:46]
<Aleks (he/him/il/lui)> moi la plupart du temps pour des "petites" modif je commit sur la branche et je laisse la CI Gitlab faire son taf
[12:30:56]
<Guillaume Bouzige> je veut test restic/remote backup....c'est impactant hein ?
[12:30:57]
<kayou> et faut savoir que les tests peuvent aussi laisser des traces après
[12:31:03]
<Aleks (he/him/il/lui)> et pour les vrais gros morceaux, c'est important d'être + precotioneux mais bon
[12:31:14]
<Aleks (he/him/il/lui)> ah
[12:31:22]
<Aleks (he/him/il/lui)> 😬
[12:31:32]
<Aleks (he/him/il/lui)> ouaip, je connais pas trop la portée de cette PR
[12:31:32]
<Guillaume Bouzige> > <@kayou:matrix.org> et faut savoir que les tests peuvent aussi laisser des traces après
avec des snapshots je peut rewind
[12:31:49]
<Guillaume Bouzige> non je vais repartir de zero justement
[12:31:54]
<Aleks (he/him/il/lui)> ah ok
[12:32:14]
<Guillaume Bouzige> enfin tenté...
[12:32:21]
<Aleks (he/him/il/lui)> bah anyway déjà avoir un truc où quand tu testes "à la main" ça marche c'est une première bonne étape
[12:32:29]
<Guillaume Bouzige> 💪🦾
[12:32:44]
<Aleks (he/him/il/lui)> parce que ensuite, faire des tests unitaires de remote backup c'est genre heuuuuuu
[12:33:03]
<Aleks (he/him/il/lui)> enfin tu mdiras ptete tu peux setup un storage chépakoi et pointer sur 127.0.0.1
[12:33:25]
<Aleks (he/him/il/lui)> mais bon c'est déjà un sacré "gros" test, plutot fonctionnel que unitaire
[12:33:29]
<Guillaume Bouzige> ca doit se faire je pense bien
[12:34:19]
<Guillaume Bouzige> to be continued...
[12:34:31]
<Aleks (he/him/il/lui)> 👍️
[12:36:08]
<Aleks (he/him/il/lui)> en tout cas hésite pas à poser des questions et remettre en cause le code, des fois y'a pleins de trucs designé de manière arbitraire sans vraie raison que ce soit fait comme ça plutot qu'autrement ... enfin des fois y'a aussi des vrais bonnes raisons implicites qu'un truc soit fait d'une certaine manière 😅
[12:36:33]
<Guillaume Bouzige> entendu, je n'hesiterai pas
[12:37:43]
<Guillaume Bouzige> merci !
[20:12:53]
<Yunohost Git/Infra notifications> [yunohost] @alexAubin pushed 1 commit to api-for-current-action: POC to add a route to fetch whats the current running action, cf plans to improve the webadmin UX ([383c524c](https://github.com/YunoHost/yunohost/commit/383c524c78b55fce70dee81610b1dbdf9e2ec7c1))
[20:12:54]
<Yunohost Git/Infra notifications> [yunohost] @alexAubin created new branch api-for-current-action
[20:15:23]
<Yunohost Git/Infra notifications> [yunohost] @alexAubin opened [pull request #1628](https://github.com/YunoHost/yunohost/pull/1628): POC to add a route to fetch whats the current running action, cf plans to improve the webadmin UX
[20:17:18]
<Yunohost Git/Infra notifications> [yunohost] @alexAubin pushed 1 commit to api-for-current-action: POC to add a route to fetch whats the current running action, cf plans to improve the webadmin UX ([7018d258](https://github.com/YunoHost/yunohost/commit/7018d258a7c4f7cb181975f49714db6ec2f5c4a1))
[20:20:01]
<Yunohost Git/Infra notifications> [yunohost] ✖️ Pipeline [#807816749](https://gitlab.com/yunohost/yunohost/-/pipelines/807816749) canceled on branch
[20:22:14]
<Yunohost Git/Infra notifications> [yunohost] @github-code-scanning[bot] [commented](https://github.com/YunoHost/yunohost/pull/1628#discussion_r1137732496) on pull request #1628 POC to add a route to fetch whats the current running action, cf plans to improve the webadmin UX: ## Except block handles BaseException
Except block directly handles BaseException.
[Show more details](https://githu...
[20:22:14]
<Yunohost Git/Infra notifications> [yunohost] @github-code-scanning[bot] [commented](https://github.com/YunoHost/yunohost/pull/1628#discussion_r1137732502) on pull request #1628 POC to add a route to fetch whats the current running action, cf plans to improve the webadmin UX: ## Unused import
Import of write_to_yaml is not used.
[Show more details](https://github.com/YunoHost/yunohost/secur...