mbox series

[COMMITTED,0/3] build fixes

Message ID 20220712095901.212353-1-richard.henderson@linaro.org
Headers show
Series build fixes | expand

Message

Richard Henderson July 12, 2022, 9:58 a.m. UTC 
This is a collection of fixes required to get CI back to green.
Given that 2 of 3 apply to cirrus, I had to push them to staging to
even try them.  Since it worked, I've now pushed them to master.


r~


Richard Henderson (1):
  gitlab-ci/cirrus: Update freebsd to python 3.9 packages

Vladimir Sementsov-Ogievskiy (2):
  iotests: fix copy-before-write for macOS and FreeBSD
  iotests/copy-before-write: specify required_fmts

 .gitlab-ci.d/cirrus/freebsd-12.vars        | 3 ++-
 .gitlab-ci.d/cirrus/freebsd-13.vars        | 3 ++-
 tests/qemu-iotests/tests/copy-before-write | 8 +++++++-
 3 files changed, 11 insertions(+), 3 deletions(-)

Comments

Daniel P. Berrangé July 12, 2022, 10:13 a.m. UTC | #1 
On Tue, Jul 12, 2022 at 03:28:58PM +0530, Richard Henderson wrote:
> This is a collection of fixes required to get CI back to green.
> Given that 2 of 3 apply to cirrus, I had to push them to staging to
> even try them.  Since it worked, I've now pushed them to master.

FWIW, it is possible to test Cirrus CI fixes in your own personal
fork too, but it needs a little extra setup first, detailed in

  .gitlab-ci.d/cirrus/README.rst

With regards,
Daniel
Richard Henderson July 12, 2022, 10:26 a.m. UTC | #2 
On 7/12/22 15:43, Daniel P. Berrangé wrote:
> On Tue, Jul 12, 2022 at 03:28:58PM +0530, Richard Henderson wrote:
>> This is a collection of fixes required to get CI back to green.
>> Given that 2 of 3 apply to cirrus, I had to push them to staging to
>> even try them.  Since it worked, I've now pushed them to master.
> 
> FWIW, it is possible to test Cirrus CI fixes in your own personal
> fork too, but it needs a little extra setup first, detailed in
> 
>    .gitlab-ci.d/cirrus/README.rst

How many of the 'github' strings therein should really be 'gitlab'?


r~
Daniel P. Berrangé July 12, 2022, 10:31 a.m. UTC | #3 
On Tue, Jul 12, 2022 at 03:56:54PM +0530, Richard Henderson wrote:
> On 7/12/22 15:43, Daniel P. Berrangé wrote:
> > On Tue, Jul 12, 2022 at 03:28:58PM +0530, Richard Henderson wrote:
> > > This is a collection of fixes required to get CI back to green.
> > > Given that 2 of 3 apply to cirrus, I had to push them to staging to
> > > even try them.  Since it worked, I've now pushed them to master.
> > 
> > FWIW, it is possible to test Cirrus CI fixes in your own personal
> > fork too, but it needs a little extra setup first, detailed in
> > 
> >    .gitlab-ci.d/cirrus/README.rst
> 
> How many of the 'github' strings therein should really be 'gitlab'?

None actually, this is all part of the (gross) hack. Cirrus CI officially
only integrates with GitHub, so we need a dummy GitHub repo there. Once
the job starts though, it actually pulls the real code from gitlab.

With regards,
Daniel
Warner Losh July 12, 2022, 2:49 p.m. UTC | #4 
On Tue, Jul 12, 2022 at 4:33 AM Daniel P. Berrangé <berrange@redhat.com>
wrote:

> On Tue, Jul 12, 2022 at 03:56:54PM +0530, Richard Henderson wrote:
> > On 7/12/22 15:43, Daniel P. Berrangé wrote:
> > > On Tue, Jul 12, 2022 at 03:28:58PM +0530, Richard Henderson wrote:
> > > > This is a collection of fixes required to get CI back to green.
> > > > Given that 2 of 3 apply to cirrus, I had to push them to staging to
> > > > even try them.  Since it worked, I've now pushed them to master.
> > >
> > > FWIW, it is possible to test Cirrus CI fixes in your own personal
> > > fork too, but it needs a little extra setup first, detailed in
> > >
> > >    .gitlab-ci.d/cirrus/README.rst
> >
> > How many of the 'github' strings therein should really be 'gitlab'?
>
> None actually, this is all part of the (gross) hack. Cirrus CI officially
> only integrates with GitHub, so we need a dummy GitHub repo there. Once
> the job starts though, it actually pulls the real code from gitlab.
>

Yea, fun times that hack... :(

Warner