Wednesday, February 22, 2023
dev@conference.yunohost.org
February
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
         
             

[07:43:30] <Yunohost Git/Infra notifications> [YunoHost demo] [🔴 Down] Request failed with status code 502
[07:53:33] <Yunohost Git/Infra notifications> [YunoHost demo] [✅ Up] 200 - OK
[15:08:14] <Yunohost Git/Infra notifications> [issues] @fflorent opened [issue #2155](https://github.com/YunoHost/issues/issues/2155): convert_app_to_packaging_v2.py does not convert updater.sh
[15:08:14] <Yunohost Git/Infra notifications> [issues] @fflorent labeled :space_invader: bug on [issue #2155](https://github.com/YunoHost/issues/issues/2155): convert_app_to_packaging_v2.py does not convert updater.sh
[15:09:34] <Yunohost Git/Infra notifications> [issues] @fflorent [commented](https://github.com/YunoHost/issues/issues/2155#issuecomment-1440224624) on [issue #2155](https://github.com/YunoHost/issues/issues/2155) convert_app_to_packaging_v2.py does not convert updater.sh: Also the example_ynh package should be updated once the issue is solved: https://github.com/YunoHost/example_ynh/blob/...
[15:10:11] *tituspijean[m] hides in bush
[15:10:27] <florent[m]> > * <@titus:pijean.ovh> hides in bush

About the issue I raised? ^^'
[15:10:46] <tituspijean[m]> Yup 😅
[15:11:20] <florent[m]> I'd be glad to help. Mayhaps a awk script could be used for that (however, I am not this kind of wizard)
[15:11:44] <Yunohost Git/Infra notifications> [issues] @alexAubin [commented](https://github.com/YunoHost/issues/issues/2155#issuecomment-1440229632) on [issue #2155](https://github.com/YunoHost/issues/issues/2155) convert_app_to_packaging_v2.py does not convert updater.sh: >This script used to invoke jq to extract the relevant keys, like upstream.code or version. The equivalent tool for that...
[15:12:14] <Aleks (he/him/il/lui)> but what's the status of that plan to centralize part of the updater thingy ? 😬
[15:13:13] <tituspijean[m]> I have yet to suceed using PyGithub to push commits :/
[15:13:18] <Aleks (he/him/il/lui)> btw `grep`/`sed` can also be okay-ish (i mean `jq` is essentially needed because all the quotes + comma are hell to parse)
[15:13:28] <Aleks (he/him/il/lui)> ah o_O
[15:13:47] <Aleks (he/him/il/lui)> bro you're updating task status and opening PRs and stuff but can't push commit ? 😅
[15:13:50] <tituspijean[m]> But I'm stubborn. We could rely on an os call
[15:13:56] <Aleks (he/him/il/lui)> dont you just need `git push` ? 😅
[15:14:28] <tituspijean[m]> > <@Alekswag:matrix.org> dont you just need `git push` ? 😅

From Python? That's a mystery for me xD
[15:14:55] <Aleks (he/him/il/lui)> `os.system('git push origin master')` ? 😬
[15:14:55] <Aleks (he/him/il/lui)> but yeah that's pretty hackish
[15:15:10] <tituspijean[m]> PyGithub offers ways to update files on the fly without even pulling repos
[15:15:26] <Aleks (he/him/il/lui)> ooh okay
[15:15:27] <tituspijean[m]> That's why I wanted to avoid the os.system thingy
[15:15:46] <tituspijean[m]> (╯°□°)╯︵ ┻━┻ IF IT WAS WORKING
[15:16:58] <Aleks (he/him/il/lui)> <whisper>at least `git push` is forge-agnostic 👀</whisper>
[15:17:51] <florent[m]> https://github.com/actions/runner-images/blob/main/images/linux/Ubuntu2204-Readme.md
> - yq 4.30.8
[15:17:59] <florent[m]> Bouhia!
[15:17:59] <tituspijean[m]> *oh you*
[15:18:21] <florent[m]> `yq` seems actually embedded in Ubuntu image provided by github actions!
[15:19:46] <florent[m]> Question: if the Github action fails, what happens?
[15:19:58] <florent[m]> Is there any kind of report?
[15:20:25] <tituspijean[m]> Yup, maintainers are notified by Github
[15:20:56] <florent[m]> However, logically some maintainers here should have been noticed already, don't they?
[15:21:35] <florent[m]> As the `manifest.json` file does not exist anymore in v2 and is still used in `updater.sh`
[15:21:36] <tituspijean[m]> Not that many apps have been converted, I guess.
[15:22:27] <florent[m]> My point being that I would propose to be sure to have error reports first before fixing stuffs, if you see what I mean
[15:22:32] <tituspijean[m]> Maybe eric has in mind if he converted apps that have an autoupdater.
[15:23:08] <florent[m]> Yeah, I had in mind Omeka-s, indeed, which has been converted recently (poke eric_G )
[15:29:04] <florent[m]> The currently converted repositories: https://github.com/search?q=org%3AYunoHost-Apps+filename%3A%22manifest.toml%22&type=code
[15:32:18] <Aleks (he/him/il/lui)> yeah i was thinking we should have some dashboard in tartiflette for v2 apps maybe
[15:32:43] <Yunohost Git/Infra notifications> [issues] @fflorent [commented](https://github.com/YunoHost/issues/issues/2155#issuecomment-1440263404) on [issue #2155](https://github.com/YunoHost/issues/issues/2155) convert_app_to_packaging_v2.py does not convert updater.sh: yq seems to be embedded in Ubuntu image in Github action: https://github.com/actions/runner-images/blob/main/images/li...
[15:32:51] <tituspijean[m]> *Tartiflette refactor imminent*
[16:04:08] <Yunohost Git/Infra notifications> [issues] @fflorent [commented](https://github.com/YunoHost/issues/issues/2155#issuecomment-1440314362) on [issue #2155](https://github.com/YunoHost/issues/issues/2155) convert_app_to_packaging_v2.py does not convert updater.sh: The fix I made for updater.sh, FWIW: https://github.com/fflorent/healthchecks_ynh/commit/34d6ad8876989e61e6e9d9470ec7a7a...
[17:47:22] <tituspijean[m]> https://aria.im/_matrix/media/v1/download/pijean.ovh/19b7f301774f2b354ffd4c7f3d34d6604a84f512b86c43e549178a2d243c6570
[17:47:26] <tituspijean[m]> no way
[17:47:41] <tituspijean[m]> > GitHub is gradually rolling out IPv6
[17:48:07] <tituspijean[m]> Hell shall freeze and pigs will fly
[18:34:19] <Yunohost Git/Infra notifications> [issues] @ericgeldmacher labeled :space_invader: bug on [issue #2156](https://github.com/YunoHost/issues/issues/2156): Decoding errors not handled when trying to read service logs.
[18:34:19] <Yunohost Git/Infra notifications> [issues] @ericgeldmacher opened [issue #2156](https://github.com/YunoHost/issues/issues/2156): Decoding errors not handled when trying to read service logs.
[18:37:32] <Yunohost Git/Infra notifications> [issues] @ericgeldmacher [commented](https://github.com/YunoHost/issues/issues/2156#issuecomment-1440586835) on [issue #2156](https://github.com/YunoHost/issues/issues/2156) Decoding errors not handled when trying to read service logs.: Looks like this may be that same issue here: #2002
[18:39:37] <Yunohost Git/Infra notifications> [issues] @ericgeldmacher [commented](https://github.com/YunoHost/issues/issues/2156#issuecomment-1440586835) on [issue #2156](https://github.com/YunoHost/issues/issues/2156) Decoding errors not handled when trying to read service logs.: Looks like this may be the same issue here: #2002
[18:40:01] <Aleks (he/him/il/lui)> > <@titus:pijean.ovh> sent an image.

what the actual fuck
[18:40:05] <Aleks (he/him/il/lui)> is it april 1st already
[18:40:35] <Aleks (he/him/il/lui)> ah "rolling out" like "enabling" ?
[18:46:15] <Aleks (he/him/il/lui)> i was thinkin it meant disabling
[18:46:16] <Yunohost Git/Infra notifications> [issues] @ericgeldmacher edited [issue #2156](https://github.com/YunoHost/issues/issues/2156): Decoding errors not handled when trying to read service logs.
[19:01:59] <Yunohost Git/Infra notifications> [issues] @ericgeldmacher edited [issue #2156](https://github.com/YunoHost/issues/issues/2156): Decoding errors not handled when trying to read service logs.
[20:34:59] <tituspijean[m]> Dante thought everything out. 😅
[20:54:20] <Yunohost Git/Infra notifications> [YunoHost demo] [🔴 Down] Request failed with status code 502
[20:59:24] <Yunohost Git/Infra notifications> [YunoHost demo] [✅ Up] 200 - OK
[21:49:12] <tituspijean[m]> (for people receiving these notifications, I'm the one who has just created a PAT for yunohost-bot on Github)
[21:56:14] <Yunohost Git/Infra notifications> [issues] @Billos labeled :birthday: feature on [issue #2157](https://github.com/YunoHost/issues/issues/2157): SSHD whitelist for Root login
[21:56:14] <Yunohost Git/Infra notifications> [issues] @Billos opened [issue #2157](https://github.com/YunoHost/issues/issues/2157): SSHD whitelist for Root login
[22:09:17] <Yunohost Git/Infra notifications> [issues] @alexAubin closed [issue #2157](https://github.com/YunoHost/issues/issues/2157): SSHD whitelist for Root login
[22:09:18] <Yunohost Git/Infra notifications> [issues] @alexAubin [commented](https://github.com/YunoHost/issues/issues/2157#issuecomment-1440874988) on [issue #2157](https://github.com/YunoHost/issues/issues/2157) SSHD whitelist for Root login: Hmf we already had https://github.com/YunoHost/yunohost/pull/1551 recently ... I feel very meh about those kind of featu...
[22:09:20] <Yunohost Git/Infra notifications> [issues] @alexAubin reopened [issue #2157](https://github.com/YunoHost/issues/issues/2157): SSHD whitelist for Root login
[23:04:13] <Yunohost Git/Infra notifications> [issues] @Billos [commented](https://github.com/YunoHost/issues/issues/2157#issuecomment-1440973341) on [issue #2157](https://github.com/YunoHost/issues/issues/2157) SSHD whitelist for Root login: By default the login on root is disabled, an option like this would avoid a bit more being locked out of a server