Re: [PATCH] ci: update FreeBSD cirrus job

From: Junio C Hamano <gitster@pobox.com>
Date: Jan 31, 2024, 22:35 UTC
Message-ID: <xmqq4jet3zdm.fsf@gitster.g>
In-Reply-To: <20240131191325.33228-1-carenas@gmail.com>

Carlo Marcelo Arenas Belón <carenas@gmail.com> writes:

Show 7 quoted lines

> FreeBSD 12 is EOL and no longer available, causing errors in this job.
>
> Upgrade to 13.2, which is the next oldest release with support and that
> should keep it for at least another 4 months.
>
> This will be upgraded again once 13.3 is released to avoid furtheer
> surprises.

Thanks.

> "Not enough compute credits to prioritize tasks!" seems to be just a
> reminder that the credit allocate for the Free Tier by Cirrus is all
> used up and which might result in additional delays getting a result.

This statement does not quite help future readers of "git log", unless they still remember that it was how Dscho noticed the problem. Either moving it under "---" line (meaning: remove from the proposed log message), or adding reference to the original report, would be more appropriate. For now, I'd amend the log message to read:

  The original report [*] of this problem mentions an error message
  "Not enough compute credits to prioritize tasks!".  It seems to be
  just a reminder that the credit allocate for the Free Tier by
  Cirrus is all used up and which might result in additional delays
  getting a result.

  [*] https://lore.kernel.org/git/d2d7da84-e2a3-a7b2-3f95-c8d53ad4dd5f@gmx.de/

or something like that while queuing.

Thanks.

Show 28 quoted lines

>
> Signed-off-by: Carlo Marcelo Arenas Belón <carenas@gmail.com>
> ---
>  .cirrus.yml | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/.cirrus.yml b/.cirrus.yml
> index b6280692d2..77346a4929 100644
> --- a/.cirrus.yml
> +++ b/.cirrus.yml
> @@ -1,7 +1,7 @@
>  env:
>    CIRRUS_CLONE_DEPTH: 1
>  
> -freebsd_12_task:
> +freebsd_task:
>    env:
>      GIT_PROVE_OPTS: "--timer --jobs 10"
>      GIT_TEST_OPTS: "--no-chain-lint --no-bin-wrappers"
> @@ -9,7 +9,7 @@ freebsd_12_task:
>      DEFAULT_TEST_TARGET: prove
>      DEVELOPER: 1
>    freebsd_instance:
> -    image_family: freebsd-12-3
> +    image_family: freebsd-13-2
>      memory: 2G
>    install_script:
>      pkg install -y gettext gmake perl5