[00:15:35]
<12b> For a big production database with needs for clustering, replication, ..., I would prefer postgresql. But for a single server setup like with yunohost, using mariadb will do the job perfectly, if backed up correctly. So using that if it is already there makes sense. It can be easier to manage for people not familiar with postgres. And it may be ligther weight to run maria db instead of postgresql. (but that could vary depending on the apps)
[07:46:15]
<florent> > For a big production database with needs for clustering, replication, ..., I would prefer postgresql. But for a single server setup like with yunohost, using mariadb will do the job perfectly, if backed up correctly. So using that if it is already there makes sense. It can be easier to manage for people not familiar with postgres. And it may be ligther weight to run maria db instead of postgresql. (but that could vary depending on the apps)
Thanks for your answer! :)
It's a small app (you send pings by a curl command), my guess is that mariadb is enough
[10:28:42]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @lapineige [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/559#issuecomment-1441520428) on [issue #559](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/559) Packaging version 2: testme
[10:28:43]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/559#issuecomment-1441520484) on [issue #559](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/559) Packaging version 2: :sunflower:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/5808/badge)](...
[12:16:56]
<rodinux[m]> > <@rodinux07:matrix.org> In diagnosis something like " - This application is currently flagged as broken on YunoHost's application catalog. This may be a temporary issue while the maintainers attempt to fix the issue. In the meantime, upgrading this app is disabled.
> "
I try a PR between master with testing to see if it come from the ci-apps-devel.yunohost.org...
[12:17:56]
<Aleks (he/him/il/lui)> > I slightly tend to prefer installing Mariadb as it is preinstalled in Yunohost. What do you think?
personally I would tend to use postgresql ... Mariadb is preinstalled in YunoHost so far but won't be starting with Bookworm
[12:18:23]
<Aleks (he/him/il/lui)> I've seen way more issues about mariadb/mysql self-corrupting compared to postgresql...
[12:18:50]
<Aleks (he/him/il/lui)> like postgresql seems more "modern" than mysql ... but i don't have any accurate argument to back this up
[12:19:40]
<Aleks (he/him/il/lui)> > <@rodinux07:matrix.org> > https://ci-apps.yunohost.org/ci/ Paheko as not yet been CIed
>
> I don't understand why now the ci have done the job, but not here https://dash.yunohost.org/appci/app/paheko ?? Status "Just got added to catalog" ??
That's because there's no level yet here :/ https://github.com/YunoHost/apps/blob/master/apps.toml#L2445
[12:20:05]
<Aleks (he/him/il/lui)> because ci-apps.yunohost.org/ is having a hard time keeping up with the queue
[12:22:37]
<rodinux[m]> Ok, I don't have to worry about it... thanks
[12:23:04]
<Yunohost Git/Infra notifications> [apps] @alexAubin created new branch tailoredflow-not-working
[12:23:04]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 1 commit to tailoredflow-not-working: Set tailoredflow as notworking ([0e76504d](https://github.com/YunoHost/apps/commit/0e76504d876385bef6811b251515506daa57f69b))
[12:23:37]
<Yunohost Git/Infra notifications> [apps] @alexAubin opened [pull request #1637](https://github.com/YunoHost/apps/pull/1637): Set tailoredflow as notworking
[12:25:00]
<Aleks (he/him/il/lui)> > <@rodinux07:matrix.org> Ok, I don't have to worry about it... thanks
Paheko's test yielded level 7 here on tuesday https://ci-apps.yunohost.org/ci/job/13741 so the level will be added during this weekend's PR about updating app levels
[12:42:51]
<florent> > <@Alekswag:matrix.org> personally I would tend to use postgresql ... Mariadb is preinstalled in YunoHost so far but won't be starting with Bookworm
Thanks for your feedback then, I rollback to postgresql
[12:44:14]
<florent> seems though a bit equivalent regarding what to chose (except when we have to manage the DB of course)
[12:53:54]
<florent> With manifest v2, BTW, do we still have to install postgresql as a dependency in [resources.apt] or is it installed automatically?
[12:53:55]
<florent> (Thanks in advance and sorry for the questions flood ^^)
[12:53:55]
<florent> Also is it always the `5432` port or can it be something else?
[12:54:56]
<Aleks (he/him/il/lui)> yes you need to declare it in the apt packages ... though that could be done automagically ... idk, maybe will change this later
[12:55:31]
<Aleks (he/him/il/lui)> the port should be 5432 i think, yeah
[12:56:15]
<florent> Thanks!
[13:28:49]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to testing: 25.0.4 ([91b64b68](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/91b64b68f982b8d767bdcb3b4da1951deb33806b))
[13:28:51]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot pushed 1 commit to testing: Auto-update README ([03c9bd6b](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/03c9bd6b88368ce97e1d8e073cc38b2c105eee09))
[13:29:04]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/558#issuecomment-1441796558) on [issue #558](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/558) testing: testme
[13:29:06]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/558#issuecomment-1441796638) on [issue #558](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/558) testing: Alrighty
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/5810/badge)](ht...
[13:29:34]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar edited [pull request #558](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/558): testing
[13:36:46]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/559#issuecomment-1441810933) on [issue #559](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/559) Packaging version 2: >Failed to provision permissions : Domain __DOMAIN__ unknown Zblerg my bad, gotta re-check that part of the code th...
[14:34:39]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 3 commits to v2 ([b952901df31f...cad440467d20](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/b952901df31f...cad440467d20))
[14:34:48]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2] 25.0.4 - Éric Gaspar
[14:34:48]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot pushed 1 commit to v2: Auto-update README ([12f4ac17](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/12f4ac17fff6953aec522179ead0155690c6e8c7))
[14:34:50]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2] Auto-update README - yunohost-bot
[14:34:51]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2] Merge branch testing into v2 - Éric Gaspar
[15:21:34]
<eric_G> Following this [issue](https://github.com/YunoHost-Apps/cryptpad_ynh/issues/102) Do we want to give this information for CryptPad telemetry? https://github.com/YunoHost-Apps/cryptpad_ynh/blob/2ddffd382bdfb203644c99149cc7e2b4d6cdb124/conf/config.js#L327
[15:22:11]
<eric_G> Guillaume Bouzige: ^
[15:24:30]
<eric_G> I haven't got a fix opinion on this. this more an open question.
[15:53:24]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to v2: Update manifest.toml ([28e39750](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/28e39750d5b5d9776de3971141079f8e1889348f))
[16:05:08]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 8 commits to enh_postgresql_migration ([ec1f643f9c76...66854d8b92ae](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/ec1f643f9c76...66854d8b92ae))
[16:05:15]
<Yunohost Git/Infra notifications> [nextcloud_ynh/enh_postgresql_migration] 25.0.4 - Éric Gaspar
[16:05:16]
<Yunohost Git/Infra notifications> [nextcloud_ynh/enh_postgresql_migration] Auto-update README - yunohost-bot
[16:05:16]
<Yunohost Git/Infra notifications> [nextcloud_ynh/enh_postgresql_migration] Merge branch testing into enh_postgresql_migration - Éric Gaspar
[16:27:15]
<florent> > <@ericg:matrix.org> Following this [issue](https://github.com/YunoHost-Apps/cryptpad_ynh/issues/102) Do we want to give this information for CryptPad telemetry? https://github.com/YunoHost-Apps/cryptpad_ynh/blob/2ddffd382bdfb203644c99149cc7e2b4d6cdb124/conf/config.js#L327
Can we let the user choose whether they want to activate telemetry (ask the question during installation)? Yunohost would then be more or less neutral about this question
[18:32:43]
<Guillaume Bouzige> > <@ericg:matrix.org> Following this [issue](https://github.com/YunoHost-Apps/cryptpad_ynh/issues/102) Do we want to give this information for CryptPad telemetry? https://github.com/YunoHost-Apps/cryptpad_ynh/blob/2ddffd382bdfb203644c99149cc7e2b4d6cdb124/conf/config.js#L327
I don't have strong opinion on it. by default the telemetry is not on, it is off, but if the user choose to turn it on then it will tell that the method of install is by yunohost. when I went to read on cryptpad doc about it, it seems to me like they created that option just for us...so why not use it then
[18:36:30]
<Guillaume Bouzige> > Can we let the user choose whether they want to activate telemetry (ask the question during installation)? Yunohost would then be more or less neutral about this question
I don't understand it that way this option doesnt activate telemetry. i like to have telemetry off by default, I don't even think of asking users about it...
[18:40:33]
<eric_G> Telemetry use to be set in the config file. It has been removed in the later versions.
[18:41:08]
<eric_G> After install I hade the telemetry set to true by default.
[18:41:29]
<eric_G> *had
[18:41:29]
<Guillaume Bouzige> > <@ericg:matrix.org> After install I hade the telemetry set to true by default.
ah shit
[18:41:54]
<Guillaume Bouzige> ok then I/we shall find a way to turn it off then
[18:43:44]
<eric_G> For me it is more an opt out telemetry rather than an opt in
[18:44:01]
<Guillaume Bouzige> what do you mean ?
[18:59:25]
<eric_G> Telemetry is on by default
[19:02:09]
<eric_G> Also the method of install option is not specifically created for YunoHost
[19:02:20]
<Guillaume Bouzige> yeah i got that, there must be a way to have it off by default
[19:02:27]
<Guillaume Bouzige> yeah it was for docker
[19:02:48]
<Guillaume Bouzige> but hell to docker yunohost is better
[19:11:56]
<Guillaume Bouzige> i though naively that `blockDailyCheck: true` will not activate telemetry
[19:13:21]
<Guillaume Bouzige> I will go speak to them about it but they are way more active during business hours
[20:47:59]
<florent> https://yunohost.org/en/packaging_apps_resources#data-dir
[20:48:23]
<florent> Is there any reason why the owner of the data_dir directory is not granted the right to create files or directory in it?
[20:49:08]
<florent> (the rights are r-x, which does not seem sufficient for that)
[20:50:31]
<florent> Don't hesitate to just tell me to create an issue ^^
[20:52:11]
<Aleks (he/him/il/lui)> eh yeah indeed that sounds not appropriate
[20:52:23]
<Aleks (he/him/il/lui)> copypasta from the install dir
[20:54:11]
<florent> If that's of any help, I created this issue: https://github.com/YunoHost/issues/issues/2160
[20:54:12]
<Aleks (he/him/il/lui)> fixed in https://github.com/YunoHost/yunohost/commit/139e54a2e52f7a8fa38df8b1c48d52204173051e
[20:54:40]
<florent> How can you people here can be so fast 😵
[20:54:50]
<florent> (are you humans?)
[20:54:56]
<florent> or kind of gods
[20:55:09]
<florent> Seriously, thank you 🙏
[20:55:23]
<Aleks (he/him/il/lui)> idk, open resources.py in browser, switch to edit, Ctrl+F "rx", edit, commit 😅
[21:28:31]
<Krakinou> Hello again, question gathered from my journey into the wonderful world of v2 packaging : regarding the nodejs install helper, it's still needed to use the ynh_install_nodejs and ynh_use_nodejs helpers? I don't see any change regarding this in the doc, but as it's a kind of dependency, not 100% sure...
[21:43:18]
<Krakinou> and question 2 : I don't see any way to set the home folder of the user anymore (similar to ynh_system_user_create --username=$app --home_dir="$final_path"), is it missing from the doc or do I have to use a "usermod -d $install_dir $app" in my scripts?
[21:44:01]
<Krakinou> question 3 : do you guy rather that I open issues in the issue tracker instead of asking random question now and then?
[21:46:36]
<tituspijean> 2) It seems undocumented on the website, but after a quick search on Github: https://github.com/YunoHost/yunohost/blob/79c9a7b294595abaf36223f9fe3bd658276249af/src/utils/resources.py#L433
[21:47:43]
<tituspijean> 3. My philisophy: bother in the chatrooms first, then open issue if there's actual bug/missing feature
[21:49:33]
<tituspijean> > <@titus:pijean.ovh> 2) It seems undocumented on the website, but after a quick search on Github: https://github.com/YunoHost/yunohost/blob/79c9a7b294595abaf36223f9fe3bd658276249af/src/utils/resources.py#L433
eeeeeh actually it is on the website! https://yunohost.org/en/packaging_apps_resources
[21:51:35]
<tituspijean> Krakinou 1. that will be for packaging v3: https://github.com/YunoHost/issues/issues/2136
[21:54:09]
<Krakinou> mééééh, I opened the resource doc page a few days ago and never bothers to refresh it 😕
[21:54:58]
<Krakinou> it actually is on the website...
[21:57:27]
<Krakinou> thanks for the answer
[21:57:47]
<tituspijean> ❤️
[22:06:25]
<Yunohost Git/Infra notifications> [wordpress_ynh] @yunohost-bot opened [pull request #211](https://github.com/YunoHost-Apps/wordpress_ynh/pull/211): Upgrade auto-updater
[22:06:25]
<Yunohost Git/Infra notifications> [wordpress_ynh] @yunohost-bot pushed 1 commit to upgrade-auto-updater: [autopatch] Upgrade auto-updater ([80eb94a5](https://github.com/YunoHost-Apps/wordpress_ynh/commit/80eb94a58d41110c788cb199797f2f648ffe055a))
[22:06:27]
<Yunohost Git/Infra notifications> [wordpress_ynh] @yunohost-bot created new branch upgrade-auto-updater
[22:06:31]
<tituspijean> Oh my, the auto-updater auto-upgrader seems to be working!
[22:15:43]
<tituspijean> aaaaand I hit a rate limit
[22:18:15]
<Yunohost Git/Infra notifications> [apps] @ericgaspar approved [pull request #1637](https://github.com/YunoHost/apps/pull/1637#pullrequestreview-1312229635) Set tailoredflow as notworking
[22:18:16]
<Yunohost Git/Infra notifications> [apps] @ericgaspar merged [pull request #1637](https://github.com/YunoHost/apps/pull/1637): Set tailoredflow as notworking
[22:18:17]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 2 commits to master ([0280c7813b0f...33da3b068f8f](https://github.com/YunoHost/apps/compare/0280c7813b0f...33da3b068f8f))
[22:18:19]
<Yunohost Git/Infra notifications> [apps/master] Set tailoredflow as notworking - Alexandre Aubin
[22:18:22]
<Yunohost Git/Infra notifications> [apps/master] Merge pull request #1637 from YunoHost/tailoredflow-not-working Set tailoredflow as notworking - Éric Gaspar
[22:18:43]
<Guillaume Bouzige> does anyone know if it works to selfhost onlyoffice as standalone, without nextcloud ?
[22:41:51]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to v2: Update manifest.toml ([c2d5eb88](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/c2d5eb88e414cb2c9925c986fb769398f2525a34))
[22:42:37]
<tituspijean> > does anyone know if it works to selfhost onlyoffice as standalone, without nextcloud ?
I guess so? https://github.com/YunoHost-Apps/onlyoffice_ynh
[22:43:27]
<tituspijean> ```
Online collaborative edition of OnlyOffice documents requires:
a server part, with two installation options:
OnlyOffice Document Server packaged for YunoHost.
The Community Document Server for Nextcloud
a client part, such as:
The ONLYOFFICE connector for Nextcloud
The ONLYOFFICE Desktop Editors
```
[22:43:52]
<tituspijean> so it would work with desktop editors as clients
[22:44:20]
<tituspijean> so, the key was a sneaky `workflow` permissions to be ticked when generating an access token. It allows access to the `.github/workflows` directory of the repos.
[22:48:29]
<Guillaume Bouzige> yeah so they won't be any web editors...how come there is in together with nextcloud but not without it
[22:49:44]
<Guillaume Bouzige> I shall investigate further on my own, I was wondering if some will offer me insights so I don't have to test it by myself...
[22:50:17]
<tituspijean> Do you have any web editor in mind?
[22:56:04]
<Guillaume Bouzige> no i am just looking for online document editing software that can be selfhosted
[22:56:22]
<Guillaume Bouzige> and accessible via a webclient
[23:20:42]
<Krakinou> hmm, I'm not clear about the disclaimer stuff : If i let the disclaimer.md file in my package, I have a warning in the linter, but if I remove it the readme is not updated anymore in the github
[23:21:20]
<Aleks (he/him/il/lui)> hmmm
[23:21:22]
<Aleks (he/him/il/lui)> is it not ?
[23:21:32]
<Krakinou> or at least, updating the description.md does not update the readme
[23:21:47]
<Aleks (he/him/il/lui)> which repo ?
[23:21:49]
<Krakinou> https://github.com/YunoHost-Apps/zwave-js-ui_ynh/commit/4915bebddd7fe17c2400022482d3ad74fd0cc06c
[23:22:18]
<Krakinou> or I miss sthg?
[23:25:29]
<Aleks (he/him/il/lui)> ah yes there's a 500 error on the webhook service, checking this
[23:26:01]
<tituspijean> I might have exceeded a rate limit with the yunobot, sorry 😕
[23:26:11]
<Aleks (he/him/il/lui)> doesnt seem related
[23:26:27]
*tituspijean is relieved 😅
[23:26:38]
<Aleks (he/him/il/lui)> `toml.decoder.TomlDecodeError: This float doesn't have a leading digit (line 42 column 1 char 1254)`
[23:27:07]
<Aleks (he/him/il/lui)> https://github.com/YunoHost-Apps/zwave-js-ui_ynh/blob/4915bebddd7fe17c2400022482d3ad74fd0cc06c/manifest.toml#L42
[23:27:11]
<Aleks (he/him/il/lui)> should be an actual string
[23:27:31]
<Krakinou> ffffffffff.......ck
[23:27:43]
<Krakinou> 🙂
[23:28:02]
<Krakinou> thanks, where do you see this error from?
[23:28:33]
<Aleks (he/him/il/lui)> in the webhook service that handle the bot thing, but it runs inside our infra
[23:28:42]
<Aleks (he/him/il/lui)> so yeah it's not easy to actually debug :/
[23:29:00]
<Aleks (he/him/il/lui)> in fact we have no warning anywhere when these fails
[23:29:21]
<Krakinou> ok, I fix it and now it's working
[23:29:27]
<Aleks (he/him/il/lui)> one way to address this could be to add a `.github/` task in each v2 repo to validate that the manifest is valid toml ...
[23:29:28]
<Krakinou> tx!
[23:29:48]
<Aleks (he/him/il/lui)> or to integrate with PyGitHub from titus ;P
[23:31:25]
<Krakinou> I remember the good ol' days, where packaging was just some bash script written on a mechanical keyboard...
[23:31:48]
<Aleks (he/him/il/lui)> (or simpler : just add a matrix notification when hook fails)
[23:32:36]
<Aleks (he/him/il/lui)> > <Krakinou> I remember the good ol' days, where packaging was just some bash script written on a mechanical keyboard...
the good old days when empty variables would lead to disastrous `rm -rf /var/www/<emptystring>` :P #pepperidgefarmremembers
[23:32:38]
<Krakinou> If the toml was not correct, I would finally have noticed when running the package_check
[23:32:56]
<Krakinou> so maybe no need to overthink it 🙂
[23:33:48]
<Krakinou> 😉
[23:50:52]
<Yunohost Git/Infra notifications> [apps] @tituspijean pushed 1 commit to autoupdater-upgrader: Fix auto-updater upgrader ([8e9b9b10](https://github.com/YunoHost/apps/commit/8e9b9b10dcf9d4b698c499055eec371e60a432a6))
[23:51:05]
<Yunohost Git/Infra notifications> [apps] @tituspijean edited [pull request #1624](https://github.com/YunoHost/apps/pull/1624): Add auto-updater upgrader
[23:52:04]
<Yunohost Git/Infra notifications> [apps] @tituspijean [commented](https://github.com/YunoHost/apps/pull/1624#issuecomment-1442585416) on [issue #1624](https://github.com/YunoHost/apps/pull/1624) Add auto-updater upgrader: Status: works, but getting some secondary rate limit block sometimes.
[23:55:01]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @lapineige [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/558#issuecomment-1442588075) on [issue #558](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/558) testing: Upgrade went fine :)