Monday, January 23, 2023
apps@conference.yunohost.org
January
Mon Tue Wed Thu Fri Sat Sun
            1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
         

[04:04:55] <Yunohost Git/Infra notifications> Application django-for-runners rises from level (unknown) to level 6 on https://ci-apps.yunohost.org/ci/job/13137
[08:34:56] <Yunohost Git/Infra notifications> [nextcloud_ynh] @MayeulC [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/549#discussion_r1083761481) on pull request #549 nginx.conf: support PATCH method: Good call. Though Im mostly wondering if there are any security considerations to have, and if other keywords need to b...
[08:38:04] <Yunohost Git/Infra notifications> [nextcloud_ynh] @MayeulC [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/549#issuecomment-1399974213) on [issue #549](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/549) nginx.conf: support PATCH method: testme
[08:38:07] <Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/549#issuecomment-1399974331) on [issue #549](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/549) nginx.conf: support PATCH method: May the CI gods be with you
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/...
[09:48:31] <Yunohost Git/Infra notifications> [apps] @Salamandar opened [pull request #1619](https://github.com/YunoHost/apps/pull/1619): Add enDI logo
[09:52:31] <Yunohost Git/Infra notifications> [apps] @ericgaspar approved [pull request #1619](https://github.com/YunoHost/apps/pull/1619#pullrequestreview-1265278985) Add enDI logo
[09:52:51] <Yunohost Git/Infra notifications> [apps] @ericgaspar merged [pull request #1619](https://github.com/YunoHost/apps/pull/1619): Add enDI logo
[09:52:51] <Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 2 commits to master ([cbe6901e946e...99f0f999a263](https://github.com/YunoHost/apps/compare/cbe6901e946e...99f0f999a263))
[09:52:54] <Yunohost Git/Infra notifications> [apps/master] Add enDI logo - Félix Piédallu
[09:52:57] <Yunohost Git/Infra notifications> [apps/master] Merge pull request #1619 from Salamandar/endi_logo Add enDI logo - Éric Gaspar
[10:02:55] <Yunohost Git/Infra notifications> [nextcloud_ynh] @lapineige approved [pull request #532](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/532#pullrequestreview-1265294524) Testing
[10:03:41] <Yunohost Git/Infra notifications> @mdic forked my_webapp_ynh to [mdic/my_webapp_ynh](https://github.com/mdic/my_webapp_ynh)
[10:07:19] <Yunohost Git/Infra notifications> [nextcloud_ynh] @lapineige [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/532#issuecomment-1400090730) on [issue #532](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/532) Testing: https://github.com/nextcloud/server/pull/35734#issuecomment-1369025742 25.0.3 should be fixing the bug, at least most of...
[10:07:32] <Yunohost Git/Infra notifications> [nextcloud_ynh] @lapineige [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/532#issuecomment-1400091029) on [issue #532](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/532) Testing: testme
[10:07:35] <Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/532#issuecomment-1400091105) on [issue #532](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/532) Testing: Alrighty
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/5266/badge)](ht...
[10:08:23] <Yunohost Git/Infra notifications> [nextcloud_ynh] @lapineige [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/542#issuecomment-1400092182) on [issue #542](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/542) changing the time between cron runs to the recommended value of 5 min: Any idea how that could increase server load ?
[10:08:27] <Yunohost Git/Infra notifications> [nextcloud_ynh] @lapineige [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/542#issuecomment-1400092182) on [issue #542](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/542) changing the time between cron runs to the recommended value of 5 min: Any idea about how that could increase server load ?
[10:09:19] <Yunohost Git/Infra notifications> [nextcloud_ynh] @lapineige [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/532#issuecomment-1400093561) on [issue #532](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/532) Testing: Oh and lets merge #530 too ?
[10:09:26] <Yunohost Git/Infra notifications> [nextcloud_ynh] @lapineige [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/530#issuecomment-1400093730) on [issue #530](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/530) Update to php8.1: testme
[10:16:44] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 10 commits to Update-to-PHP8.1 ([8b4ccc9847b5...148726cddfb2](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/8b4ccc9847b5...148726cddfb2))
[10:16:48] <Yunohost Git/Infra notifications> [nextcloud_ynh/Update-to-PHP8.1] Fix warning - Éric Gaspar
[10:16:51] <Yunohost Git/Infra notifications> [nextcloud_ynh/Update-to-PHP8.1] Fix warning - Éric Gaspar
[10:16:53] <Yunohost Git/Infra notifications> [nextcloud_ynh/Update-to-PHP8.1] Merge branch testing into Update-to-PHP8.1 - Éric Gaspar
[10:17:01] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 10 commits to testing ([8802926c279c...eb3b4e2efb6e](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/8802926c279c...eb3b4e2efb6e))
[10:17:01] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar merged [pull request #530](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/530): Update to php8.1
[10:17:02] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar dismissed review [pull request #532](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/532#pullrequestreview-1265294524): Testing
[10:17:04] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar deleted branch Update-to-PHP8.1
[10:17:05] <Yunohost Git/Infra notifications> [nextcloud_ynh/testing] Auto-update README - yunohost-bot
[10:17:10] <Yunohost Git/Infra notifications> [nextcloud_ynh/testing] Merge pull request #530 from YunoHost-Apps/Update-to-PHP8.1 Update to php8.1 - Éric Gaspar
[10:17:10] <Yunohost Git/Infra notifications> [nextcloud_ynh/testing] Merge branch testing into Update-to-PHP8.1 - Éric Gaspar
[10:18:38] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar edited [pull request #532](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/532): Testing
[10:43:26] <gredin67> wtf? https://ci-apps-dev.yunohost.org/ci/job/5215
are there still issues with package_check?
[10:46:45] <Yunohost Git/Infra notifications> [apps] @ericgaspar [commented](https://github.com/YunoHost/apps/pull/1617#discussion_r1083901951) on pull request #1617 Add moncycle.app to catalog: The app name should be the ID set in the manifest : "id": "moncycle_app", I propose, to avoide confusion, to drop the ...
[11:15:12] <Yunohost Git/Infra notifications> [gitlab_ynh] @kay0u pushed 1 commit to testing: 15.8.0 ([983f7656](https://github.com/YunoHost-Apps/gitlab_ynh/commit/983f7656fe23ac66b3566b6bd3a2c57eee2f4410))
[11:15:25] <Yunohost Git/Infra notifications> [gitlab_ynh] @yunohost-bot pushed 1 commit to testing: Auto-update README ([cefc3620](https://github.com/YunoHost-Apps/gitlab_ynh/commit/cefc3620057cd70e9b3dce93a0b651a64b3fef10))
[11:19:59] <Yunohost Git/Infra notifications> [apps] @Limezy [commented](https://github.com/YunoHost/apps/pull/1617#discussion_r1083932715) on pull request #1617 Add moncycle.app to catalog: Understood, you are right that this .app may create some confusion. What do you think about going with moncycle_app ...
[11:24:12] <Yunohost Git/Infra notifications> [gitlab_ynh] @kay0u opened [pull request #210](https://github.com/YunoHost-Apps/gitlab_ynh/pull/210): Testing
[11:24:13] <Yunohost Git/Infra notifications> [gitlab_ynh] @kay0u [commented](https://github.com/YunoHost-Apps/gitlab_ynh/pull/210#issuecomment-1400188670) on [issue #210](https://github.com/YunoHost-Apps/gitlab_ynh/pull/210) Testing: testme
[11:24:19] <Yunohost Git/Infra notifications> [gitlab_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/gitlab_ynh/pull/210#issuecomment-1400188772) on [issue #210](https://github.com/YunoHost-Apps/gitlab_ynh/pull/210) Testing: :carousel_horse:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/5267/bad...
[11:25:30] <Yunohost Git/Infra notifications> [gitlab_ynh] @kay0u [commented](https://github.com/YunoHost-Apps/gitlab_ynh/pull/209#issuecomment-1400190164) on [issue #209](https://github.com/YunoHost-Apps/gitlab_ynh/pull/209) 15.7.5: Many thanks for this PR :+1: But its superseded by #210
[11:25:31] <Yunohost Git/Infra notifications> [gitlab_ynh] @kay0u closed [pull request #209](https://github.com/YunoHost-Apps/gitlab_ynh/pull/209): 15.7.5
[11:55:37] <Yunohost Git/Infra notifications> [apps] @alexAubin [commented](https://github.com/YunoHost/apps/pull/1617#discussion_r1083966618) on pull request #1617 Add moncycle.app to catalog: (Imho i would just name it moncycle, having an _app suffix is a bit meh, considering that any yunohost app is ... an...
[11:56:23] <Aleks (he/him/il/lui)> > <@gredin67:matrix.fdn.fr> wtf? https://ci-apps-dev.yunohost.org/ci/job/5215
> are there still issues with package_check?

why x_X
[12:01:19] <Aleks (he/him/il/lui)> FYI folks : i rebuilt the base CI images to disable some services useless in 99% of cases, such as rspamd, dovecot etc to try to slightly improve perfs
[12:58:08] <Yunohost Git/Infra notifications> [apps] @Limezy [commented](https://github.com/YunoHost/apps/pull/1617#discussion_r1084024838) on pull request #1617 Add moncycle.app to catalog: I get your logic, no problem with that 👍
Let me fix my stupid CI mistake, change the repo and repo fork, amend this PR ...
[13:01:23] <Yunohost Git/Infra notifications> CI miserably crashed because another process is using the lock
[13:01:39] <Yunohost Git/Infra notifications> CI miserably crashed because another process is using the lock
[13:02:09] <Yunohost Git/Infra notifications> While testing jenkins: It looks like package_check did not finish properly ... on https://ci-apps.yunohost.org/ci/job/13151
[13:02:15] <Yunohost Git/Infra notifications> While testing snweb: It looks like package_check did not finish properly ... on https://ci-apps.yunohost.org/ci/job/13148
[13:10:53] <Yunohost Git/Infra notifications> While testing processwire: It looks like package_check did not finish properly ... on https://ci-apps.yunohost.org/ci/job/13149
[13:11:30] <Yunohost Git/Infra notifications> While testing snweb: It looks like package_check did not finish properly ... on https://ci-apps.yunohost.org/ci/job/13148
[13:36:31] <badri> Is there something wrong with the CI?
[13:36:59] <badri> It seems to be failing everything with a "[app] is already installed" or "[domain where we wanted to install the app] is already in use"
[13:37:29] <badri> (Just scrolled up and saw some action's been happening there)
[13:51:14] <Gérard Collin> > <@Alekswag:matrix.org> FYI folks : i rebuilt the base CI images to disable some services useless in 99% of cases, such as rspamd, dovecot etc to try to slightly improve perfs

Btw aleks, is it fine with you if I write a script to prepare a ramfs account and configure lxd to use it for the CI?
[13:52:35] <Gérard Collin> I'm ok to spend some time on this, but only if no-one else is doing it, and if someone adapts it to the CI server. will need to be installed there
[13:52:42] <Aleks (he/him/il/lui)> not sure what you mean but yes i guess ? Note also that kayou is experimenting a bit with this idea and knows better than me
[13:53:35] <Aleks (he/him/il/lui)> > <badri> It seems to be failing everything with a "[app] is already installed" or "[domain where we wanted to install the app] is already in use"

yeah i've seen this too, due to not being able to stop LXC / switch to other snapshot, so it stays on the same system
[13:53:43] <Gérard Collin> Ah ok. Let's avoid being two doing it
[13:54:07] <Aleks (he/him/il/lui)> (imho the it should crash instead of acting as if everything is fine in the event that it can't stop the current working LXC to switch snapshots)
[13:54:16] <kayou> > <@gcco:matrix.org> I'm ok to spend some time on this, but only if no-one else is doing it, and if someone adapts it to the CI server. will need to be installed there

FYI i use this https://brandonrozek.com/blog/lxdtmpfs/
[13:55:09] <Gérard Collin> Ok, then if you're already in this topic, I'll do something else. Happy to help if you want
[13:56:45] <kayou> and i did these changes in package_check: https://paste.yunohost.org/efokoluyim.bash
[13:57:46] <kayou> the main issue here is:
- there is not enough ram on ci-apps's VM
- images are still stored in non-ramdisk disk, so the creation of a container still takes times due to read IO
[14:39:10] <Yunohost Git/Infra notifications> [nextcloud_ynh] @lapineige [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/532#issuecomment-1400453482) on [issue #532](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/532) Testing: Upgrade went fine on my side.
[14:42:01] <eric_G[m]> is there any opposition to merge Nextcloud testing to master?
[14:45:46] <Yunohost Git/Infra notifications> [nextcloud_ynh] @lapineige [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/532#issuecomment-1400453482) on [issue #532](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/532) Testing: Upgrade went fine on my side. But I still have the quota bug :/
[14:52:56] <eric_G[m]> apparently yes 😬
[19:25:42] <limezy> > <badri> It seems to be failing everything with a "[app] is already installed" or "[domain where we wanted to install the app] is already in use"

Oh ok ! So I’m not alone… I was scratching my head out of ideas about why my package, perfectly working on my server, kept failing on the CI !
[20:18:49] <Yunohost Git/Infra notifications> [apps] @ericgaspar just made [pull request #1617](https://github.com/YunoHost/apps/pull/1617) ready for review: Add moncycle.app to catalog
[20:18:50] <Yunohost Git/Infra notifications> [apps] @ericgaspar approved [pull request #1617](https://github.com/YunoHost/apps/pull/1617#pullrequestreview-1266268166) Add moncycle.app to catalog
[20:20:35] <Yunohost Git/Infra notifications> [apps] @alexAubin approved [pull request #1617](https://github.com/YunoHost/apps/pull/1617#pullrequestreview-1266272120) Add moncycle.app to catalog: LGTM assuming the testing branch will be merged in the near future
[20:25:50] <Gérard Collin> Ah you've told package_check to use a temp storage you should have created before I guess
[20:26:09] <Gérard Collin> The ramfs thingy is done somewhere else
[20:32:33] <Yunohost Git/Infra notifications> [apps] @ericgaspar merged [pull request #1617](https://github.com/YunoHost/apps/pull/1617): Add moncycle.app to catalog
[20:32:34] <Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 3 commits to master ([99f0f999a263...2a3015557910](https://github.com/YunoHost/apps/compare/99f0f999a263...2a3015557910))
[20:32:35] <Yunohost Git/Infra notifications> [apps/master] Add moncycle.app - Limezy
[20:32:39] <Yunohost Git/Infra notifications> [apps/master] Fix app name - Éric Gaspar
[20:32:40] <Yunohost Git/Infra notifications> [apps/master] Merge pull request #1617 from Limezy/add-moncycle.app Add moncycle.app to catalog - Éric Gaspar
[20:34:12] <Yunohost Git/Infra notifications> [apps] @ericgaspar [commented](https://github.com/YunoHost/apps/pull/1617#issuecomment-1400943716) on [issue #1617](https://github.com/YunoHost/apps/pull/1617) Add moncycle.app to catalog: merged à linstant.