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

[12:27:56] <ChriChri> https://codeberg.org/flohmarkt/flohmarkt_ynh/src/commit/1c1b5c9a3845c946e4229e5f6bb5435412f6a094/scripts/_common.sh#L49
[15:37:27] <Salamandar> FFS i got so many issues with incus and yunohost
[15:37:37] <Salamandar> and domain name resolution
[15:37:51] <Salamandar> resolution works -> yunohost tools postinstall -> resolution broken
[15:40:21] <Salamandar> yeah adding the proper dns server (host incus dnsmasq ip) in etc/dnsmasq.conf resolves the issue
[15:40:51] <Salamandar> why didn't i have this issue before ? Looks like the host dnsmasq DHCP server doesn't advertise properly the DNS ?
[16:00:12] <kayou> i had the same issue with lxd
[16:00:46] <kayou> i just change my etc/resolve.conf, and set the nameserver to something else (1.1.1.1, 8.8.8.8)
[16:02:09] <kayou> IIRC, the problem is that the dnsmasq can't start correctly inside the container, because incus (or lxd) starts his own on the same port (53)
[16:51:26] <Yunohost Git/Infra notifications> [yunohost] OniriCorpe pushed 1 commit to do-not-enable-a-service-disabled-by-the-user: do not enable a service disabled by the user ([9b769cfe](https://github.com/YunoHost/yunohost/commit/9b769cfe237b7225233d0f5194c7ec0970316f38))
[16:51:33] <Yunohost Git/Infra notifications> [yunohost] OniriCorpe created new branch do-not-enable-a-service-disabled-by-the-user
[16:54:32] <Yunohost Git/Infra notifications> [yunohost] OniriCorpe pushed 1 commit to do-not-enable-a-service-disabled-by-the-user: do not enable a service disabled by the user ([266408e2](https://github.com/YunoHost/yunohost/commit/266408e23599a27ea196fbd4cdfd3ec1ea1f2871))
[16:54:37] <Yunohost Git/Infra notifications> [yunohost] OniriCorpe opened [pull request #1849](https://github.com/YunoHost/yunohost/pull/1849): do not enable a service disabled by the user
[16:56:14] <Yunohost Git/Infra notifications> [yunohost] OniriCorpe edited [pull request #1849](https://github.com/YunoHost/yunohost/pull/1849): do not enable a service disabled by the user
[17:02:28] <Yunohost Git/Infra notifications> [yunohost] OniriCorpe pushed 1 commit to do-not-enable-a-service-disabled-by-the-user: also handle reload_or_restart and restart ([95250230](https://github.com/YunoHost/yunohost/commit/95250230bc52f60caee0ea3ebf57631c2d2b8280))
[17:24:05] <Yunohost Git/Infra notifications> [pepettes] OniriCorpe labeled bug on [issue #10](https://github.com/YunoHost/pepettes/issues/10): bookworm compatibility
[17:24:06] <Yunohost Git/Infra notifications> [pepettes] OniriCorpe opened [issue #10](https://github.com/YunoHost/pepettes/issues/10): bookworm compatibility
[17:24:08] <Yunohost Git/Infra notifications> [pepettes] OniriCorpe assigned alexAubin on [issue #10](https://github.com/YunoHost/pepettes/issues/10): bookworm compatibility
[18:25:33] <Yunohost Git/Infra notifications> [yunohost] 🔴 Pipeline [#1302797631](https://gitlab.com/YunoHost/yunohost/-/pipelines/1302797631) failed on branch do-not-enable-a-service-disabled-by-the-user
[20:54:31] <Yunohost Git/Infra notifications> [yunohost] alexAubin edited review [pull request #1849](https://github.com/YunoHost/yunohost/pull/1849#pullrequestreview-2074975179): do not enable a service disabled by the user
[20:54:31] <Yunohost Git/Infra notifications> [yunohost] alexAubin [commented](https://github.com/YunoHost/yunohost/pull/1849#discussion_r1612287656) on pull request #1849 do not enable a service disabled by the user: Hmmm I think that part should be entirely removed ... the "start_on_boot" thing is precisely the systemds enabled/disab...
[21:44:50] <Yunohost Git/Infra notifications> [yunohost] OniriCorpe [commented](https://github.com/YunoHost/yunohost/pull/1849#discussion_r1612346440) on pull request #1849 do not enable a service disabled by the user: So how can we do to not re-enable an intentionally disabled service on update?
[21:46:21] <Yunohost Git/Infra notifications> [yunohost] alexAubin [commented](https://github.com/YunoHost/yunohost/pull/1849#discussion_r1612348084) on pull request #1849 do not enable a service disabled by the user: We just remove the original systemctl enable service like you already did ? P
[21:50:53] <Yunohost Git/Infra notifications> [yunohost] OniriCorpe [commented](https://github.com/YunoHost/yunohost/pull/1849#discussion_r1612353494) on pull request #1849 do not enable a service disabled by the user: hummm yes x) why is he here in the first place?
[21:51:10] <Yunohost Git/Infra notifications> [yunohost] OniriCorpe pushed 1 commit to do-not-enable-a-service-disabled-by-the-user: Update helpers/systemd Co-authored-by: Alexandre Aubin <4533074+alexAubin@users.noreply.github.com> ([8bdaedfd](https://github.com/YunoHost/yunohost/commit/8bdaedfde749ec3ca21e879d422d5183ed82e418))
[21:51:40] <Yunohost Git/Infra notifications> [yunohost] alexAubin [commented](https://github.com/YunoHost/yunohost/pull/1849#discussion_r1612354382) on pull request #1849 do not enable a service disabled by the user: Because we did not really think about the use case of "the user wants the service to be disabled" i think
[21:53:59] <Yunohost Git/Infra notifications> [yunohost] OniriCorpe pushed 1 commit to do-not-enable-a-service-disabled-by-the-user: Update helpers/systemd Co-authored-by: Alexandre Aubin <4533074+alexAubin@users.noreply.github.com> ([e8c6abb7](https://github.com/YunoHost/yunohost/commit/e8c6abb76871f853bf9561382123faea4e16caea))
[21:56:17] <Yunohost Git/Infra notifications> [yunohost] OniriCorpe [commented](https://github.com/YunoHost/yunohost/pull/1849#discussion_r1612361012) on pull request #1849 do not enable a service disabled by the user: ah, but now if a packages uses ynh_add_systemd_config at installation, the service will not be enabled :/
[21:56:35] <Yunohost Git/Infra notifications> [yunohost] OniriCorpe [commented](https://github.com/YunoHost/yunohost/pull/1849#discussion_r1612361530) on pull request #1849 do not enable a service disabled by the user: the issue is during upgrade, not install
[21:58:06] <Aleks (he/him/il/lui)> Émy - OniriCorpe: maybe we can use `if [[ ${YNH_APP_ACTION} == "upgrade" ]]` to differentiate the upgrade case
[21:59:57] <Yunohost Git/Infra notifications> [yunohost] 🔴 Pipeline [#1303075672](https://gitlab.com/YunoHost/yunohost/-/pipelines/1303075672) failed on branch do-not-enable-a-service-disabled-by-the-user
[22:01:05] <Émy - OniriCorpe> > <@Alekswag:matrix.org> Émy - OniriCorpe: maybe we can use `if [[ ${YNH_APP_ACTION} == "upgrade" ]]` to differentiate the upgrade case

hum yes 🤔
[22:01:17] <Émy - OniriCorpe> should work
[22:06:16] <Yunohost Git/Infra notifications> [yunohost] OniriCorpe pushed 1 commit to do-not-enable-a-service-disabled-by-the-user: enable the app service during install ([cfcbeb3f](https://github.com/YunoHost/yunohost/commit/cfcbeb3fa9e2a031d0571d0c5d9dd4feecdbd6f8))
[22:07:42] <Yunohost Git/Infra notifications> [yunohost] 🔴 Pipeline [#1303079374](https://gitlab.com/YunoHost/yunohost/-/pipelines/1303079374) failed on branch do-not-enable-a-service-disabled-by-the-user
[23:33:09] <Yunohost Git/Infra notifications> [yunohost] 🔴 Pipeline [#1303089479](https://gitlab.com/YunoHost/yunohost/-/pipelines/1303089479) failed on branch do-not-enable-a-service-disabled-by-the-user