[03:27:31]
<Yunohost Git/Infra notifications> Autoupdater just ran, here are the results:
- 60 pending update PRs
- 18 new apps PRs
- 4 failed apps updates: fluffychat, mobilizon, pinkarrows, screego
See the full log here: http://paste.yunohost.org/raw/epifuqebux
[07:09:47]
<Yunohost Git/Infra notifications> Job [#30901](https://ci-apps.yunohost.org/ci/job/30901) for hubzilla failed miserably :(
[07:22:52]
<gredin67> > <@gredin67:matrix.fdn.fr> Salamandar: I tested your PR https://github.com/YunoHost-Apps/fab-manager_ynh/pull/126
> but the app is not able to send email
> https://github.com/YunoHost-Apps/fab-manager_ynh/issues/131#issuecomment-2307207454
anyone else would have an idea about wrong mail config ?
[07:23:14]
<Yunohost Git/Infra notifications> Job [#30898](https://ci-apps.yunohost.org/ci/job/30898) for flohmarkt failed miserably :(
[07:51:52]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar merged [pull request #734](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/734): Doc : sorti le high performance backend de la section Onlyoffice
[07:51:54]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to testing: Update ADMIN.md (#734) Jai sorti High Performance Backend, qui était dans la section Onlyoffice, alors que cest indép... ([d1d315cb](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/d1d315cb6931a110e55641a96f5e8dd0fe605a38))
[07:53:46]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 2 commits to testing ([d1d315cb6931...7701925bd381](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/d1d315cb6931...7701925bd381))
[08:11:38]
<Yunohost Git/Infra notifications> Job [#30902](https://ci-apps.yunohost.org/ci/job/30902) for glance failed miserably :(
[08:50:03]
<Tag> https://ci-apps.yunohost.org/ci/job/30437
zzz... what!! redirect_ynh is broken now? without any modification from our side ? /o\
[08:50:54]
<nicofrand> Hi! How long does one have to wait before the broken flag is removed from an app once the CI is fixed?
[08:52:44]
<Yunohost Git/Infra notifications> App wireguard failed all tests in job [#30167](https://ci-apps.yunohost.org/ci/job/30167) :(
[08:52:56]
<Tag> App's level are refreshed on every Friday by a bot, and then a human have to merge the PR https://github.com/YunoHost/apps/pull/2602
[08:58:41]
<Yunohost Git/Infra notifications> [apps] Tagadda edited [pull request #2602](https://github.com/YunoHost/apps/pull/2602): Update app levels according to CI results
[09:16:13]
<Tag> euh, j'avoue ne rien comprendre à cette PR
Pourquoi le commentaire et les commits ne correspondent pas ? 🤔
[09:16:15]
<Tag> https://github.com/YunoHost/apps/pull/2602
[09:17:39]
<Yunohost Git/Infra notifications> App z-push failed all tests in job [#30559](https://ci-apps.yunohost.org/ci/job/30559) :(
[09:17:41]
<Aleks (he/him/il/lui)> tu veux dire les commits qui fixent ttrss, scrutiny et django_example qui ne sont pas dans la liste ?
[09:26:22]
<Yunohost Git/Infra notifications> [apps] ericgaspar edited [pull request #2602](https://github.com/YunoHost/apps/pull/2602): Update app levels according to CI results
[09:26:53]
<Yunohost Git/Infra notifications> [apps] ericgaspar edited [pull request #2602](https://github.com/YunoHost/apps/pull/2602): Update app levels according to CI results
[09:27:08]
<Yunohost Git/Infra notifications> [apps] ericgaspar edited [pull request #2602](https://github.com/YunoHost/apps/pull/2602): Update app levels according to CI results
[09:27:41]
<Yunohost Git/Infra notifications> [apps] ericgaspar edited [pull request #2602](https://github.com/YunoHost/apps/pull/2602): Update app levels according to CI results
[09:47:09]
<orhtej2> > <@tag:lostpod.me> https://ci-apps.yunohost.org/ci/job/30437
> zzz... what!! redirect_ynh is broken now? without any modification from our side ? /o\
first time running CURL tests, connecting to 127.0.0.1 redirects to HTTPS with self-signed certificate.
OTOH curl tests were supposed to ignore SSL errors?
[09:47:16]
<Yunohost Git/Infra notifications> [apps] ericgaspar edited [pull request #2602](https://github.com/YunoHost/apps/pull/2602): Update app levels according to CI results
[09:48:11]
<orhtej2> > <@gredin67:matrix.fdn.fr> anyone else would have an idea about wrong mail config ?
looks OK assuming these are valid keys, what's the issue you're getting?
[09:48:13]
<Yunohost Git/Infra notifications> [apps] ericgaspar edited [pull request #2602](https://github.com/YunoHost/apps/pull/2602): Update app levels according to CI results
[09:51:07]
<orhtej2> > first time running CURL tests, connecting to 127.0.0.1 redirects to HTTPS with self-signed certificate.
> OTOH curl tests were supposed to ignore SSL errors?
interesting that is passes for sub.domain.tld 🤔
[09:56:56]
<orhtej2> https://ci-apps-dev.yunohost.org/ci/ <- this one has a single runner all of the sudden?
[09:59:21]
<eric_G> what? https://github.com/YunoHost/apps/pull/2602 file changed: moodle 🤔
[10:06:26]
<Tag> > <@Alekswag:matrix.org> tu veux dire les commits qui fixent ttrss, scrutiny et django_example qui ne sont pas dans la liste ?
Non, la PR modifie uniquement moodle https://github.com/YunoHost/apps/pull/2602/files
et le commentaire le correspond pas https://github.com/YunoHost/apps/pull/2602
[10:08:13]
<Tag> > https://ci-apps-dev.yunohost.org/ci/ <- this one has a single runner all of the sudden?
Again the weird bug with the "first runner id" is increasing until it matches the max runner count
[10:11:56]
<Tag> ah, c'est un force push de salam andar ?
[10:17:24]
<Yunohost Git/Infra notifications> App jellyfin goes down from level 6 to 1 in job [#30577](https://ci-apps.yunohost.org/ci/job/30577)
[10:20:34]
<Yunohost Git/Infra notifications> App z-push failed all tests in job [#30559](https://ci-apps.yunohost.org/ci/job/30559) :(
[10:37:30]
<Yunohost Git/Infra notifications> App psitransfer goes down from level 6 to 1 in job [#30608](https://ci-apps.yunohost.org/ci/job/30608)
[11:13:29]
<Yunohost Git/Infra notifications> App element-call goes down from level 8 to 1 in job [#30617](https://ci-apps.yunohost.org/ci/job/30617)
[11:28:00]
<Tag> > <@yunohostinfra:matrix.org> App element-call goes down from level 8 to 1 in job [#30617](https://ci-apps.yunohost.org/ci/job/30617)
weird connection issue, restarted
[11:28:06]
<Yunohost Git/Infra notifications> App syncserver-rs rises from level 7 to 8 in job [#30649](https://ci-apps.yunohost.org/ci/job/30649) !
[11:55:50]
<Yunohost Git/Infra notifications> App element-call goes down from level 8 to 1 in job [#30617](https://ci-apps.yunohost.org/ci/job/30617)
[12:06:39]
<Yunohost Git/Infra notifications> App jellyfin goes down from level 6 to 1 in job [#30577](https://ci-apps.yunohost.org/ci/job/30577)
[13:00:18]
<nicofrand> How accurate is `When using regexp in the nginx location field (location ~ __PATH__), start the path with ^ (location ~ ^__PATH__). ` ? nextcloud_ynh uses `location ^~ __PATH__/ {` for example. Should I stick to the first way in https://github.com/YunoHost-Apps/kresus_ynh/blob/master/conf/nginx.conf#L10 ?
[13:01:41]
<nicofrand> example_ynh does not have this kind of field
[13:14:51]
<orhtej2> > How accurate is `When using regexp in the nginx location field (location ~ __PATH__), start the path with ^ (location ~ ^__PATH__). ` ? nextcloud_ynh uses `location ^~ __PATH__/ {` for example. Should I stick to the first way in https://github.com/YunoHost-Apps/kresus_ynh/blob/master/conf/nginx.conf#L10 ?
See the docs: https://nginx.org/en/docs/http/ngx_http_core_module.html#location
`~` is regex match, `^~` is prefix match
`location ~ /some_path` matches `/some_other_path/some_path`, `location ^~ /some_path` does not, `location ~ ^/some_path` does not as well.
[13:18:35]
<nicofrand> OK thanks. So, in my case, where only the file extension is important, there is nothing wrong with the current way, but to please the CI I should use `location ~ ^__PATH__`, right?
[13:22:11]
<orhtej2> > OK thanks. So, in my case, where only the file extension is important, there is nothing wrong with the current way, but to please the CI I should use `location ~ ^__PATH__`, right?
something like https://github.com/YunoHost-Apps/penpot_ynh/blob/ad8dc67c8d2a0b3ef6b76d6feae85dfd07195652/conf/nginx.conf#L100 probably
[13:23:23]
<nicofrand> What's wrong with using __PATH__ then?
[14:07:09]
<orhtej2> > What's wrong with using __PATH__ then?
linked is sublocation of `__PATH__`, which I don't use in this example as it goes on the root of domain
[14:09:18]
<nicofrand> ok, but what about kresus_ynh?
[14:09:36]
<nicofrand> Is there an official way to please the CI?
[14:12:23]
<orhtej2> > Is there an official way to please the CI?
yes, change the location to `location ^~ __PATH__`
[14:12:46]
<orhtej2> so just change `~` to `^~`
[14:19:06]
<gredin67> > looks OK assuming these are valid keys, what's the issue you're getting?
mail is not sent from the app, although the server itself does send mail correctly. I don't know how to debug.
[14:19:06]
<orhtej2> `journalctl` should show if there is auth failure for the app, see what IP the app request originates from (should from 127.0.0.1)
[14:20:25]
<nicofrand> > yes, change the location to `location ^~ __PATH__`
Just to confirm, `^~`? (linter says `start the path with ^ (location ~ ^__PATH__)`
[14:21:53]
<orhtej2> > Just to confirm, `^~`? (linter says `start the path with ^ (location ~ ^__PATH__)`
it's not a smart check TBH: https://github.com/YunoHost/package_linter/blob/677d48f50f6bdda4ea16dd31efb572371b4ba782/tests/test_configurations.py#L311
[14:22:55]
<nicofrand> raah, I'm getting tired of this CI
[14:23:39]
<nicofrand> Do I need to use `^~ __PATH__` or `~ ^__PATH__`?
[14:24:51]
<orhtej2> > Do I need to use `^~ __PATH__` or `~ ^__PATH__`?
whichever you prefer
[14:24:58]
<nicofrand> lol
[14:25:29]
<nicofrand> I'd prefer not to have to update the package just to please the CI without being hit with a bad mark TBH
[14:26:06]
<Aleks (he/him/il/lui)> `location ^~` and `location ~` are two different syntax, see orhtej2 comment before pointing to nginx doc
but in your case you're using `location ~`, and the following regex should start with `^` to prevent weird scenarios
[14:26:19]
<Aleks (he/him/il/lui)> as the linter says: `start the path with ^ (location ~ ^__PATH__)`
[14:26:47]
<@rosbeefandino:3cmr.fr> Hi all, is someone know if sliding sync working on synapse_ynh, if yes : on which branch ? , if master : is there a guide to configure old installs ? done ;)
[14:27:26]
<Aleks (he/him/il/lui)> i guess you want to read https://github.com/YunoHost-Apps/synapse_ynh/pull/439
[15:34:13]
<Yunohost Git/Infra notifications> App diacamma goes down from level 6 to 1 in job [#30668](https://ci-apps.yunohost.org/ci/job/30668)
[15:36:59]
<Aleks (he/him/il/lui)> https://i.imgflip.com/955pz0.jpg
[17:16:54]
<Émy - OniriCorpe> > <@rosbeefandino:3cmr.fr> Hi all, is someone know if sliding sync working on synapse_ynh, if yes : on which branch ? , if master : is there a guide to configure old installs ? done ;)
It’s working on testing
[18:59:18]
<orhtej2> CI plz https://ci-apps-dev.yunohost.org/ci/
[19:00:20]
<orhtej2> Why's there only one worker up?
[19:43:15]
<Yunohost Git/Infra notifications> App pihole failed all tests in job [#30684](https://ci-apps.yunohost.org/ci/job/30684) :(
[21:51:40]
<Yunohost Git/Infra notifications> [nextcloud_ynh] alexAubin merged [pull request #719](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/719): WIP: Experimental curl tests
[21:51:42]
<Yunohost Git/Infra notifications> [nextcloud_ynh] alexAubin pushed 6 commits to testing ([7701925bd381...5376a6859360](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/7701925bd381...5376a6859360))
[21:51:43]
<Aleks (he/him/il/lui)> (took the liberty of merging https://github.com/YunoHost-Apps/nextcloud_ynh/pull/719 since they should be working)
[21:51:44]
<Yunohost Git/Infra notifications> [nextcloud_ynh] alexAubin deleted branch curl-tests
[21:51:46]
<Yunohost Git/Infra notifications> [nextcloud_ynh] alexAubin edited [pull request #735](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/735): Testing
[21:52:14]
<Yunohost Git/Infra notifications> App tooljet failed all tests in job [#30692](https://ci-apps.yunohost.org/ci/job/30692) :(
[21:56:58]
<Yunohost Git/Infra notifications> [nextcloud_ynh] alexAubin pushed 2 commits to testing ([5376a6859360...978e71f00ae6](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/5376a6859360...978e71f00ae6))
[21:57:02]
<Yunohost Git/Infra notifications> [nextcloud_ynh/testing] Update ADMIN.md (#734) Jai sorti High Performance Backend, qui était dans la section Onlyoffice, alors que cest indép... - tomdereub
[21:57:06]
<Aleks (he/him/il/lui)> (and force-pushed the testing branch because the commit history was a weird mess ?)
[22:01:39]
<Yunohost Git/Infra notifications> App rustdesk-server rises from level 7 to 8 in job [#30693](https://ci-apps.yunohost.org/ci/job/30693) !
[22:43:32]
<Yunohost Git/Infra notifications> App cusdis stays at level 1 in job [#30694](https://ci-apps.yunohost.org/ci/job/30694)