[01:09:58]
<Yunohost Git/Infra notifications> App borgserver failed all tests in job [#21561](https://ci-apps.yunohost.org/ci/job/21561) :(
[01:59:23]
<Yunohost Git/Infra notifications> App digiscreen failed all tests in job [#21563](https://ci-apps.yunohost.org/ci/job/21563) :(
[03:13:51]
<Yunohost Git/Infra notifications> App facilmap goes down from level 8 to 6 in job [#21565](https://ci-apps.yunohost.org/ci/job/21565)
[10:56:34]
<thatoo> > <@Alekswag:matrix.org> can you share the actual code
```
[install]
[install.domain]
type = "domain"
[install.init_main_permission]
type = "group"
default = "visitors"
help.en = "xxx"
help.fr = "xxx"
[install.server_name]
ask.en = "xxx"
ask.fr = "xxx"
help.en = "xxx"
help.fr = "xxx"
type = "domain"
[install.registration]
ask.en = "xxx?"
ask.fr = "xxx ?"
type = "boolean"
help.en = "xxx"
help.fr = "xxx"
default = false
[install.registration_token]
ask.en = "xxx."
ask.fr = "xxx."
help.en = "xxx."
help.fr = "xxx."
type = "password"
visible = "! registration"
```
[11:50:08]
<Yunohost Git/Infra notifications> App jitsi rises from level 0 to 8 in job [#21579](https://ci-apps.yunohost.org/ci/job/21579) !
[12:16:30]
<Aleks (he/him/il/lui)> .... does anybody understand what the hell the "non-append" option of ynh_use_logrotate is about ...? Are there any case at all where we would like to "append" instead of replacing the entire configuration ?
[12:17:48]
<eric_G> no, I rely completely on upstream knowledge 😶🌫️
[12:19:33]
<Aleks (he/him/il/lui)> i don't even know what the fuck is supposed to be the "specific_user" arg for ... like why would you want stuff to be ran as root for 90% of apps, and somehow some apps do need to run the log rotation as a specific user
[12:24:23]
<Mateusz Szymański> > <@Alekswag:matrix.org> .... does anybody understand what the hell the "non-append" option of ynh_use_logrotate is about ...? Are there any case at all where we would like to "append" instead of replacing the entire configuration ?
to my understanding: all config for logrotate for app ends in `/etc/logrotate.d/$app`. Imagine you want to rotate 2 files OR update the config, just as calibreweb happens to do:
https://github.com/YunoHost-Apps/calibreweb_ynh/blob/b0b09712d841e210b26967d343d37e9a75bc5151/scripts/upgrade#L204
What you should do is
```sh
# replace file
ynh_use_logrotate <options for file1> --nonappend
# setup second file, not replacing (hence keeping config for file1)
ynh_use_logrotate <options for file2>
```
[12:24:40]
<Mateusz Szymański> (which is not necessarily what happens in linked snippet :P)
[12:25:28]
<Aleks (he/him/il/lui)> oooor we could just list several files in argument instead of having this madness, assuming the helper was designed in an appropriate way
[12:25:54]
<Mateusz Szymański> > <@Alekswag:matrix.org> oooor we could just list several files in argument instead of having this madness, assuming the helper was designed in an appropriate way
or that, yes
[12:26:06]
<Mateusz Szymański> then we patch 600 repos and we're all set
[12:26:32]
<Aleks (he/him/il/lui)> i think i'm gonna go with "silently ignore the --non-append" argument :|
[12:27:28]
<Mateusz Szymański> my stale checkout of 400+ repos shows only a handful of matches for `--nonappend` although I haven't searched for `-n`
[12:27:41]
<Mateusz Szymański> ¯\_(ツ)_/¯ so I guess we're fine
[12:27:51]
<Aleks (he/him/il/lui)> funfact :
[12:27:59]
<Aleks (he/him/il/lui)> there's `--nonappend` and `--non-append`
[12:28:07]
<Aleks (he/him/il/lui)> and there's some stupid code to handle the mess
[12:28:21]
<Aleks (he/him/il/lui)> apparently one was supposed to be deprecated but never properly adversited or whatever
[12:28:24]
<Aleks (he/him/il/lui)> and now we have both in the wild
[12:28:28]
<Mateusz Szymański> https://github.com/YunoHost/yunohost/blob/4897f72974fff30b168a40bf55bf917395cca4cc/helpers/logrotate#L20C1-L26C75 :P
[12:28:29]
<Aleks (he/him/il/lui)> and both are pretty much useless
[12:28:44]
<Aleks (he/him/il/lui)> yeah exactly
[13:00:41]
<Aleks (he/him/il/lui)> jeesus christ we have `if [ ! -f "$1" ]` with $1 being most of the time something literally like `--foo=bar`
[13:03:37]
<eric_G> Are permissions within install_dir preserved when restoring? (permission for conf file for ex.)
[13:03:55]
<eric_G> we don't need this right?
https://github.com/YunoHost-Apps/cryptpad_ynh/blob/24102d402b922bcae9a2464a96696fa4153eea1c/scripts/restore#L22
[13:11:41]
<Mateusz Szymański> > <@ericg:matrix.org> Are permissions within install_dir preserved when restoring? (permission for conf file for ex.)
well yes but actually no, when transplanting between servers permissions are restored for phony users so I'd reapply
[13:39:51]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to master: Update grocy.png ([bb7b0078](https://github.com/YunoHost/apps/commit/bb7b0078d24866a77b2b255331021f29e9e5178a))
[15:18:55]
<thatoo> > <@thatoo:defis.info> ```
> [install]
> [install.domain]
> type = "domain"
>
> [install.init_main_permission]
> type = "group"
> default = "visitors"
> help.en = "xxx"
> help.fr = "xxx"
>
> [install.server_name]
> ask.en = "xxx"
> ask.fr = "xxx"
> help.en = "xxx"
> help.fr = "xxx"
> type = "domain"
>
> [install.registration]
> ask.en = "xxx?"
> ask.fr = "xxx ?"
> type = "boolean"
> help.en = "xxx"
> help.fr = "xxx"
> default = false
>
> [install.registration_token]
> ask.en = "xxx."
> ask.fr = "xxx."
> help.en = "xxx."
> help.fr = "xxx."
> type = "password"
> visible = "! registration"
> ```
I let a code with a branch with it in case there is a solution. : https://github.com/Thatoo/conduit_ynh/blob/Thatoo-test/manifest.toml
Otherwise I'll try something else.
[15:23:15]
<Mateusz Szymański> > <@Alekswag:matrix.org> hmmm is it all apps doing that ?
IDK, was browsing through YNH-Apps feed and stumbled upon: https://github.com/YunoHost-Apps/cjdns_ynh/pull/3#issuecomment-1866433194
[15:24:58]
<Mateusz Szymański> browsing through CI logs is pure comedy gold I guess?
```
94397 INFO WARNING - /usr/bin/xinit: connection to X server lost
94398 INFO DEBUG -
94398 INFO WARNING - waiting for X server to shut down (II) Server terminated successfully (0). Closing log file.
```
[15:25:15]
<Aleks (he/him/il/lui)> > <@thatoo:defis.info> I let a code with a branch with it in case there is a solution. : https://github.com/Thatoo/conduit_ynh/blob/Thatoo-test/manifest.toml
> Otherwise I'll try something else.
can't find any app using `visible = whatever` in their manifest, so i'm assuming it's not supported
[15:25:35]
<Mateusz Szymański> https://ci-apps-dev.yunohost.org/ci/ <- most look OK
[15:25:45]
<Aleks (he/him/il/lui)> hmmm are you looking at Kodi ? 😅
[15:25:54]
<Mateusz Szymański> > <@Alekswag:matrix.org> hmmm are you looking at Kodi ? 😅
yep
[15:26:06]
<Mateusz Szymański> it's meant for YNHboxes with actual HDMI output?
[15:26:37]
<Aleks (he/him/il/lui)> hmpf yeah i tried to make Kodi work the other day and lost 30 mental health point try to configure the damn "dummy" Xorg screen thing
[15:26:53]
<Aleks (he/him/il/lui)> somehow you need Xorg running to test Kodi yes
[15:27:10]
<Aleks (he/him/il/lui)> so like, good luck with that in a CI context
[15:27:20]
<Aleks (he/him/il/lui)> but it was working in the past (the CI test) with some dummy interface
[15:27:32]
<Aleks (he/him/il/lui)> and there's some sort of doc about that kind of stuff in the internets
[15:27:39]
<Aleks (he/him/il/lui)> but couldnt get it to work
[15:28:01]
<Aleks (he/him/il/lui)> maybe some of their conf/API changed idk
[15:29:53]
<Aleks (he/him/il/lui)> but it would be nice to get it working at some point this century x_x
[15:30:44]
<Aleks (he/him/il/lui)> i mean worst part is maybe it does work in a real life context, but at the same time, totally bypassing the CI is kind of "eh"
[15:36:34]
<Aleks (he/him/il/lui)> axolotle: hmm are you familiar with this error maybe ? https://ci-apps-bookworm.yunohost.org/ci/job/1882 sounds like something related to pydantic maybe
[15:36:46]
<Aleks (he/him/il/lui)> ```
Running: yunohost app install --no-remove-on-failure --force /app_folder -a "domain=sub.domain.tld&init_main_permission=visitors&admin=package_checker&password=MySuperComplexPassword&name=my blog®istration=0&"
2270 ERROR extra fields not permitted
Error: Installation failed.
```
[15:37:38]
<Aleks (he/him/il/lui)> or could maybe due to `in key install > domain
Additional properties are not allowed ('full_domain' was unexpected)`
[15:42:26]
<axolotle> yep, extra props are forbidden
[15:42:45]
<axolotle> should it not be?
[15:43:32]
<Aleks (he/him/il/lui)> but now tests.toml was added a few minutes ago, maybe the tests are gonna be more sensible
[15:43:32]
<axolotle> or `full_domain` should be added as a DomainOption prop?
[15:44:40]
<Aleks (he/him/il/lui)> also much confusion because it's testing the master branch on the dev ci ...
[15:46:05]
<Aleks (he/him/il/lui)> the PR is called testing but is merging master into master
[15:46:07]
<Aleks (he/him/il/lui)> more psychokwak emojis
[15:46:25]
<Aleks (he/him/il/lui)> mcgonnagal_5_points_removed_from_mental_health.jpg
[15:46:52]
<Aleks (he/him/il/lui)> https://botsin.space/@scream
[15:47:14]
<Mateusz Szymański> https://raw.githubusercontent.com/Andy-Bell/catemon/master/public/images/psyduck-ascii.txt
ynh package when
[15:48:08]
<axolotle> > <@thatoo:defis.info> I let a code with a branch with it in case there is a solution. : https://github.com/Thatoo/conduit_ynh/blob/Thatoo-test/manifest.toml
> Otherwise I'll try something else.
Hum, `visible` is supposed to work with install too, but maybe the space in `! registration` is not supported by the js to python custom interpreter? idk
[15:53:23]
<Aleks (he/him/il/lui)> Mateusz Szymański: relaunched https://ci-apps-dev.yunohost.org/ci/job/12034, now it's a bit more sensible though still spooky errors/warnings in the linter
[16:15:37]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to 28.0.0: 28.0.1 ([ab33a96b](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/ab33a96bc4e08ce4e8367a8a4e8bbc293fe8d176))
[16:15:43]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot pushed 1 commit to 28.0.0: Auto-update README ([f957b2b4](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/f957b2b4ad432d9296b5124f20ec37e0aaad1a9c))
[16:16:08]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar edited [pull request #642](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/642): 28.0.1
[16:20:21]
<thatoo> no, I tried also with
`` visible = "registration" ``
and it does not work neither. I mean, the install don't display anything when I toggle "I understand that this installation might break..."
Whereas if I remove the "visible" line, then it displays.
[16:46:29]
<axolotle> hum you mean that the form isn't displayed at all in the web-admin? Propably an error somewhere then, is there anything in the console?
[16:49:21]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @lapineige opened [pull request #644](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/644): Fix patches for v28
[16:49:25]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @lapineige [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/644#issuecomment-1866634621) on [issue #644](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/644) Fix patches for v28: testme
[16:49:27]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/644#issuecomment-1866634702) on [issue #644](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/644) Fix patches for v28: Alrighty
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/12038/badge)](h...
[16:57:03]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @lapineige [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/644#issuecomment-1866644567) on [issue #644](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/644) Fix patches for v28: testme
[16:57:04]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/644#issuecomment-1866644674) on [issue #644](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/644) Fix patches for v28: Alrighty
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/12040/badge)](h...
[16:59:08]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @lapineige [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/644#issuecomment-1866647389) on [issue #644](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/644) Fix patches for v28: Fix patch fixes the issue with LoginController.php 153105 INFO DEBUG - patching file core/Controller/LoginControlle...
[16:59:53]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/644#issuecomment-1866648422) on [issue #644](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/644) Fix patches for v28: (NB : randomly jumping in, but @zamentur was pointing the other day that it is less trivial than it looks, because durin...
[17:02:58]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @lapineige [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/644#issuecomment-1866647389) on [issue #644](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/644) Fix patches for v28: First patch fixes the issue with LoginController.php 153105 INFO DEBUG - patching file core/Controller/LoginControl...
[17:02:58]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @lapineige [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/644#issuecomment-1866651864) on [issue #644](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/644) Fix patches for v28: Uh, I already thought it was trivial to fix :laughing: This is in the last-version folder, maybe it already handled ?
[17:03:19]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @lapineige [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/644#issuecomment-1866652602) on [issue #644](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/644) Fix patches for v28: > Alrighty [[Test Badge](https://camo.githubusercontent.com/b4339e8690f4ca919c20d8b56119eaa80198cef2c04f34538ae2a10c43...
[17:07:36]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/644#issuecomment-1866657097) on [issue #644](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/644) Fix patches for v28: Hmmyeah my bad it seems to already be handled since a long time :P https://github.com/YunoHost-Apps/nextcloud_ynh/blob/8...
[17:07:46]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @lapineige [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/642#issuecomment-1866657531) on [issue #642](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/642) 28.0.1: #644
[17:07:47]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @lapineige [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/642#issuecomment-1866657531) on [issue #642](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/642) 28.0.1: Please take a look at #644
[17:14:44]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @lapineige [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/644#issuecomment-1866667599) on [issue #644](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/644) Fix patches for v28: Long live Yunohost contributors :)
[18:01:55]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @lapineige [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/644#issuecomment-1866721098) on [issue #644](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/644) Fix patches for v28: Upgrades are working straight away in CI :tada: Ill try manual upgrade from v27.
[18:08:06]
<lapineige> Nextcloud 28 upgrade seem to be fixed (there were issue with patches).
https://github.com/YunoHost-Apps/nextcloud\_ynh/pull/644
Tests welcome 🙂
edit : uh wait I was a bit too hasty, Nginx shown an error at the end even if there were no issue in the middle 🤔
[18:09:24]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @lapineige [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/644#issuecomment-1866721098) on [issue #644](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/644) Fix patches for v28: ~~Upgrades are working straight away in CI :tada:~~ Ill try manual upgrade from v27. edit: uh, no, there is an error:...
[18:18:42]
<eric_G> lapineige: I have successfully updated to 28.0.1 my prod Nextcloud 😅
[18:19:12]
<lapineige> Great !
Then it's mysterious why it fails in CI, comming from v25 and v26
[18:19:35]
<eric_G> Nextcloud feels so much faster 😄
[18:19:49]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @lapineige [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/644#issuecomment-1866741263) on [issue #644](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/644) Fix patches for v28: Should we update test.toml to check also upgrade from v27 ?
[18:19:49]
<lapineige> Yeah main motivation to work on the upgrade 😄
[18:31:26]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @lapineige [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/644#issuecomment-1866764909) on [issue #644](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/644) Fix patches for v28: Change_url fails too but it seems to be a CI issue.
[18:37:35]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to testing: Update change_url ([1729962f](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/1729962f9e30235134afa0df7035a2d16f2e8f1e))
[18:38:00]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#issuecomment-1866772615) on [issue #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622) Testing / switch to packagingv2 + proper mail config: testme
[18:38:05]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#issuecomment-1866772671) on [issue #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622) Testing / switch to packagingv2 + proper mail config: Meow :cat2:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/12042/badge)]...
[18:39:55]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/643#issuecomment-1866774851) on [issue #643](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/643) Update nginx.conf: Is this related to the #628 PR?
[18:50:29]
<Aleks (he/him/il/lui)> > <@ericg:matrix.org> Nextcloud feels so much faster 😄
ah ? they did a bunch of optimization ?
[18:58:08]
<thatoo> In config_panel.toml, I wrote
```
[main.registration.registration_token]
type = "password"
bind = ":__INSTALL_DIR__/conduit.toml"
```
It does display the password I guess, at least it shows some points...
However if I change the password, yunohost beleive that nothing has changed :
> Reading config panel description and current configuration...
Checking what changed in the new configuration...
Nothing has changed
Reloading services...
[18:59:23]
<thatoo> and conduit.toml contain
```
# Enables registration. If set to false, users should register on this server with the help of a token.
allow_registration = __ALLOW_REGISTRATION__
registration_token = "__REGISTRATION_TOKEN__"
```
[19:07:41]
<thatoo> It works at install though in manifest.toml
[19:32:33]
<eric_G> > <@Alekswag:matrix.org> ah ? they did a bunch of optimization ?
Well... that's what I feel after every Nextcloud update 😬
[19:50:53]
<lapineige> > Nextcloud 28 upgrade seem to be fixed (there were issue with patches).
> https://github.com/YunoHost-Apps/nextcloud\_ynh/pull/644
> Tests welcome 🙂
>
> edit : uh wait I was a bit too hasty, Nginx shown an error at the end even if there were no issue in the middle 🤔
I didn't validate that step half an our ago, and now it seems stuck here:
> WARNING: You should check the app notifications above before continuing, there might be important stuff to know. [Press enter to continue]:
Not responding even on SSH 😕
[19:56:05]
<Aleks (he/him/il/lui)> not sure what you're talking about, is it on your own machine ? or some CI job ?
[20:02:42]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to testing: Update change_url ([84c71a56](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/84c71a56e9cbd9e72803ad10b0ed2dfa09aac7b9))
[20:03:33]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar approved [pull request #644](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/644#pullrequestreview-1793597246) Fix patches for v28
[20:03:38]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar merged [pull request #644](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/644): Fix patches for v28
[20:03:40]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 4 commits to 28.0.0 ([f957b2b4ad43...8d9bb382395e](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/f957b2b4ad43...8d9bb382395e))
[20:03:46]
<Yunohost Git/Infra notifications> [nextcloud_ynh/28.0.0] Adapt the nextcloud 28 content https://github.com/nextcloud/server/blob/e231abd9bfac66e69db810c986792f9ba1a9d581/lib/pu... - lapineige
[20:03:52]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/642#issuecomment-1866862785) on [issue #642](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/642) 28.0.1: testme
[20:03:52]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/642#issuecomment-1866862912) on [issue #642](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/642) 28.0.1: May the CI gods be with you
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/...
[20:04:22]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 3 commits to 28.0.0 ([8d9bb382395e...02f9e7f716cc](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/8d9bb382395e...02f9e7f716cc))
[20:04:26]
<Yunohost Git/Infra notifications> [nextcloud_ynh/28.0.0] Update change_url - Éric Gaspar
[20:04:32]
<Yunohost Git/Infra notifications> [nextcloud_ynh/28.0.0] Update change_url - Éric Gaspar
[20:05:28]
<lapineige> My own machine
[20:05:43]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to 28.0.0: Update tests.toml ([263bc166](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/263bc166ff42b6b1c5bce2c9b5827444fc1da908))
[20:05:47]
<lapineige> SSH is not even giving a timeout
[20:10:25]
<Aleks (he/him/il/lui)> still confused, it asks to press Enter, can't you press Enter ? x_X
[20:13:44]
<lapineige> I did. Twice
[20:15:58]
<Aleks (he/him/il/lui)> :| ...
[20:17:44]
<lapineige> On another server, I tried 7min ago, it's already done, pressing enter worked instantly
[20:18:32]
<lapineige> A massive bunch on the Files view, totally rewriten, which makes it fast.
The rest isn't improved much, performance-wise.
edit : oh, it also have brand new icons that are much clearer :D
[20:33:00]
<lapineige> And on a third server, same, perfect upgrade.
[20:33:05]
<lapineige> On a fast connection, it seems a bit faster indeed (with some lazy load on transparency that is very weird…), but that not that significant imho.
[20:37:58]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @lapineige [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/642#issuecomment-1866900809) on [issue #642](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/642) 28.0.1: Update went well on 2 servers, x86 and ARM architectures. Lets go for a merge once CI is happy ?
[20:48:15]
<lapineige> > I didn't validate that step half an our ago, and now it seems stuck here:
>
> > WARNING: You should check the app notifications above before continuing, there might be important stuff to know. [Press enter to continue]:
>
> Not responding even on SSH 😕
Well, I'm going to pull the plug, I'll tell you how it went 😅
[20:53:34]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @Thatoo closed [pull request #643](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/643): Update nginx.conf
[20:53:34]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @Thatoo [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/643#issuecomment-1866915176) on [issue #643](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/643) Update nginx.conf: Oh my god... I forgot I PR it already... I should go to sleep.
[20:55:58]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/643#issuecomment-1866917286) on [issue #643](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/643) Update nginx.conf: I understand, the PR was merged very quickly, which is unusual for nextcloud PR
[20:57:21]
<lapineige> Annnnnd… I'm in Nextcloud 28 \\o/
So it seems that the upgrade in indeed complete at that point, a crash here doesn't break anything *so far*
[21:05:40]
<eric_G> also, the patch correction seems to have fixed the redirect logout issue for me cf https://github.com/YunoHost-Apps/nextcloud\_ynh/pull/636
[21:06:16]
<eric_G> also, the patch correction seems to have fixed the redirect logout issue for me
[21:09:36]
<lapineige> it's funny 😂
[22:05:41]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar merged [pull request #642](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/642): 28.0.1
[22:05:54]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#issuecomment-1866988896) on [issue #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622) Testing / switch to packagingv2 + proper mail config: testme
[22:05:57]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#issuecomment-1866988936) on [issue #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622) Testing / switch to packagingv2 + proper mail config: Alrighty
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/12044/badge)](h...
[22:07:06]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 34 commits to v2_PostgreSQL ([1ab867a7b4de...1ebbf754078f](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/1ab867a7b4de...1ebbf754078f))
[22:07:23]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2_PostgreSQL] Update tests.toml - Éric Gaspar
[22:07:29]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597#issuecomment-1866990408) on [issue #597](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597) V2 postgresql: testme
[22:07:30]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597#issuecomment-1866990462) on [issue #597](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597) V2 postgresql: May the CI gods be with you
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/...
[22:07:51]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 23 commits to enh-faster-permissions-setter ([f0f71baf2d2f...42924bfd0cc2](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/f0f71baf2d2f...42924bfd0cc2))
[22:07:55]
<Yunohost Git/Infra notifications> [nextcloud_ynh/enh-faster-permissions-setter] Update tests.toml - Éric Gaspar
[22:07:59]
<Yunohost Git/Infra notifications> [nextcloud_ynh/enh-faster-permissions-setter] Merge pull request #642 from YunoHost-Apps/28.0.0 28.0.1 - eric_G
[22:26:13]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/637#issuecomment-1867006532) on [issue #637](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/637) [enh] Faster permissions setter on big datadir: testme
[22:26:18]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/637#issuecomment-1867006591) on [issue #637](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/637) [enh] Faster permissions setter on big datadir: :rocket:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/12046/badge)](ht...
[22:26:21]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/636#issuecomment-1867007795) on [issue #636](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/636) [fix] redirect logout: strangely #644 PR has solved my redirect logout issue