[09:31:12]
<Salamandar> is it normal ? is it fatal ?
[09:31:13]
<Salamandar> OK i did a sed -i to replace 7.3 by 8.1. Let's hope it's enough.
[09:36:27]
<Salamandar> Attention : update-alternatives: erreur: l'alternative /usr/bin/php7.4 pour php n'est pas enregistrée ; pas de configuration
Attention : [Error] Upgrade failed.
FFFFFFFFF
[09:43:13]
<Salamandar> yunohost should depend on php7.4…
[09:44:39]
<Salamandar> because right now yunohost is broken because I manually uninstalled php7.4 because apt reported it as unused (and nothing in /etc/php/7.4/fpm/pool.d)
[09:44:39]
<Salamandar> (like an apt dependency)
[09:45:59]
<David Sterry @weex@c4.social> Ecko reports as broken due to https://ci-apps.yunohost.org/ci/job/13266 . Is that `lsblk: /dev/sda2: not a block device` warning relevant?
[09:46:03]
<tituspijean> David Sterry @weex@c4.social: no, the relevant part is further down, with the ruby commands.
[09:46:04]
<tituspijean> But it's still pretty cryptic. The full log might help (address is at the very bottom of the report)
[09:46:10]
<David Sterry @weex@c4.social> > <@titus:pijean.ovh> But it's still pretty cryptic. The full log might help (address is at the very bottom of the report)
Ok, I'll dig in. Thanks!
[09:46:32]
<Yunohost Git/Infra notifications> Application uptime-kuma rises from level (unknown) to level 8 on https://ci-apps.yunohost.org/ci/job/13437
[09:49:21]
<Yunohost Git/Infra notifications> Application encryptor-decryptor rises from level (unknown) to level 6 on https://ci-apps.yunohost.org/ci/job/13443
[09:50:21]
<Yunohost Git/Infra notifications> Application pgadmin completely failed the continuous integration tests on https://ci-apps.yunohost.org/ci/job/13220
[09:50:21]
<Yunohost Git/Infra notifications> Application gitlist stays at level 1 on https://ci-apps.yunohost.org/ci/job/13408
[09:50:21]
<Yunohost Git/Infra notifications> Application encryptor-decryptor rises from level (unknown) to level 7 on https://ci-apps.yunohost.org/ci/job/13443
[09:51:44]
<Yunohost Git/Infra notifications> Application moncycle stays at level 2 on https://ci-apps.yunohost.org/ci/job/13294
[12:08:30]
<Salamandar> @Aleks (he/him/il/lui) maybe ^^'' I tried restarting the job, but same error again and again
[12:08:30]
<Salamandar> I'm really unlucky…
https://ci-apps-dev.yunohost.org/ci/job/5446
[12:08:37]
<Salamandar> nice thank you <3
[12:08:37]
<Salamandar> that's ok
[12:08:37]
<Salamandar> ooooh
[12:08:37]
<Salamandar> oh !
[12:08:37]
<Salamandar> ok :(
[12:08:38]
<Aleks (he/him/il/lui)> sounds like it's an appv2 but the CI is still in 11.0 so that's failing miserably :D
[12:08:38]
<Aleks (he/him/il/lui)> will trigger the CI base image rebuild
[12:08:38]
<Aleks (he/him/il/lui)> Salamandar: re-restarted it again and it's running
[12:08:38]
<Yunohost Git/Infra notifications> Application rspamdui stays at level 1 on https://ci-apps.yunohost.org/ci/job/13449
[13:27:55]
<Aleks (he/him/il/lui)> zbfgkbfl i cant rebuild the images right now because https://uk.lxd.images.canonical.com/images/debian/bullseye/amd64/default/20230202_09:15/ is empty x_x
[13:34:21]
<Aleks (he/him/il/lui)> tricked it
[15:04:44]
<Aleks (he/him/il/lui)> hmmm so it's a bit weird but next ci-apps-dev jobs should be running 11.1 i think
[15:05:30]
<Aleks (he/him/il/lui)> we shall keep a look on https://ci-apps-dev.yunohost.org/ci/job/5447
[15:37:02]
<tituspijean> > <@Alekswag:matrix.org> we shall keep a look on https://ci-apps-dev.yunohost.org/ci/job/5447
All good?
[15:37:18]
<Aleks (he/him/il/lui)> omg :3
[15:38:09]
<Aleks (he/him/il/lui)> let's see how the upgrade goes 👀
[15:38:10]
<Aleks (he/him/il/lui)> especially upgrade from v1
[15:38:10]
<tituspijean> So lean
[15:38:23]
<tituspijean> Hum... btw, it's "provisioning" with one "n" 😛
[15:38:44]
<Aleks (he/him/il/lui)> ogod
[15:38:45]
<Aleks (he/him/il/lui)> noooo
[15:38:54]
<Aleks (he/him/il/lui)> 11.1 released is ruined
[15:40:21]
<tituspijean> "Je ne comprends pas pourquoi vous vous évertuez à écrire en anglais (et mal !) alors que le projet est francophone !" 🙃
[15:49:01]
<tituspijean> 🥁
[15:49:40]
<tituspijean> oooof that's a failure
[15:56:19]
<Aleks (he/him/il/lui)> ah it didn't fail as much as i expected :D
[15:57:04]
<tituspijean> *task failed successfully*
[15:58:28]
<Aleks (he/him/il/lui)> sounds like $install_dir is not defined or something
[15:58:58]
<Salamandar> > <@titus:pijean.ovh> "Je ne comprends pas pourquoi vous vous évertuez à écrire en anglais (et mal !) alors que le projet est francophone !" 🙃
Hahahaa
[15:59:07]
<Salamandar> reminds me of some discussions about french with maniack
[15:59:40]
<Aleks (he/him/il/lui)> le débat sur est-ce qu'il faut écrire empaqueteur ou packageur en français
[16:00:17]
<Salamandar> > <@Alekswag:matrix.org> le débat sur est-ce qu'il faut écrire empaqueteur ou packageur en français
empaqueteur berk
[16:00:17]
<Aleks (he/him/il/lui)> et est-ce que weblate a le droit d'overwrite les chaines fr dans les apps si le packageur a définit une chaine fr
[16:00:31]
<Salamandar> mainteneur ?
[16:00:36]
<Aleks (he/him/il/lui)> empaqueteureuse*
[16:01:21]
<Salamandar> > <@Alekswag:matrix.org> empaqueteureuse*
ogod
[16:01:21]
<Salamandar> > <@titus:pijean.ovh> Je suis partisan du *emballeur* 😛
lol
[16:01:21]
<Aleks (he/him/il/lui)> omg x)
[16:01:21]
<Salamandar> empaqueteur·euse
[16:01:21]
<Salamandar> wait
[16:01:21]
<tituspijean> Je suis partisan du *emballeur* 😛
[16:01:52]
<Salamandar> > <@Salamandar:matrix.org> empaqueteur·euse
(we wrote our company's contracts with markdown, and we have a git branch with jinja templating for tune words for male/female/nonbinary)
[16:02:32]
<Aleks (he/him/il/lui)> :o
[16:03:01]
<tituspijean> like a `sed s/eur/eur·euse/`? 😆
[16:03:32]
<Salamandar> that allows less median dots in the document. I'm not against median dots, but _if_ we can remove them properly, that can give a better document
[16:03:34]
<Salamandar> jinja+pandoc FTW
[16:03:35]
<Salamandar> > <@titus:pijean.ovh> like a `sed s/eur/eur·euse/`? 😆
yeah but not only
[16:03:36]
<Salamandar> wayt
[16:03:36]
<Salamandar> wiat
[16:03:37]
<Salamandar> wait*
[16:04:25]
<Salamandar> https://aria.im/_matrix/media/v1/download/matrix.org/RQCnbpfszODgEpOSoZBfzEqB
[16:05:40]
<Salamandar> We don't really care if the "versionned" document is crappy to read because we generate markdown + pdf, and those really are the documents we care about
[16:05:53]
<Salamandar> Also jinja is nice
[16:05:56]
<Salamandar> https://aria.im/_matrix/media/v1/download/matrix.org/fyfxhoTNeEEmyiHIjqcHbear
[16:06:51]
<Salamandar> so a bit more than just a sed @tituspijean ;)
[16:06:51]
<Salamandar> capitalize -> first letter is capitalized
[16:11:50]
<tituspijean> The "æ" for a/e reminds me of the "@" for a/o in spanish
[16:13:56]
<Salamandar> oh they really do that ? :o
[16:14:27]
<Salamandar> > <@titus:pijean.ovh> The "æ" for a/e reminds me of the "@" for a/o in spanish
we don't have anyone non-binary around us, so we aren't even sure it's "valid"/"accepted"… we'll see when required
[16:14:40]
<tituspijean> 'twas from one of my teacher, maybe not a reliable source 😆
[16:15:05]
<tituspijean> > <@Salamandar:matrix.org> we don't have anyone non-binary around us, so we aren't even sure it's "valid"/"accepted"… we'll see when required
never seen it, but makes sense ^^
[16:15:12]
<Salamandar> ahah ^^
[16:15:41]
<Salamandar> https://ci-apps-dev.yunohost.org/ci/job/5447
OMG that's green first try
[16:17:41]
<tituspijean> well, upgrade is borken 😛
[16:17:44]
<Salamandar> yeah
[16:17:49]
<Salamandar> that's what i'm reading
[16:18:11]
<Salamandar> 21304 WARNING AssertionError: Setting file /etc/yunohost/apps/:/settings.yml does not exists ?
looks like manifestv1 has been broken in the process ? :o
[16:18:21]
<Salamandar> wait no
[16:18:31]
<Aleks (he/him/il/lui)> hmmm yeah i can't really dig it right now,, $install_dir is probably undefined because it's not set a setting ... something to do with the automagic legacy migration
[16:18:32]
<Salamandar> it's doing like it was manifestv1… although it's mv2
[16:19:36]
<Salamandar> > 21321 WARNING sed: can't read /opt/node_n/ynh_app_version: No such file or directory
wait what ?my app doesn't use node
[16:19:37]
<Salamandar> ah maybe it's just when sourcing the files
[16:20:35]
<Salamandar> Ah no my fault, it's using node alright
[16:21:43]
<Aleks (he/him/il/lui)> hm the issue with the upgrade is probably because we call https://github.com/YunoHost/yunohost/blob/dev/src/app.py#L677 (which loads the settings as bash vars) before the `AppResourceManager().apply()` line 694 which does the automagic setting migration
[16:22:44]
<Aleks (he/him/il/lui)> but we need `env_dict` for the operation_logger creation
[16:24:00]
<Aleks (he/him/il/lui)> but i'm not sure it's a good idea to dump the entire setting dict as part of the logger metadata 🤔
[16:24:05]
<Aleks (he/him/il/lui)> aaaaaaa
[16:24:08]
<Salamandar> xD
[16:55:12]
<eric_G[m]> I am trying to use `convert_app_to_packaging_v2.py` script
[16:55:13]
<eric_G[m]> I get:
```
Traceback (most recent call last):
File "/Users/eric/Documents/packaging_v2/convert_app_to_packaging_v2.py", line 340, in <module>
open(args.app_path + "/manifest.toml", "w").write(_dump_v2_manifest_as_toml(manifest))
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/eric/Documents/packaging_v2/convert_app_to_packaging_v2.py", line 147, in _dump_v2_manifest_as_toml
from tomlkit import document, nl, table, dumps, comment
ModuleNotFoundError: No module named 'tomlkit'
```
[16:57:25]
<eric_G[m]> when I `pip install tomlkit` I get `Requirement already satisfied: pip in /Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages (22.3)`
[17:08:24]
<Aleks (he/him/il/lui)> hmmm
[17:08:49]
<Aleks (he/him/il/lui)> Maybe it's `pip3 install tomlkit`
[17:08:49]
<Aleks (he/him/il/lui)> (pip3 because python3 etc)
[17:08:51]
<Aleks (he/him/il/lui)> ah nvm it's already using python3
[17:08:51]
<Aleks (he/him/il/lui)> hmmm
[17:09:12]
<Aleks (he/him/il/lui)> and are you able to run `python3 -c 'import tomlkit'` ?
[17:09:30]
<Aleks (he/him/il/lui)> or maybe you're running the script with `python` instead of `python3` ?
[17:09:43]
<Aleks (he/him/il/lui)> (and `python` maybe be binded to `python2` by default idk)
[17:17:58]
<eric_G[m]> thanks! `pip3 install tomlkit` did the job
[17:18:43]
<Aleks (he/him/il/lui)> ah \o/
[18:46:39]
<Salamandar> > Info: Creating a safety backup prior to the upgrade
> Info: The archive will contain about 13.6GB of data.
Hmmm… looks like this pre-upgrade backup contains way more data than just the app
[18:47:30]
<Salamandar> In : /home/yunohost.backup/tmp/cinny-pre-upgrade1/backup.csv
```
"/etc/ldap/ldap.conf","conf/ldap/ldap.conf"
"/home/larchuto","data/home/larchuto"
"/home/raphael","data/home/raphael"
"/home/salamandar","data/home/salamandar"
"/home/synapse","data/home/synapse"
"/home/yunohost.multimedia","data/multimedia"
"/etc/yunohost/firewall.yml","conf/ynh/firewall.yml"
"/etc/yunohost/current\_host","conf/ynh/current\_host"
"/etc/yunohost/domains","conf/ynh/domains"
"/etc/yunohost/settings.yml","conf/ynh/settings.yml"
"/etc/dkim","conf/ynh/dkim"
"/etc/yunohost/certs","conf/ynh/certs"
"/var/mail","data/mail"
"/var/lib/metronome","data/xmpp/var\_lib\_metronome"
"/var/xmpp-upload","data/xmpp/var\_xmpp-upload"
"/home/yunohost.backup/tmp/cinny-pre-upgrade1/conf/manually\_modified\_files/manually\_modified\_files\_list","conf/manually\_modified\_files/home/yunohost.backup/tmp/cinny-pre-upgrade1/conf/manually\_modified\_files/manually\_modified\_files\_list"
"/etc/ssowat/conf.json.persistent","conf/manually\_modified\_files/etc/ssowat/conf.json.persistent"
"/usr/share/yunohost/hooks/restore/05-conf\_ldap","hooks/restore/05-conf\_ldap"
"/usr/share/yunohost/hooks/restore/17-data\_home","hooks/restore/17-data\_home"
"/usr/share/yunohost/hooks/restore/18-data\_multimedia","hooks/restore/18-data\_multimedia"
"/usr/share/yunohost/hooks/restore/20-conf\_ynh\_settings","hooks/restore/20-conf\_ynh\_settings"
"/usr/share/yunohost/hooks/restore/21-conf\_ynh\_certs","hooks/restore/21-conf\_ynh\_certs"
"/usr/share/yunohost/hooks/restore/23-data\_mail","hooks/restore/23-data\_mail"
"/usr/share/yunohost/hooks/restore/27-data\_xmpp","hooks/restore/27-data\_xmpp"
"/usr/share/yunohost/hooks/restore/50-conf\_manually\_modified\_files","hooks/restore/50-conf\_manually\_modified\_files"
"/var/www/cinny","apps/cinny/backup/var/www/cinny"
"/etc/nginx/conf.d/cinny.salamandar.fr.d/cinny.conf","apps/cinny/backup/etc/nginx/conf.d/cinny.salamandar.fr.d/cinny.conf"
"/home/yunohost.backup/tmp/cinny-pre-upgrade1/backup.csv","backup.csv"
"/home/yunohost.backup/tmp/cinny-pre-upgrade1/info.json","info.json"
"/home/yunohost.backup/tmp/cinny-pre-upgrade1/apps/cinny","apps/cinny"
"/home/yunohost.backup/tmp/cinny-pre-upgrade1/conf","conf"
"/home/yunohost.backup/tmp/cinny-pre-upgrade1/data","data"
```
[18:48:06]
<Salamandar> https://aria.im/_matrix/media/v1/download/matrix.org/iOhRjgFcBLAglhazjzRUkxtk
[18:48:07]
<Salamandar> manifestv1 to manifestv2 might not be fully operational yet ?
[18:48:35]
<Salamandar> (do you want me to open a ticket ?)
[18:50:09]
<Aleks (he/him/il/lui)> yup 👍️
[18:51:02]
<Aleks (he/him/il/lui)> maybe we need to add some `ignore_system` here https://github.com/YunoHost/yunohost/blob/dev/src/app.py#L650 idk
[19:10:55]
<Salamandar> IMHO `system` should be disabled by default. I would find it logic to do "positive" selections of everything you want in your backup. But that's my opinion ;)
[19:16:39]
<Aleks (he/him/il/lui)> it's disabled by default when you specify an app on the command line
[19:16:44]
<Aleks (he/him/il/lui)> but calling the function is different
[19:16:55]
<Aleks (he/him/il/lui)> something to do with argparse injecting None or []
[19:17:07]
<Aleks (he/him/il/lui)> (compared to the default value of the python arg)
[19:31:46]
<yalh76> @room Meeting o clock => https://pad.lqdn.fr/p/yunohost-02-02-2023
[19:39:50]
<Salamandar> > <@Alekswag:matrix.org> but calling the function is different
ah :D
[19:40:01]
<Salamandar> > <@Alekswag:matrix.org> (compared to the default value of the python arg)
i see thanks :)
[19:57:13]
<Aleks (he/him/il/lui)> > <@Salamandar:matrix.org> > Info: Creating a safety backup prior to the upgrade
> > Info: The archive will contain about 13.6GB of data.
>
> Hmmm… looks like this pre-upgrade backup contains way more data than just the app
actually can you elaborate the context
[19:57:22]
<Aleks (he/him/il/lui)> like i thought this was from Cinny's test ?
[19:57:30]
<Salamandar> nope, on my instance
[19:57:38]
<Salamandar> upgrading from manifestv1 to manifestv2
[19:57:44]
<Aleks (he/him/il/lui)> hmokay o_O
[20:16:14]
<Yunohost Git/Infra notifications> [example_ynh/master] Propagate changes related to manifestv2 - Alexandre Aubin
[20:16:14]
<Yunohost Git/Infra notifications> [example_ynh] @alexAubin pushed 2 commits to master ([7a077746cfc6...749b78c79f26](https://github.com/YunoHost/example_ynh/compare/7a077746cfc6...749b78c79f26))
[20:16:14]
<Yunohost Git/Infra notifications> [example_ynh] @alexAubin deleted branch manifestv2
[20:16:14]
<Yunohost Git/Infra notifications> [example_ynh] @alexAubin merged [pull request #179](https://github.com/YunoHost/example_ynh/pull/179): WIP: Manifest/packaging v2
[20:16:16]
<Yunohost Git/Infra notifications> [example_ynh/master] Merge pull request #179 from YunoHost/manifestv2 WIP: Manifest/packaging v2 - Alexandre Aubin
[20:16:27]
<Yunohost Git/Infra notifications> [example_ynh] @alexAubin pushed 2 commits to master ([749b78c79f26...f93c42019dfc](https://github.com/YunoHost/example_ynh/compare/749b78c79f26...f93c42019dfc))
[20:16:28]
<Yunohost Git/Infra notifications> [example_ynh] @alexAubin merged [pull request #199](https://github.com/YunoHost/example_ynh/pull/199): Replace check_process with tests.toml
[20:16:30]
<Yunohost Git/Infra notifications> [example_ynh] @alexAubin deleted branch tests_toml
[20:16:31]
<Yunohost Git/Infra notifications> [example_ynh/master] Replace check_process with tests.toml - Alexandre Aubin
[20:16:34]
<Yunohost Git/Infra notifications> [example_ynh/master] Merge pull request #199 from YunoHost/tests_toml Replace check_process with tests.toml - Alexandre Aubin
[21:15:01]
<Yunohost Git/Infra notifications> [apps] @Salamandar opened [pull request #1627](https://github.com/YunoHost/apps/pull/1627): Add initial json schemas for validation of TOML app files (manifest.toml, tests.toml)
[21:18:08]
<Yunohost Git/Infra notifications> [apps] @Salamandar edited [pull request #1627](https://github.com/YunoHost/apps/pull/1627): Add initial json schemas for validation of TOML app files (manifest.toml, tests.toml)
[21:18:45]
<Yunohost Git/Infra notifications> [apps] @Salamandar edited [pull request #1627](https://github.com/YunoHost/apps/pull/1627): Add initial json schemas for validation of TOML app files (manifest.toml, tests.toml)
[21:26:30]
<tituspijean> Mmmh I have an issue upgrading Yunorunner on v11.1: https://paste.yunohost.org/raw/iroyoraxex
`fatal: detected dubious ownership in repository at '/var/www/yunorunner'` (owner is `yunorunner`)
[21:29:08]
<Aleks (he/him/il/lui)> hmpf yeah that shit is gonna cause a shitload of issues
[21:29:46]
<Aleks (he/him/il/lui)> some recent release of git which ended up in Debian Stable (how?) now has this behavior where it checks for ownership / unix perms
[21:29:48]
<Aleks (he/him/il/lui)> and will refuse to do stuff if it aint right
[21:32:22]
<tituspijean> I ran the suggested command (`git config --global --add safe.directory /var/www/yunorunner`), to no avail 😕
[21:32:56]
<Aleks (he/him/il/lui)> but i already saw a similar thing on the forum today
[21:32:57]
<Aleks (he/him/il/lui)> like some sort of npm/composer/?? triggering some package git clone and exploding also because of this
[21:33:00]
<tituspijean> (mmmh, as `root`, so I guess it should be ran as `$app`?)
[21:33:11]
<Aleks (he/him/il/lui)> which is probably okay but like ... why does such a "major" (in terms of UX etc) ended up in regular debian upgrades x_x
[21:33:12]
<Aleks (he/him/il/lui)> "thanksfully" we don't have that many app based on git
[21:33:22]
<Aleks (he/him/il/lui)> (zbgrlb got network disconnect)
[21:38:10]
<Aleks (he/him/il/lui)> > <@titus:pijean.ovh> (mmmh, as `root`, so I guess it should be ran as `$app`?)
yeah idk it's not clear to me what git wants exactly :/
[21:38:10]
<tituspijean> (╯°□°)╯︵ ┻━┻ ` fatal: $HOME not set`
[21:38:10]
<Aleks (he/him/il/lui)> omg xD
[21:38:10]
<tituspijean> Currently testing this: https://github.com/YunoHost-Apps/yunorunner_ynh/tree/fix-git
[21:38:15]
<Aleks (he/him/il/lui)> what about git clone as yunorunner maybe ?
[21:41:07]
<tituspijean> ah. the commands are run as `root`, I guess that's the issue.
[21:42:36]
<tituspijean> P#TAIN DE M. `error: could not lock config file /home/yunorunner/.gitconfig: Permission denied`
[21:42:37]
<tituspijean> sorry.
[21:42:47]
<Aleks (he/him/il/lui)> 😬
[21:46:58]
<tituspijean> Fixed
[21:47:59]
<tituspijean> so the issue is indeed due to the fact that the git commands were run as `root` in a directory owned by `$app`.
Solved by using the `ynh_exec_as` helper. No need to add global git config.
[21:57:40]
<tituspijean> Mmmh testing branch is 21 commits behing master... can I recreate it?
[22:00:00]
<Aleks (he/him/il/lui)> sure
[22:06:37]
<Aleks (he/him/il/lui)> 😅
[22:06:37]
<tituspijean> ```
Warning: Warning: Permanently added 'github.com,140.82.121.4' (ECDSA) to the list of known hosts.
Warning: git@github.com: Permission denied (publickey).
Warning: fatal: Could not read from remote repository.
Warning: Please make sure you have the correct access rights
Warning: and the repository exists.
```
[22:06:37]
<tituspijean> aaaaand now there's an issue if the remotes are set with ssh addresses instead of plain http
[22:06:37]
<tituspijean> Rage quitting for tonight 🙂
[22:48:40]
<eric_G[m]> https://ci-apps-dev.yunohost.org/ci/job/5454
[22:49:31]
<eric_G[m]> using packaging_format V2...
[22:51:12]
<eric_G[m]> I have this error in the Bookstack CI test `WARNING Variable $finalpath wasn't initialized when trying to replace __FINALPATH__ in /etc/php/8.1/fpm/pool.d`
[22:53:06]
<Aleks (he/him/il/lui)> ugh i though i fixed that one x_x
[22:53:07]
<Aleks (he/him/il/lui)> lemme check
[22:57:39]
<Aleks (he/him/il/lui)> hmyeah fix coming up in next version
[23:06:58]
<Aleks (he/him/il/lui)> rebuilding CI base image so we get the various fixes in ci-apps-dev