[00:40:52]
<Yunohost Git/Infra notifications> [package_check] @alexAubin pushed 1 commit to support-v2-apps: Typo >_> https://github.com/YunoHost/package_check/commit/1a204c659383a76d2f9d4fb15b66ec4b4d52b753
[01:03:17]
<Yunohost Git/Infra notifications> [CI_package_check] @alexAubin pushed 1 commit to master: Add 10 second tempo when another app is using the lock to prevent yunorunner from cancelling all jobs etc... https://github.com/YunoHost/CI_package_check/commit/d6bee6cee0bef22cbe43b0498aaedd90ebd7d9ca
[01:03:51]
<Yunohost Git/Infra notifications> [package_check] @alexAubin pushed 1 commit to support-v2-apps: Fixes from the battlefield https://github.com/YunoHost/package_check/commit/8cd12dab76a789e3b84fbaddf511cade8caf1629
[05:59:36]
<Yunohost Git/Infra notifications> Application simple-hash-generator rises from level (unknown) to level 7 on https://ci-apps.yunohost.org/ci/job/12931
[06:55:23]
<Guillaume Bouzige> woah the app page of ci is back !! great thanks to have a clearer vision on apps catalog...https://ci-apps.yunohost.org/ci/apps/
[07:29:26]
<Yunohost Git/Infra notifications> [apps] @ericgaspar approved pull request #1611 13ft : Add potential_alternative_to https://github.com/YunoHost/apps/pull/1611#pullrequestreview-1251023663
[07:29:32]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 3 commits to master: https://github.com/YunoHost/apps/compare/a5653675b4c8...4f9dbc287724
[07:29:32]
<Yunohost Git/Infra notifications> [apps] @ericgaspar merged pull request #1611: 13ft : Add potential_alternative_to https://github.com/YunoHost/apps/pull/1611
[07:29:35]
<Yunohost Git/Infra notifications> [apps/master] 13ft : Add potential_alternative_to - lapineige
[07:29:38]
<Yunohost Git/Infra notifications> [apps/master] Update apps.json - Éric Gaspar
[07:29:42]
<Yunohost Git/Infra notifications> [apps/master] Merge pull request #1611 from lapineige/patch-7 13ft : Add potential_alternative_to - Éric Gaspar
[07:35:30]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to master: Create flusio.png https://github.com/YunoHost/apps/commit/13cb05136f24bcef424e4c1acf2554d79b68a131
[09:32:25]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to master: Add logos https://github.com/YunoHost/apps/commit/47701dcfdebd6fd8f5c99017ac511f964d365097
[09:34:34]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to master: Create monitorix.png https://github.com/YunoHost/apps/commit/2a7fa12b49279a51e353c8e56aee074a53739a65
[09:35:17]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to master: Create mumble-web.png https://github.com/YunoHost/apps/commit/b8aa0c6fec68d7e13cecb4aa127b8a573c3271f2
[09:39:28]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to master: Create pterodactyl.png https://github.com/YunoHost/apps/commit/5d4e139dc8c65733978e90153c347e2a79240b8e
[09:51:11]
<selfhoster1312> it's back, but still showing only 1 running task?!
[13:02:29]
<Aleks (he/him/il/lui)> > woah the app page of ci is back !! great thanks to have a clearer vision on apps catalog...https://ci-apps.yunohost.org/ci/apps/
was it down ? 🤔
[13:05:50]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to master: Create chitchatter.png https://github.com/YunoHost/apps/commit/afc388aa5fb9622160eaa6c059dc39940295b885
[13:14:06]
<tituspijean> > <@Alekswag:matrix.org> was it down ? 🤔
Yup 😕
[13:36:57]
<tituspijean> Grmbl `Error: In use` is back on dev CI
[13:40:44]
<Salamandar> Hey @florent, is it you fflorent on github ?
[13:41:07]
<florent> > <@Salamandar:matrix.org> Hey @florent, is it you fflorent on github ?
Yes, it's me, why? :)
[13:41:27]
<florent> (hope I haven't made anything wrong ^^')
[13:41:35]
<Salamandar> Ah good :) I wanted to install elasticsearch7_ynh, but I can't : unbound variable in the install script
[13:41:43]
<florent> Oh!?
[13:42:04]
<Salamandar> What's weird is, the app is lvl 6, so it should be ok…
[13:42:18]
<Salamandar> ```
2023-01-17 11:36:24,247: WARNING - ./install: line 119: container: unbound variable
```
[13:42:19]
<florent> Any log by chance?
[13:42:31]
<florent> erg!!!!!
[13:42:35]
<Salamandar> And indeed i can't find where $container was supposed to be declared
[13:42:52]
<florent> meta-erg!
[13:42:58]
<Salamandar> :D
[13:43:13]
<Salamandar> it's coming from https://github.com/YunoHost-Apps/elasticsearch7_ynh/commit/9bf49e7ad00ff4f41a67f0395545c9e0ddbdd0d4
[13:43:54]
<florent> OK, $container is a special variable that is supposed to be set when you're using LXC
[13:43:55]
<Salamandar> Ah yes, so that explains why the CI is working fine
[13:43:56]
<florent> But we use an option to forbid undeclared vars
[13:43:59]
<Salamandar> you should do something like `${container:-}` to cope with an undefined variable
[13:44:06]
<florent> And foolish am I, I worked with LXD when testing ^^'
[13:44:35]
<florent> > <@Salamandar:matrix.org> you should do something like `${container:-}` to cope with an undefined variable
Thanks a lot!
[13:44:36]
<Salamandar> https://aria.im/_matrix/media/v1/download/matrix.org/AxcKLbOPTupKMzDuhYpGeZnu
[13:44:48]
<Aleks (he/him/il/lui)> :O
[13:44:55]
<Salamandar> This is the cheat sheet for the awfully non-clear bash variable substitutions
[13:45:04]
<Salamandar> > And foolish am I, I worked with LXD when testing ^^'
Ahah :)
[13:45:15]
<Aleks (he/him/il/lui)> "awfully non-clear" ? How dare you ?
[13:45:20]
<Aleks (he/him/il/lui)> How dare you stay so polite
[13:45:24]
<Salamandar> > <@Alekswag:matrix.org> How dare you stay so polite
hahahaha
[13:45:59]
<Aleks (he/him/il/lui)> gretahowdareyou.gif
[13:46:31]
<Salamandar> BTW @Aleks (he/him/il/lui) can I use the manifestv2 branch of the example_ynh repository or is it still WIP ?
[13:47:14]
<Aleks (he/him/il/lui)> hmmmm there was some movement since then but most of it is still up to date
[13:47:18]
<Salamandar> Ok !
[13:48:03]
<Salamandar> I'll try to rebase it on master, but appart from that i'll write a yunohost package for 2 softwares i want :p
[13:50:03]
<Aleks (he/him/il/lui)> To get the ~full picture I would recommend:
- reading https://github.com/YunoHost/doc/blob/resources-doc/pages/06.contribute/10.packaging_apps/14.advanced/03.packaging_v2/packaging_v2.md
- running https://github.com/YunoHost/test_apps/blob/master/manifestv2_app_ynh/convert_to_packaging_v2.py on your favourite app and then reviewing the changes in the various files
- then running the package linter on it
- the package linter will surely complain about the old doc/DISCLAIMER.md to be split into ADMIN.md, PRE_INSTALL.md (or other info which are now in the manifest.toml directly)
- and I also introduced yesteday the `tests.toml` to replace `check_process`, but now is only illustrated in https://github.com/YunoHost-Apps/helloworld_ynh/blob/master/tests.toml
[13:51:04]
<Salamandar> Thanks, will do !
[14:00:59]
<florent> Here are the fix:
- https://github.com/YunoHost-Apps/elasticsearch7_ynh/pull/4
- https://github.com/YunoHost-Apps/elasticsearch8_ynh/pull/2
- https://github.com/YunoHost-Apps/opensearch_ynh/pull/3
[14:01:40]
<florent> It seems the testing branch of elasticsearch7\_ynh works in my own server
[14:01:46]
<florent> (without LXC ^^')
[14:03:07]
<florent> Or... At least, it was installed correctly :/
[14:05:02]
<florent> Grumf
```
janv. 17 15:00:38 domain.tld systemd[1]: /etc/systemd/system/elasticsearch7.service:14: EnvironmentFile= path is not absolute, ignoring: @path.env@
```
[14:05:57]
<florent> (that's a bit frustrating when you thought having well tested that previously 😅)
[14:11:37]
<florent> OK, sounds like the `@...@` variable are gradle-specific:
https://github.com/elastic/elasticsearch/blob/be7c7415627377a1b795400fb8dfcc6cbdf0e322/distribution/build.gradle#L422-L445
I'll propose a patch soon, but I don't have time right now
[14:15:36]
<Aleks (he/him/il/lui)> this looks too much like a pokemon
[14:15:36]
<Aleks (he/him/il/lui)> `@...@`
[14:15:45]
<florent> Weird, I cannot open any issue on this repo o_O :
https://github.com/YunoHost-Apps/elasticsearch7_ynh
[14:15:55]
<Aleks (he/him/il/lui)> https://www.pokepedia.fr/images/thumb/d/da/Mimiqui_%28Forme_D%C3%A9guis%C3%A9e%29-SL.png/473px-Mimiqui_%28Forme_D%C3%A9guis%C3%A9e%29-SL.png?20160719133320
[14:17:11]
<florent> That's maybe because I asked for unforking from opensearch_ynh, but still that's weird O_o
[14:25:58]
<Salamandar> 11556 WARNING https://artifacts.elastic.co/downloads/elasticsearch/elasticsearch-8.6.0-linux-x86_64.tar.gz:
11557 WARNING 2023-01-17 14:14:19 ERROR 403: Forbidden.
[14:26:07]
<Salamandar> weird because on firefox i can dl this url
[14:26:15]
<Salamandar> https://ci-apps-dev.yunohost.org/ci/job/5163
[14:26:21]
<Salamandar> ci-specific bug ?
[14:26:29]
<Aleks (he/him/il/lui)> @_@
[14:27:05]
<Aleks (he/him/il/lui)> are you reproducing this ?
[14:27:35]
<Aleks (he/him/il/lui)> (i mean when re-running the job several times)
[14:27:42]
<Salamandar> let's see…
[14:27:49]
<tituspijean> Banned IP? :/
[14:43:57]
<Salamandar> maybe ?
[14:43:59]
<Salamandar> failed again
[14:48:34]
<tituspijean> I have no issues running the `curl` command directly from the CI server, very weird.
[14:54:46]
<florent> > <@titus:pijean.ovh> I have no issues running the `curl` command directly from the CI server, very weird.
Maybe using wget? `wget --tries 3 --no-dns-cache --timeout 900 --no-verbose https://artifacts.elastic.co/downloads/elasticsearch/elasticsearch-8.6.0-linux-x86_64.tar.gz`
[14:55:06]
<florent> This commands works on my machine
[14:57:47]
<tituspijean> Same, it works. So it's not its IP being banned
[14:59:03]
<florent> Do anyone here (from the core team) have access to the settings of this repo?
https://github.com/YunoHost-Apps/elasticsearch7_ynh
I don't have any mean to let people create issues =/
[15:00:13]
<tituspijean> I have just set you as maintainer. Do you have access?
[15:00:24]
<tituspijean> I have just set you as maintainer. Do you have access to its settings now?
[15:07:25]
<florent> Thank you
[15:07:47]
<florent> It seems the issues are back. Also thank you if activated it!
[15:32:21]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 1 commit to toml-all-the-things: TOML all the things ? https://github.com/YunoHost/apps/commit/964ec6692193baf400704ac70c9eefe4cc2c5965
[15:32:22]
<Yunohost Git/Infra notifications> [apps] @alexAubin created new branch toml-all-the-things
[15:33:45]
<Yunohost Git/Infra notifications> [apps] @alexAubin opened pull request #1612: TOML all the things ? https://github.com/YunoHost/apps/pull/1612
[15:35:35]
<Yunohost Git/Infra notifications> [apps] @alexAubin edited pull request #1612: TOML all the things ? https://github.com/YunoHost/apps/pull/1612
[15:35:59]
<Aleks (he/him/il/lui)> what do you think folks ? ^ 👀
https://github.com/YunoHost/apps/blob/toml-all-the-things/apps.toml
[15:41:45]
<eric_G[m]> much better than YAML
[15:42:03]
<eric_G[m]> or JSON...
[15:42:41]
<eric_G[m]> eh yes we have JSON 😶🌫️
[15:44:30]
<eric_G[m]> so, more readable than JSOM
[15:45:50]
<Aleks (he/him/il/lui)> we have JSON *and* YML to maximize confusion 😬
[15:45:52]
<eric_G[m]> *JSON
[15:45:54]
<eric_G[m]> damn
[15:46:48]
<eric_G[m]> I am in favor of TOML
[15:49:57]
<Salamandar> @Aleks (he/him/il/lui) hum…
[15:49:57]
<Salamandar> https://github.com/toml-lang/toml/issues/220
[15:50:14]
<Salamandar> Looks like the manifestv2 having dots in keys is bad practice :'(
[15:50:40]
<Aleks (he/him/il/lui)> :|
[15:50:47]
<Salamandar> > <@ericg:matrix.org> I am in favor of TOML
YAML is amazing but cluttered with issues (like norway being detected as `false`)… We need a "simple-yaml"
[15:51:05]
<Aleks (he/him/il/lui)> wut
[15:51:21]
<Salamandar> I searched for that because my TOML editor was detecting those dots as errors
[15:51:47]
<Salamandar> > <@Alekswag:matrix.org> wut
yeah, the country code for norway is `no`… see where it's going ?
[15:51:58]
<Aleks (he/him/il/lui)> ugh
[15:52:42]
<Salamandar> If i use double quotes the red lines go away in my toml editor
[15:52:50]
<Aleks (he/him/il/lui)> you mean `foo = norway` ? or `foo = "norway"` ?
[15:53:11]
<Salamandar> Sorry i meant about the dot in the toml keys
[15:53:22]
<Salamandar> ```
"ram.build" = "0M"
"ram.runtime" = "0M"
```
[15:54:11]
<Salamandar> But for yaml, it's if you have :
`countries: [fr, uk, no]`
its' valid yaml and gives this in python : `{"countries": ["fr", "uk", False]}`
[15:54:23]
<Aleks (he/him/il/lui)> well using that syntax it will load it as literally a key named "ram.build" instead of "ram" being a dict with "build" as a key of the dict, i think ...
[15:54:53]
<Salamandar> > <@Alekswag:matrix.org> well using that syntax it will load it as literally a key named "ram.build" instead of "ram" being a dict with "build" as a key of the dict, i think ...
Ah ! I really thought you _wanted_ a variable named `ram.build`
[15:55:03]
<Aleks (he/him/il/lui)> > <@Salamandar:matrix.org> But for yaml, it's if you have :
> `countries: [fr, uk, no]`
> its' valid yaml and gives this in python : `{"countries": ["fr", "uk", False]}`
hmokay but we're talking about using toml ? 😬
[15:55:37]
<Salamandar> > <@Alekswag:matrix.org> hmokay but we're talking about using toml ? 😬
Yes yes, i was just replying about json and yaml
[15:55:44]
<Aleks (he/him/il/lui)> oh okay 👍️
[15:56:09]
<Aleks (he/him/il/lui)> interesting fact about yaml then, i though it was a pretty decent format but sounds like every format has it quirks hehe
[15:56:17]
<Salamandar> yeah :s
[15:56:54]
<Salamandar> So, back to toml : should it be :
```
[ram]
build = "0M"
runtime = "0M"
```
?
[15:58:11]
<Aleks (he/him/il/lui)> anyway about the `foo.bar` notation in toml : yeah i'm aware there are some annoying issues with it, like when you dump a toml in python using `toml.dump()`, it will insist in having e.g.
```toml
[foo.description]
en = "EN description"
fr = "FR description"
```
instead of
```toml
description.en = "EN description"
description.fr = "FR descirption"
```
[15:58:39]
<Aleks (he/him/il/lui)> i mean switching to the "recomended" (?) 1st notation is going to be absolute hell for every place we have translated strings
[15:59:00]
<Aleks (he/him/il/lui)> (though one could also argue the proper way to handle translation would be to use an external tool, gettext etc)
[15:59:10]
<Salamandar> > <@Alekswag:matrix.org> i mean switching to the "recomended" (?) 1st notation is going to be absolute hell for every place we have translated strings
yeah i understand that
[15:59:41]
<Aleks (he/him/il/lui)> but i don't quite understand why `foo.bar` would not be recognized as proper TOML by syntax highlighting if TOML does support it :|
[15:59:42]
<Salamandar> IMHO translations should be handled somewhere else than the config file itself
[16:12:27]
<Salamandar> :p
[16:12:41]
<Salamandar> my editor is now happy with your snippet so seems legit
[16:21:24]
<Aleks (he/him/il/lui)> (i would also argue that the fact that there is more than one way to write stuff in TOML is a feature, because it allows to choose which syntax in more convenient depending on the use case)
[16:22:48]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to master: Create plateau.png https://github.com/YunoHost/apps/commit/b115bbf92f01df5df1675725a61831f386a98e2a
[19:21:29]
<Yunohost Git/Infra notifications> Application statpingng completely failed the continuous integration tests on https://ci-apps.yunohost.org/ci/job/12947
[19:21:32]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 1 commit to statpingng-id: statping ng : the app in manifest should be the same as the manifest id ... https://github.com/YunoHost/apps/commit/af6615c96e5e71e5d8f89993236e068a826209b5
[19:21:32]
<Yunohost Git/Infra notifications> [apps] @alexAubin created new branch statpingng-id
[19:21:33]
<Yunohost Git/Infra notifications> [apps] @alexAubin opened pull request #1613: statping ng : the app in manifest should be the same as the manifest id ... https://github.com/YunoHost/apps/pull/1613
[22:09:06]
<Yunohost Git/Infra notifications> Application synapse-admin rises from level (unknown) to level 6 on https://ci-apps.yunohost.org/ci/job/12953
[22:09:49]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to master: Create bookwyrm.png https://github.com/YunoHost/apps/commit/abcc8bb7a62388d172229d153c3998e921e12c90
[22:09:54]
<Yunohost Git/Infra notifications> Application technitium-dns rises from level (unknown) to level 7 on https://ci-apps.yunohost.org/ci/job/12957
[22:48:21]
<Yunohost Git/Infra notifications> Application bookwyrm stays at level 2 on https://ci-apps.yunohost.org/ci/job/12629
[23:31:48]
<Yunohost Git/Infra notifications> Application thelounge stays at level 1 on https://ci-apps.yunohost.org/ci/job/12959