Thursday, August 29, 2024
dev@conference.yunohost.org
August
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
 
             

[00:01:36] <Yunohost Git/Infra notifications> 🏗️ Starting build for yunohost/11.2.29+202408282230 for bullseye/unstable/all ...
[00:08:41] <Yunohost Git/Infra notifications> ✔️ Completed build for yunohost/11.2.29+202408282230 for bullseye/unstable/all.
[01:38:48] <Yunohost Git/Infra notifications> [yunohost] 🔴 Pipeline [#1431261248](https://gitlab.com/YunoHost/yunohost/-/pipelines/1431261248) failed on branch dev
[08:50:53] <Yunohost Git/Infra notifications> [yunohost-admin] Axolotle pushed 1 commit to bookworm: fix: ConfigPanel help message ([7859b221](https://github.com/YunoHost/yunohost-admin/commit/7859b221fc57c4f5c4e07c79f76407c1b17d4887))
[09:00:31] <Yunohost Git/Infra notifications> 🏗️ Starting build for yunohost-admin/12.0.1+202408290900 for bookworm/unstable/all ...
[09:04:39] <Yunohost Git/Infra notifications> ✔️ Completed build for yunohost-admin/12.0.1+202408290900 for bookworm/unstable/all.
[11:33:02] <selfhoster1312> can i use package_check with a custom ynh-dev environment ?
[12:01:20] <Aleks (he/him/il/lui)> Assuming you're able to install lxd/incus inside it, I suppose
[13:22:17] <selfhoster1312> is there a reason it wouldn't be possible for package_check to go from base image, add a ynh-dev use-git step and then use that?
[13:23:06] <selfhoster1312> (it seems simple to me but maybe i'm missing something obvious)
[13:23:38] <Aleks (he/him/il/lui)> i don't know what you mean ... you mean to simplify running package_check locally ?
[13:24:15] <selfhoster1312> yeah i mean: if i make a yunohost patch, how am i supposed to verify it works on a real package? so i'd like to use package_check with that modified yunohost branch
[13:24:28] <selfhoster1312> (maybe i'm approaching this from the wrong angle ;))
[13:25:08] <Aleks (he/him/il/lui)> personally I just run `yunohost app install` and possibly other commands like upgrade / backup ... no need to run the full package check suite
[13:25:43] <selfhoster1312> ok
[15:48:19] <Yunohost Git/Infra notifications> [yunohost] alexAubin merged [pull request #1939](https://github.com/YunoHost/yunohost/pull/1939): docs: ynh_install_app_dependencies -> ynh_apt_install_dependencies
[15:48:20] <Yunohost Git/Infra notifications> [yunohost] alexAubin pushed 2 commits to dev ([e3e8b903c77b...7a04462ccde9](https://github.com/YunoHost/yunohost/compare/e3e8b903c77b...7a04462ccde9))
[15:56:44] <selfhoster1312> https://github.com/YunoHost/yunohost/blob/dev/helpers/helpers.v2.1.d/apt#L30
[15:56:50] <selfhoster1312> https://xmpp-upload.kl.netlib.re/upload/2oOeZmbZFLUE7009/here-be-dragons.jpg
[15:57:25] <Aleks (he/him/il/lui)> not even close to the config panel regexes 😬
[15:58:06] <selfhoster1312> oh my
[15:58:38] <Aleks (he/him/il/lui)> https://github.com/YunoHost/yunohost/blob/dev/helpers/helpers.v2.1.d/templating#L305-L333
[15:58:39] <selfhoster1312> « parse, don't validate » 🤯️
[15:58:44] <Aleks (he/him/il/lui)> let me sing you the song of the config panel people
[15:58:57] <selfhoster1312> waaaat
[15:58:59] <Aleks (he/him/il/lui)> `echo "$value" | sed "s/'/\\\\\\\\'/g"`
[15:59:43] <selfhoster1312> why do we even have that ynh_write_var_in_file ? how is it better than a simple bash string replace with a placeholder?
[15:59:58] <Aleks (he/him/il/lui)> 😬
[16:00:09] <Aleks (he/him/il/lui)> do you really wanna know
[16:00:22] <selfhoster1312> well it's not so used by apps: https://github.com/search?q=org%3Ayunohost-apps%20ynh_write_var_in_file&type=code
[16:00:28] <selfhoster1312> i'm afraid to do the same search in helpers :D
[16:00:41] <Aleks (he/him/il/lui)> it's definitely used in the helpers 😬
[16:00:56] <Yunohost Git/Infra notifications> 🏗️ Starting build for yunohost/11.2.29+202408291600 for bullseye/unstable/all ...
[16:01:12] <selfhoster1312> not so much actually
[16:01:22] <selfhoster1312> we could probably get rid of this monstrosity?
[16:01:40] <Aleks (he/him/il/lui)> also i wouldnt rely on github search, it's just complete batshit
[16:01:41] <selfhoster1312> i mean most use i see is for JSON/YAML keys we're probably better off with jq/yq no ?
[16:01:45] <selfhoster1312> (yes it is :()
[16:01:52] <Aleks (he/him/il/lui)> at it does find it in the apps ?
[16:01:54] <selfhoster1312> (and it brings up tons of archived repos code...)
[16:02:03] <selfhoster1312> yes
[16:02:08] <selfhoster1312> see my link above
[16:02:32] <Aleks (he/him/il/lui)> hmm grep.app only finds 2 repo wtf https://grep.app/search?q=ynh_write_var_in_file&filter[repo.pattern][0]=yunohost-apps
[16:03:31] <Yunohost Git/Infra notifications> ✔️ Completed build for yunohost/11.2.29+202408291600 for bullseye/unstable/all.
[16:05:24] <Salamandar> > <@Alekswag:matrix.org> hmm grep.app only finds 2 repo wtf https://grep.app/search?q=ynh_write_var_in_file&filter[repo.pattern][0]=yunohost-apps

what is that
[16:05:39] <Aleks (he/him/il/lui)> grep.app 👌
[16:05:55] <Salamandar> is it better than github search ?
[16:05:59] <Aleks (he/him/il/lui)> i thought so but now i'm doubtful
[16:06:21] <Aleks (he/him/il/lui)> or maybe github's search is just less crap than before
[16:13:01] <selfhoster1312> oh my god why am i reading the config helpers
[16:13:28] <Aleks (he/him/il/lui)> i'm sorry for your mental sanity
[16:14:00] <selfhoster1312> i'm sorry for yours?!
[16:14:04] <Aleks (he/him/il/lui)> shouldnt have mentionned it 😬
[16:14:40] <selfhoster1312> now i know why even installing my_webapp or reverse_proxy is so slow... you have to spawn thousands of subprocesses for just basic config/template u_u
[16:14:58] <selfhoster1312> and evaluate many regexes
[16:15:42] <Aleks (he/him/il/lui)> not sure that's relevant during install, yunohost app install doesn't call anything related to config panel (those helpers are supposedly only involved in the config panel get / apply)
[16:16:14] <selfhoster1312> ok my bad that's not why everything is so slow then
[16:18:14] <selfhoster1312> (i mean 13s to apply an nginx config on a modern server is... less than satisfying)
[16:18:25] <selfhoster1312> (sorry i feel like a broken record always mentioning how everything is so slow)
[16:19:55] <Aleks (he/him/il/lui)> if by 13s you mean "running yunohost app install <app>" then it doesn't just apply an nginx conf
[16:20:20] <selfhoster1312> yeah it's making a .deb package, templating stuff...
[16:20:45] <selfhoster1312> it's still incredibly slow :-/
[16:21:17] <Aleks (he/him/il/lui)> downloads the app's code, create the app directory with its settings, configure ssowat, creates a system user, possibly creates a database, ...
[16:22:26] <Aleks (he/him/il/lui)> it's not like people install 50 my_webapp a day so they don't care wether it takes 0.2s or 13s
[16:26:40] <selfhoster1312> > # time yunohost user create test15 -d kl.netlib.re -F test15 -p test1314
> ...
> real 0m22,424s

:'(
[16:26:49] <selfhoster1312> anyway i'll keep chasing those ms :P
[16:27:57] <selfhoster1312> is there a reason something like this should not be done in app.py directly?
https://github.com/YunoHost-Apps/my_webapp_ynh/blob/master/scripts/config#L126
[16:28:19] <selfhoster1312> installing the debian dependencies...
[17:10:51] <Yunohost Git/Infra notifications> [yunohost] 🔴 Pipeline [#1432458127](https://gitlab.com/YunoHost/yunohost/-/pipelines/1432458127) failed on branch dev
[17:41:45] <Yunohost Git/Infra notifications> [nextcloud_ynh] ci-apps-dev: level 1 https://ci-apps-dev.yunohost.org/ci/job/18610 on commit https://github.com/YunoHost-Apps/nextcloud_ynh/commit/95a0b94bda61a8a692985225b3f9aacccd18628e "curl tests: auto-test assets on the home page" by @GitHub on branches curl-tests
[18:33:18] <Yunohost Git/Infra notifications> [issues] selfhoster1312 labeled :birthday: feature on [issue #2435](https://github.com/YunoHost/issues/issues/2435): Move archived/outdated apps to another Github organization
[18:33:18] <Yunohost Git/Infra notifications> [issues] selfhoster1312 opened [issue #2435](https://github.com/YunoHost/issues/issues/2435): Move archived/outdated apps to another Github organization
[18:45:22] <Yunohost Git/Infra notifications> [issues] selfhoster1312 [commented](https://github.com/YunoHost/issues/issues/2435#issuecomment-2318611755) on [issue #2435](https://github.com/YunoHost/issues/issues/2435) Move archived/outdated apps to another Github organization: It apparently can be done in a single line of bash using the official gh client with an API token with the proper perm...
[18:45:46] <selfhoster1312> so i made up a one-liner to do the migration
[18:45:47] <selfhoster1312> :)
[18:47:19] <Yunohost Git/Infra notifications> [issues] alexAubin [commented](https://github.com/YunoHost/issues/issues/2435#issuecomment-2318615688) on [issue #2435](https://github.com/YunoHost/issues/issues/2435) Move archived/outdated apps to another Github organization: Would really stick to just archiving the repo ... There should be a flag like -archive:true to remove those from searc...
[18:55:01] <Yunohost Git/Infra notifications> [issues] selfhoster1312 [commented](https://github.com/YunoHost/issues/issues/2435#issuecomment-2318632270) on [issue #2435](https://github.com/YunoHost/issues/issues/2435) Move archived/outdated apps to another Github organization: Its org:yunohost-apps NOT is:archived QUERY which is not exactly easy to remember or type. If there is a setting on t...