Wednesday, October 26, 2022
apps@conference.yunohost.org
October
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
           

[08:11:39] <Yunohost Git/Infra notifications> [wordpress_ynh] @github-actions[bot] pushed 1 commit to ci-auto-update-v6.0.3: Upgrade to version 6.0.3 https://github.com/YunoHost-Apps/wordpress_ynh/commit/cd0b672211038802500f07be7a305e9dea0667c5
[08:11:48] <Yunohost Git/Infra notifications> [wordpress_ynh] @yunohost-bot pushed 1 commit to ci-auto-update-v6.0.3: Auto-update README https://github.com/YunoHost-Apps/wordpress_ynh/commit/6c89f03559e4f7ede236be18d80d8d38b5d2ebc6
[08:18:38] <Yunohost Git/Infra notifications> [gitlab_ynh] @kay0u merged pull request #204: Testing https://github.com/YunoHost-Apps/gitlab_ynh/pull/204
[08:18:38] <Yunohost Git/Infra notifications> [gitlab_ynh] @kay0u pushed 7 commits to master: https://github.com/YunoHost-Apps/gitlab_ynh/compare/e40a4ba4151d...b11b42cc997a
[08:18:41] <Yunohost Git/Infra notifications> [gitlab_ynh/master] 15.5.1 - Kay0u
[08:18:44] <Yunohost Git/Infra notifications> [gitlab_ynh/master] Auto-update README - yunohost-bot
[08:18:46] <Yunohost Git/Infra notifications> [gitlab_ynh/master] Merge pull request #204 from YunoHost-Apps/testing Testing - Kayou
[09:56:19] <Guillaume Bouzige> Hello appteam

I am still stuck and cant push my PR for cryptpad app update.
The trailing slash caused by helper `ynh_permission_url --permission="main" --add_url=$domain --auth_header=true` resulting in an extra trailing slash in domain added in `/etc/ssowat/conf.json`

I would love to solve that and move on to more packaging fun
[11:58:56] <Yunohost Git/Infra notifications> [nextcloud_ynh] @LAfricain commented on issue #380 how to access nextcloud locally: I have the same issue. I would like to access nextcloud locally but Im all the time redirected to the yunohost server. https://github.com/YunoHost-Apps/nextcloud_ynh/issues/380#issuecomment-1291923952
[13:00:52] <MayeulC> Well, new issue with the CI this time: `!!! Another analyseCI process is currently using the lock ./CI-1.lock !!!`
[13:52:16] <Aleks (he/him/il/lui)> > Hello appteam
>
> I am still stuck and cant push my PR for cryptpad app update.
> The trailing slash caused by helper `ynh_permission_url --permission="main" --add_url=$domain --auth_header=true` resulting in an extra trailing slash in domain added in `/etc/ssowat/conf.json`
>
> I would love to solve that and move on to more packaging fun

zblerg yeah i don't have much time to look into it but i believe that's a bug in the core due to the fact that we never tested extensively that kind of use case
[13:54:42] <Aleks (he/him/il/lui)> maybe the issue is around here https://github.com/YunoHost/yunohost/blob/dev/src/permission.py#L920
[13:59:31] <Guillaume Bouzige> > <@Alekswag:matrix.org> maybe the issue is around here https://github.com/YunoHost/yunohost/blob/dev/src/permission.py#L920

