[00:51:22]
<Yunohost Git/Infra notifications> Application calckey completely failed the continuous integration tests on https://ci-apps.yunohost.org/ci/job/13024
[01:21:28]
<Yunohost Git/Infra notifications> Application mastodon completely failed the continuous integration tests on https://ci-apps.yunohost.org/ci/job/12453
[04:12:01]
<Yunohost Git/Infra notifications> Application code-server rises from level (unknown) to level 6 on https://ci-apps.yunohost.org/ci/job/13029
[07:01:00]
<florent> > <@Alekswag:matrix.org> hmmm what would be the use case though ?
ElasticSearch which is used only by apps on the same server
[07:01:35]
<florent> (I may add security later and provide nginx at that point)
[07:54:51]
<florent> tituspijean: you wondered why elasticsearch was failing being downloaded on the CI. I get the same issue with OpenSearch when running the package check locally:
```
1650 DEBUG + '[' -n https://artifacts.opensearch.org/releases/bundle/opensearch/2.4.1/opensearch-2.4.1-linux-x64.tar.gz ']'
1650 DEBUG + local out
1650 DEBUG ++ wget --tries 3 --no-dns-cache --timeout 900 --no-verbose --output-document=/var/cache/yunohost/download/opensearch/amd64.tar.gz https://artifacts.opensearch.org/releases/bundle/opensearch/2.4.1/opensearch-2.4.1-linux-x64.tar.gz
Return code: 137
Working time: 1 minute, 4 seconds (08:42:45)
Error: Installation failed.
Return code: 1
Working time: 0s (08:42:47)
```
[07:55:38]
<florent> I wonder if that's not due to the size of the tarball (>500Mo) and some limit of disk space on the container
[07:56:21]
<florent> The first installation passes though, that's after installing a second instance or when upgrading that it fails
[08:09:52]
<florent> Hmm, first run yesterday, and it failed. Second run some minutes later, it failed again. Now, it seems like it's working. 🙄
[08:14:02]
<tituspijean> Return code 137 looks like the wget process got killed by the system
[08:14:03]
<Gérard Collin> > Hmm, first run yesterday, and it failed. Second run some minutes later, it failed again. Now, it seems like it's working. 🙄
Yunohost Git/Infra notifications: I get that kind of random errors as well when testing locally
[08:33:25]
<tituspijean> > <@titus:pijean.ovh> Return code 137 looks like the wget process got killed by the system
Which could make sense if the downloaded file is heavy
[08:33:25]
<florent> > <@gcco:matrix.org> Yunohost Git/Infra notifications: I get that kind of random errors as well when testing locally
I mean, I run the check on my personal computer ^^'
[08:33:31]
<florent> run all these checks
[08:44:52]
<Yunohost Git/Infra notifications> Application shlink stays at level 2 on https://ci-apps.yunohost.org/ci/job/13034
[08:49:37]
<Yunohost Git/Infra notifications> Application x-prober completely failed the continuous integration tests on https://ci-apps.yunohost.org/ci/job/13035
[09:05:16]
<gredin67> https://aria.im/_matrix/media/v1/download/matrix.fdn.fr/ZwjPkwIeOfHDFxcDaWSSaXFk
[09:11:32]
<gredin67> The diagnosis is run successfully, but not on the newly created domain. I had to go through the webadmin menu and re-run the diagnosis to get it run on the newly created domain. Then I could go back to the Certificate page and click on "install let's encrypt"
[09:53:57]
<MayeulC> This is HTTP basic auth (as implemented by radarr, for instance; I have multiple apps affected).
Headers are the usual ones for basic auth: the server first responds with `www-authenticate: Basic realm="Radarr"`, and then the web browser will request every web page with `Authentication: Basic base64(username:password)`
ssowat 11.0.11; yunohost 11.0.11, it's a `multi_webapp` with `"auth_header": true, "public": true`
I guess I could set it to `false`?
[09:54:39]
<MayeulC> > <@titus:pijean.ovh> MayeulC can you be more specific on said app, your YunoHost+SSO versions, on which headers it uses, and if it has the `auth_header=false` in `/etc/ssowat/conf.json`?
Sorry, my message was meant as a reply to this
[10:05:23]
<gredin67> in yunohost testing 11.1.1 and previous version, it's impossible to click on "install a let's encrypt certificate" although the diagnosis has been run successfully after clicking on the link "the diagnosis page" in the above page.
[10:59:26]
<Aleks (he/him/il/lui)> > <@gredin67:matrix.fdn.fr> The diagnosis is run successfully, but not on the newly created domain. I had to go through the webadmin menu and re-run the diagnosis to get it run on the newly created domain. Then I could go back to the Certificate page and click on "install let's encrypt"
Yes it's not specific to 11.1, there's at least 2 tickets about this ...
[11:17:17]
<Yunohost Git/Infra notifications> [gitlab_ynh] @cbldev opened pull request #209: 15.7.5 https://github.com/YunoHost-Apps/gitlab_ynh/pull/209
[11:19:06]
<Yunohost Git/Infra notifications> [gitlab_ynh] @cbldev commented on issue #209 15.7.5: gogogadgetoci https://github.com/YunoHost-Apps/gitlab_ynh/pull/209#issuecomment-1398243934
[12:38:00]
<Salamandar> Hey btw if that can be useful to other maintainers that want to know the ram usage of their install script…
https://gist.github.com/Salamandar/d31a639de17ae9beab8b43cf6f48efe2
[12:38:13]
<Salamandar> It should be extended to disk usage :p
[12:38:46]
<Aleks (he/him/il/lui)> 👀
[12:38:54]
<Salamandar> (npm build -> 1.2GB of ram 😭)
[12:39:09]
<Aleks (he/him/il/lui)> feeling like integrating this in package_check ? ;P
[12:39:29]
<Salamandar> > <@Alekswag:matrix.org> feeling like integrating this in package_check ? ;P
Yes I was actually thinking about taht !!
[12:39:32]
<Aleks (he/him/il/lui)> \o/
[13:01:48]
<Yunohost Git/Infra notifications> @Salamandar forked package_check to [Salamandar/package_check](https://github.com/Salamandar/package_check)
[13:02:33]
<Yunohost Git/Infra notifications> [package_check] @Salamandar opened [pull request #135](https://github.com/YunoHost/package_check/pull/135): Use text instead of integers for stop_timer argument
[13:04:48]
<Yunohost Git/Infra notifications> [package_check] @Salamandar opened [pull request #136](https://github.com/YunoHost/package_check/pull/136): Add metrics measurements to tests runs
[13:05:05]
<Salamandar> > <@Alekswag:matrix.org> feeling like integrating this in package_check ? ;P
done: https://github.com/YunoHost/package_check/pull/136
[13:05:43]
<Aleks (he/him/il/lui)> very speed :o
[13:08:56]
<Yunohost Git/Infra notifications> [package_check] @alexAubin merged [pull request #135](https://github.com/YunoHost/package_check/pull/135): Use text instead of integers for stop_timer argument
[13:08:56]
<Yunohost Git/Infra notifications> [package_check] @alexAubin pushed 2 commits to master ([7aa254d9fbd3...6197e36e217c](https://github.com/YunoHost/package_check/compare/7aa254d9fbd3...6197e36e217c))
[13:08:59]
<Yunohost Git/Infra notifications> [package_check/master] Use text instead of integers for stop_timer argument - Félix Piédallu
[13:09:04]
<Yunohost Git/Infra notifications> [package_check/master] Merge pull request #135 from Salamandar/int_to_enum Use text instead of integers for stop_timer argument - Alexandre Aubin
[13:11:54]
<Aleks (he/him/il/lui)> shall we yolomerge and then we can iterate later on the other point you mention ?
[13:20:58]
<Salamandar> > <@Alekswag:matrix.org> shall we yolomerge and then we can iterate later on the other point you mention ?
hmmmm i didn't test it so idk, if you're willing to rollback if it breaks ci-apps-dev xD
[13:21:51]
<Aleks (he/him/il/lui)> thank god you didn't test it i was thinking "wtf this dude is too much of a wizard he's able to confidently develop on package_check ?! what is this"
[13:22:21]
<Salamandar> ahahah
[13:26:55]
<Salamandar> ok so i just killed my pc by trying to test this script
[13:27:14]
<Salamandar> `</dev/zero head -c $((1024**3*7)) | tail` -> eat 7GB of RAM that i didn't have
[13:38:36]
<Salamandar> Well that didn't work @Aleks (he/him/il/lui)
[13:38:58]
<Salamandar> I fixed it and you might not like how it's done :p
[13:39:33]
<Aleks (he/him/il/lui)> ah ? xD
[13:39:47]
<Salamandar> yeah, go read the code, you'll understand
[13:44:17]
<Aleks (he/him/il/lui)> `declare -p resources > /dev/shm/metrics_vars` ?
[13:44:18]
<Aleks (he/him/il/lui)> dafuq :D
[13:44:23]
<Aleks (he/him/il/lui)> wut iz dis
[13:44:53]
<Aleks (he/him/il/lui)> ah the thread/new process wasnt sharing var with the parent ?
[13:45:08]
<Aleks (he/him/il/lui)> what's `/dev/shm/` o.O
[13:47:38]
<Aleks (he/him/il/lui)> that sounds neat
[13:47:38]
<Salamandar> it prints the "declaration" of resources
```
~ ❯❯❯ declare -A pouet=( \[a]=1 )
~ ❯❯❯ declare -p pouet
typeset -A pouet=( \[a]=1 )
```
[13:47:44]
<Salamandar> > <@Alekswag:matrix.org> what's `/dev/shm/` o.O
sharedmem
[13:47:44]
<Salamandar> > <@Alekswag:matrix.org> ah the thread/new process wasnt sharing var with the parent ?
No… :'(
[13:48:01]
<Aleks (he/him/il/lui)> sounds like you are a bash wizerd after all
[13:48:01]
<Salamandar> security = 0
[13:48:07]
<Aleks (he/him/il/lui)> why is that ?
[13:48:08]
<Salamandar> :'(
[13:48:18]
<Salamandar> because anyone can `cat /dev/shm/metrics_vars`
[13:48:30]
<Salamandar> it's as secure as /tmp i think
[13:50:44]
<Aleks (he/him/il/lui)> eh, can't we set write permissions ?
[13:51:32]
<Aleks (he/him/il/lui)> let's use asymetric cryptography to ensure authenticity #overengineering
[13:51:49]
<Salamandar> ahaahhahaha
[13:52:09]
<Salamandar> well there's the user/group set, rw-r--r, that's the bare minimum
[13:52:10]
<Salamandar> but that's all
[13:52:25]
<Aleks (he/him/il/lui)> anyway there's also $TEST_CONTEXT which can be used as a tmp space
[13:52:30]
<Aleks (he/him/il/lui)> ah
[13:52:31]
<Salamandar> OK… so my laptop's ssh key was removed from github.
[13:52:34]
<Aleks (he/him/il/lui)> sounds fine then ?
[13:52:37]
<Aleks (he/him/il/lui)> wut
[13:52:38]
*Salamandar is changing his github password
[13:55:59]
<Salamandar> > <@Alekswag:matrix.org> anyway there's also $TEST_CONTEXT which can be used as a tmp space
ah neat
[13:56:05]
<Salamandar> i would prefer this
[13:57:02]
<Yunohost Git/Infra notifications> [package_check] @Salamandar edited [pull request #136](https://github.com/YunoHost/package_check/pull/136): Add metrics measurements to tests runs
[13:57:28]
<Aleks (he/him/il/lui)> (maybe you need to `export` the var to have it available in the child process)
[14:06:17]
<Salamandar> > <@Alekswag:matrix.org> (maybe you need to `export` the var to have it available in the child process)
hmmm
[14:06:26]
<Salamandar> but then the child process won't be able to set the variable in its parent process
[14:08:03]
<Aleks (he/him/il/lui)> yes i was meaning to be able to use/read $TEST_CONTEXT ;)
[14:08:17]
<Salamandar> ah !
[14:08:18]
<Salamandar> ok ok
[14:08:19]
<Salamandar> maybe
[14:10:13]
<Yunohost Git/Infra notifications> Application horde stays at level 4 on https://ci-apps.yunohost.org/ci/job/13047
[14:10:40]
<Yunohost Git/Infra notifications> While testing isso: It looks like package_check did not finish properly ... on https://ci-apps.yunohost.org/ci/job/13048
[14:11:48]
<Aleks (he/him/il/lui)> (zbgkfb boring lock issues)
[14:23:04]
<Yunohost Git/Infra notifications> Application meilisearch stays at level 4 on https://ci-apps.yunohost.org/ci/job/13049
[15:23:31]
<Yunohost Git/Infra notifications> [apps] @yunohost-bot created new branch update_app_levels
[15:24:00]
<Aleks (he/him/il/lui)> (am i doing refactorings again ? you cant prove it)
[15:41:56]
<Yunohost Git/Infra notifications> [apps] @yunohost-bot opened [pull request #1615](https://github.com/YunoHost/apps/pull/1615): Update app levels accoring to CI results
[15:41:56]
<Yunohost Git/Infra notifications> [apps] @yunohost-bot pushed 1 commit to update_app_levels: Update app levels according to CI results ([d654aa19](https://github.com/YunoHost/apps/commit/d654aa19f21427ffcb3a955ca2b5d9916b9767c3))
[15:42:40]
<Yunohost Git/Infra notifications> [apps] @alexAubin [commented](https://github.com/YunoHost/apps/pull/1615#issuecomment-1398576883) on [issue #1615](https://github.com/YunoHost/apps/pull/1615) Update app levels accoring to CI results: (NVM this is me testing the new script)
[15:42:41]
<Yunohost Git/Infra notifications> [apps] @alexAubin closed [pull request #1615](https://github.com/YunoHost/apps/pull/1615): Update app levels accoring to CI results
[15:47:25]
<Yunohost Git/Infra notifications> [apps] @alexAubin deleted branch update_app_levels
[15:47:25]
<Aleks (he/him/il/lui)> so i'm adding a new python script to apps to replace this horror : https://github.com/YunoHost/yunorunner/blob/master/maintenance/update_level_apps.sh
[15:47:25]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 1 commit to toml-all-the-things: Add a new script to update levels according to CI results and create the corresponding PR ([7904db0c](https://github.com/YunoHost/apps/commit/7904db0c6a6c399e5e39ae0c8988f8af80faa327))
[15:47:25]
<Aleks (he/him/il/lui)> which will also point out outdated / missing tests (which can be due to a various number of things, such as mismatching id in manifest and catalog, among other things)
[16:02:57]
<Salamandar> "Python all the bash things !"
[16:05:58]
<Gérard Collin> > <@Salamandar:matrix.org> "Python all the bash things !"
Salamandar: JavaScript all the python stuff !! 😀
[16:06:10]
<Salamandar> o no
[16:07:46]
<Aleks (he/him/il/lui)> u wot m8
[16:08:45]
<tituspijean> YunoHost - NodeJS edition
[16:10:08]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 24 commits to toml-all-the-things ([7904db0c6a6c...0aa3c9a5a53b](https://github.com/YunoHost/apps/compare/7904db0c6a6c...0aa3c9a5a53b))
[16:10:14]
<Yunohost Git/Infra notifications> [apps/toml-all-the-things] Merge pull request #1614 from Salamandar/patch-2 Add Endi to catalog - Éric Gaspar
[16:10:17]
<Yunohost Git/Infra notifications> [apps/toml-all-the-things] fix vaultwarden logo - Éric Gaspar
[16:10:21]
<Yunohost Git/Infra notifications> [apps/toml-all-the-things] Merge remote-tracking branch origin/master into toml-all-the-things - Alexandre Aubin
[16:14:49]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 1 commit to toml-all-the-things: update_app_levels: autosort keys ([cd4e6f0c](https://github.com/YunoHost/apps/commit/cd4e6f0cbc2168e5f8899d08fef0703b3888ed61))
[16:18:40]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 1 commit to toml-all-the-things: ci: missing toml python module, lets try using python 3.11 idk ([2c42bb2c](https://github.com/YunoHost/apps/commit/2c42bb2c38d11e6efd6b01fdbd660382966a56ae))
[16:20:17]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 1 commit to toml-all-the-things: ci: mokay we need to explicitly install python toml i guess ([bdad939c](https://github.com/YunoHost/apps/commit/bdad939c580d2c5e3690ca60c1b25e3cef599c38))
[16:21:19]
<Aleks (he/him/il/lui)> ooooh there's an `yq` that parses yaml, xml and toml ? :O
[16:24:09]
<Aleks (he/him/il/lui)> meh fuck it we need a real python script
[16:33:16]
<Salamandar> ah ah
[16:33:47]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 1 commit to toml-all-the-things: ci: Add a proper catalog linter instead of bash mess ([485861e7](https://github.com/YunoHost/apps/commit/485861e7a887e423c0bc01f5bd76351462e2472d))
[16:34:43]
<Aleks (he/him/il/lui)> theeere we go
[16:34:51]
<Aleks (he/him/il/lui)> mergin' o/
[16:35:07]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 12 commits to master ([d81d17c795af...0b67b0711c18](https://github.com/YunoHost/apps/compare/d81d17c795af...0b67b0711c18))
[16:35:07]
<Yunohost Git/Infra notifications> [apps] @alexAubin merged [pull request #1612](https://github.com/YunoHost/apps/pull/1612): TOML all the things ?
[16:35:11]
<Yunohost Git/Infra notifications> [apps/master] ci: mokay we need to explicitly install python toml i guess - Alexandre Aubin
[16:35:12]
<Yunohost Git/Infra notifications> [apps] @alexAubin deleted branch toml-all-the-things
[16:35:15]
<Yunohost Git/Infra notifications> [apps/master] ci: Add a proper catalog linter instead of bash mess - Alexandre Aubin
[16:35:16]
<Yunohost Git/Infra notifications> [apps/master] Merge pull request #1612 from YunoHost/toml-all-the-things TOML all the things ? - Alexandre Aubin
[16:37:35]
<Aleks (he/him/il/lui)> such modernity
[16:38:08]
<Aleks (he/him/il/lui)> how many things are going to break because of this ? place your bets
[16:57:21]
<Yunohost Git/Infra notifications> [package_linter] @alexAubin pushed 1 commit to master: The apps catalog is in json now ([078f29de](https://github.com/YunoHost/package_linter/commit/078f29de39b4e1d70138e829382dfe66f3f1bc2a))
[16:57:44]
<Aleks (he/him/il/lui)> uh i meant toml*
[16:59:44]
<Aleks (he/him/il/lui)> so i've set the new update app cron to run at 6pm
[16:59:53]
<Aleks (he/him/il/lui)> maaaaaybe it's gonna work on first try ?
[17:00:35]
<Aleks (he/him/il/lui)> ah the server is late 1 hour
[17:00:55]
<Aleks (he/him/il/lui)> fukit let's hack the cron definition
[17:07:20]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 1 commit to master: update_app_level: use absolute path for github token fetching ([e0832609](https://github.com/YunoHost/apps/commit/e0832609b16a841bd76027fb4ae1acf32cf6ae57))
[17:08:15]
<Yunohost Git/Infra notifications> [apps] @yunohost-bot pushed 1 commit to update_app_levels: Update app levels according to CI results ([34e62799](https://github.com/YunoHost/apps/commit/34e627990d18cbaad80bcfa3b529a7028b920857))
[17:08:15]
<Yunohost Git/Infra notifications> [apps] @yunohost-bot opened [pull request #1616](https://github.com/YunoHost/apps/pull/1616): Update app levels accoring to CI results
[17:08:16]
<Yunohost Git/Infra notifications> [apps] @yunohost-bot created new branch update_app_levels
[17:08:21]
<Aleks (he/him/il/lui)> #totallyfirsttry
[17:09:50]
<Aleks (he/him/il/lui)> zomg there's a typo in the title
[17:10:02]
<Yunohost Git/Infra notifications> [apps] @alexAubin edited [pull request #1616](https://github.com/YunoHost/apps/pull/1616): Update app levels according to CI results
[17:10:48]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 1 commit to master: Typo /o\ ([d7f0f40e](https://github.com/YunoHost/apps/commit/d7f0f40e885667f61186b7ff38eb00819a720636))
[17:11:45]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 1 commit to master: update_app_levels: Minor quirk in major/minor regression definition ([a6465fd4](https://github.com/YunoHost/apps/commit/a6465fd4215ca84980280e73a99df5cc47135aaf))
[17:12:54]
<Yunohost Git/Infra notifications> [apps] @alexAubin edited [pull request #1616](https://github.com/YunoHost/apps/pull/1616): Update app levels according to CI results
[18:07:30]
<Gérard Collin> > <@Salamandar:matrix.org> "Python all the bash things !"
Or, python (with basic object model) handling logic and files modifications and calling shell scripts just for running commands.
[18:38:07]
<Salamandar> who talked about Zammad here again ?
I'm starting the package
but it's crap, they need Ruby 3
debian 11 is at ruby 2.7
Or they provide a 630MB on disk .deb that includes ruby, gems etc
[18:40:18]
<Yunohost Git/Infra notifications> [apps] @ericgaspar edited [pull request #1616](https://github.com/YunoHost/apps/pull/1616): Update app levels according to CI results
[18:43:08]
<Yunohost Git/Infra notifications> [apps] @ericgaspar edited [pull request #1616](https://github.com/YunoHost/apps/pull/1616): Update app levels according to CI results
[18:44:32]
<badri> Hi all! What's the syntax for naming a patchfile in a YunoHost package?
[18:44:41]
<Yunohost Git/Infra notifications> [apps] @ericgaspar edited [pull request #1616](https://github.com/YunoHost/apps/pull/1616): Update app levels according to CI results
[18:44:53]
<badri> The docs say `sources/patches/${src_id}-*.patch`
[18:45:43]
<badri> But what is `$src_id` exactly? I want to patch the `Gemfile` in the root directory, so would `sources/patches/Gemfile-something.patch` work?
[18:45:58]
<Salamandar> The name of the app.src it refers to
[18:46:08]
<Gérard Collin> > <badri> Hi all! What's the syntax for naming a patchfile in a YunoHost package?
I think the src-id is the name of your source file. It's app in the exemple
[18:46:23]
<badri> Oh right! I'll try that
[18:46:25]
<badri> Thanks 🙂
[18:46:33]
<Salamandar> So patches would usually be named app-001-some-patch-on-gemfile.patch
[18:46:53]
<eric_G> https://ci-apps.yunohost.org/ci/job/13048
[18:46:57]
<eric_G> `! The license id 'MIT' is not registered in https://spdx.org/licenses/. `
[18:47:06]
<eric_G> What! oO
[18:47:40]
<badri> > So patches would usually be named app-001-some-patch-on-gemfile.patch
Gotcha 🙂
[18:49:15]
<eric_G> and again for this license: https://ci-apps.yunohost.org/ci/job/13050
[18:52:26]
<Yunohost Git/Infra notifications> [apps] @ericgaspar edited [pull request #1616](https://github.com/YunoHost/apps/pull/1616): Update app levels according to CI results
[18:54:15]
<Yunohost Git/Infra notifications> [apps] @ericgaspar edited [pull request #1616](https://github.com/YunoHost/apps/pull/1616): Update app levels according to CI results
[18:55:38]
<Yunohost Git/Infra notifications> [apps] @ericgaspar edited [pull request #1616](https://github.com/YunoHost/apps/pull/1616): Update app levels according to CI results
[19:00:49]
<Yunohost Git/Infra notifications> [my_webapp_ynh] @ericgaspar pushed 1 commit to testing: Fix linter ([849d85b1](https://github.com/YunoHost-Apps/my_webapp_ynh/commit/849d85b1c7fcff34f071273a66b659647a1440e2))
[19:01:18]
<Yunohost Git/Infra notifications> [my_webapp_ynh] @ericgaspar pushed 2 commits to new-PHP-version ([76805f12cc37...49cab9e4b300](https://github.com/YunoHost-Apps/my_webapp_ynh/compare/76805f12cc37...49cab9e4b300))
[19:01:22]
<Yunohost Git/Infra notifications> [my_webapp_ynh/new-PHP-version] Fix linter - Éric Gaspar
[19:01:26]
<Yunohost Git/Infra notifications> [my_webapp_ynh/new-PHP-version] Merge branch testing into new-PHP-version - Éric Gaspar
[19:01:28]
<Yunohost Git/Infra notifications> [my_webapp_ynh] @ericgaspar pushed 2 commits to extra_php-fpm ([45c0c4fa5554...dca89aa5dc54](https://github.com/YunoHost-Apps/my_webapp_ynh/compare/45c0c4fa5554...dca89aa5dc54))
[19:01:32]
<Yunohost Git/Infra notifications> [my_webapp_ynh/extra_php-fpm] Fix linter - Éric Gaspar
[19:01:35]
<Yunohost Git/Infra notifications> [my_webapp_ynh/extra_php-fpm] Merge branch testing into extra_php-fpm - Éric Gaspar
[19:02:20]
<Yunohost Git/Infra notifications> [apps] @ericgaspar edited [pull request #1616](https://github.com/YunoHost/apps/pull/1616): Update app levels according to CI results
[19:02:23]
<Yunohost Git/Infra notifications> [apps] @Limezy opened [pull request #1617](https://github.com/YunoHost/apps/pull/1617): Add moncycle.app to catalog
[19:06:49]
<Yunohost Git/Infra notifications> [apps] @ericgaspar edited [pull request #1616](https://github.com/YunoHost/apps/pull/1616): Update app levels according to CI results
[19:07:28]
<Yunohost Git/Infra notifications> [apps] @ericgaspar edited [pull request #1616](https://github.com/YunoHost/apps/pull/1616): Update app levels according to CI results
[19:10:10]
<florent> > <@Salamandar:matrix.org> who talked about Zammad here again ?
> I'm starting the package
> but it's crap, they need Ruby 3
> debian 11 is at ruby 2.7
> Or they provide a 630MB on disk .deb that includes ruby, gems etc
Hello! It's a me! <del>Mario</del> Florent
[19:12:18]
<Yunohost Git/Infra notifications> [apps] @ericgaspar edited review [pull request #1617](https://github.com/YunoHost/apps/pull/1617#pullrequestreview-1264193819): Add moncycle.app to catalog
[19:12:19]
<Yunohost Git/Infra notifications> [apps] @ericgaspar [commented](https://github.com/YunoHost/apps/pull/1617#discussion_r1082954273) on pull request #1617 Add moncycle.app to catalog: moncycle.app is that the name of the app upstream?
[19:12:55]
<florent> You may use rvm for setting the right ruby version. Or maybe check the mastodon_ynh package which has the same issue regarding the use of Ruby
[19:13:56]
<Yunohost Git/Infra notifications> [apps] @ericgaspar [commented](https://github.com/YunoHost/apps/pull/1616#issuecomment-1398823695) on [issue #1616](https://github.com/YunoHost/apps/pull/1616) Update app levels according to CI results: I dont know why [Dupral](https://ci-apps.yunohost.org/ci/apps/drupal/latestjob) and [Isso](https://ci-apps.yunohost.org...
[19:14:38]
<florent> This may help with Ruby: https://github.com/YunoHost-Apps/Experimental_helpers/blob/master/ynh_install_ruby/ynh_install_ruby__2
[19:24:02]
<Yunohost Git/Infra notifications> Application ifconfig-io rises from level (unknown) to level 7 on https://ci-apps.yunohost.org/ci/job/13054
[19:33:40]
<Salamandar> > This may help with Ruby: https://github.com/YunoHost-Apps/Experimental_helpers/blob/master/ynh_install_ruby/ynh_install_ruby__2
> ```
> test -x /usr/bin/ruby && mv /usr/bin/ruby /usr/bin/ruby_rbenv
> ```
Wait wat
[19:43:49]
<florent> There is a rollback afterwards:
https://github.com/YunoHost-Apps/Experimental_helpers/blob/fe4757bf54ddb3a5ac1a71b88b976934f81695cd/ynh_install_ruby/ynh_install_ruby__2#L211
[19:44:20]
<florent> However, yeah, if the function fails, the system may be broken with no ruby binary
[19:57:00]
<Yunohost Git/Infra notifications> [apps] @yalh76 [commented](https://github.com/YunoHost/apps/pull/1616#issuecomment-1398863978) on [issue #1616](https://github.com/YunoHost/apps/pull/1616) Update app levels according to CI results: > I dont know why [Dupral](https://ci-apps.yunohost.org/ci/apps/drupal/latestjob) and [Isso](https://ci-apps.yunohost.o...
[20:01:10]
<Yunohost Git/Infra notifications> [apps] @yalh76 edited [pull request #1616](https://github.com/YunoHost/apps/pull/1616): Update app levels according to CI results
[20:02:07]
<Yunohost Git/Infra notifications> [apps] @yalh76 edited [pull request #1616](https://github.com/YunoHost/apps/pull/1616): Update app levels according to CI results
[20:03:23]
<Yunohost Git/Infra notifications> [apps] @yalh76 pushed 1 commit to update_app_levels: Fix petrolette level ([bfe8b944](https://github.com/YunoHost/apps/commit/bfe8b944ca998bfe98bd6934ed02a7f7ccdd27cb))
[20:03:27]
<Yunohost Git/Infra notifications> [apps] @yalh76 edited [pull request #1616](https://github.com/YunoHost/apps/pull/1616): Update app levels according to CI results
[20:04:38]
<Yunohost Git/Infra notifications> [apps] @yalh76 edited [pull request #1616](https://github.com/YunoHost/apps/pull/1616): Update app levels according to CI results
[20:06:31]
<Yunohost Git/Infra notifications> [my_webapp_ynh] @fflorent [commented](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/101#issuecomment-1398873193) on [issue #101](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/101) Allow to configure extra nginx conf in app.d/ directory: Run package_check locally for a5d3f51aa5253ce7f96cd0f419bbd312e86761ed, I got this: ===============================...
[20:09:38]
<Yunohost Git/Infra notifications> [my_webapp_ynh] @fflorent [commented](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/101#issuecomment-1398876643) on [issue #101](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/101) Allow to configure extra nginx conf in app.d/ directory: testme
[20:09:41]
<Yunohost Git/Infra notifications> [my_webapp_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/101#issuecomment-1398876701) on [issue #101](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/101) Allow to configure extra nginx conf in app.d/ directory: May the CI gods be with you
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/...
[20:17:59]
<Yunohost Git/Infra notifications> [apps] @yalh76 [commented](https://github.com/YunoHost/apps/pull/1617#discussion_r1083011886) on pull request #1617 Add moncycle.app to catalog: it seems that the master branch is far from working
[20:18:02]
<Yunohost Git/Infra notifications> [apps] @yalh76 changes_requested [pull request #1617](https://github.com/YunoHost/apps/pull/1617#pullrequestreview-1264287426) Add moncycle.app to catalog
[20:18:03]
<Yunohost Git/Infra notifications> [apps] @yalh76 edited review [pull request #1617](https://github.com/YunoHost/apps/pull/1617#pullrequestreview-1264287426): Add moncycle.app to catalog
[20:18:08]
<Yunohost Git/Infra notifications> WARNING: unknown pull_request action: converted_to_draft
[20:34:12]
<Salamandar> > However, yeah, if the function fails, the system may be broken with no ruby binary
Yes that's what I'm thinking about
[20:37:29]
<florent> Hmm, try/catch in shell would be neat
[20:42:46]
<Salamandar> lol yeah indeed
[21:52:10]
<Yunohost Git/Infra notifications> [apps] @alexAubin [commented](https://github.com/YunoHost/apps/pull/1617#discussion_r1083081681) on pull request #1617 Add moncycle.app to catalog: (the testing one seems more promising)
[21:59:40]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 1 commit to master: list_builder: fix subtags not being a list, breaking yunohost ([9c1b1e69](https://github.com/YunoHost/apps/commit/9c1b1e69061144883f7b16d296ce74fe838c99f2))
[22:56:48]
<Gérard Collin> > Hmm, try/catch in shell would be neat
It seems like something similar exist: https://stackoverflow.com/a/43418467
[22:56:49]
<Gérard Collin> Some comments:
'''You can use trap:
try { block A } catch { block B } finally { block C }
translates to:
(
set -Ee
function _catch {
block B
exit 0 # optional; use if you don't want to propagate (rethrow) error to outer shell
}
function _finally {
block C
}
trap _catch ERR
trap _finally EXIT
block A
)
'''
[22:56:49]
<Gérard Collin> There is even a bash framework for error handling: https://github.com/niieani/bash-oo-framework#error-handling-with-exceptions-and-throw