[00:11:21]
<Yunohost Git/Infra notifications> [example_ynh] @Salamandar opened [pull request #226](https://github.com/YunoHost/example_ynh/pull/226): Rework upgrade, remove the if upgrade_type == UPGRADE_APP test.
[00:13:37]
<Yunohost Git/Infra notifications> [example_ynh] @zamentur [commented](https://github.com/YunoHost/example_ynh/pull/226#issuecomment-1885965258) on [issue #226](https://github.com/YunoHost/example_ynh/pull/226) Rework upgrade, remove the if upgrade_type == UPGRADE_APP test.: You miss a part of the code: https://github.com/YunoHost/yunohost/blob/6fdcf03e9200f25431e998b030602c74c2adbae8/helpers/...
[00:16:22]
<Yunohost Git/Infra notifications> [example_ynh] @zamentur [commented](https://github.com/YunoHost/example_ynh/pull/226#issuecomment-1885965258) on [issue #226](https://github.com/YunoHost/example_ynh/pull/226) Rework upgrade, remove the if upgrade_type == UPGRADE_APP test.: You miss a part of the code: https://github.com/YunoHost/yunohost/blob/6fdcf03e9200f25431e998b030602c74c2adbae8/helpers/...
[00:17:15]
<Yunohost Git/Infra notifications> [example_ynh] @zamentur [commented](https://github.com/YunoHost/example_ynh/pull/226#issuecomment-1885965258) on [issue #226](https://github.com/YunoHost/example_ynh/pull/226) Rework upgrade, remove the if upgrade_type == UPGRADE_APP test.: You miss a part of the code: https://github.com/YunoHost/yunohost/blob/6fdcf03e9200f25431e998b030602c74c2adbae8/helpers/...
[00:19:13]
<Aleks (he/him/il/lui)> > <@Salamandar:matrix.org> TBH now i'm questionning the whole "if [ "$upgrade_type" == "UPGRADE_APP" ]; then" code
yeah same shit as always with packaging, stuff not really thoroughly designed, somebody just started to write this one day and everybody copypasted
[00:19:46]
<Aleks (he/him/il/lui)> unecessarily complex and nobody really understand why it's done this way
[00:21:18]
<Aleks (he/him/il/lui)> you'll be happy to discover that there is a third value, UPGRADE_FULL, it's not even clear if the code goes inside the "if" block or not in that case: https://github.com/YunoHost/yunohost/blob/dev/src/app.py#L672
[00:23:50]
<Aleks (he/him/il/lui)> there's also "UPGRADE_FORCED"
[00:23:50]
<Aleks (he/him/il/lui)> but then it's mashed up again in https://github.com/YunoHost/yunohost/blob/dev/helpers/utils#L998
[00:23:50]
<Aleks (he/him/il/lui)> so TL;DR probably nobody ever checks if it's UPGRADE_FULL or UPGRADE_FORCED
[00:23:51]
<Salamandar> > <@Alekswag:matrix.org> you'll be happy to discover that there is a third value, UPGRADE_FULL, it's not even clear if the code goes inside the "if" block or not in that case: https://github.com/YunoHost/yunohost/blob/dev/src/app.py#L672
I opened an issue about that already
[00:23:51]
<Salamandar> > <@Alekswag:matrix.org> and this entire thing is probably pointless because we could upgrade the source in all cases
Yup
[00:23:51]
<Aleks (he/him/il/lui)> and this entire thing is probably pointless because we could upgrade the source in all cases
[00:24:02]
<Yunohost Git/Infra notifications> [example_ynh] @Salamandar [commented](https://github.com/YunoHost/example_ynh/pull/226#issuecomment-1885975310) on [issue #226](https://github.com/YunoHost/example_ynh/pull/226) Rework upgrade, remove the if upgrade_type == UPGRADE_APP test.: > You miss a part of the code: https://github.com/YunoHost/yunohost/blob/6fdcf03e9200f25431e998b030602c74c2adbae8/helper...
[00:24:02]
<Salamandar> https://github.com/YunoHost/example_ynh/pull/226
[00:25:36]
<Salamandar> Ah no bad link
[00:25:40]
<Salamandar> https://github.com/YunoHost/issues/issues/2307
[00:26:08]
<Yunohost Git/Infra notifications> [example_ynh] @Salamandar [commented](https://github.com/YunoHost/example_ynh/pull/226#issuecomment-1885975310) on [issue #226](https://github.com/YunoHost/example_ynh/pull/226) Rework upgrade, remove the if upgrade_type == UPGRADE_APP test.: > You miss a part of the code: https://github.com/YunoHost/yunohost/blob/6fdcf03e9200f25431e998b030602c74c2adbae8/helper...
[00:26:08]
<Yunohost Git/Infra notifications> [example_ynh] @alexAubin [commented](https://github.com/YunoHost/example_ynh/pull/226#discussion_r1448136586) on pull request #226 Rework upgrade, remove the if upgrade_type == UPGRADE_APP test.: not sure to understand what that variable is used for then ? x_X
[00:27:19]
<Aleks (he/him/il/lui)> tbh i don't know any app using this var for anything else than conditional upgrade of sources
[00:27:19]
<Aleks (he/him/il/lui)> lemme scan the catalog
[00:28:27]
<Aleks (he/him/il/lui)> yeah
[00:28:48]
<Aleks (he/him/il/lui)> honestly let's just trash this thing
[00:33:13]
<Yunohost Git/Infra notifications> [example_ynh] @alexAubin [commented](https://github.com/YunoHost/example_ynh/pull/226#discussion_r1448139659) on pull request #226 Rework upgrade, remove the if upgrade_type == UPGRADE_APP test.: No app is using this variable for anything except the same copypasta of the example app, honestly lets just remove the ...
[00:34:49]
<Yunohost Git/Infra notifications> [example_ynh] @alexAubin [commented](https://github.com/YunoHost/example_ynh/pull/225#discussion_r1448141259) on pull request #225 On upgrade, pass --full_replace, --keep to ynh_setup_source, with the according doc.: On the one hand, yes On the other hand, people should reaaaaaaaaally be careful with this because if we yolowrite this ...
[03:03:36]
<Yunohost Git/Infra notifications> App endi failed all tests in job [#22106](https://ci-apps.yunohost.org/ci/job/22106) :(
[07:49:30]
<Yunohost Git/Infra notifications> App sonarr rises from level 0 to 8 in job [#22113](https://ci-apps.yunohost.org/ci/job/22113) !
[08:18:36]
<Yunohost Git/Infra notifications> App osada failed all tests in job [#22125](https://ci-apps.yunohost.org/ci/job/22125) :(
[08:23:42]
<Yunohost Git/Infra notifications> App overleaf failed all tests in job [#22126](https://ci-apps.yunohost.org/ci/job/22126) :(
[09:17:30]
<tituspijean> > <@Alekswag:matrix.org> Nextcloud be like, let me sing you the song of my people:
I always see this kind of logs like a musical canon:
```
I'm downlooooooaaaaaaading
I'm downlooooooaaaaaaading
Hear me I'm downlooooooaaaaaaading
```
[09:46:56]
<hook> ❤️🔥
[15:49:28]
<Yunohost Git/Infra notifications> [example_ynh] @Salamandar [commented](https://github.com/YunoHost/example_ynh/pull/226#issuecomment-1886710593) on [issue #226](https://github.com/YunoHost/example_ynh/pull/226) Rework upgrade, remove the if upgrade_type == UPGRADE_APP test.: @zamentur I reverted the use of upgrade_type=(ynh_check_app_version_changed), just commented this time.
[15:53:06]
<Yunohost Git/Infra notifications> [example_ynh] @Salamandar [commented](https://github.com/YunoHost/example_ynh/pull/226#discussion_r1448554388) on pull request #226 Rework upgrade, remove the if upgrade_type == UPGRADE_APP test.: Now its commented.
[15:59:17]
<Yunohost Git/Infra notifications> [example_ynh] @alexAubin approved [pull request #226](https://github.com/YunoHost/example_ynh/pull/226#pullrequestreview-1815553107) Rework upgrade, remove the if upgrade_type == UPGRADE_APP test.
[16:01:02]
<Yunohost Git/Infra notifications> [example_ynh] @alexAubin merged [pull request #226](https://github.com/YunoHost/example_ynh/pull/226): Rework upgrade, remove the if upgrade_type == UPGRADE_APP test.
[16:02:30]
<Yunohost Git/Infra notifications> [example_ynh] @alexAubin pushed 2 commits to master ([bbbc1df6dda3...baab7fbd761e](https://github.com/YunoHost/example_ynh/compare/bbbc1df6dda3...baab7fbd761e))
[16:03:45]
<Yunohost Git/Infra notifications> [example_ynh/master] Rework upgrade, remove the if upgrade_type == UPGRADE_APP test. Indeed, even if the upgrade is only a ynh revision,... - Félix Piédallu
[16:05:21]
<Yunohost Git/Infra notifications> [example_ynh/master] Merge pull request #226 from Salamandar/remove_if_upgrade_type Rework upgrade, remove the if upgrade_type == UPGRADE_A... - Alexandre Aubin
[16:06:12]
<Émy - OniriCorpe> can someone reopen this issue please? :/ https://github.com/YunoHost-Apps/synapse_ynh/issues/412
[17:36:47]
<Aleks (he/him/il/lui)> done
[17:51:09]
<eric_G> I don't think we are going to get that much more testing on Nextcloud...
[18:01:38]
<kayou> I forgot to give feedback, everything is working on my side too (with the patch on nginx)
[18:01:41]
<Aleks (he/him/il/lui)> https://aria.im/_matrix/media/v1/download/matrix.org/ChSqihjHsQLAKcaxGhQZaKJQ
[18:02:19]
<lapineige> I can do it if you wish 😄
[18:04:21]
<eric_G> I hope the support room is ready 😬
[18:08:57]
<Aleks (he/him/il/lui)> (i think)
[18:09:31]
<Aleks (he/him/il/lui)> maybe it worked for you when you ran it on your machine because it was not using v2 packaging at that time
[18:10:04]
<Aleks (he/him/il/lui)> hence the apt dependency update happened later
[18:10:35]
<Aleks (he/him/il/lui)> i'm not sure
[18:10:46]
<Aleks (he/him/il/lui)> or maybe both dependencies were there for some reason
[18:11:17]
<eric_G> Does that mean that we need different version of `php-mysql` according to what version we are updating from? Or we just need to add , php8.2-mysql as a library to be installed
[18:11:23]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to v2_PostgreSQL: Update manifest.toml ([6ebebe88](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/6ebebe88a60ff84ee5f0ae65e84841bf836cb9ec))
[18:11:35]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597#issuecomment-1887688158) on [issue #597](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597) V2 postgresql: :v:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/12585/badge)](https:/...
[18:12:00]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597#issuecomment-1887688052) on [issue #597](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597) V2 postgresql: testme
[18:14:08]
<Aleks (he/him/il/lui)> > <@ericg:matrix.org> Does that mean that we need different version of `php-mysql` according to what version we are updating from? Or we just need to add , php8.2-mysql as a library to be installed
i think we just need to temporarily make sure that phpX.Y-mysql is there before we start running the migration
[18:14:19]
<Aleks (he/him/il/lui)> also i'm confused with the fact that older versions of nextcloud need older versions of php dependencies
[18:15:36]
<Aleks (he/him/il/lui)> ah but nvm that's handled by the exec\_occ command/wrapper
[18:25:28]
<Aleks (he/him/il/lui)> (apparently with the merge of nextcloud v2 we precisely reached 70% apps v2 :O)
[18:34:51]
<Émy - OniriCorpe> > <@ericg:matrix.org> 📞 YunoHost hotline, how can I help you?
How can I not help you please ?
[18:59:25]
<Aleks (he/him/il/lui)> > Does that implies building it again when it's only a package update ? Not doing it could save time
one the one hand : yes, on the other hand : it has always been just premature optimization and as salamandar points out, a "package update" could precisely be about fixing an issue related to the build
[18:59:31]
<Yunohost Git/Infra notifications> App redirect rises from level 6 to 8 in job [#22129](https://ci-apps.yunohost.org/ci/job/22129) !
[19:24:20]
<eric_G> https://ci-apps-dev.yunohost.org/ci/job/12585 🎉
[19:24:54]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to testing: Merge pull request #622 from YunoHost-Apps/testing Testing / switch to packaging V2 + proper mail config ([3e60bacf](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/3e60bacfa5f4f3eaed7962a06a3880ffd6f2117e))
[19:25:35]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 2 commits to v2_PostgreSQL ([6ebebe88a60f...8a6b529e0618](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/6ebebe88a60f...8a6b529e0618))
[19:26:53]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2_PostgreSQL] Merge pull request #622 from YunoHost-Apps/testing Testing / switch to packaging V2 + proper mail config - eric_G
[19:26:54]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2_PostgreSQL] Merge branch testing into v2_PostgreSQL - Éric Gaspar
[19:27:26]
<kayou> > <@Alekswag:matrix.org> maybe it worked for you when you ran it on your machine because it was not using v2 packaging at that time
or because another app requires phpX.Y-mysql
[19:28:11]
<eric_G> And now we have the tests passing https://ci-apps-dev.yunohost.org/ci/job/12585
[19:28:19]
<eric_G> ready to merge 😬
[19:30:48]
<lapineige> > <@Alekswag:matrix.org> (apparently with the merge of nextcloud v2 we precisely reached 70% apps v2 :O)
It was a perfect merge, that's a proof.
[19:32:40]
<lapineige> > <@ericg:matrix.org> ready to merge 😬
I would wait at least one month to make sure the current version is fine
[20:56:13]
<kayou> > <@ericg:matrix.org> 📞 YunoHost hotline, how can I help you?
Did you turn on your computer?
[20:56:13]
<eric_G> 📞 YunoHost hotline, how can I help you?
[20:56:13]
<eric_G> > <@kayou:matrix.org> Did you turn on your computer?
First thing first 😂
[20:57:07]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin approved [pull request #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#pullrequestreview-1816146176) Testing / switch to packaging V2 + proper mail config
[20:57:07]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar merged [pull request #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622): Testing / switch to packaging V2 + proper mail config
[20:57:07]
<kayou> 🫣
[20:57:07]
<Yunohost Git/Infra notifications> [nextcloud_ynh/master] Update ADMIN.md - Alexandre Aubin
[20:57:07]
<eric_G> `At least 1 approving review is required to merge this pull request. ` 👈️
[20:57:07]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 242 commits to master ([9c6d1eea8e0a...3e60bacfa5f4](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/9c6d1eea8e0a...3e60bacfa5f4))
[20:57:07]
<Aleks (he/him/il/lui)> izi
[20:57:36]
<kayou> Next step psql?
[20:57:36]
<eric_G> FYI, I am off for the week
[20:57:36]
<Aleks (he/him/il/lui)> #PostgresqlAllTheApps
[20:57:37]
<eric_G> > <@ericg:matrix.org> FYI, I am off for the week
(at least pretending) 😶🌫️
[20:58:15]
<eric_G> Aleks (he/him/il/lui): I don't get the issue with `phpX.Y-mysql/psql` dependency on Nextcloud
[21:02:34]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to v2_PostgreSQL: Update upgrade ([ab68a3a2](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/ab68a3a2eb7dc5272415c6ca3178832ee5cd7656))
[21:19:59]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to v2_PostgreSQL: cleaning ([d7568ca2](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/d7568ca230cf4109ad00e9e66d9ab779159b66d8))
[21:20:28]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/593#issuecomment-1887943291) on [issue #593](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/593) Update install to avoid previewgenerator error: Closing with #622
[21:20:28]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597#issuecomment-1887940447) on [issue #597](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/597) V2 postgresql: testme
[21:20:46]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar closed [pull request #593](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/593): Update install to avoid previewgenerator error
[21:20:46]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 42 commits to enh-configpanel ([20b5c886b109...b28bb0dff1f2](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/20b5c886b109...b28bb0dff1f2))
[21:21:50]
<Yunohost Git/Infra notifications> [nextcloud_ynh/enh-configpanel] Update ADMIN_fr.md - Alexandre Aubin
[21:23:21]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 17 commits to enh-faster-permissions-setter ([0eace9945be3...be7370e5c744](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/0eace9945be3...be7370e5c744))
[21:23:21]
<Yunohost Git/Infra notifications> [nextcloud_ynh/enh-faster-permissions-setter] Update ADMIN_fr.md - Alexandre Aubin
[21:23:47]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/626#issuecomment-1887962337) on [issue #626](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/626) "api url" didnt changed when moving domain: This should be fixed with #622
[21:23:47]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar closed [issue #572](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/572): Deleting a user from YNH does not remove completely from Nextcloud DB, causing issues with Nextcloud Mail
[21:23:48]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/616#issuecomment-1887963996) on [issue #616](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/616) [enh] Cache control on mjs files: This should be fixed with #622
[21:23:48]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/572#issuecomment-1887967578) on [issue #572](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/572) Deleting a user from YNH does not remove completely from Nextcloud DB, causing issues with Nextcloud Mail: should be fixed with #578
[21:24:27]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar closed [issue #546](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/546): Nextcloud may support 32bits for >=v26
[21:25:14]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/546#issuecomment-1887968782) on [issue #546](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/546) Nextcloud may support 32bits for >=v26: 32bit seems to be supported... closing
[21:25:51]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/449#issuecomment-1887972900) on [issue #449](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/449) Upgrading from 21 to 22 hangs: I this still an issue?
[21:25:51]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/540#issuecomment-1887971502) on [issue #540](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/540) Create an occ script to easely launch occ command: Command to use is now displayed in ADMIN.md page: https://github.com/YunoHost-Apps/nextcloud_ynh/blob/master/doc/ADMIN...
[21:28:12]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to v2_PostgreSQL: Create PRE_UPGRADE.md ([4e9a1aa1](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/4e9a1aa1a8217c5b7494cad6e7ec26ca93d453e0))
[21:28:13]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to v2_PostgreSQL: Update PRE_UPGRADE.md ([7394835d](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/7394835d4c0528974714ef86bc3531fa787d550e))
[21:29:58]
<tituspijean> I'm pretty proud of this commit's title: https://github.com/YunoHost-Apps/ghost_ynh/pull/297/commits/ce63f5591ea691c22fa22544cf8367b4a9aadbb5
[21:33:15]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar closed [issue #343](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/343): Improve Talk with local TURN/STUN
[21:47:54]
<lapineige> I tried the last release of the nextcloud app...
`./upgrade: line 298: upgrade.d/upgrade.28.sh: No such file or directory` in 3 servers
[21:48:01]
<lapineige> It worked a day ago 🤔
[21:48:19]
<lapineige> What happened 🤔
[21:50:08]
<eric_G> ah some failures! I was starting to get weird
[21:56:44]
<lapineige> maybe it's because I did the upgrade before
[21:56:56]
<lapineige> https://paste.yunohost.org/raw/igahulatov
[21:57:49]
<Aleks (he/him/il/lui)> `YNH_APP_MANIFEST_VERSION: 27.1.4~ynh1`
[21:58:16]
<Aleks (he/him/il/lui)> 🤔
[21:59:06]
<Aleks (he/him/il/lui)> are you trying to downgrade the app maybe
[22:01:51]
<tituspijean> 28 < 27, 🐰 is trying to break the maths
[22:02:18]
<eric_G> thorough testing
[22:10:46]
<lapineige> but... i'm forcing the upgrade to master ??
[22:10:56]
<lapineige> Wasn't it merged ?
[22:12:31]
<tituspijean> current manifest version is indeed v28. Are you sure your catalog is up to date ?
[22:12:31]
<eric_G> it is not yet propose as upgrade somehow...
[22:13:02]
<eric_G> That is why nobody complains about Nextcloud yet
[22:13:03]
<tituspijean> my server displayed the upgrade though 😲
[22:14:35]
<Aleks (he/him/il/lui)> Maybe if you run it from cli the caralog aint implicitly updated
[22:14:35]
<eric_G> ah no I am updated already 🙄
[22:15:12]
<Aleks (he/him/il/lui)> Only the webadmin re-fetches the catalogue when you reach to the update interface
[22:15:57]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/107#issuecomment-1888055266) on [issue #107](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/107) Switch to PostgreSQL database?: #597 should be functional.
[22:20:16]
<lapineige> > <@titus:pijean.ovh> current manifest version is indeed v28. Are you sure your catalog is up to date ?
I ran the upgrade command in CLI, but not with the url
[22:20:41]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar closed [issue #448](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/448): Cant change app url
[22:20:41]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/448#issuecomment-1888061165) on [issue #448](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/448) Cant change app url: Closing with #622. Please reopen if needed
[22:21:18]
<lapineige> > <@Alekswag:matrix.org> Only the webadmin re-fetches the catalogue when you reach to the update interface
So I did : `yunohost tools update`
[22:21:19]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar closed [issue #561](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/561): Nexcloud install fails: Username is invalid because files already exist for this user
[22:22:34]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/626#issuecomment-1887962337) on [issue #626](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/626) "api url" didnt changed when moving domain: This should be fixed with #622. Please confirm
[22:28:13]
<Yunohost Git/Infra notifications> @ericgaspar edited repository nextcloud_ynh: Nextcloud package for YunoHost https://github.com/YunoHost-Apps/nextcloud_ynh
[22:31:30]
<eric_G> https://nextcloud.com/blog/kiteworks-acquires-owncloud-dracoon/ 🙄
[22:33:21]
<lapineige> > <@Alekswag:matrix.org> Only the webadmin re-fetches the catalogue when you reach to the update interface
Good to know. And an issue, don't you think so ?
[22:33:21]
<lapineige> > So I did : `yunohost tools update`
and it did it
[22:56:24]
<Aleks (he/him/il/lui)> ¯\_(ツ)_/¯ i guess
[22:56:24]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/638#discussion_r1449482246) on pull request #638 Nextcloud parameters in config panel: suggestion help = "Allow to redirect on a custom form to require a password reset (for example to ask you to ...
[22:56:24]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar edited review [pull request #638](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/638#pullrequestreview-1816939742): Nextcloud parameters in config panel
[22:56:24]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/638#discussion_r1449483993) on pull request #638 Nextcloud parameters in config panel: suggestion ask = "To configure the default files created for new users, you can do it by logging on Nextcloud...
[22:56:24]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar changes_requested [pull request #638](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/638#pullrequestreview-1816939742) Nextcloud parameters in config panel
[22:56:24]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/638#discussion_r1449485958) on pull request #638 Nextcloud parameters in config panel: suggestion cp -a --no-target-directory install_dir/core/skeleton "data_dir/skeleton/default"
[22:56:24]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/638#discussion_r1449485847) on pull request #638 Nextcloud parameters in config panel: suggestion create_external_storage "data_dir/skeleton" "New users default files" "admins"
[22:56:32]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/638#discussion_r1449486821) on pull request #638 Nextcloud parameters in config panel: suggestion "skeletondirectory": "__DATA_DIR__/skeleton/{lang}",
[22:56:32]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/638#issuecomment-1888090824) on [issue #638](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/638) Nextcloud parameters in config panel: testme
[22:56:32]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/638#issuecomment-1888090870) on [issue #638](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/638) Nextcloud parameters in config panel: Alrighty
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/12588/badge)](h...
[22:56:32]
<Aleks (he/him/il/lui)> i think there was a ticket in the issue tracker at some point like "refresh catalog before app operations" or something
[22:57:00]
<lapineige> Especially if we expect it to do it, as in web UI
[22:57:56]
<Aleks (he/him/il/lui)> weeeell yes but eeeeeeeh in the webadmin you start by "entering the upgrade interface" which triggers the tools update thing zblblbl
[22:58:18]
<Aleks (he/him/il/lui)> the real equivalent would be if we had an upgrade button in the app info page, which we probably should have
[22:59:12]
<tituspijean> > <@ericg:matrix.org> https://nextcloud.com/blog/kiteworks-acquires-owncloud-dracoon/ 🙄
I had to use Kiteworks at some point for work, and it always felt like a bad owncloud alternative. Looks like Kiteworks end up thinking the same 😆
[22:59:19]
<Aleks (he/him/il/lui)> but in the that case it would not have advertise an upgrade because the version of the cache was still lower than the current install, i would argue the real issue is about yolo-using --force 😬
[23:01:03]
<tituspijean> > <@titus:pijean.ovh> I had to use Kiteworks at some point for work, and it always felt like a bad owncloud alternative. Looks like Kiteworks end up thinking the same 😆
correction, they did *not* feel the same
> It seems that the decision is made to move ownCloud customers over to Kiteworks:
[23:02:30]
<Aleks (he/him/il/lui)> > <@Alekswag:matrix.org> but in the that case it would not have advertise an upgrade because the version of the cache was still lower than the current install, i would argue the real issue is about yolo-using --force 😬
but tl;dr yeah we could probably auto-refresh the app cache if it's older than 12 hours idk
[23:02:33]
<lapineige> > <@Alekswag:matrix.org> but in the that case it would not have advertise an upgrade because the version of the cache was still lower than the current install, i would argue the real issue is about yolo-using --force 😬
It would not upgrade even to the updated version because it was the same version number, as I upgraded to the PR instead. Hence the --force
[23:03:18]
<lapineige> > <@titus:pijean.ovh> correction, they did *not* feel the same
> > It seems that the decision is made to move ownCloud customers over to Kiteworks:
And here is how ownCloud died
[23:03:31]
<lapineige> I guess we can archive the repository
[23:03:41]
<Aleks (he/him/il/lui)> wut
[23:05:03]
<Aleks (he/him/il/lui)> but i though owncloud was supposed to be nextcloud-but-actually-focused-on-file-sync-rather-than-doing-everything :'(
[23:05:03]
<Aleks (he/him/il/lui)> https://www.youtube.com/watch?v=h7khqQCC2XM
[23:05:03]
<lapineige> > <@Alekswag:matrix.org> weeeell yes but eeeeeeeh in the webadmin you start by "entering the upgrade interface" which triggers the tools update thing zblblbl
yeah but you basically go to the screen where you can make an upgrade, you're not aware that it checks for new updates in the background
[23:06:52]
<lapineige> I don't really see why it would not be the case in CLI
[23:07:21]
<lapineige> Basically why would you avoid updating the app catalog before triggering an upgrade ?
(on user perspective, because otherwise we save server load/bandwith)
[23:10:21]
<Aleks (he/him/il/lui)> > Basically why would you avoid updating the app catalog before triggering an upgrade ?
> (on user perspective, because otherwise we save server load/bandwith)
because `yunohost tools upgrade apps --all` (or whatever the real syntax is) upgrade every app, calling in turn `app_upgrade()`, and if every call to `app_upgrade()` refreshes the catalog everytime it's called, you would be refreshing the catalog even when it was refreshed like 2 minutes ago so you end up having to implement cache management so it's less trivial and what happens if actually a new version of the app is released between the time the available app upgrade is displayed and the moment the actual call to `app_upgrade()` happens aaaaaaaaa
[23:10:29]
<Aleks (he/him/il/lui)> so yeah it's doable but not straightforward 😬
[23:12:01]
<lapineige> > what happens if actually a new version of the app is released between the time the available app upgrade is displayed and the moment the actual call to app\_upgrade() happens aaaaaaaaa
Just as now, nothing ? (if cache is long enough)
And in worst case... you have a fresher update ?
[23:13:13]
<lapineige> > because yunohost tools upgrade apps --all (or whatever the real syntax is) upgrade every app, calling in turn app\_upgrade(), and if every call to app\_upgrade() refreshes the catalog everytime it's called, you would be refreshing the catalog even when it was refreshed like 2 minutes ago so you end up having to implement cache management
Or call the command once at the beggining (of yunohost tool upgrade, not "app_upgrade()"), no matter the amount of apps to upgrade ?
[23:14:14]
<Aleks (he/him/il/lui)> idk this is like calling `apt update` automatically every time you run `apt install` or `apt dist-upgrade` , you could think it's nice but in practice it really ends up being annoying x_x