On 10/13/23 09:14, Antoine Beaupré wrote:
On 2023-10-13 09:05:35, Kyle Fazzari wrote:
On 10/13/23 08:26, Julian Andres Klode wrote:
Also please do not run apt update in the background or try to
calculate dist upgrades, that is evil and you're breaking stuff.
If you want to check for updates, make sure the periodic apt service
is configured to run. You are entitled to one run per day. If you
do not operate the mirror infrastructure please do not run your own
updates out of band.
A fair critique, although as I mentioned in an earlier email, this
collector cannot do its job if it's running on a significantly
out-of-date cache. At the same time, if feels out of scope for this
Debian package to ensure the periodic apt service is configured to run.
Feels like a rock and a hard place. Thoughts?
I think this is a deployment issue. People who provision this package
should *not* expect it to run `apt update`: I certainly didn't, and the
previous (shell) implementation didn't either.
We have other tools that continuously pull mirrors, and as jak stated,
APT can be configured to do so as well (although I'm not sure what the
canonical way of doing so).
So let's not do this here.
I don't entirely agree, but disagreement is okay. I do at least
recommend accompanying this with a cache age statistic, as we discussed
earlier.
Kyle
