[00:01:12]
<orhtej2> > <@oniricorpe:im.emelyne.eu> is it better for a package to do
> `grep -P "Port\s+\d+" /etc/ssh/sshd_config | grep -P -o "\d+"`
> or
> `yunohost settings get security.ssh.ssh_port` ?
The latter I guess?
[02:31:29]
<Émy - OniriCorpe> tbh i'm feeling dirty
`pattern.regexp = '^(?!\.)^(\.?[-a-zA-Z0-9]){1,256}(\.[a-zA-Z0-9]{1,6}){1}$'`
[03:49:52]
<Yunohost Git/Infra notifications> App webtrees goes down from level 8 to 6 in job [#23940](https://ci-apps.yunohost.org/ci/job/23940)
[08:01:01]
<Salamandar> > <@oniricorpe:im.emelyne.eu> `root committed now` x)
Ah yes, that was a huge issue, things were not working for a few days, so i did not go through PR ><'
[08:01:18]
<Salamandar> also it was a stupid issue, it was storing relative path instead of absolute
[09:07:06]
<Yunohost Git/Infra notifications> [package_linter] @kay0u approved [pull request #134](https://github.com/YunoHost/package_linter/pull/134#pullrequestreview-1900310060) Fix nginx test
[09:11:17]
<Yunohost Git/Infra notifications> [package_linter] @kay0u pushed 2 commits to master ([daf913007a64...78089533432b](https://github.com/YunoHost/package_linter/compare/daf913007a64...78089533432b))
[09:11:18]
<Yunohost Git/Infra notifications> [package_linter] @kay0u merged [pull request #134](https://github.com/YunoHost/package_linter/pull/134): Fix nginx test
[09:11:24]
<Yunohost Git/Infra notifications> [package_linter/master] Fix nginx test grep will return 0 when it finds matches. In our case, when it finds matches, we need to print the warni... - Salamandar
[09:23:00]
<Yunohost Git/Infra notifications> App scrutiny goes down from level 8 to 6 in job [#23951](https://ci-apps.yunohost.org/ci/job/23951)
[10:27:54]
<Yunohost Git/Infra notifications> [apps] @Salamandar opened [pull request #2058](https://github.com/YunoHost/apps/pull/2058): Use local app cache to reduce github api calls
[10:35:58]
<Yunohost Git/Infra notifications> Job [#23946](https://ci-apps.yunohost.org/ci/job/23946) for fluffychat failed miserably :(
[10:39:25]
<Yunohost Git/Infra notifications> [apps] @Salamandar opened [pull request #2059](https://github.com/YunoHost/apps/pull/2059): Split apps_cache call, run apps_cache update when updating sources, a…
[12:20:57]
<Yunohost Git/Infra notifications> App mautrix_signal rises from level 0 to 6 in job [#23954](https://ci-apps.yunohost.org/ci/job/23954) !
[14:29:50]
<Yunohost Git/Infra notifications> [apps] @alexAubin merged [pull request #2058](https://github.com/YunoHost/apps/pull/2058): Use local app cache to reduce github api calls
[14:29:52]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 2 commits to master ([fb12240090c5...e6478327e2da](https://github.com/YunoHost/apps/compare/fb12240090c5...e6478327e2da))
[14:29:59]
<Yunohost Git/Infra notifications> [apps/master] Use local app cache to reduce github api calls - Félix Piédallu
[14:30:31]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 2 commits to master ([e6478327e2da...55f3829cb816](https://github.com/YunoHost/apps/compare/e6478327e2da...55f3829cb816))
[14:30:37]
<Yunohost Git/Infra notifications> [apps] @alexAubin merged [pull request #2059](https://github.com/YunoHost/apps/pull/2059): Split apps_cache call, run apps_cache update when updating sources, a…
[14:55:30]
<gredin67> is there a syntax error here ? https://github.com/YunoHost-Apps/fab-manager_ynh/blob/version-2/scripts/_common.sh#L265-L270
[15:00:00]
<Aleks (he/him/il/lui)> https://xyproblem.info/ what's your actual issue
[15:08:18]
<Yunohost Git/Infra notifications> App question2answer rises from level 2 to 6 in job [#23956](https://ci-apps.yunohost.org/ci/job/23956) !
[15:14:15]
<gredin67> > <@gredin67:matrix.fdn.fr> is there a syntax error here ? https://github.com/YunoHost-Apps/fab-manager_ynh/blob/version-2/scripts/_common.sh#L265-L270
I don't understand anything of this package
[15:14:50]
<gredin67> I'm even wondering if installing ruby still makes sense. The current install method seems to be docker-based https://raw.githubusercontent.com/sleede/fab-manager/master/setup/setup.sh
[15:15:22]
<Yunohost Git/Infra notifications> [dokuwiki_ynh] @yunohost-bot pushed 1 commit to ci-auto-update-release-2024-02-06: Upgrade to vrelease-2024-02-06 ([d0aa2822](https://github.com/YunoHost-Apps/dokuwiki_ynh/commit/d0aa2822c8206a4191b3c608932a41b76f930bbc))
[15:15:24]
<Yunohost Git/Infra notifications> [dokuwiki_ynh] @yunohost-bot created new branch ci-auto-update-release-2024-02-06
[15:15:28]
<Yunohost Git/Infra notifications> [dokuwiki_ynh] @yunohost-bot pushed 1 commit to ci-auto-update-release-2024-02-06: Auto-update README ([18bbe9e3](https://github.com/YunoHost-Apps/dokuwiki_ynh/commit/18bbe9e38e1ddfce7ce80c16b5e9a1cedd633103))
[15:15:48]
<Aleks (he/him/il/lui)> the actual service launches bundle / rubyonrails so that seems to make sense : https://github.com/YunoHost-Apps/fab-manager_ynh/blob/version-2/conf/fab-manager-app.service#L13
[15:16:43]
<gredin67> any clue how to get verbose debug message to know where to start ?
[15:17:40]
<Aleks (he/him/il/lui)> but i don't even know what you are trying to debug 😬
[15:17:40]
<gredin67> > <@Alekswag:matrix.org> the actual service launches bundle / rubyonrails so that seems to make sense : https://github.com/YunoHost-Apps/fab-manager_ynh/blob/version-2/conf/fab-manager-app.service#L13
but what about upstream ?
[15:18:18]
<Aleks (he/him/il/lui)> many upstream ship Docker images but YunoHost apps are most often not based on docker because reasons
[15:18:20]
<gredin67> the install https://github.com/YunoHost-Apps/fab-manager_ynh/pull/88
[15:19:35]
<Aleks (he/him/il/lui)> there's a link to the detailed log in the job output : `Info: This log is now available via https://paste.yunohost.org/raw/qanaxumuwi`
[15:20:07]
<Aleks (he/him/il/lui)> ```
2024-02-24 22:42:21,215: DEBUG - + ynh_exec_as fab-manager env RAILS_ENV=production PATH=/opt/rbenv/versions/fab-manager/bin:/opt/node_n/n/versions/node/18/bin:/opt/node_n/bin:/opt/rbenv/shims:/opt/rbenv/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin LD_PRELOAD=/usr/lib/x86_64-linux-gnu/libjemalloc.so bin/bundle exec rake db:migrate
2024-02-24 22:42:21,215: DEBUG - + local user=fab-manager
2024-02-24 22:42:21,215: DEBUG - + shift 1
2024-02-24 22:42:21,215: DEBUG - ++ whoami
2024-02-24 22:42:21,216: DEBUG - + [[ fab-manager = root ]]
2024-02-24 22:42:21,216: DEBUG - + sudo -u fab-manager env RAILS_ENV=production PATH=/opt/rbenv/versions/fab-manager/bin:/opt/node_n/n/versions/node/18/bin:/opt/node_n/bin:/opt/rbenv/shims:/opt/rbenv/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin LD_PRELOAD=/usr/lib/x86_64-linux-gnu/libjemalloc.so bin/bundle exec rake db:migrate
2024-02-24 22:42:26,837: DEBUG - rake aborted!
2024-02-24 22:42:26,841: DEBUG - KeyError: key not found: "DEFAULT_HOST"
2024-02-24 22:42:26,841: DEBUG - /var/www/fab-manager/config/initializers/sentry.rb:31:in `fetch'
2024-02-24 22:42:26,841: DEBUG - /var/www/fab-manager/config/initializers/sentry.rb:31:in `block in <main>'
2024-02-24 22:42:26,845: DEBUG - /var/www/fab-manager/config/initializers/sentry.rb:30:in `<main>'
2024-02-24 22:42:26,845: DEBUG - /var/www/fab-manager/config/environment.rb:5:in `<main>'
2024-02-24 22:42:26,845: DEBUG - bin/bundle:3:in `load'
2024-02-24 22:42:26,845: DEBUG - bin/bundle:3:in `<main>'
2024-02-24 22:42:26,845: DEBUG - Tasks: TOP => db:migrate => db:load_config => environment
```
[15:21:25]
<gredin67> > <@Alekswag:matrix.org> many upstream ship Docker images but YunoHost apps are most often not based on docker because reasons
yes, so when there is no install instructions except docker-based from upstream, the only way to package is to reverse-engineer the docker package and "translate" it in bash ?
[15:21:43]
<Aleks (he/him/il/lui)> sounds like there's supposed to be a .env with eg https://github.com/sleede/fab-manager/blob/master/env.example
[15:21:54]
<Aleks (he/him/il/lui)> > <@gredin67:matrix.fdn.fr> yes, so when there is no install instructions except docker-based from upstream, the only way to package is to reverse-engineer the docker package and "translate" it in bash ?
yup
[15:23:21]
<Aleks (he/him/il/lui)> until we have YoloGENPT
[15:38:22]
<gredin67> > <@Alekswag:matrix.org> sounds like there's supposed to be a .env with eg https://github.com/sleede/fab-manager/blob/master/env.example
something like this ? https://github.com/YunoHost-Apps/fab-manager_ynh/pull/98/files
[15:39:37]
<Aleks (he/him/il/lui)> yup
[15:39:59]
<Aleks (he/him/il/lui)> though of course SECRET_KEY_BASE should probably not be hardcoded
[15:42:36]
<Aleks (he/him/il/lui)> and POSTGRES_HOST etc are confusing, i'm not sure to understand why there would be a _PASSWORD setting, but not the psql DB name ... _HOST seems to suggest we're talking about the machine, which in our case should just be localhost or 127.0.0.1
[15:45:44]
<Yunohost Git/Infra notifications> [apps] @alexAubin created new branch revert-2059-scripts
[15:45:44]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 1 commit to revert-2059-scripts: Revert "Split apps_cache call, run apps_cache update when updating sources, a…" ([a275908b](https://github.com/YunoHost/apps/commit/a275908b7b205e25a08860163940ee020e716a1f))
[15:45:45]
<Yunohost Git/Infra notifications> [apps] @alexAubin opened [pull request #2060](https://github.com/YunoHost/apps/pull/2060): Revert "Split apps_cache call, run apps_cache update when updating sources, a…"
[15:46:29]
<Yunohost Git/Infra notifications> [apps] @alexAubin merged [pull request #2060](https://github.com/YunoHost/apps/pull/2060): Revert "Split apps_cache call, run apps_cache update when updating sources, a…"
[15:46:29]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 2 commits to master ([55f3829cb816...ff68844e3d7a](https://github.com/YunoHost/apps/compare/55f3829cb816...ff68844e3d7a))
[15:46:29]
<Salamandar> > <@yunohostinfra:matrix.org> [apps] @alexAubin pushed 1 commit to revert-2059-scripts: Revert "Split apps_cache call, run apps_cache update when updating sources, a…" ([a275908b](https://github.com/YunoHost/apps/commit/a275908b7b205e25a08860163940ee020e716a1f))
thanks
[15:46:35]
<Yunohost Git/Infra notifications> [apps/master] Revert "Split apps_cache call, run apps_cache update when updating sources, a…" - Alexandre Aubin
[15:46:44]
<Yunohost Git/Infra notifications> [apps] @alexAubin deleted branch revert-2059-scripts
[18:38:49]
<gredin67> Salamandar: I see you're working on the signal and facebook python bridges. Beware that those are deprecated for their Go counterpart.
[18:43:32]
<gredin67> I think prioritary would be to created a mautrix_go_ynh meta/template package inspired by the mautrix_whatsapp_ynh one. And then to update all Go bridges to inherit/fork/... from this template.
[18:46:11]
<gredin67> > <@Alekswag:matrix.org> and POSTGRES_HOST etc are confusing, i'm not sure to understand why there would be a _PASSWORD setting, but not the psql DB name ... _HOST seems to suggest we're talking about the machine, which in our case should just be localhost or 127.0.0.1
https://github.com/sleede/fab-manager/blob/484fbde4248059028454498163dd4b6ffdd727d1/doc/environment.md?plain=1
[19:44:48]
<Salamandar> > <@gredin67:matrix.fdn.fr> Salamandar: I see you're working on the signal and facebook python bridges. Beware that those are deprecated for their Go counterpart.
yeah, I know, but I wanted to first move them to packagingv2 :)
[20:14:45]
<gredin67> I thought starting from a go package which is already on v2 would be more efficient than upgrading a python v1 package to Go v2. Except for the sources, config.yaml and config panel, it's a copy-paste.
[20:23:03]
<Yunohost Git/Infra notifications> [apps] @kay0u approved [pull request #2045](https://github.com/YunoHost/apps/pull/2045#pullrequestreview-1901922849) store: init new locales for weblate
[20:23:09]
<Yunohost Git/Infra notifications> [apps] @kay0u merged [pull request #2045](https://github.com/YunoHost/apps/pull/2045): store: init new locales for weblate
[20:23:09]
<Yunohost Git/Infra notifications> [apps] @kay0u pushed 2 commits to master ([ff68844e3d7a...72e34994e183](https://github.com/YunoHost/apps/compare/ff68844e3d7a...72e34994e183))
[20:23:10]
<Yunohost Git/Infra notifications> [apps] @kay0u deleted branch init-locales-for-weblate
[20:23:14]
<Yunohost Git/Infra notifications> [apps/master] store: init new locales - OniriCorpe
[20:24:30]
<Émy - OniriCorpe> > <@yunohostinfra:matrix.org> [apps] @kay0u approved [pull request #2045](https://github.com/YunoHost/apps/pull/2045#pullrequestreview-1901922849) store: init new locales for weblate
kayou: it was necessary finally?
[20:34:38]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @rodinux opened [issue #668](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/668): unable to upgrade to olstable branch
[20:35:54]
<Yunohost Git/Infra notifications> [apps] @alexAubin merged [pull request #2047](https://github.com/YunoHost/apps/pull/2047): store: bypass_ratelimit if staff
[20:35:55]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 2 commits to master ([72e34994e183...03c38d4a9b3e](https://github.com/YunoHost/apps/compare/72e34994e183...03c38d4a9b3e))
[20:35:58]
<Yunohost Git/Infra notifications> [apps] @alexAubin deleted branch bypass_ratelimit
[20:36:01]
<Yunohost Git/Infra notifications> [apps/master] bypass_ratelimit if staff - OniriCorpe
[20:41:32]
<Yunohost Git/Infra notifications> [apps] @kay0u pushed 1 commit to master: Add pot file ([722dd036](https://github.com/YunoHost/apps/commit/722dd0362028bdf76f31f2b12c02806deb2a854f))
[20:52:41]
<Yunohost Git/Infra notifications> [apps] @kay0u pushed 1 commit to master: add message.pot ([14ba7c46](https://github.com/YunoHost/apps/commit/14ba7c469c0c9b65f550c9428d5370f70eaeb998))
[20:52:45]
<Yunohost Git/Infra notifications> App bicbucstriim rises from level 0 to 7 in job [#23960](https://ci-apps.yunohost.org/ci/job/23960) !
[20:53:56]
<Yunohost Git/Infra notifications> [apps] @yunohost-bot opened [pull request #2061](https://github.com/YunoHost/apps/pull/2061): Translations update from Weblate
[20:57:00]
<Yunohost Git/Infra notifications> [apps] @kay0u approved [pull request #2061](https://github.com/YunoHost/apps/pull/2061#pullrequestreview-1901976543) Translations update from Weblate
[20:57:00]
<Yunohost Git/Infra notifications> [apps] @kay0u merged [pull request #2061](https://github.com/YunoHost/apps/pull/2061): Translations update from Weblate
[20:57:00]
<Yunohost Git/Infra notifications> [apps] @kay0u pushed 2 commits to master ([14ba7c469c0c...0363cc80c5b6](https://github.com/YunoHost/apps/compare/14ba7c469c0c...0363cc80c5b6))
[20:58:54]
<Yunohost Git/Infra notifications> [apps] @yunohost-bot opened [pull request #2062](https://github.com/YunoHost/apps/pull/2062): Translations update from Weblate
[21:02:45]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 1 commit to master: Fix sourcesautoupdate.sh fucking up the cron job ([9c19ea5d](https://github.com/YunoHost/apps/commit/9c19ea5de65a989658b4183186058807f087bf2f))
[21:15:22]
<Yunohost Git/Infra notifications> [apps] @OniriCorpe created new branch fixes-for-weblate
[21:15:41]
<Yunohost Git/Infra notifications> [apps] @OniriCorpe opened [pull request #2063](https://github.com/YunoHost/apps/pull/2063): fixes for weblate
[21:24:28]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 2 commits to master ([9c19ea5de65a...e374f461450b](https://github.com/YunoHost/apps/compare/9c19ea5de65a...e374f461450b))
[22:11:38]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @rodinux closed [issue #668](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/668): unable to upgrade to olstable branch
[22:12:25]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @rodinux [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/668#issuecomment-1965391667) on [issue #668](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/668) unable to upgrade to olstable branch: The issue is not related to the package.