how can I test ? I fork the core and deploy my fork right ?
[14:01:23] <Aleks (he/him/il/lui)> it's not fixed, i'm just guessing roughly piece of code to investigate .. i you're confident with python etc, you could set up a core dev env with https://github.com/yunohost/ynh-dev (cf also https://yunohost.org/dev ) and try to dig the issue
[14:03:06] <Aleks (he/him/il/lui)> the code that builds the ssowat conf.json is here https://github.com/YunoHost/yunohost/blob/dev/src/app.py#L1530-L1549 and somehow the format of the base `"url"` is right (no trailing slash when path = '/') but not the `additional_urls` one (trailing slash is there when path = '/')
[14:03:46] <Guillaume Bouzige> i see, digging over there might as well lead me to a quick dirty workaround
[14:04:05] <Guillaume Bouzige> but just to delay a futur bug when/if the thing gets corrected
[17:43:25] <Tag> https://ci-apps-dev.yunohost.org/ci/job/3976
[17:43:27] <Tag> blblblbl
[17:44:28] <Tag> I guess I'll change the lxd storage to lvm
[17:45:02] <Tag> what do you think yalh76 ?
[17:45:56] <Tag> instead of dir
[17:49:02] <yalh76> > <@tag:lostpod.me> instead of dir

I don't know if lvm storage is better than btrfs or dir
[17:49:49] <Tag> i've had issue with dir a lot... i don't know 🤷
we could reformat the 100GB drive to whaterever filesystem
[17:49:51] <yalh76> but you can try
[17:50:14] <Tag> yep I guess I'll try that this week
[17:51:36] <Tag> (IIRC we can't use ZFS because it can only rollback to the last snapshot, and we rollback to the first)
[17:52:20] <yalh76> in a CI the files to save, i case of reinstallation from scratch are:
- /var/www/yunorunner/db.sqlite
- /home/CI_package_check/logs/
- /home/CI_package_check/badges/
- /home/CI_package_check/summary/
[17:52:49] <Tag> thanks o/ I guess I'll run a ynh backup ?
[17:53:17] <yalh76> nope ... backup will only do the job for yunorunner, not the CI_package_check files
[17:53:29] <Tag> meh okay
[17:53:56] <yalh76> BTW, ci-apps-dev is still working at scaleway, if needed ^^
[17:55:08] <Tag> I guess I can do this with the CI still running
[17:57:21] <yalh76> > <@tag:lostpod.me> https://ci-apps-dev.yunohost.org/ci/job/3976

issue solved doing admin stuff:
```
rm -rf /var/lib/lxd/storage-pools/default/containers-snapshots/ynh-appci-bullseye-amd64-stable-test-2
rm -rf /var/lib/lxd/storage-pools/default/containers/ynh-appci-bullseye-amd64-stable-test-2
lxc delete ynh-appci-bullseye-amd64-stable-test-2
```
[18:16:45] <Yunohost Git/Infra notifications> [apps] @yalh76 approved pull request #1532 Change peachpub to notworking https://github.com/YunoHost/apps/pull/1532#pullrequestreview-1157066324
[18:16:55] <Yunohost Git/Infra notifications> [apps] @yalh76 merged pull request #1532: Change peachpub to notworking https://github.com/YunoHost/apps/pull/1532
[18:16:56] <Yunohost Git/Infra notifications> [apps] @yalh76 pushed 2 commits to master: https://github.com/YunoHost/apps/compare/e560fd32aa32...132c6d46f84b
[18:17:05] <Yunohost Git/Infra notifications> [apps/master] Change peachpub to notworking - notplants
[18:17:09] <Yunohost Git/Infra notifications> [apps/master] Merge pull request #1532 from mhfowler/master Change peachpub to notworking - yalh76
[21:22:26] <nicofrand> Hi! Where can I learn about the new packaging format please?
[21:23:21] <eric_G[m]> nicofrand: https://github.com/YunoHost/apps/tree/master/tools/packaging_v2
[21:23:36] <eric_G[m]> https://github.com/YunoHost-Apps/helloworld_ynh
[21:23:55] <eric_G[m]> https://github.com/YunoHost/test_apps/tree/master/manifestv2_app_ynh
[21:24:27] <eric_G[m]> https://github.com/YunoHost/doc/blob/resources-doc/pages/06.contribute/10.packaging_apps/11.helpers/packaging_apps_resources.md
[21:25:03] <nicofrand> Thanks!