[23:03:30]
<Aleks (he/him/il/lui)> > <@Salamandar:matrix.org> (but keep the `before_install` image, i'm sure it has its uses)
yeah that's for core CI
[23:03:31]
<Aleks (he/him/il/lui)> (it feels like we're reinventing docker builds but for LXC)
[23:03:31]
<Aleks (he/him/il/lui)> but clearly we could have a different image build plumbing
[23:03:31]
<Aleks (he/him/il/lui)> mainly for performance / cache / not spending time to reinstall all the deps in the CI obviously
[23:03:31]
<Aleks (he/him/il/lui)> yeah they get flagged as manually installed
[23:03:31]
<Aleks (he/him/il/lui)> https://github.com/YunoHost/yunohost/blob/dev/.gitlab/ci/install.gitlab-ci.yml#L24
[23:03:42]
<Aleks (he/him/il/lui)> there's probably an option during postinstall
[23:03:42]
<Aleks (he/him/il/lui)> 😬
[23:03:42]
<Aleks (he/him/il/lui)> zblerg
[23:03:42]
<Aleks (he/him/il/lui)> there is not
[23:03:42]
<Aleks (he/him/il/lui)> oooor
[23:03:42]
<Aleks (he/him/il/lui)> (wont be available to do much until at least wednesday)
[23:03:42]
<Aleks (he/him/il/lui)> yea
[23:04:41]
<Yunohost Git/Infra notifications> [appstore] Couldn't pull, maybe local changes are present?
[23:04:42]
<Yunohost Git/Infra notifications> [update_app_levels] Updating apps level failed miserably!
[23:04:55]
<Salamandar> https://aria.im/_matrix/media/v1/download/matrix.org/wTCzQnvLncgirYIjQIHCFkye
[23:04:55]
<Salamandar> hmmm
[23:04:55]
<Salamandar> how are we supposed to automatize that ?
[23:04:55]
<Salamandar> another question around the lxd image builder
[23:04:56]
<Salamandar> but the way yunohost and its deps are installed is not "clean", what i mean is those packages are installed "explicitly", not "as deps". That's why I want `dev` to be run with unpatched install.sh
[23:04:56]
<Salamandar> 100% that
[23:05:04]
<Yunohost Git/Infra notifications> [lxd_img_builder/py] fix dest_dir -> dest_file - Félix Piédallu
[23:05:05]
<Yunohost Git/Infra notifications> [lxd_img_builder] Salamandar pushed 2 commits to py ([52e39d83a5ad...3825ddee1f8e](https://github.com/YunoHost/lxd_img_builder/compare/52e39d83a5ad...3825ddee1f8e))
[23:05:05]
<Salamandar> that can be added though
[23:05:05]
<Yunohost Git/Infra notifications> [lxd_img_builder] Salamandar pushed 5 commits to py ([119b85ddeba8...52e39d83a5ad](https://github.com/YunoHost/lxd_img_builder/compare/119b85ddeba8...52e39d83a5ad))
[23:05:05]
<Yunohost Git/Infra notifications> [lxd_img_builder/py] Update README for py - Félix Piédallu
[23:05:14]
<Yunohost Git/Infra notifications> [appstore/apps-tools] Couldn't pull, maybe local changes are present?
[23:05:14]
<Yunohost Git/Infra notifications> [apps-tools] Couldn't pull, maybe local changes are present?
[23:05:15]
<Yunohost Git/Infra notifications> [yunohost] Salamandar opened [pull request #1966](https://github.com/YunoHost/yunohost/pull/1966): Allow bypassing the terms of services at postinstall time
[23:05:16]
<Yunohost Git/Infra notifications> [appstore] Couldn't pull, maybe local changes are present?
[23:05:16]
<Salamandar> Untested though
[23:05:16]
<Yunohost Git/Infra notifications> [yunorunner] OniriCorpe opened [issue #68](https://github.com/YunoHost/yunorunner/issues/68): the CI script should remove the data dir on app removal
[23:05:17]
<Salamandar> > <@yunohostinfra:matrix.org> [update_app_levels] Updating apps level failed miserably!
that's (again) because the branch already existed upstream. Dunno if we want the script to make a push --force…
[23:05:17]
<Yunohost Git/Infra notifications> [yunohost] Salamandar pushed 1 commit to bypass_tos: Allow bypassing the terms of services at postinstall time ([654d7489](https://github.com/YunoHost/yunohost/commit/654d7489dab198f5a87ce4c1f6874f61342a7bcb))
[23:05:18]
<Yunohost Git/Infra notifications> [lxd_img_builder] Salamandar opened [pull request #9](https://github.com/YunoHost/lxd_img_builder/pull/9): Reimplement image_builder in python, refactor recipes.
[23:05:18]
<Yunohost Git/Infra notifications> [lxd_img_builder] Salamandar pushed 1 commit to py: Add update_all_images.sh script ([71a22db5](https://github.com/YunoHost/lxd_img_builder/commit/71a22db5c7b57b9954ee4c3d54289283ec0d0918))
[23:05:18]
<Salamandar> yeah
[23:05:18]
<Salamandar> I don't see one
[23:05:18]
<Salamandar> > <@yunohostinfra:matrix.org> [yunohost] Salamandar opened [pull request #1966](https://github.com/YunoHost/yunohost/pull/1966): Allow bypassing the terms of services at postinstall time
tested now :D
[23:05:26]
<Yunohost Git/Infra notifications> [lxd_img_builder] Salamandar pushed 2 commits to py ([3825ddee1f8e...7a2cd01bbe54](https://github.com/YunoHost/lxd_img_builder/compare/3825ddee1f8e...7a2cd01bbe54))
[23:05:26]
<Yunohost Git/Infra notifications> [lxd_img_builder] Salamandar pushed 1 commit to master: Publish to simplestreams if an environment variable exists ([34427a89](https://github.com/YunoHost/lxd_img_builder/commit/34427a8901a90f054873b7b0125a9c060808ee1e))
[23:05:26]
<Yunohost Git/Infra notifications> [lxd_img_builder] Salamandar pushed 1 commit to py: Delete legacy image_builder ([3dd3cc84](https://github.com/YunoHost/lxd_img_builder/commit/3dd3cc84fa19305e1de9d0524884164b169afa7d))
[23:05:26]
<Yunohost Git/Infra notifications> [lxd_img_builder] Salamandar pushed 1 commit to py: Add a script that removes stale images from the simplestreams repo ([57b7539e](https://github.com/YunoHost/lxd_img_builder/commit/57b7539e96ce84310a6df59c088ebe82a1f4c595))
[23:05:26]
<m606> > <@yunohostinfra:matrix.org> [yunohost] Salamandar pushed 1 commit to bypass_tos: Allow bypassing the terms of services at postinstall time ([654d7489](https://github.com/YunoHost/yunohost/commit/654d7489dab198f5a87ce4c1f6874f61342a7bcb))
I guess you already know but just in case in 12.4.0.1 I still see the TOS-related pending migration (that appeared with 12.4.0) without actual link to TOS displayed
[23:05:27]
<Salamandar> huh
[23:05:27]
<Salamandar> that's something else…
[23:05:44]
<Salamandar> dunno how it worked at first
[23:05:44]
<Salamandar> that's also why it couldn't pull the appstore code actually…
[23:05:44]
<Yunohost Git/Infra notifications> [lxd_img_builder] Salamandar merged [pull request #7](https://github.com/YunoHost/lxd_img_builder/pull/7): Publish to simplestreams if SIMPLESTREAMS_REPO environment variable is passed
[23:05:44]
<Yunohost Git/Infra notifications> [lxd_img_builder/py] Add image_builder.py - Félix Piédallu
[23:05:44]
<Salamandar> but now it should be fine
[23:05:44]
<Salamandar> it was trying to push on master
[23:05:44]
<Salamandar> hmmmm
[23:05:44]
<Yunohost Git/Infra notifications> [lxd_img_builder] Salamandar pushed 5 commits to py ([3dd3cc84fa19...31bdc9ee9b01](https://github.com/YunoHost/lxd_img_builder/compare/3dd3cc84fa19...31bdc9ee9b01))
[23:05:44]
<Yunohost Git/Infra notifications> [lxd_img_builder] Salamandar deleted branch py
[23:05:44]
<Salamandar> OK, I was reading on the wrong server. it's on the new server now. And it was… missing the ssh key to pull/push via ssh
[23:05:45]
<Yunohost Git/Infra notifications> [lxd_img_builder] Salamandar merged [pull request #9](https://github.com/YunoHost/lxd_img_builder/pull/9): Reimplement image_builder in python, refactor recipes.
[23:05:45]
<Yunohost Git/Infra notifications> [lxd_img_builder] Salamandar pushed 7 commits to master ([34427a8901a9...4d4e63683a8d](https://github.com/YunoHost/lxd_img_builder/compare/34427a8901a9...4d4e63683a8d))
[23:06:03]
<Émy - OniriCorpe> this step is SUPER slow on the CI x____x
`Preinstalling apt dependencies before creating the initial snapshot...`
[23:06:04]
<Émy - OniriCorpe> zfs 👀
[23:07:12]
<Yunohost Git/Infra notifications> [appstore/apps-tools] Couldn't pull, maybe local changes are present?
[23:07:12]
<Salamandar> i think the hdds are slow :Dw
[23:07:12]
<Yunohost Git/Infra notifications> [apps-tools] Couldn't pull, maybe local changes are present?
[23:07:18]
<Aleks (he/him/il/lui)> not sure to understand the challenge ?
[23:07:18]
<Aleks (he/him/il/lui)> aaah
[23:07:18]
<Tag> > <@Salamandar:matrix.org> i think the hdds are slow :Dw
yeah :/
[23:07:18]
<Aleks (he/him/il/lui)> ah you mean in python ?
[23:07:19]
<Aleks (he/him/il/lui)> neat 📸
[23:07:25]
<orhtej2> > <@Salamandar:matrix.org> why is the line twice in the appstore cron ?
Maintenance autoadds itself to cron, faulty check?
[23:07:25]
<Aleks (he/him/il/lui)> arent there any logs ?
[23:07:25]
<orhtej2> Check if cron runs in same env
[23:07:35]
<Yunohost Git/Infra notifications> [autoupdate_app_sources] App sources auto-update failed miserably!
[23:07:38]
<tituspijean> 🚭️
[23:07:48]
<Yunohost Git/Infra notifications> [appstore/apps-tools] Couldn't pull, maybe local changes are present?
[23:07:48]
<Yunohost Git/Infra notifications> [apps-tools] Couldn't pull, maybe local changes are present?
[23:07:57]
<Yunohost Git/Infra notifications> [appstore/apps-tools] Couldn't pull, maybe local changes are present?
[23:07:57]
<Yunohost Git/Infra notifications> [apps-tools] Couldn't pull, maybe local changes are present?
[23:08:28]
<Yunohost Git/Infra notifications> [appstore/apps-tools] Couldn't pull, maybe local changes are present?
[23:08:28]
<Salamandar> > <@oniricorpe:im.emelyne.eu> zfs 👀
yeah but that doesn't accelerate writes, only reads
[23:08:28]
<Yunohost Git/Infra notifications> [apps-tools] Couldn't pull, maybe local changes are present?
[23:08:33]
<Yunohost Git/Infra notifications> [yunohost] alexAubin pushed 2 commits to bookworm ([d0005e66ae53...6c364016bd90](https://github.com/YunoHost/yunohost/compare/d0005e66ae53...6c364016bd90))
[23:08:33]
<Yunohost Git/Infra notifications> [yunohost] alexAubin deleted branch bypass_tos
[23:08:34]
<Yunohost Git/Infra notifications> [yunohost] 🔴 Pipeline [#1493211198](https://gitlab.com/YunoHost/yunohost/-/pipelines/1493211198) failed on branch bookworm
[23:08:34]
<Yunohost Git/Infra notifications> [lxd_img_builder] Salamandar pushed 1 commit to master: Add logging to file capabilities ([81404183](https://github.com/YunoHost/lxd_img_builder/commit/81404183e2838ef13ba4e9a660ef21349b79652e))
[23:08:34]
<Yunohost Git/Infra notifications> [lxd_img_builder] Salamandar pushed 1 commit to master: mkdir logs, add to gitignore ([5747f51c](https://github.com/YunoHost/lxd_img_builder/commit/5747f51cde447c5ed049e21e97ea09d05b94a902))
[23:08:34]
<Yunohost Git/Infra notifications> [lxd_img_builder] Salamandar pushed 1 commit to master: Fix update_all_images for better logging ([c773feb7](https://github.com/YunoHost/lxd_img_builder/commit/c773feb71c430406049ae075d93843f1306f5ed2))
[23:08:45]
<Yunohost Git/Infra notifications> [wordpress_ynh] ci-apps-bookworm-dev: level 0 https://ci-apps-bookworm-dev.yunohost.org/ci/job/3412 on commit https://github.com/YunoHost-Apps/wordpress_ynh/commit/b97a41e880a52fd6fc7b2d17ef6088fbd85d29c9 "Update manifest.toml" by @Éric Gaspar on branches testing
[23:08:45]
<Salamandar> ¯\_(ツ)_/¯
[23:08:45]
<Salamandar> was someone on the server ? :p
[23:08:45]
<Salamandar> ```
root@apps:/var/www/appstore/apps_tools# ls -lah
total 360K
drwxr-xr-x 15 appstore www-data 4.0K Oct 12 14:02 .
```
[23:08:45]
<Salamandar> ah no my bad it's 16:20 not 14:20
[23:08:45]
<Salamandar> let's leave it and see if it reproduces in 4 hours
[23:08:45]
<Salamandar> that's just after the script ran
[23:08:45]
<Yunohost Git/Infra notifications> test message
[23:08:45]
<Salamandar> \shrug
[23:08:45]
<Salamandar> wait a minute
[23:08:46]
<Salamandar> (yeah the only challenge here was prettiness tbh)
[23:08:46]
<Salamandar> so that's pretty
[23:08:46]
<Salamandar> just gotta check if it works with interactive commands…
[23:08:46]
<Salamandar> yeah
[23:08:47]
<Yunohost Git/Infra notifications> 🏗️ Starting build for yunohost/12.0.4.1+202410121045 for bookworm/unstable/all ...
[23:08:47]
<Salamandar> i redirect the logs to a file with lines prefixed
[23:08:47]
<Yunohost Git/Infra notifications> ✔️ Completed build for yunohost/12.0.4.1+202410121045 for bookworm/unstable/all.
[23:08:47]
<Salamandar> https://aria.im/_matrix/media/v1/download/matrix.org/QzsqYwmZElEFLSJPLWZVqlrv
[23:08:47]
<Yunohost Git/Infra notifications> [apps-tools] Couldn't pull, maybe local changes are present?
[23:08:47]
<Yunohost Git/Infra notifications> [appstore/apps-tools] Couldn't pull, maybe local changes are present?
[23:08:48]
<Salamandar> nah not anymore
[23:08:48]
<Salamandar> probably a faulty copy paste
[23:08:48]
<Salamandar> yes but they don't show that
[23:08:48]
<Yunohost Git/Infra notifications> [yunohost] alexAubin merged [pull request #1966](https://github.com/YunoHost/yunohost/pull/1966): Allow bypassing the terms of services at postinstall time
[23:08:48]
<Salamandar> https://github.com/YunoHost/lxd_img_builder/blob/4cbdbfbe720ae70b5f42d882fe008d57e1f609ef/image_builder.py#L36
[23:08:48]
<Salamandar> oh wait
[23:08:48]
<Salamandar> maybe there are two git instances running at the same time. That would explain.
[23:08:49]
<Salamandar> it works manually ???
[23:08:49]
<Salamandar> https://github.com/YunoHost-Apps/appstore_ynh/commit/c2366b6fd9ac8d6b00431d8ecac23e55808a2f65
[23:08:56]
<Salamandar> ```
root@apps:/var/www/appstore/appstore# cat /etc/cron.d/apps_tools
# Every 4 hours
0 */4 * * * appstore /bin/bash /var/www/appstore/apps_tools/maintenance.sh rebuild_catalog
# Everyday at 01:30 UTC
30 1 * * * appstore /bin/bash /var/www/appstore/apps_tools/maintenance.sh autoupdate_app_sources
# Every friday at 6 PM UTC
0 17 * * 5 appstore /bin/bash /var/www/appstore/apps_tools/maintenance.sh update_app_levels
root@apps:/var/www/appstore/appstore# cat /etc/cron.d/appstore
# Every 4 hours
0 */4 * * * appstore /var/www/appstore/update_catalog.sh
# Every day
0 */4 * * * appstore /var/www/appstore/update_catalog.sh
```
[23:08:56]
<Salamandar> and that would be my fault :D
[23:08:56]
<Salamandar> why is the line twice in the appstore cron ?
[23:08:57]
<Yunohost Git/Infra notifications> [yunohost] Salamandar merged [pull request #1967](https://github.com/YunoHost/yunohost/pull/1967): Fix #1966
[23:08:57]
<Yunohost Git/Infra notifications> [yunohost] Salamandar pushed 1 commit to bypass_tos: Allow bypassing the terms of services at postinstall time ([c19062c2](https://github.com/YunoHost/yunohost/commit/c19062c28cc73631bdb81e1c3bfaa079b64e1f26))
[23:08:57]
<Yunohost Git/Infra notifications> [lxd_img_builder/simplestreams] Split long command on multiple lines - Félix Piédallu
[23:08:57]
<Yunohost Git/Infra notifications> [yunohost] Salamandar [commented](https://github.com/YunoHost/yunohost/pull/1966#issuecomment-2408603807) on [issue #1966](https://github.com/YunoHost/yunohost/pull/1966) Allow bypassing the terms of services at postinstall time: Ah crap was already merged.
[23:08:57]
<Yunohost Git/Infra notifications> [yunohost] Salamandar created new branch bypass_tos
[23:08:57]
<Yunohost Git/Infra notifications> [lxd_img_builder] Salamandar pushed 2 commits to simplestreams ([82ca2bac6729...ac7053f1f1a6](https://github.com/YunoHost/lxd_img_builder/compare/82ca2bac6729...ac7053f1f1a6))
[23:08:57]
<Salamandar> > <@yunohostinfra:matrix.org> [apps-tools] Couldn't pull, maybe local changes are present?
i don't get it
[23:08:58]
<Yunohost Git/Infra notifications> 🏗️ Starting build for yunohost/12.0.4.1+202410121545 for bookworm/unstable/all ...
[23:08:58]
<Yunohost Git/Infra notifications> ✔️ Completed build for yunohost/12.0.4.1+202410121545 for bookworm/unstable/all.
[23:08:58]
<Yunohost Git/Infra notifications> [yunohost] Salamandar opened [pull request #1967](https://github.com/YunoHost/yunohost/pull/1967): Fix #1966
[23:08:58]
<Yunohost Git/Infra notifications> [yunohost] Salamandar pushed 1 commit to bookworm: Fix: terms and services should be terms of services... ([05bc7072](https://github.com/YunoHost/yunohost/commit/05bc7072571346ecb767eba31ccf54c6af9d9503))
[23:08:58]
<Yunohost Git/Infra notifications> [yunohost] Salamandar deleted branch bypass_tos
[23:08:58]
<Yunohost Git/Infra notifications> [yunohost] Salamandar pushed 1 commit to bypass_tos: Fix: terms and services should be terms of services... ([b8ce3702](https://github.com/YunoHost/yunohost/commit/b8ce37027bdd16f717c5e2d917ec569a7d0a6ea1))
[23:08:59]
<Yunohost Git/Infra notifications> [yunohost] tituspijean [commented](https://github.com/YunoHost/yunohost/pull/1966#issuecomment-2408596138) on [issue #1966](https://github.com/YunoHost/yunohost/pull/1966) Allow bypassing the terms of services at postinstall time: It should be terms-of-services 😅
[23:09:00]
<Salamandar> saw that yesterday but was too tired to fix
[23:09:01]
<Salamandar> yeah
[23:09:02]
<Yunohost Git/Infra notifications> [lxd_img_builder] Salamandar pushed 2 commits to master ([259d728b9549...fd3f01c6f63c](https://github.com/YunoHost/lxd_img_builder/compare/259d728b9549...fd3f01c6f63c))
[23:09:02]
<Yunohost Git/Infra notifications> [lxd_img_builder/master] Fix the patch_gitlab_runner script - Félix Piédallu
[23:09:02]
<Yunohost Git/Infra notifications> [lxd_img_builder/simplestreams] Split long command on multiple lines - Félix Piédallu
[23:09:02]
<Yunohost Git/Infra notifications> [lxd_img_builder] Salamandar pushed 1 commit to simplestreams: Publish to simplestreams if an environment variable exists ([82ca2bac](https://github.com/YunoHost/lxd_img_builder/commit/82ca2bac67295bb6c75acec12b0c3f5513e7919e))
[23:09:03]
<Salamandar> my brain probably thought of "terms and conditions"
[23:09:03]
<Yunohost Git/Infra notifications> [lxd_img_builder] Salamandar pushed 1 commit to py: start working on python version ([7c4a387d](https://github.com/YunoHost/lxd_img_builder/commit/7c4a387d2f9a33f440beb4220b997de11f6f879d))
[23:09:03]
<Yunohost Git/Infra notifications> [appstore] Couldn't pull, maybe local changes are present?
[23:09:03]
<Salamandar> fixed
[23:09:08]
<Salamandar> thanks for the reminder @titus:pijean.ovh :D
[23:09:09]
<Salamandar> FYI, about incus-simplestreams https://github.com/lxc/incus/issues/1303
[23:09:09]
<Salamandar> https://aria.im/_matrix/media/v1/download/matrix.org/PEkCiddhqzzZicpsuUZlZBxC
[23:09:09]
<Yunohost Git/Infra notifications> [lxd_img_builder] Salamandar pushed 1 commit to master: prune_incus_simplestreams: cleanup old versions as well ([d2ad5701](https://github.com/YunoHost/lxd_img_builder/commit/d2ad57015e16d81d0df6cc621121a3867ef2580d))
[23:10:27]
<Salamandar> Thank you whoever did that dialog <3