Re: Use windows VMs instead of windows containers on the CI

Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com>

From: Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com>
To: Thomas Munro <thomas.munro@gmail.com>
Cc: Andres Freund <andres@anarazel.de>, Nazir Bilal Yavuz <byavuz81@gmail.com>, Justin Pryzby <pryzby@telsasoft.com>, pgsql-hackers@postgresql.org
Date: 2023-02-04T12:26:08Z
Lists: pgsql-hackers
On Fri, Feb 3, 2023 at 3:27 PM Thomas Munro <thomas.munro@gmail.com> wrote:
>
> On Fri, Feb 3, 2023 at 6:57 PM Andres Freund <andres@anarazel.de> wrote:
> > And pushed!  I think an improvement in CI times of this degree is pretty
> > awesome.
>
> +1
>
> A lot of CI compute time is saved.  The Cirrus account[1] was
> previously hitting the 4 job limit all day long, and now it's often
> running 1 or 2 jobs when I look, and it has space capacity to start a
> new job immediately if someone posts a new patch.  I'll monitor it
> over the next few days but it looks great.
>
> [1] https://cirrus-ci.com/github/postgresql-cfbot/postgresql

Oh, wow! This commit drastically improved testing time on Windows. It
was Windows tests that were always behind in my github repo's CI, now
I can see it got much faster. Thanks for working on this.

Windows tests were taking around 27min
(https://cirrus-ci.com/build/6060448332644352) before the patch, but
it came down to 13min (https://cirrus-ci.com/build/6528980879147008)
after the patch - Yay! 2X improvement :).

--
Bharath Rupireddy
PostgreSQL Contributors Team
RDS Open Source Databases
Amazon Web Services: https://aws.amazon.com



Commits

  1. ci: Use windows VMs instead of windows containers