[00:01:39]
<Yunohost Git/Infra notifications> [Apps tools error] Error while updating %s: %s
[17:11:26]
<Yunohost Git/Infra notifications> [Apps tools error] Error while updating %s: %s
[17:14:18]
<Yunohost Git/Infra notifications> [Apps tools error] Error while updating %s: %s
[17:14:30]
<Bram> Someone forgot to string format 😅
[17:15:03]
<orhtej2> > <@Bram_:matrix.org> Someone forgot to string format 😅
But it should work as logger should do the formatting: https://github.com/YunoHost/apps/blob/71588cee4f8ef69316ecf08467ef6e85ce0f71e4/tools/list_builder.py#L68
[17:15:24]
<Bram> > But it should work as logger should do the formatting: https://github.com/YunoHost/apps/blob/71588cee4f8ef69316ecf08467ef6e85ce0f71e4/tools/list_builder.py#L68
indeed, that's weird
[17:17:12]
<Yunohost Git/Infra notifications> [Apps tools error] Error while updating %s: %s
[17:21:30]
<Yunohost Git/Infra notifications> [issues] ketsapiwiq [commented](https://github.com/YunoHost/issues/issues/1438#issuecomment-2083058691) on [issue #1438](https://github.com/YunoHost/issues/issues/1438) Specification draft for vulnerabilities indexing and detection ?: An additional step we could take is adding a security scan of the root filesytem as a CI step when testing apps levels (...
[17:21:51]
<Yunohost Git/Infra notifications> [Apps tools error] Error while updating %s: %s
[17:21:57]
<Yunohost Git/Infra notifications> [issues] ketsapiwiq [commented](https://github.com/YunoHost/issues/issues/1438#issuecomment-2083058691) on [issue #1438](https://github.com/YunoHost/issues/issues/1438) Specification draft for vulnerabilities indexing and detection ?: An additional step we could take is adding a security scan of the root filesytem as a CI step when testing apps levels (...
[17:21:57]
<Yunohost Git/Infra notifications> [issues] ketsapiwiq [commented](https://github.com/YunoHost/issues/issues/1438#issuecomment-2083058691) on [issue #1438](https://github.com/YunoHost/issues/issues/1438) Specification draft for vulnerabilities indexing and detection ?: An additional step we could take is adding a security scan of the root filesytem as a CI step when testing apps levels (...
[17:22:03]
<tituspijean> Bram: orhtej2 I've just added some debugging in production to see what's happening. Though I dunno how to trigger that.
[17:22:04]
<tituspijean> ah, every 4h hours, that's the cronjob.
[17:22:06]
<tituspijean> (╯°□°）╯︵ ┻━┻
[17:22:06]
<Yunohost Git/Infra notifications> [Apps tools error] Error while updating %s: %s
[17:22:06]
<tituspijean> the high-tech debugging was: `logging.error("Error while updating %s: %s (debug: name "+str(name)+", error "+str(err)+")", name, err)`
[17:22:20]
<Yunohost Git/Infra notifications> [ynh-dev] yorffuoj opened [pull request #81](https://github.com/YunoHost/ynh-dev/pull/81): Update doc when incus-admin group is not present
[17:22:27]
<orhtej2> https://github.com/YunoHost/apps/blob/71588cee4f8ef69316ecf08467ef6e85ce0f71e4/tools/app_caches.py#L80
Again, looks Ok
[17:22:27]
<orhtej2> > <@titus:pijean.ovh> the high-tech debugging was: `logging.error("Error while updating %s: %s (debug: name "+str(name)+", error "+str(err)+")", name, err)`
Perhaps it's not running from master?
[17:22:27]
<orhtej2> Or is a different job? There are two matches
[17:57:08]
<Yunohost Git/Infra notifications> [issues] ketsapiwiq [commented](https://github.com/YunoHost/issues/issues/1438#issuecomment-2083058691) on [issue #1438](https://github.com/YunoHost/issues/issues/1438) Specification draft for vulnerabilities indexing and detection ?: An additional step we could take is adding a security scan of the root filesytem as a CI step when testing apps levels (...
[18:08:45]
<Yunohost Git/Infra notifications> [ynh-dev] OniriCorpe approved [pull request #81](https://github.com/YunoHost/ynh-dev/pull/81#pullrequestreview-2029212629) Update doc when incus-admin group is not present
[18:09:24]
<Yunohost Git/Infra notifications> [ynh-dev] OniriCorpe merged [pull request #81](https://github.com/YunoHost/ynh-dev/pull/81): Update doc when incus-admin group is not present
[18:09:33]
<Yunohost Git/Infra notifications> [ynh-dev] OniriCorpe [commented](https://github.com/YunoHost/ynh-dev/pull/81#issuecomment-2083352089) on [issue #81](https://github.com/YunoHost/ynh-dev/pull/81) Update doc when incus-admin group is not present: thanks
[18:47:31]
<Yunohost Git/Infra notifications> [Apps tools error] App caches: Error while updating %s: %s
[18:48:39]
<tituspijean> > https://github.com/YunoHost/apps/blob/71588cee4f8ef69316ecf08467ef6e85ce0f71e4/tools/app_caches.py#L80
> Again, looks Ok
that's the one failing, I have added the "App caches" string in the previous message
[18:51:55]
<Yunohost Git/Infra notifications> [Apps tools error] App caches: Error while updating %s: %s (debug: name jump, error Cmd(git) failed due to: exit code(128)
cmdline: git clone -v --depth=5 --single-branch --branch=master -- https://github.com/YunoHost-Apps/jump_ynh /var/www/app_yunohost/apps/.apps_cache/jump
stderr: Cloning into /var/www/app_yunohost/apps/.apps_cache/jump...
POST git-upload-pack (353 bytes)
warning: Could not find remote branch master to clone.
fatal: Remote branch master not found in upstream origin
)
[18:52:22]
<tituspijean> *sigh*
[19:00:03]
<Yunohost Git/Infra notifications> [yunohost] tituspijean converted to draft the [pull request #1647](https://github.com/YunoHost/yunohost/pull/1647): [enh] exclude .well-known subpaths from conflict checks
[19:01:01]
<Yunohost Git/Infra notifications> [yunohost] tituspijean [commented](https://github.com/YunoHost/yunohost/pull/1647#issuecomment-2083454255) on [issue #1647](https://github.com/YunoHost/yunohost/pull/1647) [enh] exclude .well-known subpaths from conflict checks: After further consideration, I would push for a dedicated "well-known path" resource, rather than patching the current "...
[19:05:57]
<tituspijean> Temporary fixed in the app repo (https://github.com/YunoHost-Apps/jump\_ynh/issues/1), but it does not explain why the logging does not process the name and error vars correctly
[19:11:34]
<tituspijean> Ah, I think I got it.
[19:12:33]
<tituspijean> https://github.com/YunoHost/apps/blob/71588cee4f8ef69316ecf08467ef6e85ce0f71e4/tools/appslib/logging_sender.py#L55 : calls for LogRecord.msg, which is the raw string before replacement of the "%s"
[19:13:03]
<tituspijean> https://docs.python.org/3/library/logging.html#logrecord-attributes
should we use LogRecord.message instead?
[19:14:50]
<tituspijean> but I'm a bit confused about "This is set when Formatter.format() is invoked.", and when it is invoked, if at all in the current code.
[19:17:55]
<tituspijean> (yolotesting my theory)
[19:18:36]
<tituspijean> mmmh maybe it won't complain, now that there is a cache for jump_ynh
[19:19:00]
<Yunohost Git/Infra notifications> [Apps tools error] [App caches] Error while updating jump: Cmd(git) failed due to: exit code(128)
cmdline: git fetch -v --force -- origin master
stderr: fatal: couldnt find remote ref master
[19:19:02]
<tituspijean> ah yisss
[19:19:21]
<tituspijean> I've never been so glad to see an error message
[19:19:46]
<Émy - OniriCorpe> 🎉🥲
[20:02:32]
<Yunohost Git/Infra notifications> [Apps tools error] [App caches] Error while updating jump: Cmd(git) failed due to: exit code(128)
cmdline: git fetch -v --force -- origin master
stderr: fatal: couldnt find remote ref master
[21:06:47]
<Tag> https://toot.aquilenet.fr/@paquerette/112274156251241191