[00:09:30]
<Yunohost Git/Infra notifications> [Apps tools error] [List builder] Error while updating readflow: No cache yet for readflow
[01:07:19]
<Yunohost Git/Infra notifications> [Apps tools error] [List builder] Error while updating readflow: No cache yet for readflow
[02:05:47]
<Yunohost Git/Infra notifications> [Apps tools error] [List builder] Error while updating readflow: No cache yet for readflow
[03:03:06]
<Yunohost Git/Infra notifications> [Apps tools error] [List builder] Error while updating readflow: No cache yet for readflow
[04:08:29]
<Yunohost Git/Infra notifications> [Apps tools error] [List builder] Error while updating readflow: No cache yet for readflow
[05:01:09]
<Yunohost Git/Infra notifications> [Apps tools error] [List builder] Error while updating readflow: No cache yet for readflow
[06:03:34]
<Yunohost Git/Infra notifications> [Apps tools error] [List builder] Error while updating readflow: No cache yet for readflow
[07:06:11]
<Yunohost Git/Infra notifications> [Apps tools error] [List builder] Error while updating readflow: No cache yet for readflow
[08:03:34]
<Yunohost Git/Infra notifications> [Apps tools error] [List builder] Error while updating readflow: No cache yet for readflow
[09:09:23]
<Yunohost Git/Infra notifications> [Apps tools error] [List builder] Error while updating readflow: No cache yet for readflow
[10:03:52]
<Yunohost Git/Infra notifications> [Apps tools error] [List builder] Error while updating readflow: No cache yet for readflow
[10:57:18]
<Yunohost Git/Infra notifications> [issues] Thatoo [commented](https://github.com/YunoHost/issues/issues/2568#issuecomment-2830091462) on [issue #2568](https://github.com/YunoHost/issues/issues/2568) feedback when "Fetching source index from": How would you rename it then?
[11:06:44]
<Yunohost Git/Infra notifications> [Apps tools error] [List builder] Error while updating readflow: No cache yet for readflow
[12:06:31]
<Yunohost Git/Infra notifications> [Apps tools error] [List builder] Error while updating readflow: No cache yet for readflow
[13:02:56]
<Yunohost Git/Infra notifications> [Apps tools error] [List builder] Error while updating readflow: No cache yet for readflow
[14:06:26]
<Yunohost Git/Infra notifications> [Apps tools error] [List builder] Error while updating readflow: No cache yet for readflow
[15:07:58]
<Yunohost Git/Infra notifications> [Apps tools error] [List builder] Error while updating readflow: No cache yet for readflow
[15:13:52]
<Tagada> On sait pourquoi il dit ça ? :(
[15:14:30]
<Salamandar> je suis pas allé regarder
[15:14:34]
<Aleks (he/him/il/lui)> ça ressemble à un truc où il a pas réussi à git clone ou alors spa la bonne branche
[15:15:26]
<Salamandar> maybe the script supposed to update the cache doesn't run
[15:19:24]
<Yunohost Git/Infra notifications> [Apps tools error] [List builder] Error while updating readflow: No cache yet for readflow
[15:38:01]
<Thomas> J'ai mis main au lieu de master comme branche par défaut (première fois que je fais ça), ça aurait un lien ?
[15:38:54]
<Aleks (he/him/il/lui)> oui il faut préciser explicitement dans le apps.toml comme ici https://github.com/YunoHost/apps/blob/main/apps.toml#L404
[15:44:00]
<Thomas> Ok okay, vu que la branche par défaut a été remplacée dans example_ynh je pensais que c'était ok
[15:45:11]
<Thomas> Possible que quelqu'un adapte (pour éviter que je fasse une pr pour si peu), sinon je la fais ce soir
[15:45:55]
<Tagada> Thomas: pour quel repo ?
[15:46:35]
<Aleks (he/him/il/lui)> Thomas is not in the apps group https://github.com/orgs/YunoHost/teams/apps 🙀
[15:46:42]
<Aleks (he/him/il/lui)> unacceptable.jpg
[15:47:04]
<Thomas> > <@tag:lostpod.me> Thomas: pour quel repo ?
readflow
[15:47:19]
<Tagada> ah oui je sais pô lire :')
[15:49:55]
<Aleks (he/him/il/lui)> (invite sent for the apps team on the yunohost org, will also add orhtej)
[15:50:36]
<Thomas> Thanks Tag!
[15:50:40]
<Aleks (he/him/il/lui)> ah and rodinux
[15:50:52]
<Thomas> > <@Alekswag:matrix.org> (invite sent for the apps team on the yunohost org, will also add orhtej)
Oooh nice thanks Aleks!
[15:51:19]
<Aleks (he/him/il/lui)> thanks for your work bruh /o\
[15:52:27]
<Tagada> j'ai relancé le machin
[15:52:57]
<Tagada> bon y'ai toujours un `ERROR:root:[App caches] Error while updating influxdb_v2: Cmd('git') failed due to: exit code(128)` mais ça ne râle plus pour readflow
[15:57:09]
<Thomas> https://github.com/YunoHost-Apps/influxdb_v2_ynh has a master branch, not main
[15:58:15]
<Aleks (he/him/il/lui)> but it's declared as main here so it's the reversed issue : https://github.com/YunoHost/apps/blob/main/apps.toml#L2052
[16:06:48]
<Salamandar> ah hum oui je comprends la confusion
[16:06:51]
<Salamandar> we shall update the code that reads apps.toml to fetch the remote's HEAD to know which branch to pull
[16:06:52]
<Salamandar> instead of defaulting to master
[16:06:58]
<rodinux> thanks
[16:11:40]
<rodinux> Du coup les branches sont remplacées ? main <=> master ?
[16:21:42]
<Aleks (he/him/il/lui)> seulement pour certains repo genre celui du catalogue ou des outils, pas les dépôts des apps, mais un jour ce serait cool de tout passer à `main`
[16:22:29]
<Salamandar> ouaip
[16:25:29]
<Thomas> Qu'est-ce qui bloque pour le faire pour les apps ?
[16:26:15]
<Aleks (he/him/il/lui)> par exemple, le fait que quand tu deletes la branche master, ça peut genre fermer toutes les PRs qui visent de merger dans master
[16:26:41]
<Aleks (he/him/il/lui)> ou alors ptete y'a moyen de dire à github de changer la branche principale mais du coup il faut passer par l'API et boucler sur toutes les apps
[16:26:52]
<Aleks (he/him/il/lui)> et ptete on a des trucs hardcodés à certains endroits
[16:27:30]
<Thomas> Ah j'avais pas pensé, merci pour l'info
[16:30:25]
<rodinux> J'ai une question, est-l possible d'ajouter dans une app un diagnosis ignore ??
[16:35:45]
<Aleks (he/him/il/lui)> quel est ton vrai problème
[16:39:20]
<rodinux> bah, déjà il faut que je fixe autre chose... j'essaye de fixer un config_panel pour node_exporter, j'avais dans l'ideal ajouter un diagnosis ignore à la fin `yunohost diagnosis ignore --filter ports port=$port` car je met un hook iptables qui bloque l'IPv6 pour ce port et garde que l'IPv4 du serveur à écouter localement ou externe...
[16:40:52]
<rodinux> J'y ai passé beaucoup de temps, je suis content car le panel fonctionne après des tests...
[16:42:54]
<Aleks (he/him/il/lui)> ben du coup yep tu peux ajouter un diagnosis ignore
[16:43:04]
<rodinux> par contre je galère pour que l'upgrade fonctionne car j'ai ajouter une variable, il est possible de sourcer les helpers dans le fichier `_common.sh` ou c'est déconseillé...
[16:43:50]
<rodinux> J'imagine que non, ce fichier doit rester détaché ?
[16:44:15]
<Aleks (he/him/il/lui)> shaaaaare the cooode
[16:45:23]
<rodinux> https://github.com/YunoHost-Apps/node_exporter_ynh/blob/panel_conf/scripts/_common.sh
[16:46:47]
<rodinux> si j'upgrade depuis la version d'avant, il ne connaît pas ce setting
[16:48:13]
<Aleks (he/him/il/lui)> les settings sont chargés dans l'environnement des scripts donc c'est pas utile d'aller essayer de lire la valeur avec `ynh_app_setting_get`. Si t'es pas sur que le setting existe, tu peux écrire ton test avec:
```
if [[ -z "${ip_prometheus_server:-}" ]]
```
(note la syntaxe `${variable:-}` qui est la truc horrible de bash pour dire "la valeur de la variable, ou bien chaine vide si cette variable n'existe pas", ce qui permet de ne pas déclencher l'erreur de "la variable n'existe pas"
[16:50:12]
<Aleks (he/him/il/lui)> si dans des versions précédentes, cette variable n'existait pas, alors idéalement il faut la définir pendant l'upgrade
typiquement en appelant `ynh_app_setting_set_default --key=ip_prometheus_server --value=<la valeur par défaut>` au début du script d'upgrade
[16:50:45]
<Aleks (he/him/il/lui)> qui du coup définira le setting si il existe pas, ou touchera a rien si il existe déjà
[16:51:03]
<Aleks (he/him/il/lui)> (c'est la version "moderne" de ce genre de syntaxe : https://github.com/YunoHost-Apps/nextcloud_ynh/blob/master/scripts/upgrade#L28 )
[16:51:35]
<Aleks (he/him/il/lui)> et du coup si tu définit correctement le nouveau setting dans ton script d'upgrade, t'as rien de spécial à faire dans common.sh puisque le setting existera toujours apriori
[16:53:11]
<rodinux> dans mes tests j'en avais besoin, mais peut-être plus nécessaire...
[16:56:43]
<rodinux> Par contre si j'ajoute `ynh_app_setting_set_default --key=ip_prometheus_server --value="127.0.0.1"` mais que la version avait changer cette variable ? elle va être écrasée ?
[16:57:27]
<Aleks (he/him/il/lui)> non si le setting existe déjà ça change rien
[17:28:36]
<rodinux> par contre si je met un diagnosis ignore ça casse dans le CI, ou alors c'est un script post_app_install et post_app_remove pour unignore ?
[17:29:06]
<Aleks (he/him/il/lui)> heuuuu beh faut juste faire le symétrique dans le script de remove
[17:29:19]
<Aleks (he/him/il/lui)> et dans l'absolu y'a pas de raison que ça interfère avec la CI
[17:30:25]
<rodinux> il met une erreur phython car il n'a pas eu de diagnosis encore... ou bien il faut que je mette un diagnosis run avant...
[17:35:43]
<Aleks (he/him/il/lui)> zgrompf
[17:36:22]
<Aleks (he/him/il/lui)> mouai eventuellement pour pas que ca prenne 3 plombes tu peux lui dire de juste faire le diagnosique de "ip" et "ports" maybe
[17:37:12]
<rodinux> Sinon j'oublie et je met juste une explication...
[17:38:59]
<rodinux> un doc/ADMIN.md
[17:48:41]
<Yunohost Git/Infra notifications> [yunohost] yalh76 pushed 1 commit to ynh_setup_source_git: Improving git ([a9477f1e](https://github.com/YunoHost/yunohost/commit/a9477f1e6236f38ea01ce2d942e76ffe7303b44d))
[17:52:45]
<Yunohost Git/Infra notifications> [yunohost] yalh76 [commented](https://github.com/YunoHost/yunohost/pull/2085#issuecomment-2825068921) on [issue #2085](https://github.com/YunoHost/yunohost/pull/2085) WIP: Implement git in ynh_setup_source: 1. Yes, the goal is to use the commit id not the branch neither the tag 2. The change from sha256 to sha1... not sure,...
[18:10:08]
<Yunohost Git/Infra notifications> [yunohost] yalh76 [commented](https://github.com/YunoHost/yunohost/pull/2085#issuecomment-2831094254) on [issue #2085](https://github.com/YunoHost/yunohost/pull/2085) WIP: Implement git in ynh_setup_source: @alexAubin seems quite ok now Tested on https://github.com/YunoHost-Apps/peertube-search-index_ynh/pull/47
[18:49:36]
<Yunohost Git/Infra notifications> [yunohost] Josue-T pushed 1 commit to inscrease_dkim_key: Inscrease DKIM key ([bcac00dc](https://github.com/YunoHost/yunohost/commit/bcac00dcbcbed8ed85d054185f08af74ece53c5a))
[18:49:38]
<Yunohost Git/Infra notifications> [yunohost] Josue-T created new branch inscrease_dkim_key
[18:56:30]
<Yunohost Git/Infra notifications> [yunohost] Josue-T opened [pull request #2098](https://github.com/YunoHost/yunohost/pull/2098): Inscrease DKIM key
[19:08:56]
<Yunohost Git/Infra notifications> [yunohost] Josue-T [commented](https://github.com/YunoHost/yunohost/pull/2097#discussion_r2060759836) on pull request #2097 Add support for multi db support: Note that for now I did a simple solution. With the current implementation we will end with something like this: - myap...