Re: CI and test improvements
От | Thomas Munro |
---|---|
Тема | Re: CI and test improvements |
Дата | |
Msg-id | CA+hUKGL=z6-kU80nPTjWqV=Xo=mpM36Lca=ngTr4LqcCf=tHzQ@mail.gmail.com обсуждение исходный текст |
Ответ на | Re: CI and test improvements (Justin Pryzby <pryzby@telsasoft.com>) |
Ответы |
Re: CI and test improvements
Re: CI and test improvements |
Список | pgsql-hackers |
On Wed, Nov 23, 2022 at 11:57 AM Justin Pryzby <pryzby@telsasoft.com> wrote: > [PATCH 02/10] cirrus/macos: switch to "macos_instance" / M1.. Duelling patches. Bilal's patch[1] uses the matrix feature to run the tests on both Intel and ARM, which made sense when he proposed it, but according to Cirrus CI warnings, the Intel instances are about to go away. So I think we just need your smaller change to switch the instance type. As for the pathname change, there is another place that knows where Homebrew lives, in ldap/001_auth. Fixed in the attached. That test just SKIPs if it can't find the binary, making it harder to notice. Standardising our approach here might make sense for a later patch. As for the kerberos test, Bilal's patch may well be a better idea (it adds MacPorts for one thing), and so I'll suggest rebasing that, but here I just wanted the minimum mechanical fix to avoid breaking on the 1st of Jan. I plan to push this soon if there are no objections. Then discussion of Bilal's patch can continue. > [PATCH 03/10] cirrus/macos: update to macos ventura I don't know any reason not to push this one too, but it's not time critical. [1] https://www.postgresql.org/message-id/flat/CAN55FZ2R%2BXufuVgJ8ew_yDBk48PgXEBvyKNvnNdTTVyczbQj0g%40mail.gmail.com
Вложения
В списке pgsql-hackers по дате отправления: