[00:19:30]
<Yunohost Git/Infra notifications> App glitchsoc rises from level 6 to 8 in job [#20298](https://ci-apps.yunohost.org/ci/job/20298) !
[00:53:46]
<Yunohost Git/Infra notifications> App halcyon failed all tests in job [#20309](https://ci-apps.yunohost.org/ci/job/20309) :(
[00:59:03]
<Yunohost Git/Infra notifications> App horde failed all tests in job [#20310](https://ci-apps.yunohost.org/ci/job/20310) :(
[01:03:26]
<Yunohost Git/Infra notifications> App garage stays at level 3 in job [#20308](https://ci-apps.yunohost.org/ci/job/20308)
[01:35:10]
<Yunohost Git/Infra notifications> App osjs failed all tests in job [#20313](https://ci-apps.yunohost.org/ci/job/20313) :(
[02:49:48]
<Yunohost Git/Infra notifications> App matrix-appservice-irc failed all tests in job [#20321](https://ci-apps.yunohost.org/ci/job/20321) :(
[05:29:33]
<lapineige> Yes because it was replaced by the official one. My hosting is outdated (I only used it for dev purposes) so it should no longer be used
[07:33:47]
<eric_G> I've been puzzled by this failing test. Any thoughts on it? https://ci-apps.yunohost.org/ci/job/19744. As I recall, the repository has not changed since the previous successful test for Miniflux.
[07:39:00]
<Yunohost Git/Infra notifications> App miniflux stays at level 3 in job [#19744](https://ci-apps.yunohost.org/ci/job/19744)
[07:44:35]
<orhtej2> > <@ericg:matrix.org> I've been puzzled by this failing test. Any thoughts on it? https://ci-apps.yunohost.org/ci/job/19744. As I recall, the repository has not changed since the previous successful test for Miniflux.
Permissions are not set somewhere, likely for logs
[07:44:44]
<orhtej2> in restore compared to install
[07:48:56]
<eric_G> Thanks! I have commented on those lines: https://github.com/YunoHost-Apps/miniflux_ynh/blob/cad1157eed5d2392addbdf8840ced11a51fa37be/conf/systemd.service#L12-L13. Let's see if that fixes the issue.
[08:09:55]
<orhtej2> > <@ericg:matrix.org> Thanks! I have commented on those lines: https://github.com/YunoHost-Apps/miniflux_ynh/blob/cad1157eed5d2392addbdf8840ced11a51fa37be/conf/systemd.service#L12-L13. Let's see if that fixes the issue.
Good for testing but not for merging, right?
[08:10:33]
<eric_G> why not?
[08:11:29]
<orhtej2> Because you're getting rid of logs?
[08:12:04]
<orhtej2> permissions on /var/log are set by ynh_setup_logrotate
[08:12:37]
<orhtej2> which is not part of restore
[09:52:12]
<miro5001> > <@ericg:matrix.org> I've been puzzled by this failing test. Any thoughts on it? https://ci-apps.yunohost.org/ci/job/19744. As I recall, the repository has not changed since the previous successful test for Miniflux.
I had a similar issue with indico services failing fixed by https://github.com/YunoHost-Apps/indico_ynh/blob/103544f2090720dc4341ef384dfe35f1d69b2cf3/scripts/restore#L33
[10:03:04]
<Yunohost Git/Infra notifications> [nextcloud_ynh] rodinux [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/822#issuecomment-3101991566) on [issue #822](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/822) fix settings enable_mail_forward in install and upgrade scripts: @ericgaspar @kay0u do you think is correct like this ?
[10:09:00]
<orhtej2> > <@miro5001:matrix.org> I had a similar issue with indico services failing fixed by https://github.com/YunoHost-Apps/indico_ynh/blob/103544f2090720dc4341ef384dfe35f1d69b2cf3/scripts/restore#L33
A bit different than helpers but workable I guess https://github.com/YunoHost/yunohost/blob/82e804f5c5ce185f0a750ffb2f7e93251ddc6ef4/helpers/helpers.v2.1.d/logrotate#L43
[10:14:28]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u approved [pull request #822](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/822#pullrequestreview-3042279734) fix settings enable_mail_forward in install and upgrade scripts: LGTM anyway
[10:14:29]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u edited review [pull request #822](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/822#pullrequestreview-3042279734): fix settings enable_mail_forward in install and upgrade scripts
[10:14:30]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/822#discussion_r2222027629) on pull request #822 fix settings enable_mail_forward in install and upgrade scripts: This case was an old backward compatible fix, I think there is no need to use it for new booleans. Did you see something...
[10:27:51]
<miro5001> Backup doesn't include log files so restore won't create these files, and on a fresh machine they don't exist. I expected the service to create the $app.log in /var/log/$app, but didn't. So I had to touch logfile for it to work
[10:30:25]
<orhtej2> > <@miro5001:matrix.org> Backup doesn't include log files so restore won't create these files, and on a fresh machine they don't exist. I expected the service to create the $app.log in /var/log/$app, but didn't. So I had to touch logfile for it to work
That's what the linked code does, ~the permissions
[10:37:15]
<miro5001> Yes, I saw it, but permissions were good. The problem was that /var/log/$app/$app-celery.log and /var/log/$app/$app-uwsgi.log were not created. So services complained that these files were not found. All other apps I know create the log file if it doesn't exist.
[10:50:47]
<eric_G> shouldn't the log rights be `chown -R "$app:$app" "/var/log/$app"` instead of `chown -R "$app:www-data" "/var/log/$app"` (however, it may not be the issue)
[10:52:47]
<Aleks (he/him/il/lui)> urgh yes T_T
[10:52:58]
<Aleks (he/him/il/lui)> hmgnnnnn packagingv3333333333333
[10:57:50]
<Yunohost Git/Infra notifications> [nextcloud_ynh] rodinux [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/822#discussion_r2222170775) on pull request #822 fix settings enable_mail_forward in install and upgrade scripts: yesterday we have upgrade an instance with the enable_forward_email activated, but after the upgrade it was desactivated...
[11:19:47]
<Yunohost Git/Infra notifications> App miniflux rises from level 3 to 8 in job [#20335](https://ci-apps.yunohost.org/ci/job/20335) !
[11:44:16]
<eric_G> thanks orhtej2
[11:57:08]
<orhtej2> > <@ericg:matrix.org> thanks orhtej2
Always welcome
[14:51:08]
<Yunohost Git/Infra notifications> App snweb rises from level 0 to 7 in job [#20251](https://ci-apps.yunohost.org/ci/job/20251) !
[18:02:52]
<Yunohost Git/Infra notifications> [nextcloud_ynh] rodinux pushed 1 commit to v30: Update manifest.toml version 30.0.13 v30 ([cf336bba](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/cf336bba12866be57fd4f53b582f711fbde3af46))
[18:13:41]
<Yunohost Git/Infra notifications> [apps] fflorent opened [pull request #3052](https://github.com/YunoHost/apps/pull/3052): Add Docs to the app catalog
[18:17:12]
<florent[m]> ^ Hmm, the CI fails, I guess because I use a branch of my fork. Is this important?
[18:58:04]
<Paprika> Has anyone faced an issue where an app has permission issues after restore during CI tests?
I couldn't find the cause because the restore script does `chown -R` on the `/var/log/$app` directory, yet the app cannot access `/var/log/$app/$app.log`, which is mega weird. As a temporary workaround, I simply went ant opted for `systemd` for logging, but this is the first time I faced this.
[19:05:52]
<tituspijean[m]> It might help if you share which app you are writing about :)
[19:20:29]
<Yunohost Git/Infra notifications> [apps] ericgaspar merged [pull request #3052](https://github.com/YunoHost/apps/pull/3052): Add Docs to the app catalog
[19:20:30]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to main: Add Docs to the app catalog (#3052) ([7fa059e4](https://github.com/YunoHost/apps/commit/7fa059e47b98cc2371279f8b89090faac03fd712))
[19:21:06]
<Paprika> I am packaging `wizarr`, at the moment on github [here](https://github.com/botagas/wizarr_ynh/tree/testing)
[19:26:08]
<Yunohost Git/Infra notifications> [apps_tools] tituspijean [commented](https://github.com/YunoHost/apps_tools/pull/40#discussion_r2223636449) on pull request #40 Add screenshots back to app READMEs: Do the _ need to be escaped? Im getting this when trying to run the script manually: (venv) root@pijean:/home/ti...
[19:30:21]
<tituspijean[m]> Can you share the CI logs on the affected commits? (It's great to use your own CI though :D)
[19:32:03]
<Paprika> Actually, I recently deleted those runs with ciclic, but I can do another CI run on the main branch to get the logs for you actually, it will just take me some time before it finishes the tests
[19:34:19]
<Yunohost Git/Infra notifications> [apps_tools] alexAubin pushed 1 commit to readd-screenshots: Update README.md.j2: no _() needed ([f38cb876](https://github.com/YunoHost/apps_tools/commit/f38cb87686fc35c2634197369b8e7d444acc2621))
[19:35:08]
<Yunohost Git/Infra notifications> [apps_tools] tituspijean approved [pull request #40](https://github.com/YunoHost/apps_tools/pull/40#pullrequestreview-3044605339) Add screenshots back to app READMEs
[19:35:32]
<Yunohost Git/Infra notifications> [apps_tools] tituspijean merged [pull request #40](https://github.com/YunoHost/apps_tools/pull/40): Add screenshots back to app READMEs
[19:35:33]
<Yunohost Git/Infra notifications> [apps_tools] tituspijean pushed 1 commit to main: Add screenshots back to app READMEs (#40) ([e642bae5](https://github.com/YunoHost/apps_tools/commit/e642bae5f2636a56d348f59fb2c6b0e864ceeff8))
[19:37:12]
<Yunohost Git/Infra notifications> [apps_tools] alexAubin pushed 1 commit to main: Fix readme generator tests ? ([48adbaeb](https://github.com/YunoHost/apps_tools/commit/48adbaebf3b53622c330c68684dc9469de6edea1))
[19:46:14]
<Yunohost Git/Infra notifications> [apps_tools] alexAubin [commented](https://github.com/YunoHost/apps_tools/pull/42#discussion_r2223686822) on pull request #42 Add changelog bot command: suggestion changelog = re.search(f"{command}\s*(.*)", fullbody, re.DOTALL).group(1).rstrip()
[19:46:57]
<Yunohost Git/Infra notifications> [apps_tools] alexAubin [commented](https://github.com/YunoHost/apps_tools/pull/42#discussion_r2223687891) on pull request #42 Add changelog bot command: eeh wut :thinking:
[19:47:16]
<Yunohost Git/Infra notifications> [apps_tools] tituspijean pushed 1 commit to changelog: Handle the case where we directly go to a newline after changelog Co-authored-by: Alexandre Aubin <4533074+alexAubin@u... ([d21ae199](https://github.com/YunoHost/apps_tools/commit/d21ae1997f83980812bb5e24e17ee23451682854))
[19:47:24]
<Yunohost Git/Infra notifications> [apps_tools] alexAubin [commented](https://github.com/YunoHost/apps_tools/pull/42#discussion_r2223689196) on pull request #42 Add changelog bot command: suggestion author=Actor("yunohost-bot", "yunohost@yunohost.org"),
[19:47:56]
<Yunohost Git/Infra notifications> [apps_tools] tituspijean [commented](https://github.com/YunoHost/apps_tools/pull/42#discussion_r2223690639) on pull request #42 Add changelog bot command: Bad copy-paste is bad.
[19:48:05]
<Yunohost Git/Infra notifications> [apps_tools] tituspijean [commented](https://github.com/YunoHost/apps_tools/pull/42#discussion_r2223691102) on pull request #42 Add changelog bot command: suggestion
[19:48:12]
<Yunohost Git/Infra notifications> [apps_tools] tituspijean pushed 1 commit to changelog: Ooops ([7081a8c1](https://github.com/YunoHost/apps_tools/commit/7081a8c1728754ca7d1dd20b40df746ed8492d20))
[19:48:29]
<Yunohost Git/Infra notifications> [apps_tools] tituspijean pushed 1 commit to changelog: Ooops² Co-authored-by: Alexandre Aubin <4533074+alexAubin@users.noreply.github.com> ([f94136a1](https://github.com/YunoHost/apps_tools/commit/f94136a14e902e9f6cdfd7db0b3da16bf974674e))
[19:52:07]
<Yunohost Git/Infra notifications> [nextcloud_ynh] rodinux [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/822#issuecomment-3104638817) on [issue #822](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/822) fix settings enable_mail_forward in install and upgrade scripts: well, after testing upgrading with these I have the same issue, why ??
[19:55:27]
<Yunohost Git/Infra notifications> [apps_tools] alexAubin pushed 1 commit to changelog: Update webhook.py: save indent level + fix return values that were not actually used ([2056a081](https://github.com/YunoHost/apps_tools/commit/2056a081723dc9cd3c99110d6fec333251d9fc69))
[19:59:17]
<Yunohost Git/Infra notifications> [nextcloud_ynh] rodinux [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/822#issuecomment-3104638817) on [issue #822](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/822) fix settings enable_mail_forward in install and upgrade scripts: well, after testing upgrading with these I have the same issue, why ?? https://paste.yunohost.org/raw/kajirosegu
[20:10:20]
<Yunohost Git/Infra notifications> [nextcloud_ynh] alexAubin [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/822#discussion_r2223731274) on pull request #822 fix settings enable_mail_forward in install and upgrade scripts: suggestion ldapEmail=(exec_occ ldap:show-config "" | grep ldapEmailAttribute) if [ "(echo ldapEmail | grep -Po "ma...
[20:40:20]
<miro5001> I have introduced mail_user to manifest :
```
[install.mail_user]
ask.en = "Choose the application mail system"
ask.fr = "Choisissez l'email système de l'application"
type = "select"
choices.noreply="noreply@"
choices.contact="contact@"
default = "noreply"
```
So I added to upgrade script :
```
ynh_app_setting_set_default --key=mail_user --value=noreply
```
I get this error in the upgrade run on ci :
```
19580 WARNING /usr/share/yunohost/helpers.v2.1.d/setting: line 88: mail_user: invalid variable name
```
What am I doing wrong?
Here is the PR https://github.com/YunoHost-Apps/ojs_ynh/pull/18
[20:41:09]
<miro5001> https://ci-apps-dev.yunohost.org/ci/job/11349
[21:03:32]
<Salamandar> your server is ko
[21:03:38]
<Salamandar> omg people hosting the ci on their machine
[21:10:43]
<Paprika> ty for the headsup, all good now
[21:11:10]
<Paprika> didn't realize I had static assets blocked on cf
[21:11:59]
<Paprika> Just for you CI logs: https://yunorunner.tiesiog.lt/job/2
[21:12:13]
<Paprika> Just for you [CI logs](https://yunorunner.tiesiog.lt/job/2)
[21:12:30]
<Paprika> Just for you [CI logs](https://yunorunner.tiesiog.lt/job/1)
[21:12:35]
<Paprika> Just for you [CI logs](https://yunorunner.tiesiog.lt/job/2)
[21:19:31]
<orhtej2> > <@botagiuks:tiesiog.lt> Just for you: https://yunorunner.tiesiog.lt/job/2
I mean it states Logs are inaccessible, not sure if parent dir shouldn't be 750?
[21:21:43]
<orhtej2> Ah this doesn't use logrotate
[21:23:09]
<orhtej2> @Paprika where is gunicorn conf supposed to come from? It's shipped with sources?
[21:23:21]
<Yunohost Git/Infra notifications> [apps_tools] tituspijean [commented](https://github.com/YunoHost/apps_tools/pull/42#issuecomment-3104873897) on [issue #42](https://github.com/YunoHost/apps_tools/pull/42) Add changelog bot command: Lets go
[21:23:29]
<Yunohost Git/Infra notifications> [apps_tools] tituspijean merged [pull request #42](https://github.com/YunoHost/apps_tools/pull/42): Add changelog bot command
[21:25:21]
<Yunohost Git/Infra notifications> [apps_tools] tituspijean deleted branch changelog
[21:26:02]
<orhtej2> > <@yunohostinfra:matrix.org> [apps_tools] tituspijean deleted branch changelog
How often does the bot restart?
[21:27:01]
<Yunohost Git/Infra notifications> [nextcloud_ynh] rodinux pushed 1 commit to adjust_mail_forward: try fix ldap_email-attribute ([698abea5](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/698abea509c20281455da7bb7fc03f259fe5e2ff))
[21:34:19]
<Paprika> Shipped with source, and I change it on install with `cat`.
[21:34:41]
<Paprika> I may need some more education on that regard
[21:35:57]
<Paprika> It's super useful for testing when you just want to go to sleep 🤣
[21:41:12]
<Yunohost Git/Infra notifications> [nextcloud_ynh] rodinux [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/822#discussion_r2223884100) on pull request #822 fix settings enable_mail_forward in install and upgrade scripts: Ok, thanks a lot. first test with an upgrade was OK, not sure if I put the code on the correct place...
[21:53:44]
<tituspijean[m]> do not test, I made a mistake in the code :)
[21:54:59]
<tituspijean[m]> (it has already been restarted, but I was assuming we were using PyGithub, while the webhooks use GitPython. I'll fix that later this week)
[22:07:43]
<miro5001> I ended up using :
```
if [[ "$mail_user" != "noreply" && "$mail_user" != "contact" ]]; then
mail_user="noreply"
fi
```
And the ci test passed. Tomorrow real-world test
[22:44:47]
<Yunohost Git/Infra notifications> [nextcloud_ynh] rodinux pushed 1 commit to adjust_mail_forward: unused value settings ([6909349b](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/6909349b5c1fa0ee4be48b4dea9d913261742fa0))
[22:52:16]
<Yunohost Git/Infra notifications> [nextcloud_ynh] rodinux [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/822#discussion_r2223973748) on pull request #822 fix settings enable_mail_forward in install and upgrade scripts: ok you are right, not needed
[22:59:25]
<Yunohost Git/Infra notifications> [nextcloud_ynh] rodinux [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/822#issuecomment-3105066299) on [issue #822](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/822) fix settings enable_mail_forward in install and upgrade scripts: ok its fine like this, I tested it... excepting, I am not sure, if sometimes when enabling enable_forward_email a new...
[23:31:51]
<Yunohost Git/Infra notifications> App django-fritzconnection goes down from level 6 to 3 in job [#20342](https://ci-apps.yunohost.org/ci/job/20342)
[23:33:59]
<Yunohost Git/Infra notifications> App mosquitto failed all tests in job [#20344](https://ci-apps.yunohost.org/ci/job/20344) :(
[23:40:23]
<Yunohost Git/Infra notifications> App opentracker failed all tests in job [#20346](https://ci-apps.yunohost.org/ci/job/20346) :(