Wednesday, January 08, 2025
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
   
             

[01:31:23] <Yunohost Git/Infra notifications> [apps] y​alh76 created new branch appflowy
[01:31:44] <Yunohost Git/Infra notifications> [apps] y​alh76 pushed 1 commit to appflowy: AppFlowy ([3afce307](https://github.com/YunoHost/apps/commit/3afce307ad9c57a4cb9ea3b00ecdaab6a34c192d))
[01:31:53] <Yunohost Git/Infra notifications> [apps] y​alh76 opened [pull request #2763](https://github.com/YunoHost/apps/pull/2763): AppFlowy
[07:22:24] <Yunohost Git/Infra notifications> [cryptpad_ynh] J​osue-T [commented](https://github.com/YunoHost-Apps/cryptpad_ynh/pull/234#discussion_r1906586796) on pull request #234 Testing: And is it ok with: yunohost user permission list cryptpad
[09:03:33] <Yunohost Git/Infra notifications> [penpot_ynh] o​rhtej2 merged [pull request #80](https://github.com/YunoHost-Apps/penpot_ynh/pull/80): Upgrade to v2.4.0
[09:04:35] <Yunohost Git/Infra notifications> [penpot_ynh] o​rhtej2 deleted branch ci-auto-update-2.4.0
[09:05:07] <Yunohost Git/Infra notifications> [penpot_ynh] o​rhtej2 opened [pull request #81](https://github.com/YunoHost-Apps/penpot_ynh/pull/81): v2.4.0
[10:23:58] <Yunohost Git/Infra notifications> [cryptpad_ynh] r​odinux [commented](https://github.com/YunoHost-Apps/cryptpad_ynh/pull/234#discussion_r1906956643) on pull request #234 Testing: yes yunohost user permission list cryptpad permissions: cryptpad.main: allowed: - all_users - ...
[10:31:53] <Yunohost Git/Infra notifications> [cryptpad_ynh] r​odinux pushed 1 commit to testing: Update manifest.toml fix the post migration to delete old sandbox domain ([0afc6104](https://github.com/YunoHost-Apps/cryptpad_ynh/commit/0afc610495815d2f3f30d935970927fd30c9a57d))
[10:33:00] <Yunohost Git/Infra notifications> [cryptpad_ynh] r​odinux [commented](https://github.com/YunoHost-Apps/cryptpad_ynh/pull/236#issuecomment-2577336378) on [issue #236](https://github.com/YunoHost-Apps/cryptpad_ynh/pull/236) Testing: Can you approve @Ddataa ??
[10:52:29] <Yunohost Git/Infra notifications> [cryptpad_ynh] D​dataa approved [pull request #236](https://github.com/YunoHost-Apps/cryptpad_ynh/pull/236#pullrequestreview-2536821109) Testing
[10:53:42] <Yunohost Git/Infra notifications> [cryptpad_ynh] D​dataa merged [pull request #236](https://github.com/YunoHost-Apps/cryptpad_ynh/pull/236): Testing
[11:20:11] <Yunohost Git/Infra notifications> [cryptpad_ynh] J​osue-T [commented](https://github.com/YunoHost-Apps/cryptpad_ynh/pull/234#discussion_r1907028262) on pull request #234 Testing: And with yunohost user permission list -f cryptpad ?
[13:09:06] <Yunohost Git/Infra notifications> [cryptpad_ynh] r​odinux [commented](https://github.com/YunoHost-Apps/cryptpad_ynh/pull/234#discussion_r1907158165) on pull request #234 Testing: permissions: cryptpad.main: additional_urls: sandbox-domain.tld allowed: - all_users - visi...
[13:10:08] <Yunohost Git/Infra notifications> [penpot_ynh] o​rhtej2 merged [pull request #81](https://github.com/YunoHost-Apps/penpot_ynh/pull/81): v2.4.0
[13:10:56] <Yunohost Git/Infra notifications> [cryptpad_ynh] r​odinux [commented](https://github.com/YunoHost-Apps/cryptpad_ynh/pull/234#discussion_r1907160457) on pull request #234 Testing: we should add something to correct this additional_urls ?
[14:01:36] <Paprika> Thanks for the few tips. Managed to improve the code, made it so random string is generated for both the secret and password (if not given by the user during pre-install), and gave it a couple of tests to make sure it installs, upgrades, removes, restores and backups properly.

Feels good to make this work despite not having that much knowledge in programming overall.
[14:25:11] <Paprika> I assume the next step is to add it to the app catalog. As far as I read, I am supposed to clone the apps repo, add my app info, logo, and ask for a pull request.

What about the auto README and testing with yunorunner?

I was only able to do a package linter check on the package, but couldn't get package_checker to work due to LXD/Incus issues (refuses to work, most likely due to Proxmox/LXC incompatibilities).
[14:49:03] <miro5001> There are some points that need to be reviewed. I will try to make comments tonight. I am still at work.
[14:49:05] <Paprika> I assume the next step is to add it to the app catalog. As far as I read, I am supposed to clone the apps repo, add my app info, logo, and ask for a pull request.

What about the auto README and testing with yunorunner?

I was only able to do a package linter check on the package, but couldn't get package\_checker to work due to LXD/Incus issues (refuses to work, most likely due to Proxmox/LXC incompatibilities. If only there was a way to run package_checker natively on an LXC).
[14:51:04] <Paprika> No problem, just new to the process. Focus on what's important to you.
I am mostly using this for my own sake, so not in a hurry in any shape or form.
[14:59:46] <Paprika> No problem, just new to the process. Focus on what's important to you.
I am mostly using this for my own sake, so not in a hurry in any shape or form.
Will push my final changes to master once I am finished with tweaking the restore script a bit further.
[15:06:27] <Paprika> No problem, just new to the process. Focus on what's important to you.
I am mostly using this for my own sake, so not in a hurry in any shape or form.
Will push my final changes to master once I am finished with tweaking the restore script a bit further.
Last change might be to make sure to pull the latest release from original code repo, and then I might proceed to packaging Janitorr for YNH.
[15:20:48] <Paprika> No problem, just new to the process. Focus on what's important to you.
I am mostly using this for my own sake, so not in a hurry in any shape or form.
Will push my final changes to master once I am finished with tweaking the restore script a bit further.
Last change might be to make sure to always pull the latest release from original code repo, and then I might proceed to packaging Janitorr for YNH.
[16:15:17] <orhtej2> > <@botagiuks:tiesiog.lt> No problem, just new to the process. Focus on what's important to you.
> I am mostly using this for my own sake, so not in a hurry in any shape or form.
> Will push my final changes to master once I am finished with tweaking the restore script a bit further.
> Last change might be to make sure to always pull the latest release from original code repo, and then I might proceed to packaging Janitorr for YNH.

I would recommend transferring the app over to yunohost-apps org, you can retain maintainer status but this way you can get ci, hooks etc
[17:12:34] <limezy> Hi all ! I face a bug when trying to accessing a config panel in CLI :
Warning: It looks like you called the helper using positional arguments instead of keyword arguments ?
Error: Failed to read config panel values.
Do you have an idea of what's happening ?
Thanks !
[17:52:20] <limezy> https://github.com/YunoHost-Apps/moncycle_ynh/pull/58
[17:52:20] <orhtej2> > <@limezy:matrix.org> Hi all ! I face a bug when trying to accessing a config panel in CLI :
> Warning: It looks like you called the helper using positional arguments instead of keyword arguments ?
> Error: Failed to read config panel values.
> Do you have an idea of what's happening ?
> Thanks !

Which app
[17:55:04] <Aleks (he/him/il/lui)> should try running the equivalent command from cli with --debug
[17:57:32] <Yunohost Git/Infra notifications> [apps] y​unohost-bot opened [pull request #2764](https://github.com/YunoHost/apps/pull/2764): Add hypebot to wishlist
[17:57:32] <Yunohost Git/Infra notifications> [apps] y​unohost-bot labeled Wishlist on [pull request #2764](https://github.com/YunoHost/apps/pull/2764): Add hypebot to wishlist
[19:07:51] <Paprika> I'll take a look someday on doing that. Thanks for the heads-up.
I don't know much about hooks at the moment, and ci, so I'll have to investigate.
[19:08:26] <Paprika> I have managed to get package\_check running, but can't find enough info on it. For instance, how to set what particular YNH version to test, and whatnot.
[19:10:36] <Paprika> I have managed to get package\_check running, but can't find enough info on it. ~~For instance, how to set what particular YNH version to test, and whatnot.~~ Seems like that is not possible.
[19:10:39] <Paprika> I have managed to get package\_check running, but can't find enough info on it. <del>For instance, how to set what particular YNH version to test, and whatnot.</del> Seems like that is not possible.
[20:02:37] <orhtej2> > <@botagiuks:tiesiog.lt> I have managed to get package\_check running, but can't find enough info on it. <del>For instance, how to set what particular YNH version to test, and whatnot.</del> Seems like that is not possible.

I mean there are ci instances running for bullseye, bookworm and trixie
[20:02:38] <Yunohost Git/Infra notifications> [cryptpad_ynh] J​osue-T [commented](https://github.com/YunoHost-Apps/cryptpad_ynh/pull/234#discussion_r1907766674) on pull request #234 Testing: Well now I remember this story around this. Because Yunohost dont really cleanly handle (for now) the subdomain, so for...
[20:30:49] <orhtej2> > <@limezy:matrix.org> https://github.com/YunoHost-Apps/moncycle_ynh/pull/58

testing here: https://github.com/YunoHost-Apps/moncycle_ynh/pull/59
[20:30:50] <orhtej2> (`$install_dir` is autopopulated, no need to fetch it again)
[20:30:53] <Paprika> Thanks, I think I understood how package_check works now, ran a couple of tests on bookworm. Seems like it all went splendid.
[20:42:21] <Yunohost Git/Infra notifications> [borg_ynh] o​rhtej2 created new branch orhtej2-patch-1
[20:42:21] <Yunohost Git/Infra notifications> [borg_ynh] o​rhtej2 pushed 1 commit to orhtej2-patch-1: Update backup_method ([56ec3348](https://github.com/YunoHost-Apps/borg_ynh/commit/56ec3348d68308ea8c556402140099f6de02c139))
[20:44:29] <Yunohost Git/Infra notifications> [borg_ynh] o​rhtej2 opened [pull request #188](https://github.com/YunoHost-Apps/borg_ynh/pull/188): Strip work_dir from paths inside the backed up repo
[20:45:58] <Yunohost Git/Infra notifications> [borg_ynh] o​rhtej2 edited [pull request #188](https://github.com/YunoHost-Apps/borg_ynh/pull/188): Strip work_dir from paths inside the backup archive
[20:47:22] <Yunohost Git/Infra notifications> [borg_ynh] a​lexAubin approved [pull request #188](https://github.com/YunoHost-Apps/borg_ynh/pull/188#pullrequestreview-2538265692) Strip work_dir from paths inside the backup archive
[20:47:39] <Yunohost Git/Infra notifications> [borg_ynh] a​lexAubin [commented](https://github.com/YunoHost-Apps/borg_ynh/pull/188#issuecomment-2578634848) on [issue #188](https://github.com/YunoHost-Apps/borg_ynh/pull/188) Strip work_dir from paths inside the backup archive: smol poke @zamentur in case you can confirm its alright
[21:13:51] <orhtej2> > <@limezy:matrix.org> https://github.com/YunoHost-Apps/moncycle_ynh/pull/58

I took the liberty of merging the fix to testing, I'll leave releasing to master to you as I see the changes are more involved
[22:18:38] <Paprika> Took a lot of trial and error, but managed to setup a development environment with yunohost and package_check. Wanted to setup yunorunner as well, but not sure how is it supposed to work. Barely any documentation on it. Is it supposed to have some kind of webUI, or it's just CLI?
[22:24:52] <orhtej2> > <@botagiuks:tiesiog.lt> Took a lot of trial and error, but managed to setup a development environment with yunohost and package_check. Wanted to setup yunorunner as well, but not sure how is it supposed to work. Barely any documentation on it. Is it supposed to have some kind of webUI, or it's just CLI?

what you're doing is valiant but unnecessary, typical flow looks like this: https://github.com/YunoHost-Apps/moncycle_ynh/pull/58 - app is in YNH-Apps org, you create pr, type `!testme` and magic happens automatically, no need for local setup
[22:26:43] <Paprika> Oh yeah I do realize that's the workflow, but I believe I am ways off before I can move it to apps org. Plus, I just do a ton of testing aside my own apps, so I need a local testing environment in general, not just for yunohost.
[23:07:34] <Yunohost Git/Infra notifications> [joplin_ynh] y​unohost-bot opened [pull request #71](https://github.com/YunoHost-Apps/joplin_ynh/pull/71): Upgrade to v3.2.8
[23:13:03] <orhtej2> > <@yunohostinfra:matrix.org> [joplin_ynh] y​unohost-bot opened [pull request #71](https://github.com/YunoHost-Apps/joplin_ynh/pull/71): Upgrade to v3.2.8

I was just about done testing 3.2.7
[23:13:53] <orhtej2> And this thing grinds ci to halt with its hefty 7gb backup
[23:17:13] <orhtej2> > And this thing grinds ci to halt with its hefty 7gb backup

I have a feeling it builds more than server, especially given dockerfile but i'm too tired lately to investigate
[23:17:20] <Yunohost Git/Infra notifications> Autoupdater just ran, here are the results:

- 15 pending update PRs
- 10 new apps PRs
- 2 failed apps updates: flood, stremio

See the full log here: https://paste.yunohost.org/raw/utagekager
[23:19:40] <Yunohost Git/Infra notifications> [package_check] y​alh76 closed [pull request #130](https://github.com/YunoHost/package_check/pull/130): Implement LXD helper
[23:19:41] <Yunohost Git/Infra notifications> [package_check] y​alh76 deleted branch LXD