[01:35:12]
<Yunohost Git/Infra notifications> App prometheus goes down from level 8 to 6 in job [#22033](https://ci-apps.yunohost.org/ci/job/22033)
[02:45:57]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#issuecomment-1884093487) on [issue #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622) Testing / switch to packaging V2 + proper mail config: (all tests passing now - except for linter warnings ofc)
[03:02:49]
<Yunohost Git/Infra notifications> App sonarr failed all tests in job [#22037](https://ci-apps.yunohost.org/ci/job/22037) :(
[06:53:01]
<kayou> > <@yunohostinfra:matrix.org> [nextcloud_ynh] @alexAubin [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#issuecomment-1884093487) on [issue #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622) Testing / switch to packaging V2 + proper mail config: (all tests passing now - except for linter warnings ofc)
and now what? should we ask some testing on the forum? or should we press the button?
[07:31:19]
<Yunohost Git/Infra notifications> App civicrm_drupal7 failed all tests in job [#22055](https://ci-apps.yunohost.org/ci/job/22055) :(
[07:33:59]
<Yunohost Git/Infra notifications> App cjdns failed all tests in job [#22056](https://ci-apps.yunohost.org/ci/job/22056) :(
[07:47:24]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 3 commits to v2_PostgreSQL ([1ebbf754078f...63bd60d6f318](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/1ebbf754078f...63bd60d6f318))
[07:48:00]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 10 commits to v2_PostgreSQL ([63bd60d6f318...f1354451d1c3](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/63bd60d6f318...f1354451d1c3))
[07:48:03]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597#issuecomment-1884342872) on [issue #597](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597) V2 postgresql: testme
[07:49:01]
<lapineige> I can test it on 3 servers
[07:49:36]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to oldstable: Update manifest.json ([28bab2ff](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/28bab2ff724c4cf8e6cafa7cb48a90ecbad5e968))
[07:54:11]
<eric_G> > <@kayou:matrix.org> and now what? should we ask some testing on the forum? or should we press the button?
Asking for testing in the forum seems a good idea even though user seems to prefer testing the master branch... 😶🌫️
[07:56:48]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @Limezy [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597#issuecomment-1884352487) on [issue #597](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597) V2 postgresql: The CI launching bot seems down
[08:15:58]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @lapineige [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#issuecomment-1884375655) on [issue #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622) Testing / switch to packaging V2 + proper mail config: Well it even fixed my buggy server than gave a 502 out of nowhere before the upgrade :smiley:
[08:22:42]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @lapineige [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#issuecomment-1884384104) on [issue #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622) Testing / switch to packaging V2 + proper mail config: Second server updated without any issue. And the third one. All good
[08:23:14]
<lapineige> 3 servers upgraded without any issue. I'm fine pressing the green button when you're ok with it 🙂
[08:51:19]
<Aleks (he/him/il/lui)> \o/
[08:51:22]
<Aleks (he/him/il/lui)> i'm good with it
[08:52:15]
<eric_G> > 3 servers upgraded without any issue. I'm fine pressing the green button when you're ok with it 🙂
I am pressing the green button for you to press the green button 😬
[08:58:45]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin approved [pull request #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#pullrequestreview-1812708014) Testing / switch to packaging V2 + proper mail config
[09:02:03]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#issuecomment-1884435925) on [issue #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622) Testing / switch to packaging V2 + proper mail config: Sure P Ideally wait for at least one upgrade test to complete but yeah [we-are-go-for-launch-apollo13](https://github....
[09:02:04]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#issuecomment-1884435925) on [issue #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622) Testing / switch to packaging V2 + proper mail config: [we-are-go-for-launch-apollo13](https://github.com/YunoHost-Apps/nextcloud_ynh/assets/4533074/2bed4aee-3325-48d5-a081-a...
[09:12:39]
<eric_G> I made a call for testing in the forum https://forum.yunohost.org/t/nextcloud-a-cloud-for-personal-or-group-use/9223/547
[09:33:49]
<Yunohost Git/Infra notifications> App discourse failed all tests in job [#22064](https://ci-apps.yunohost.org/ci/job/22064) :(
[09:33:55]
<Aleks (he/him/il/lui)> annnnnnnd ci-apps-dev is completely broken and no idea why it's exploding since yesterday for some reason
[09:37:06]
<hook> Hmmm, does anyone else have MAM issues with Metronome?
[09:38:36]
<eric_G> hook: wrong channel?
[09:39:00]
<hook> Wrong channel. Sorry :]
[09:55:05]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597#issuecomment-1884524682) on [issue #597](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597) V2 postgresql: :sunflower:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/12562/badge)]...
[09:55:10]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597#issuecomment-1884524445) on [issue #597](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597) V2 postgresql: testme
[11:28:17]
<lapineige> > <@ericg:matrix.org> I made a call for testing in the forum https://forum.yunohost.org/t/nextcloud-a-cloud-for-personal-or-group-use/9223/547
Ok let's way for let say 3 tests :)
[11:31:56]
<Salamandar> > <@ericg:matrix.org> I made a call for testing in the forum https://forum.yunohost.org/t/nextcloud-a-cloud-for-personal-or-group-use/9223/547
> Info: [####++++++++++......] > Reconfiguring Nextcloud...
Warning: File /var/www/nextcloud/config/config.php has been manually modified since the installation or last upgrade. So it has been duplicated in /var/cache/yunohost/appconfbackup//var/www/nextcloud/config/config.php.backup.20240110.123049
Not sure if it's relevant or if it's a missing ynh_file_checksum_delete ?
[11:35:23]
<Salamandar> OMG NEXTCLOUD 28 IS SO MUCH FASTER
[11:36:13]
<Salamandar> what happened ?
[11:36:58]
<Salamandar> oh is it the move to postgresql ?
[11:36:59]
<Aleks (he/him/il/lui)> hmpf yes and no, the thing is that this file is likely to get modified by nextcloud itself, so it can't really be tracked wether or not the config.php was manually modified, it's not that relevant
[11:39:27]
<Aleks (he/him/il/lui)> it's not using postgresql yet
[11:40:43]
<Salamandar> ah ok
[11:41:12]
<Salamandar> everything is faster, even the image viewer (so not really javascript / php, just file downloading)
[11:41:13]
<Salamandar> haha
[11:41:33]
<eric_G> That must be the transition to package v2 😅
[11:43:55]
<Salamandar> btw @ericg:matrix.org , i see you opened a lot of "stale" merge requests for manifestv2. We could coordinate to merge them together if you want :D
[11:46:54]
<Mateusz Szymański> > <@Salamandar:matrix.org> everything is faster, even the image viewer (so not really javascript / php, just file downloading)
really? it struggled to load galleries in 27.x on my potato VPS. Will test later then
[11:47:00]
<Salamandar> it would be nice to just remove manifestv1 altogether :D
[11:47:26]
<Salamandar> > <@orhtej2:matrix.org> really? it struggled to load galleries in 27.x on my potato VPS. Will test later then
mine is not really a potato, it's a helios64, but only 4GB ram…
[11:47:27]
<Salamandar> nice that org-wide search
[11:47:27]
<Salamandar> 157
[11:47:28]
<lapineige> > <@Salamandar:matrix.org> OMG NEXTCLOUD 28 IS SO MUCH FASTER
Full rewrite of the files view frontend
[11:47:28]
<lapineige> > <@Alekswag:matrix.org> 515 PR opened on all yunohost-apps 😐️
Could be bad and good :p
[11:47:30]
<Aleks (he/him/il/lui)> what do you mean you can't see them ? 😬
[11:47:30]
<Aleks (he/him/il/lui)> 154* ?
[11:47:30]
<Aleks (he/him/il/lui)> 515 PR opened on all yunohost-apps 😐️
[11:47:30]
<Aleks (he/him/il/lui)> Salamandar: i try to give it a go from time to time but it's hard because the remaining v2 PR are of course the one which are not straightforward to get into working state, and then you start opening up the code and further refactoring half the package ... but I usually look at https://github.com/pulls?q=is%3Aopen+is%3Apr+org%3AYunoHost-apps+archived%3Afalse+sort%3Aupdated-desc+ , possibly add "2" in the search bar
[11:47:32]
<eric_G> I didn't count but doesn't seem to be 154 to me
[11:47:32]
<eric_G> I am using https://dash.yunohost.org/appci/branch/stable but can't see 157 apps in v1
[11:47:32]
<eric_G> How can I grab a list of the 157 apps still in v2?
[11:47:32]
<eric_G> > <@Salamandar:matrix.org> it would be nice to just remove manifestv1 altogether :D
almost there...
[11:49:07]
<hook> > it's not using postgresql yet
Will the move to Postgres be automatic on Nextcloud app upgrade (when that will be due)?
[11:49:50]
<eric_G> ok... manually counting 145 apps 😶🌫️
[11:49:50]
<Aleks (he/him/il/lui)> 424 PR if we remove yunohost-bot : https://github.com/pulls?q=is%3Aopen+is%3Apr+org%3AYunoHost-apps+archived%3Afalse+sort%3Aupdated-desc+-author%3Ayunohost-bot
[11:50:43]
<Salamandar> BTW i got some weird warnings from package_checker on unattended_upgrades manifestv2, @Alekswag:matrix.org can you check it out ?
[11:50:43]
<Salamandar> https://dash.yunohost.org/appci/branch/stable says 154 indeed
[11:50:43]
<Salamandar> https://ci-apps-dev.yunohost.org/ci/job/12548
[11:50:46]
<Salamandar> oh
[11:50:46]
<Aleks (he/him/il/lui)> that's because the config script is probably broken
[11:50:46]
<Salamandar> 119 WARNING Unknown key 'name' found in config panel
[11:50:46]
<Salamandar> ok
[11:50:46]
<Salamandar> 213 WARNING AssertionError: Setting file /etc/yunohost/apps//settings.yml does not exists ?
[11:51:08]
<Aleks (he/him/il/lui)> the `scripts/config` looks like complete legacy trash and should be rewritten from scratch
[11:52:36]
<Aleks (he/him/il/lui)> `main.global_config.email_type` zzzzzz
[12:31:59]
<lapineige> > <@Salamandar:matrix.org> everything is faster, even the image viewer (so not really javascript / php, just file downloading)
Some back-end optimisations were done on that side
[12:33:33]
<lapineige> > <@Salamandar:matrix.org> 119 WARNING Unknown key 'name' found in config panel
Same in archivist... Strange
[12:34:03]
<lapineige> > <hook> > it's not using postgresql yet
> Will the move to Postgres be automatic on Nextcloud app upgrade (when that will be due)?
Yes. But we're not here yet. It's testable I believe. eric_G: ?
[12:35:30]
<Aleks (he/him/il/lui)> > Same in archivist... Strange
yes, same thing, `scripts/config` last change is 4 years ago, hasnt been moved to the new config panel format (despite `version = "1.0"` in config_panel.toml but that's not enough), should just be trashed and rewritten from scratch ...
[12:35:44]
<Aleks (he/him/il/lui)> also same `main.global_config.email_type` bullshit ~_~
[13:01:08]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to v2_PostgreSQL: add psql pwd ([98d24a99](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/98d24a9930c2b9297b72ffde8587afc14c630f9b))
[13:08:50]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to v2_PostgreSQL: cleaning ([92ca8877](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/92ca88779ab8c102603090c1cce18f9cff77bf98))
[13:14:00]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar edited review [pull request #597](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597#pullrequestreview-1813197290): V2 postgresql
[13:14:08]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597#discussion_r1447368554) on pull request #597 V2 postgresql: Do we need this line?
[13:14:23]
<hook> >> <hook> > it's not using postgresql yet
>> Will the move to Postgres be automatic on Nextcloud app upgrade (when that will be due)?
>
> Yes. But we're not here yet. It's testable I believe. eric_G: ?
Sweet!
I ran Nextcloud on Postgres on and Olimex Lime 2 for years, so I am looking forward to it. But I'll pass on testing, I had my fill of ownCloud/Nextcloud shenanigans on those boards.
[13:15:53]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597#issuecomment-1884524682) on [issue #597](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597) V2 postgresql: :sunflower:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/12562/badge)]...
[13:15:54]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597#issuecomment-1884524445) on [issue #597](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597) V2 postgresql: testme
[13:18:20]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @Salamandar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#issuecomment-1884834400) on [issue #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622) Testing / switch to packaging V2 + proper mail config: Excellent, working on my server
[13:28:55]
<eric_G> rodinux: logs path for Nextcloud: `/home/yunohost.app/nextcloud/data/nextcloud.log`
[13:34:31]
<kayou> `chown -R nextcloud: /home/yunohost.app/nextcloud` is suuuuper long on my server
[13:34:31]
<kayou> There is no markdown anymore on matrix? O.o
[13:36:50]
<eric_G> do you have acces to the Nextcloud log page? -> `domain.tld/settings/admin/logging`
[13:36:57]
<eric_G> I get an empty page
[13:41:00]
<kayou> I'm still waiting for the upgrade to finish
[13:42:23]
<eric_G> https://aria.im/_matrix/media/v1/download/matrix.org/NnCGxtPxQqhgvVwVukwfooBd
[13:48:26]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to v2_PostgreSQL: Update config.json ([cb57f51e](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/cb57f51e104d932143585dda6e032297f2e26286))
[14:11:52]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @rodinux [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#issuecomment-1884919826) on [issue #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622) Testing / switch to packaging V2 + proper mail config: Hello, for me something wrong after the upgrade, no logs and a blank page for https://domain.tld/nextcloud/settings/adm...
[14:28:04]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @rodinux [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#issuecomment-1884929350) on [issue #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622) Testing / switch to packaging V2 + proper mail config: I have try after reading this issue here https://github.com/nextcloud/logreader/issues/1073 sudo -u nextcloud php8.2...
[14:28:04]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to v2_PostgreSQL: Update upgrade ([5c1f0576](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/5c1f0576372257f403d7f4ed463da5f440691051))
[14:28:06]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @kay0u [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/637#discussion_r1447453291) on pull request #637 [enh] Faster permissions setter on big datadir: On my server, the main issue is this line. Idk how to speed up this chown though
[14:28:06]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @kay0u edited review [pull request #637](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/637#pullrequestreview-1813343036): [enh] Faster permissions setter on big datadir
[14:28:08]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @rodinux [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#issuecomment-1884924073) on [issue #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622) Testing / switch to packaging V2 + proper mail config: the page blank [Capture d’écran du 2024-01-10 15-10-22](https://github.com/YunoHost-Apps/nextcloud_ynh/assets/11272759...
[14:32:08]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @rodinux [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#issuecomment-1884958460) on [issue #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622) Testing / switch to packaging V2 + proper mail config: Now I not sure if I remove the upgrade to restore the backup, or I keep it for now to resolve with you what should be wr...
[14:33:19]
<kayou> > <@ericg:matrix.org> I get an empty page
Same
[14:33:57]
<kayou> Probably an nginx conf
[14:48:18]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#issuecomment-1884990482) on [issue #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622) Testing / switch to packaging V2 + proper mail config: >no logs and a blank page for https://domain.tld/nextcloud/settings/admin/logging Which page is blank exactly ? Youre ...
[14:49:29]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @rodinux [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#issuecomment-1884992862) on [issue #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622) Testing / switch to packaging V2 + proper mail config: > > no logs and a blank page for https://domain.tld/nextcloud/settings/admin/logging > > Which page is blank exactly ? ...
[14:51:48]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @rodinux [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#issuecomment-1884996275) on [issue #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622) Testing / switch to packaging V2 + proper mail config: there is no nextcloud.log file also /var/www/nextcloud# sudo -u nextcloud php8.2 --define apc.enable_cli=1 occ log:t...
[14:53:28]
<Aleks (he/him/il/lui)> sounds like that's why Nextcloud is way faster : displaying a blank page is super fast ? 😬
[14:55:11]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @rodinux [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#issuecomment-1885003054) on [issue #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622) Testing / switch to packaging V2 + proper mail config: Perhaps I need do a sudo -u nextcloud php8.2 --define apc.enable_cli=1 occ log:manage ?? I try sudo -u nextcloud ph...
[15:00:46]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @rodinux [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#issuecomment-1885012885) on [issue #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622) Testing / switch to packaging V2 + proper mail config: 2 options * or I remove nextcloud and restore the backup to come back with the 27.1.4 (hope without damage) * or I try...
[15:10:23]
<Mateusz Szymański> [ ] `markdown`
[15:10:23]
<Mateusz Szymański> ## markdown
[15:10:26]
<Mateusz Szymański> > <@kayou:matrix.org> There is no markdown anymore on matrix? O.o
you need to enable it in preferences in your client of choice
[15:11:24]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @rodinux [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#issuecomment-1885032333) on [issue #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622) Testing / switch to packaging V2 + proper mail config: Perhaps something to do with nginx and mjs ? I see nothing here https://domain.tld/nextcloud/apps/logreader/js/logreade...
[15:12:19]
<kayou> Ty, it was working out of the box before
[15:13:25]
<Salamandar> > <@kayou:matrix.org> `chown -R nextcloud: /home/yunohost.app/nextcloud` is suuuuper long on my server
`test markdown`
[15:13:27]
<Salamandar> :D
[15:25:14]
<lapineige> > <@Alekswag:matrix.org> also same `main.global_config.email_type` bullshit ~_~
I don't know what it is ^^
[15:25:47]
<lapineige> > <@kayou:matrix.org> There is no markdown anymore on matrix? O.o
It does, but it's not always straightforward
[15:29:23]
<Aleks (he/him/il/lui)> > I don't know what it is ^^
it's related to the format of email sent by the old ynh_send_stuff_to_admin which is just replaced by the PRE/POST_INSTALL.md etc
[15:29:27]
<Aleks (he/him/il/lui)> like you could chose wether to receive the email as html or as raw text, big yay
[15:29:39]
<eric_G> > <@kayou:matrix.org> Probably an nginx conf
or a permission issue... I have a similar warning in Firefox console `The resource from “https://SERVER_DOMAIN/apps/logreader/js/logreader-main.mjs?v=da1c8fd3-5” was blocked due to MIME type (“text/html”) mismatch (X-Content-Type-Options: nosniff)`. cf this issue: https://help.nextcloud.com/t/apps-files-blocked-due-to-mime-type-solved/66151
[15:37:50]
<Aleks (he/him/il/lui)> can you click on the request and check what kind of content is in the "Reponse" ?
[15:38:06]
<Aleks (he/him/il/lui)> i mean it's probably something like a 404 or whatever and totally not .mjs content
[15:38:19]
<Aleks (he/him/il/lui)> which in turn probably means something not properly configured in the nginx config
[15:38:31]
<Aleks (he/him/il/lui)> but that's a rough guess
[15:40:29]
<kayou> According to the nextcloud's nginx conf, after adding
```
include mime.types;
types {
text/javascript js mjs;
}
```
in the nginx conf, it's better: the page loads, but I still have an error "impossible de charger les entrées du journal", and the log list is empty
[15:40:39]
<kayou> Sorry for the delay, I'm @work
[15:51:51]
<eric_G> after adding to NGINX config
```
include mime.types;
types {
text/javascript js mjs;
}
```
I have the logs showing as well
[15:58:15]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to testing: Update nginx.conf ([874a0c51](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/874a0c516a58a19bbe6ec94598c08d90eb43af76))
[15:58:16]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to testing: Add .mjs as a file extension ([3e946265](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/3e9462656ebd60b0e5179b3ed342f9fcf9e67c10))
[15:58:16]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar dismissed review [pull request #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#pullrequestreview-1812708014): Testing / switch to packaging V2 + proper mail config
[16:02:21]
<lapineige> > <@ericg:matrix.org> I get an empty page
Disable the metadata app
[16:02:26]
<lapineige> It's the one throwing too many warnings. Because it's not compatible with Nextcloud 28
[16:11:26]
<Yunohost Git/Infra notifications> App mastodon goes down from level 8 to 6 in job [#22073](https://ci-apps.yunohost.org/ci/job/22073)
[16:20:46]
<Yunohost Git/Infra notifications> Job [#22067](https://ci-apps.yunohost.org/ci/job/22067) for framaforms failed miserably :(
[17:06:19]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @rodinux [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#issuecomment-1885250797) on [issue #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622) Testing / switch to packaging V2 + proper mail config: If I add your line it do strange things I think it must be already good with location ~ \.(?:css|js|mjs|svg|gif|p...
[17:50:17]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin pushed 1 commit to testing: Update ADMIN.md ([2f11e708](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/2f11e708b6027bce7951b06b042f32076533a4a0))
[17:55:36]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin pushed 1 commit to testing: Update ADMIN_fr.md ([b4f66f67](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/b4f66f67c5a634696a0020e241b88edcf4d80a88))
[18:03:04]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot pushed 1 commit to v2_PostgreSQL: Auto-update README ([ce616827](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/ce61682761e25227b99f2306a64f8bd200767ece))
[18:03:12]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin pushed 1 commit to v2_PostgreSQL: Update manifest.toml: bump version to have a version reference corresponding to the mysql->postgresql transition ([e32aaf6c](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/e32aaf6ce9c1e6216d252201aed802349c5e26d5))
[18:05:23]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597#discussion_r1447745794) on pull request #597 V2 postgresql: Im not sure we can really rely on the mysqlpwd ... Its supposed to be automatically deleted and renamed to db_pwd du...
[18:10:41]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597#discussion_r1447748767) on pull request #597 V2 postgresql: what about sonthing like Mattermost https://github.com/YunoHost-Apps/mattermost_ynh/blob/52d95b9785edbe71e84fb9b99a26be...
[18:12:01]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar edited a [comment](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597#discussion_r1447748767) on pull request #597 V2 postgresql: what about something like Mattermost https://github.com/YunoHost-Apps/mattermost_ynh/blob/52d95b9785edbe71e84fb9b99a26b...
[18:12:55]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597#discussion_r1447751522) on pull request #597 V2 postgresql: Now that we are in packaging v2, the DB should already be provisioned before the upgrade happens so eeeeh
[18:13:39]
<Aleks (he/him/il/lui)> (i'm reworking the DB migration snippet)
[18:14:09]
<Yunohost Git/Infra notifications> App pluxml goes down from level 8 to 1 in job [#22078](https://ci-apps.yunohost.org/ci/job/22078)
[18:23:31]
<kayou> Fyi when I put my nose in this migration, I find out that a lot of app may break, so I gave up
[18:23:39]
<kayou> ¯\_(ツ)_/¯
[18:25:29]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597#discussion_r1447766607) on pull request #597 V2 postgresql: suggestion # If were moving through version 28.0.1~ynh2 (in which the switch to psql is made) if ynh_compare_current...
[18:25:47]
<Aleks (he/him/il/lui)> > <@kayou:matrix.org> Fyi when I put my nose in this migration, I find out that a lot of app may break, so I gave up
you mean the mysql->postgresql migration ?
[18:26:07]
<kayou> Yes
[18:26:36]
<kayou> But there were the story of 4 bytes migration too
[19:17:25]
<Aleks (he/him/il/lui)> what exactly makes you say that app may break tho ?
[19:17:25]
<Aleks (he/him/il/lui)> i had a feeling that it sounded too easy yeah 🤔
[19:17:26]
<kayou> IIRC, a "clean" instant can migrate without any issue, but if you install an ~old nextcloud version, upgrade, then migrate, the migration crashes
[19:17:29]
<eric_G> how old?
[19:17:35]
<kayou> Or something like that
[19:17:35]
<kayou> Old like, we were on NC 18, and I tried to install NC 16, upgrade to 18, migrate
[19:17:39]
<eric_G> you mean a pre covid version 😬
[19:17:39]
<Aleks (he/him/il/lui)> #There'sNoBefore
[19:17:46]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597#issuecomment-1885406333) on [issue #597](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597) V2 postgresql: testme
[19:17:46]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597#discussion_r1447777637) on pull request #597 V2 postgresql: lets test
[19:18:08]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to v2_PostgreSQL: Update scripts/upgrade Co-authored-by: Alexandre Aubin <4533074+alexAubin@users.noreply.github.com> ([e724e982](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/e724e9820702a09130b77c7cead584909372ed8f))
[19:18:08]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597#issuecomment-1885406407) on [issue #597](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597) V2 postgresql: :sunflower:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/12567/badge)]...
[19:47:12]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin pushed 1 commit to v2_PostgreSQL: Update upgrade: grompf ([5149bcbe](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/5149bcbee4918e6335a7103b7d0f9a5c02d80a9e))
[20:13:53]
<Yunohost Git/Infra notifications> [package_check] @alexAubin pushed 1 commit to master: yolo: attempt to preinstall apt dependencies in the initial lxc snapshot ([94abddf5](https://github.com/YunoHost/package_check/commit/94abddf58d99c4dd22749d2d646296446d19294c))
[20:18:33]
<Yunohost Git/Infra notifications> [package_check] @alexAubin pushed 1 commit to master: Need those classic option for noninteractive apt etc.. ([2def397a](https://github.com/YunoHost/package_check/commit/2def397a8a7004f6a452bfb643134ba4ef299d58))
[20:23:18]
<Yunohost Git/Infra notifications> [package_check] @alexAubin pushed 1 commit to master: Zgrompf ([1dda43ee](https://github.com/YunoHost/package_check/commit/1dda43eef45a60460ecc842e5419b07083d15f30))
[20:27:54]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 5 commits to v2_PostgreSQL ([5149bcbee491...15385967e476](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/5149bcbee491...15385967e476))
[20:27:55]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2_PostgreSQL] Update ADMIN.md - Alexandre Aubin
[20:47:15]
<Yunohost Git/Infra notifications> [package_check] @alexAubin pushed 1 commit to master: Woopsies ([ac8108a3](https://github.com/YunoHost/package_check/commit/ac8108a3523e3812ff4d5a67507545c2ec25028b))
[20:47:15]
<Yunohost Git/Infra notifications> [package_check] @alexAubin pushed 1 commit to master: Restructure some stuff to improve UX, ideally dont want to wait for the apt deps to install before being able to know w... ([8e267559](https://github.com/YunoHost/package_check/commit/8e267559a7da9c114beba318d7d43723594e971d))
[20:48:44]
<Yunohost Git/Infra notifications> Job [#22087](https://ci-apps.yunohost.org/ci/job/22087) for pgadmin failed miserably :(
[20:49:33]
<Aleks (he/him/il/lui)> must ... resist ... starting to rewrite yunorunner and package_check from scratch using celery >_>
[20:49:33]
<Aleks (he/him/il/lui)> zzzz locks
[20:51:07]
<eric_G> > <@Alekswag:matrix.org> must ... resist ... starting to rewrite yunorunner and package_check from scratch using celery >_>
is that before or after implementing telemetry? 😶🌫️
[20:51:09]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin pushed 1 commit to v2_PostgreSQL: Update upgrade: aaaaaa ([1827a858](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/1827a8584a276e8756e8819562c5cd35937661e1))
[20:51:40]
<Raphael> Hello, I would like to know what is the best way to verify a package, apparently lint is a bit considered deprecated, right?
[20:52:06]
<Aleks (he/him/il/lui)> what do you mean by "verifying a package" and "lint is deprecated" ? The standard workflow is to test the app using the CI system ?
[20:52:10]
<Aleks (he/him/il/lui)> packaging_testing
[20:52:22]
<Aleks (he/him/il/lui)> sry : https://yunohost.org/packaging_testing
[20:53:40]
<Raphael> Linter, not lint btw
[20:53:40]
<Raphael> Yes, I meant between the three way to do it
[20:56:15]
<Aleks (he/him/il/lui)> the linter and the "actual install/remove/backup/upgrade tests" are complementary to each other
[20:56:15]
<Aleks (he/him/il/lui)> not sure what are the three ways you refer to 😬
[20:57:08]
<Raphael> Ok, they are complementary, I thought they were alternatives.
[21:06:40]
<Salamandar> Hi
[21:06:40]
<Yunohost Git/Infra notifications> [package_check] @alexAubin pushed 1 commit to master: Make sure to generate the psql password to prevent warnings ([9627a8c0](https://github.com/YunoHost/package_check/commit/9627a8c0a3417b85adcb5c7ed635d76c99efa591))
[21:06:56]
<Salamandar> `ynh_setup_source --dest_dir="$install_dir" --keep=".env storage/oauth-private.key storage/oauth-public.key storage/private_uploads public/uploads"`
should remove for exemple the app subdir right ?
[21:07:12]
<Mateusz Szymański> > <@Salamandar:matrix.org> `ynh_setup_source --dest_dir="$install_dir" --keep=".env storage/oauth-private.key storage/oauth-public.key storage/private_uploads public/uploads"`
> should remove for exemple the app subdir right ?
not without `--full_replace=1`
[21:07:16]
<Salamandar> oooooh
[21:07:17]
<Salamandar> okay
[21:07:26]
<Salamandar> bug here then https://github.com/YunoHost-Apps/snipeit_ynh/blob/master/scripts/upgrade
[21:07:39]
<Mateusz Szymański> (this `keep` with no `full_replace` should throw a warning iMO), common source of confusion
[21:07:53]
<Salamandar> > <@orhtej2:matrix.org> (this `keep` with no `full_replace` should throw a warning iMO), common source of confusion
no. --keep should *enable* full_replace.
[21:09:01]
<Mateusz Szymański> > <@ashledombos:matrix.org> Hello, I would like to know what is the best way to verify a package, apparently lint is a bit considered deprecated, right?
The proof of pudding is in the eating or whatever the idiom goes, install the app on real system, backup restore do whatever's supposed to be supported and see if it works ;)
CI only catches obvious errors like 'app after install does not start at all'
[21:09:52]
<Mateusz Szymański> > <@Salamandar:matrix.org> no. --keep should *enable* full_replace.
whatever floats your boat I guess, I see appeal in keeping the conf rather than regenerating it, OTOH what if app ships breaking change?
[21:10:57]
<Salamandar> > <@orhtej2:matrix.org> whatever floats your boat I guess, I see appeal in keeping the conf rather than regenerating it, OTOH what if app ships breaking change?
I think you didn't understand what I said ^^ IMHO if you pass --keep it should implicitly enable the --full_replace flag too ^^
[21:11:31]
<Salamandar> but yeah a warning should be added to the package_checker imho
[21:12:10]
<Mateusz Szymański> > <@Salamandar:matrix.org> I think you didn't understand what I said ^^ IMHO if you pass --keep it should implicitly enable the --full_replace flag too ^^
ahhhhhhhhh hmmmmmmmmm I see your point and I'm not sure if I agree, being explicit about 'I'm going to remove all in that folder' is not too much to ask from packager
[21:12:44]
<Mateusz Szymański> > <@Alekswag:matrix.org> there are probably legit case where you don't want to full-replace, because the install dir may contain data or asset or whatever, but you still don't want the archive to overwrite the config with default stuff or whatever
thing is `keep` without `full_replace` is noop and likely a packaging error
[21:12:44]
<Salamandar> yeah i get what you mean
[21:12:48]
<Aleks (he/him/il/lui)> there are probably legit case where you don't want to full-replace, because the install dir may contain data or asset or whatever, but you still don't want the archive to overwrite the config with default stuff or whatever
[21:12:49]
<Aleks (he/him/il/lui)> (though you could also argue data should be outside the install dir, docker-style kindof)
[21:12:55]
<Mateusz Szymański> > <@Alekswag:matrix.org> (though you could also argue data should be outside the install dir, docker-style kindof)
if only all apps supported it x_x
[21:13:06]
<Aleks (he/him/il/lui)> yeah..
[21:13:25]
<Salamandar> > <@Alekswag:matrix.org> there are probably legit case where you don't want to full-replace, because the install dir may contain data or asset or whatever, but you still don't want the archive to overwrite the config with default stuff or whatever
oooh
[21:13:26]
<Salamandar> alright
[21:13:31]
<Salamandar> i see what you mean
[21:13:34]
<Aleks (he/him/il/lui)> maybe we could do some mount bind magic but that's gonna require some engineering
[21:14:07]
<Salamandar> > <@Alekswag:matrix.org> maybe we could do some mount bind magic but that's gonna require some engineering
and maybe systemd hardening would break that
[21:14:12]
<Salamandar> although with bind mounts, maybe not…
[21:14:18]
<Aleks (he/him/il/lui)> if we did more hardening than the current basis yeah
[21:15:23]
<Aleks (he/him/il/lui)> but there's clearly a big underlying topic of "it would really simplify things to have immutable install_dirs that can be thrown away during upgrade and just re-plug the config+data and voila", docker-style
[21:16:03]
<Salamandar> oh yeah
[21:16:09]
<Aleks (he/him/il/lui)> #PackagingV4Wishlist
[21:16:50]
<Salamandar> OK so yolocommiting this on master on snipe-it app…
[21:17:23]
<Salamandar> TBH I thought ynh_setup_source was overwriting the whole directory…
[21:17:48]
<Salamandar> https://github.com/YunoHost/example_ynh/blob/1e94494f246bc3bebc4a58c8161eabd38dabe68d/scripts/upgrade#L76
[21:17:57]
<Salamandar> This line should be updated to add --full_replace=1, to let packagers know it's available
[21:18:12]
<Salamandar> because that's what you want 90% of the time
[21:18:55]
<Mateusz Szymański> I mean it [is documented](https://yunohost.org/en/packaging_apps_helpers#ynh-setup-source) so....
[21:19:01]
<Salamandar> yes yes, i know
[21:19:51]
<eric_G> who reads the doc anyway? 😬
[21:19:51]
<Salamandar> but in my head I went "ok if the example does that, it's probably with a 'sane' behaviour, so it is cleaning everything up"
[21:20:01]
<Salamandar> I'll have to go around every app i've ever packaged/updated to add --full_replace…
[21:20:23]
<Salamandar> > <@ericg:matrix.org> who reads the doc anyway? 😬
well me, except when it's almost obvious for me… :(
[21:21:03]
<eric_G> maybe I should read more the doc... I learning a lot tonight
[21:22:15]
<eric_G> I have almost define a standard of using `--keep` on every package 😊
[21:23:30]
<Yunohost Git/Infra notifications> Job [#22089](https://ci-apps.yunohost.org/ci/job/22089) for borg failed miserably :(
[21:27:07]
<Mateusz Szymański> > <@ericg:matrix.org> I have almost define a standard of using `--keep` on every package 😊
this way when someone comes in to fix the obvious bug of missing `-r` the required `--keep` is already there!
[21:29:40]
<Salamandar> ahah
[21:30:53]
<Salamandar> btw I have some small merge requests on example_ynh, who's the "owner" ? maybe @ericg:matrix.org ?
[21:32:25]
<Mateusz Szymański> > <@Salamandar:matrix.org> btw I have some small merge requests on example_ynh, who's the "owner" ? maybe @ericg:matrix.org ?
just create MR, someone will come and merge it
[21:32:30]
<eric_G> nop, example_ynh is not part of my territory yet
[21:32:50]
<eric_G> but I own the rest 😂
[21:32:57]
<Salamandar> they're already waiting for 2 months, that's why i'm asking :D
[21:33:00]
<Salamandar> > <@ericg:matrix.org> nop, example_ynh is not part of my territory yet
ahaha
[21:33:02]
<Aleks (he/him/il/lui)> wtf we did not merge https://github.com/YunoHost/example_ynh/pull/210 @_@
[21:33:19]
<Salamandar> "everything the shadow does not touch"
[21:33:20]
<Salamandar> > <@Alekswag:matrix.org> wtf we did not merge https://github.com/YunoHost/example_ynh/pull/210 @_@
it's already kinda merged i think
[21:33:21]
*Aleks (he/him/il/lui) has not open example_ynh/pulls in a long time
[21:34:32]
<Salamandar> > <@Salamandar:matrix.org> it's already kinda merged i think
ah no, i misread gitg output
[21:34:44]
<Yunohost Git/Infra notifications> [example_ynh] @alexAubin approved [pull request #215](https://github.com/YunoHost/example_ynh/pull/215#pullrequestreview-1814224463) Reorganize steps in scripts to have more coherent scripts one to another
[21:36:05]
<Yunohost Git/Infra notifications> [example_ynh] @alexAubin [commented](https://github.com/YunoHost/example_ynh/pull/216#discussion_r1448004432) on pull request #216 Create log files at install and restore steps: I think that one is now handled by the logrotate helper ... Assuming the app calls the logrotate helper ... But its n...
[21:37:01]
<Yunohost Git/Infra notifications> [example_ynh] @alexAubin edited a [comment](https://github.com/YunoHost/example_ynh/pull/216#discussion_r1448004432) on pull request #216 Create log files at install and restore steps: I think that one is now handled by the logrotate helper ... Assuming the app calls the logrotate helper ... But its n...
[21:37:46]
<Yunohost Git/Infra notifications> App pgadmin goes down from level 7 to 1 in job [#22087](https://ci-apps.yunohost.org/ci/job/22087)
[21:38:46]
<Yunohost Git/Infra notifications> [example_ynh] @alexAubin [commented](https://github.com/YunoHost/example_ynh/pull/217#discussion_r1448007101) on pull request #217 Backup and restore the log directory: Hmmmmmm 95% yes but then Im wondering about for example synapse for which logs have an absolutely unreasonable size ......
[21:39:46]
<Yunohost Git/Infra notifications> [example_ynh] @alexAubin approved [pull request #221](https://github.com/YunoHost/example_ynh/pull/221#pullrequestreview-1814231611) Fix SC2086 and various other shellcheck errors
[21:40:04]
<Yunohost Git/Infra notifications> [example_ynh] @alexAubin approved [pull request #217](https://github.com/YunoHost/example_ynh/pull/217#pullrequestreview-1814231962) Backup and restore the log directory
[21:45:50]
<Yunohost Git/Infra notifications> [example_ynh] @Salamandar [commented](https://github.com/YunoHost/example_ynh/pull/216#discussion_r1448021048) on pull request #216 Create log files at install and restore steps: Indeed. But the real issues happens when youre using fail2ban but not logrotate (even though you just should use logrot...
[21:46:20]
<Yunohost Git/Infra notifications> [example_ynh] @Salamandar [commented](https://github.com/YunoHost/example_ynh/pull/217#discussion_r1448021819) on pull request #217 Backup and restore the log directory: Indeed Ill add a comment.
[21:46:23]
<Aleks (he/him/il/lui)> so preinstallation of apt dependencies in the initial snapshot during CI tests is now implemented, should save up some time and bandwidth (both for us and the Debian project)
[21:50:30]
<Salamandar> ty <3
[21:50:30]
<Salamandar> > <@Alekswag:matrix.org> so preinstallation of apt dependencies in the initial snapshot during CI tests is now implemented, should save up some time and bandwidth (both for us and the Debian project)
oooooh
[22:13:51]
<Yunohost Git/Infra notifications> [example_ynh/master] Reorganize steps in scripts to have more coherent scripts one to another - Félix Piédallu
[22:13:51]
<Yunohost Git/Infra notifications> [example_ynh] @ericgaspar merged [pull request #215](https://github.com/YunoHost/example_ynh/pull/215): Reorganize steps in scripts to have more coherent scripts one to another
[22:14:28]
<Salamandar> <3
[22:14:52]
<Yunohost Git/Infra notifications> [example_ynh] @ericgaspar pushed 6 commits to master ([0a732cbcc4fd...9b0f4dcaae53](https://github.com/YunoHost/example_ynh/compare/0a732cbcc4fd...9b0f4dcaae53))
[22:14:52]
<Yunohost Git/Infra notifications> [example_ynh] @ericgaspar merged [pull request #221](https://github.com/YunoHost/example_ynh/pull/221): Fix SC2086 and various other shellcheck errors
[22:14:54]
<Yunohost Git/Infra notifications> [example_ynh/master] Merge branch master into fix-SC2086 - OniriCorpe
[22:14:58]
<Yunohost Git/Infra notifications> [example_ynh/master] Merge branch master into fix-SC2086 - eric_G
[22:15:40]
<Yunohost Git/Infra notifications> [example_ynh] @ericgaspar pushed 2 commits to master ([9b0f4dcaae53...021125aa33d0](https://github.com/YunoHost/example_ynh/compare/9b0f4dcaae53...021125aa33d0))
[22:15:40]
<Yunohost Git/Infra notifications> [example_ynh] @ericgaspar merged [pull request #217](https://github.com/YunoHost/example_ynh/pull/217): Backup and restore the log directory
[22:15:45]
<Yunohost Git/Infra notifications> [example_ynh/master] Backup and restore the log directory - Félix Piédallu
[22:16:32]
<Yunohost Git/Infra notifications> [example_ynh] @Salamandar [commented](https://github.com/YunoHost/example_ynh/pull/216#issuecomment-1885829179) on [issue #216](https://github.com/YunoHost/example_ynh/pull/216) Create log files at install and restore steps: Closing in favor of #217
[22:16:38]
<Yunohost Git/Infra notifications> [example_ynh] @Salamandar closed [pull request #216](https://github.com/YunoHost/example_ynh/pull/216): Create log files at install and restore steps
[22:17:09]
<Yunohost Git/Infra notifications> [example_ynh] @Salamandar [commented](https://github.com/YunoHost/example_ynh/pull/198#issuecomment-1885830087) on [issue #198](https://github.com/YunoHost/example_ynh/pull/198) Create application log directory in restore: It should be closed now that #217 is merged
[22:20:08]
<Yunohost Git/Infra notifications> [example_ynh] @alexAubin closed [pull request #198](https://github.com/YunoHost/example_ynh/pull/198): Create application log directory in restore
[22:22:23]
<Aleks (he/him/il/lui)> Nextcloud be like, let me sing you the song of my people:
[22:22:43]
<Aleks (he/him/il/lui)> https://aria.im/_matrix/media/v1/download/matrix.org/IteAYKFQjqhimgwyGwteidYH
[22:32:16]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597#issuecomment-1885847031) on [issue #597](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597) V2 postgresql: + sudo -u nextcloud php8.1 --define apc.enable_cli=1 occ --no-interaction --no-ansi db:convert-type --all-apps --cle...
[22:45:23]
<Salamandar> > <@Alekswag:matrix.org> Capture d’écran du 2024-01-10 23-22-03.png
ahah
[22:46:47]
<eric_G> > <@Alekswag:matrix.org> sent an image.
it is beautiful when it doesn't fail
[22:53:21]
<Yunohost Git/Infra notifications> [example_ynh] @ericgaspar approved [pull request #210](https://github.com/YunoHost/example_ynh/pull/210#pullrequestreview-1814340880) Simplified add fpm config
[22:54:44]
<Yunohost Git/Infra notifications> [example_ynh] @Salamandar opened [pull request #224](https://github.com/YunoHost/example_ynh/pull/224): Codingstyle
[22:57:49]
<Yunohost Git/Infra notifications> [example_ynh] @ericgaspar approved [pull request #224](https://github.com/YunoHost/example_ynh/pull/224#pullrequestreview-1814346874) Codingstyle
[22:57:53]
<Yunohost Git/Infra notifications> [example_ynh] @ericgaspar pushed 4 commits to master ([021125aa33d0...744c3fd04f7a](https://github.com/YunoHost/example_ynh/compare/021125aa33d0...744c3fd04f7a))
[22:57:54]
<Yunohost Git/Infra notifications> [example_ynh] @ericgaspar merged [pull request #210](https://github.com/YunoHost/example_ynh/pull/210): Simplified add fpm config
[22:57:57]
<Yunohost Git/Infra notifications> [example_ynh/master] fix - Éric Gaspar
[22:58:03]
<Yunohost Git/Infra notifications> [example_ynh/master] fpm config: cf YunoHost 11.2, remove the necessity to handle fpm_usage / footprint manually - Alexandre Aubin
[22:58:11]
<Yunohost Git/Infra notifications> [example_ynh] @ericgaspar pushed 4 commits to master ([744c3fd04f7a...bbbc1df6dda3](https://github.com/YunoHost/example_ynh/compare/744c3fd04f7a...bbbc1df6dda3))
[22:58:11]
<Yunohost Git/Infra notifications> [example_ynh] @ericgaspar merged [pull request #224](https://github.com/YunoHost/example_ynh/pull/224): Codingstyle
[23:00:33]
<Yunohost Git/Infra notifications> [example_ynh] @ericgaspar closed [pull request #195](https://github.com/YunoHost/example_ynh/pull/195): Document php extra php fpm.conf
[23:00:34]
<Yunohost Git/Infra notifications> [example_ynh] @ericgaspar [commented](https://github.com/YunoHost/example_ynh/pull/195#issuecomment-1885881704) on [issue #195](https://github.com/YunoHost/example_ynh/pull/195) Document php extra php fpm.conf: closing in favor of #210
[23:01:52]
<Salamandar> @ericg:matrix.org those instamerges :o
[23:02:30]
<eric_G> yes, I noticed that I had merge power on this repo 😬
[23:03:12]
<Salamandar> :D
[23:03:12]
<Yunohost Git/Infra notifications> [example_ynh] @alexAubin deleted branch simplified-add-fpm-config
[23:18:36]
<Yunohost Git/Infra notifications> [example_ynh] @Salamandar opened [pull request #225](https://github.com/YunoHost/example_ynh/pull/225): On upgrade, pass --full_replace, --keep to ynh_setup_source, with the according doc.
[23:18:53]
<Salamandar> @ericg:matrix.org this (last ?) merge request is what triggered the discussion tonight :D
[23:51:00]
<Salamandar> TBH now i'm questionning the whole "if [ "$upgrade_type" == "UPGRADE_APP" ]; then" code
[23:51:03]
<Salamandar> is it bad ? :D
[23:52:44]
<Salamandar> because if upgrade\_type is NOT upgrade\_app, that means it's a "package update"
[23:52:53]
<Salamandar> because if upgrade__type is NOT upgrade__app, that means it's a "package update"
[23:53:03]
<Yunohost Git/Infra notifications> App jackett stays at level 1 in job [#22096](https://ci-apps.yunohost.org/ci/job/22096)
[23:53:12]
<Salamandar> because if upgrade`_`type is NOT upgrade`_`app, that means it's a "package update"
[23:53:18]
<Salamandar> but maybe the package update FIXES the source extraction ?
[23:53:40]
<Salamandar> In that case we WANT to execute the code inside this if..else test