diff mbox series

ci: update FreeBSD cirrus job

Message ID 20240131191325.33228-1-carenas@gmail.com (mailing list archive)
State Accepted
Commit 81fffb66d3f702c269a5aee6e403c3ec62fc0277
Headers show
Series ci: update FreeBSD cirrus job | expand

Commit Message

Carlo Marcelo Arenas Belón Jan. 31, 2024, 7:13 p.m. UTC
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.

"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.

Signed-off-by: Carlo Marcelo Arenas Belón <carenas@gmail.com>
---
 .cirrus.yml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Junio C Hamano Jan. 31, 2024, 10:35 p.m. UTC | #1
Carlo Marcelo Arenas Belón <carenas@gmail.com> writes:

> 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.

>
> 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
Junio C Hamano Jan. 31, 2024, 10:57 p.m. UTC | #2
Carlo Marcelo Arenas Belón <carenas@gmail.com> writes:

> This will be upgraded again once 13.3 is released to avoid furtheer

I've typofixed this to "further" while queuing; no need to resend
the patch only to fix it.
diff mbox series

Patch

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