[07:26:41]
<Yunohost Git/Infra notifications> Application tracim completely failed the continuous integration tests on https://ci-apps.yunohost.org/ci/job/13469
[08:51:10]
<Yunohost Git/Infra notifications> [package_check] @gcollin [commented](https://github.com/YunoHost/package_check/pull/138#issuecomment-1415381517) on [issue #138](https://github.com/YunoHost/package_check/pull/138) Added optional support for ramfs: @alexAubin whenever you have time to check this one.
[09:16:12]
<eric_G[m]> Is this still needed in `nginx.conf`? :
```
# Include SSOWAT user panel.
include conf.d/yunohost_panel.conf.inc;
```
[10:29:56]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @zamentur pushed 6 commits to testing ([79ac36bc280f...56cc5a2983a5](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/79ac36bc280f...56cc5a2983a5))
[10:29:57]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @zamentur merged [pull request #526](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/526): Fix upgrade of old versions <=v21 and for v22 to v23
[10:30:00]
<Yunohost Git/Infra notifications> [nextcloud_ynh/testing] Fix v22 to v23 upgrade - AlexF
[10:30:04]
<Yunohost Git/Infra notifications> [nextcloud_ynh/testing] [fix] PHP version to recommended value - ljf (zamentur)
[10:30:13]
<Yunohost Git/Infra notifications> [nextcloud_ynh/testing] [fix] PHP version during upgrade - ljf (zamentur)
[10:31:02]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar opened [pull request #555](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/555): Testing
[10:31:34]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @zamentur [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/535#issuecomment-1415650792) on [issue #535](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/535) opcache.interned_strings_buffer higher 8: Finally 16Mb is not enough... On 32GB server i have a setup that put the same warning with 16MB...
[10:39:29]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar edited [pull request #555](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/555): Testing
[10:39:46]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar edited [pull request #555](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/555): Testing
[11:03:49]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/535#issuecomment-1415714106) on [issue #535](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/535) opcache.interned_strings_buffer higher 8: shall we increase the value to 32?
[12:00:56]
<eric_G[m]> Why do we need `ynh_systemd_action --service_name=nginx --action=reload` on `restore` script but not in the `install` nor in the `upgrade` script?
[12:13:56]
<Yunohost Git/Infra notifications> [example_ynh] @ericgaspar opened [pull request #200](https://github.com/YunoHost/example_ynh/pull/200): Add dummies
[12:18:42]
<Aleks (he/him/il/lui)> > <@ericg:matrix.org> Why do we need `ynh_systemd_action --service_name=nginx --action=reload` on `restore` script but not in the `install` nor in the `upgrade` script?
because during install/upgrade, ynh_add_nginx_config takes care of it ... during restore we instead restore stuff using the full path of the file instead of just calling ynh_add_nginx_config (and therefore not backing up the nginx config during backup) ... which imho should be The Way™ ... but i think the intention is to cover cases where admins edited their conf themselves T_T
[12:30:10]
<eric_G[m]> How can I set 3 ports in `manifest.toml` v2?
[12:33:20]
<Aleks (he/him/il/lui)> cf https://yunohost.org/fr/packaging_apps_resources#example-4
[12:35:04]
<Aleks (he/him/il/lui)> like
```toml
[resources.port]
main.default = 1234 # corresponds to $port
foo.default = 2345 # corresponds to $port_foo
bar.default = 3456 # corresponds to $port_bar
```
[12:35:55]
<Aleks (he/him/il/lui)> though the `default` value is not really needed, as it would pick a random value instead (and usually this is just for internal reverse proxy) but at least this allows to create the `foo` and `bar` keys ;P
[12:37:59]
<Aleks (he/him/il/lui)> but if you need a specific value you should use `foo.fixed = true` and if you need it to be exposed on the firewall, you'll want `foo.exposed = 'TCP'`
[12:39:16]
<eric_G[m]> for Galène I have set this to:
```
[resources.port]
main.default = 8095
ldap_port.default = 8096
turn_port.default = 1194
turn_port.exposed = "Both"
```
[12:40:01]
<Aleks (he/him/il/lui)> no need to call them `ldap_port`, just `ldap`, otherwise the setting will be `$port_ldap_port`
[12:44:21]
<eric_G[m]> OK. Now I have:
```
[resources.port]
main.default = 8095
ldap.default = 8096
turn.default = 1194
turn.exposed = "Both"
```
[12:45:30]
<eric_G[m]> and i am using `$port_ldap`and `$port_turn\`
[12:46:40]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @lapineige [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/535#issuecomment-1415824968) on [issue #535](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/535) opcache.interned_strings_buffer higher 8: @zamentur you might 32bits ?
[13:07:06]
<Aleks (he/him/il/lui)> >Be warned, if you want to continue playing 20euro_ynh game, now you need to update to YunoHost 11.1.2
xD
[13:11:11]
<Yunohost Git/Infra notifications> Application 20euros rises from level (unknown) to level 8 on https://ci-apps.yunohost.org/ci/job/13472
[13:15:39]
<eric_G[m]> ^Boum
[13:15:40]
<tituspijean> > <@Alekswag:matrix.org> >Be warned, if you want to continue playing 20euro_ynh game, now you need to update to YunoHost 11.1.2
>
> xD
Best incentive, ever.
[13:16:18]
<eric_G[m]> wait for the next one... Galène
[13:56:13]
<eric_G[m]> I don't understand what's going on with this failed test? https://ci-apps-dev.yunohost.org/ci/job/5468
[14:02:00]
<tituspijean> this, maybe? https://github.com/YunoHost-Apps/galene_ynh/blob/galene_v2/tests.toml#L9-L10
[14:02:12]
<tituspijean> It complains about duplicated keys
[14:02:27]
<Aleks (he/him/il/lui)> yup
[14:02:59]
<eric_G[m]> `exclude ="install.nourl, setup_sub_dir"`
[14:03:22]
<tituspijean> not a list with `[]`?
[14:03:53]
<Aleks (he/him/il/lui)> you probably mean `exclude = ["install.subdir"]` (nourl is not performed anyway because it will automatically understand that this is not a webapp)
[14:04:40]
<Aleks (he/him/il/lui)> i'm adding it to package_check readme though may not be the optimal place
[14:04:40]
<Aleks (he/him/il/lui)> annnd the list of test id is not really documented
[14:07:02]
<Yunohost Git/Infra notifications> [package_check] @alexAubin pushed 1 commit to master: README: add test ids that can be used in exclude/only statement ([9c5e62ad](https://github.com/YunoHost/package_check/commit/9c5e62ad6d08da152f3e3eb5087f6e7510347f0d))
[14:07:03]
<Aleks (he/him/il/lui)> https://github.com/YunoHost/package_check/blob/master/README.md#test-ids
[14:10:09]
<eric_G[m]> Ok, new errors: https://ci-apps-dev.yunohost.org/ci/job/5468
[14:13:29]
<Aleks (he/him/il/lui)> ah, here https://github.com/YunoHost-Apps/galene_ynh/blob/galene_v2/tests.toml#L15 you probably meant : `args.group_name = "stuff"`, `args.group_description = "other stuff"`
[14:15:28]
<Aleks (he/him/il/lui)> doesnt seem to work ? 😅
[14:16:34]
<Aleks (he/him/il/lui)> hmyeah the code is buggy
[14:17:40]
<Aleks (he/him/il/lui)> fixing it
[14:19:54]
<Aleks (he/him/il/lui)> here we go
[14:19:55]
<Yunohost Git/Infra notifications> [package_check] @alexAubin pushed 1 commit to master: tests.toml: fix issue when manifest contains questions with no default ([83d4d8f2](https://github.com/YunoHost/package_check/commit/83d4d8f292a4f2517bf93f91369903cf82bed95a))
[14:20:03]
<Aleks (he/him/il/lui)> but now i really gotta move 😅
[14:28:53]
<Aleks (he/him/il/lui)> okay, seem to sort of be running :P
[14:29:15]
<Aleks (he/him/il/lui)> aaaaaa
[14:30:23]
<eric_G[m]> Package linter is happy
[14:30:30]
<eric_G[m]> but not the rest...
[14:30:40]
<Aleks (he/him/il/lui)> omg it's `ports` and not `port` x_x
[14:30:45]
<Aleks (he/him/il/lui)> for the resource name
[14:30:47]
<Aleks (he/him/il/lui)> typo in the doc :|
[14:31:11]
<Aleks (he/him/il/lui)> https://github.com/YunoHost-Apps/galene_ynh/blob/galene_v2/manifest.toml#L74
[14:32:04]
<eric_G[m]> a nice live debugging session that we have 😅
[14:32:15]
<Aleks (he/him/il/lui)> yah 😅
[14:32:38]
<Aleks (he/him/il/lui)> well it would have been suspicious that everything just works 😅
[14:35:52]
<eric_G[m]> ok the `port`thing if fixed. Now I have `'Create permission 'galene'' could not be completed` but this could be a bug in the script himself.
[14:57:47]
<eric_G[m]> ... and for this app something with deprovisionning apt fails: https://ci-apps-dev.yunohost.org/ci/job/5469
[15:19:41]
<Aleks (he/him/il/lui)> > <@ericg:matrix.org> ok the `port`thing if fixed. Now I have `'Create permission 'galene'' could not be completed` but this could be a bug in the script himself.
ah that's related to the fact that it's a full domain app (no path question)
[15:19:51]
<Aleks (he/him/il/lui)> gotta think about it
[16:09:02]
<Yunohost Git/Infra notifications> While testing calckey: It looks like package_check did not finish properly ... on https://ci-apps.yunohost.org/ci/job/13473
[17:00:32]
<Yunohost Git/Infra notifications> [apps] @yunohost-bot opened [pull request #1628](https://github.com/YunoHost/apps/pull/1628): Update app levels according to CI results
[17:00:32]
<Yunohost Git/Infra notifications> [apps] @yunohost-bot pushed 1 commit to update_app_levels: Update app levels according to CI results ([4d09170e](https://github.com/YunoHost/apps/commit/4d09170e8c1654cac1361cb8d4a15259e2a514b8))
[17:00:34]
<Yunohost Git/Infra notifications> [apps] @yunohost-bot created new branch update_app_levels
[17:53:53]
<Yunohost Git/Infra notifications> Application calckey stays at level 1 on https://ci-apps.yunohost.org/ci/job/13474
[19:55:48]
<Aleks (he/him/il/lui)> > <@ericg:matrix.org> ok the `port`thing if fixed. Now I have `'Create permission 'galene'' could not be completed` but this could be a bug in the script himself.
pushed a fix, rebuilding the app image now (11.1.5.4)
[20:06:41]
<Aleks (he/him/il/lui)> > <@ericg:matrix.org> ... and for this app something with deprovisionning apt fails: https://ci-apps-dev.yunohost.org/ci/job/5469
annnd found the issue for that one too ... the remove_php_conf helpers triggers a removal of apt deps because of historical reasons ... we should probably get rid of this but in the meantime i skip this for appsv2
[20:24:21]
<Aleks (he/him/il/lui)> restarted https://ci-apps-dev.yunohost.org/ci/job/5468 , at least it's going through the actual install now ;P
[20:25:02]
<Aleks (he/him/il/lui)> uuuh somebody rerestarted it /o\
[20:25:24]
<eric_G[m]> Damn, I have restarted 🙄
[20:25:28]
<Aleks (he/him/il/lui)> :D
[20:25:47]
<eric_G[m]> not touching anything
[20:31:39]
<Aleks (he/him/il/lui)> zbfglbf the lock thingy is a nightmare .. i hope to have some time at some point to deploy the new yunorunner/package check version which should fix the issue but i'm at FOSDEM/OFFDEM all weekend
[20:32:49]
<Aleks (he/him/il/lui)> gonna kill the old process in CLI é_è
[20:34:44]
<Aleks (he/him/il/lui)> ah but now there's 3 actual job ongoing 🤔
[20:43:49]
<eric_G[m]> https://aria.im/_matrix/media/v1/download/matrix.org/eiLFIKUrzwzJFVRtzRSasAUf
[20:47:32]
<eric_G[m]> is FOSDEM in Belgium?
[20:50:07]
<tituspijean> yup, in Brussels
[20:51:04]
<tituspijean> Aleks (he/him/il/lui) I'm trying to decipher yunorunner's run.py to no avail: does package_check report the commit it is processing back to yunorunner?
[20:51:47]
<Aleks (he/him/il/lui)> Hmm i dont think so ?
[20:52:20]
<Aleks (he/him/il/lui)> The commit you see at the top of logs is from some package check logic
[20:52:39]
<tituspijean> ah, nor the level when the job is done, you mentioned that yesterday
[20:52:42]
<tituspijean> ok I see
[20:53:08]
<Aleks (he/him/il/lui)> Yeah
[20:53:28]
<Aleks (he/him/il/lui)> But it could be doable to parse the result.json
[21:21:02]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to testing: Update manifest.json ([e1e1be0a](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/e1e1be0a053418cd8a6c3327a27b858c924f27be))
[21:21:02]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot pushed 1 commit to testing: Auto-update README ([795b4492](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/795b4492d688c12bab6b5ac1c4b99961211fe351))
[21:21:08]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/555#issuecomment-1416415337) on [issue #555](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/555) Testing: testme
[21:21:12]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/555#issuecomment-1416415371) on [issue #555](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/555) Testing: :carousel_horse:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci.pijean.ovh/ci/api/job/418/badge)](https:/...
[21:21:18]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/555#issuecomment-1416415510) on [issue #555](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/555) Testing: Fingers crossed
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/5477/bad...
[21:28:26]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to v2: fix ([8fd043f7](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/8fd043f76c95b7219d594cffcb9b9a098f911def))
[21:28:26]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot pushed 1 commit to v2: Auto-update README ([d6f4ccbb](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/d6f4ccbb016d800f2ddb533efdcac65d622cf7f1))
[21:36:03]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 2 commits to v2 ([d6f4ccbb016d...c2d6c0fb0852](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/d6f4ccbb016d...c2d6c0fb0852))
[21:36:06]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2] Fix - Éric Gaspar
[21:36:10]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2] Merge branch v2 of https://github.com/YunoHost-Apps/nextcloud_ynh into v2 - Éric Gaspar
[21:37:03]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar opened [pull request #556](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/556): V2
[21:37:11]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/556#issuecomment-1416427928) on [issue #556](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/556) V2: testme
[21:37:15]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/556#issuecomment-1416427955) on [issue #556](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/556) V2: Fingers crossed
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/5478/bad...
[21:37:15]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/556#issuecomment-1416427956) on [issue #556](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/556) V2: :rocket:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci.pijean.ovh/ci/api/job/419/badge)](https://ci.pije...
[21:38:07]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 55 commits to v2 ([c2d6c0fb0852...3d4c67ec3ffc](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/c2d6c0fb0852...3d4c67ec3ffc))
[21:38:11]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2] Update manifest.json - Éric Gaspar
[21:38:11]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot pushed 1 commit to v2: Auto-update README ([754d3211](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/754d3211b5f30d705b8c0247c006bc41c419843f))
[21:38:13]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2] Auto-update README - yunohost-bot
[21:38:20]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2] Merge branch testing into v2 - Éric Gaspar
[21:50:26]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to testing: Add description ([41dac63c](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/41dac63c3a985b54c4eb96ce37e881a5433a3f73))
[21:50:30]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot pushed 1 commit to testing: Auto-update README ([654f71b6](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/654f71b62b73521f9ceab59bc29d487df9c2de7a))
[21:55:28]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar closed [pull request #556](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/556): V2