MIME-Version: 1.0
References: <3ydjipcr7kbss57nvi67noplncqhesl5eyb6wgol4ccjxynspv@yatlykpribmm>
 <DIM49A100EY1.2YFS2A6WAI3ZJ@jeltef.nl>
 <CAN55FZ30Np67cATsqYxF1SsP598VoRv4hJQZ4w9RA3Qe55prnQ@mail.gmail.com>
 <CAN55FZ13uX0cLSbgtSnnFeh5sTLeMr7+8UzmqpU6QjOtrRJTLg@mail.gmail.com>
 <CAN4CZFO4usEzFQoYzEywvOgoagW=U4yhpB4Oq-a7bUCR53djHA@mail.gmail.com>
In-Reply-To: 
 <CAN4CZFO4usEzFQoYzEywvOgoagW=U4yhpB4Oq-a7bUCR53djHA@mail.gmail.com>
From: Nazir Bilal Yavuz <byavuz81@gmail.com>
Date: Thu, 28 May 2026 20:07:43 +0300
Message-ID: 
 <CAN55FZ2mGpEgpUYmzbH5YTxVQWWhp6edT8aqoaR-FrH4eEH8SQ@mail.gmail.com>
Subject: Re: Heads Up: cirrus-ci is shutting down June 1st
To: Zsolt Parragi <zsolt.parragi@percona.com>
Cc: pgsql-hackers@lists.postgresql.org
Content-Type: text/plain; charset="UTF-8"
Archived-At: 
 <https://www.postgresql.org/message-id/CAN55FZ2mGpEgpUYmzbH5YTxVQWWhp6edT8aqoaR-FrH4eEH8SQ%40mail.gmail.com>
Precedence: bulk

Hi,

Thank you for looking into this!

On Thu, 28 May 2026 at 02:10, Zsolt Parragi <zsolt.parragi@percona.com> wrote:
>
> Hello!
>
> I didn't try the workflow on github yet, I only have a few comments
> based on the yaml file:
>
> +on:
> +  push:
> +    branches: [ "*" ]
> +
>
> Should this be "**", otherwise we ignore branches with a "/" in it?

You are right. I completely removed the 'branches:' part, now it
should match with every branch.


> +      SANITIZER_FLAGS: -fsanitize=address
>
> is this used anywhere? Seems like it's directly included into CFLAGS/LDFLAGS.

It wasn't used but I put it there for visibility since in Github
Actions you can't refer to other environment variables (Perhaps a
comment was needed for this). We don't use this now since all linux
tasks are merged in the v3 [1].

[1] https://postgr.es/m/CAN55FZ1-qiOWtQH5o6Q_7LJ7S3Ef_hfDE068uP0hGjB3gzwghg%40mail.gmail.com


-- 
Regards,
Nazir Bilal Yavuz
Microsoft