From: | Andres Freund <andres(at)anarazel(dot)de> |
---|---|
To: | Nazir Bilal Yavuz <byavuz81(at)gmail(dot)com> |
Cc: | pgsql-hackers(at)postgresql(dot)org, Thomas Munro <thomas(dot)munro(at)gmail(dot)com>, Justin Pryzby <pryzby(at)telsasoft(dot)com>, Daniel Gustafsson <daniel(at)yesql(dot)se> |
Subject: | Re: Cirrus-ci is lowering free CI cycles - what to do with cfbot, etc? |
Date: | 2023-08-23 21:48:03 |
Message-ID: | 20230823214803.evia2vpup6uzn4c3@awork3.anarazel.de |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hi,
On 2023-08-23 23:55:15 +0300, Nazir Bilal Yavuz wrote:
> On Wed, 23 Aug 2023 at 09:58, Andres Freund <andres(at)anarazel(dot)de> wrote:
> > I'm hoping to push this fairly soon, as I'll be on vacation the last week of
> > August. I'll be online intermittently though, if there are issues, I can react
> > (very limited connectivity for middday Aug 29th - midday Aug 31th though). I'd
> > appreciate a quick review or two.
>
> Patch looks good to me besides some minor points.
Thanks for looking!
> v3-0004-ci-Prepare-to-make-compute-resources-for-CI-confi.patch:
> diff --git a/.cirrus.star b/.cirrus.star
> + """The main function is executed by cirrus-ci after loading
> .cirrus.yml and can
> + extend the CI definition further.
> +
> + As documented in .cirrus.yml, the final CI configuration is composed of
> +
> + 1) the contents of this file
>
> Instead of '1) the contents of this file' comment, '1) the contents
> of .cirrus.yml file' could be better since this comment appears in
> .cirrus.star file.
Good catch.
> + if repo_config_url != None:
> + print("loading additional configuration from
> \"{}\"".format(repo_config_url))
> + output += config_from(repo_config_url)
> + else:
> + output += "n# REPO_CI_CONFIG_URL was not set\n"
>
> Possible typo at output += "n# REPO_CI_CONFIG_URL was not set\n".
Fixed.
> v3-0008-ci-switch-tasks-to-debugoptimized-build.patch:
> Just thinking of possible optimizations and thought can't we create
> something like 'buildtype: xxx' to override default buildtype using
> .cirrus.star? This could be better for PG developers. For sure that
> could be the subject of another patch.
We could, but I'm not sure what the use would be?
Greetings,
Andres Freund
From | Date | Subject | |
---|---|---|---|
Next Message | Daniel Gustafsson | 2023-08-23 21:54:03 | Re: Fix error handling in be_tls_open_server() |
Previous Message | Jacob Champion | 2023-08-23 21:47:55 | Re: Fix error handling in be_tls_open_server() |