Re: [PATCH v1 3/4] mirror: Reduce frequency of retrieving results archive

[Luis Chamberlain <mcgrof@kernel.org>]

On Mon, Mar 10, 2025 at 02:32:07PM -0400, cel@kernel.org wrote:
> From: Chuck Lever <chuck.lever@oracle.com>
> 
> Each run takes longer than 10 seconds, which generates a "git
> process is still running" warning in the system journal every
> minute.
> 
> Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
> ---
>  playbooks/roles/linux-mirror/templates/mirrors.yaml.j2 | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/playbooks/roles/linux-mirror/templates/mirrors.yaml.j2 b/playbooks/roles/linux-mirror/templates/mirrors.yaml.j2
> index dabbadcc6528..45e79349a3d4 100644
> --- a/playbooks/roles/linux-mirror/templates/mirrors.yaml.j2
> +++ b/playbooks/roles/linux-mirror/templates/mirrors.yaml.j2
> @@ -36,7 +36,7 @@ mirrors:
>    - short_name: "kdevops-results-archive"
>      url: "{{ defaults_kdevops_results_archive_git }}"
>      target: "kdevops-results-archive.git"
> -    refresh: "10s"
> +    refresh: "10m"

We don't poke kernel.org here ever, this is a github tree.

The question here is how often we want this updated to make CI updates
take effect. It is why I went so low here. I think such a low value only
makes sense for mirrors which won't need to leverage leverage
kdevops-results-archive to push updates to that tree, as otherwise their
tree will be out of sync. So, I think a bool to lower this default might
be nice, disabled by default, and a new defconfig for mirror-ci might be
useful which has that bool enabled. This way folks who wish to help with
CI infra and use kdevops-results-archive to push results there, can just
have their mirror setup use:

make defconfig-mirror-ci

Without this we risk that a CI run, once it has test results to upload,
won't be able to push the results. Because it will git fetch prior, but
its using the mirror.

  Luis