Re: [ovs-dev] [PATCH ovn v2 branch-22.12] CI: Remove .cirrus.yml

2022-12-12 Thread Dumitru Ceara
On 12/12/22 07:32, Ales Musil wrote:
> On Fri, Dec 9, 2022 at 3:48 PM Dumitru Ceara  wrote:
> 
>> On 12/9/22 10:09, Ales Musil wrote:
>>> Because we have enabled Cirrus CI remove the
>>> old config from branches to prevent it from failing.
>>>
>>> Signed-off-by: Ales Musil 
>>> ---
>>> It should be backported all the way down to 21.12.
>>>
>>> v2: Add forgotten Signed-off-by.
>>> ---
>>>  .cirrus.yml | 31 ---
>>>  1 file changed, 31 deletions(-)
>>>  delete mode 100644 .cirrus.yml
>>
>> Thanks for the patch but this breaks the build.  This is from the
>> ovsrobot run:
>>
>>
>> https://github.com/ovsrobot/ovn/actions/runs/3656074515/jobs/6178100493#step:13:3206
>>
>> make[1]: *** No rule to make target '.cirrus.yml', needed by 'all-am'.
>> Stop.
>>
>> Regards,
>> Dumitru
>>
> 
> Yeah I managed to post this patch in broken state twice. It should be fixed
> in v3.

No worries, thanks for working on this in the first place!

I'll have a look at v3 soon.

Thanks,
Dumitru

___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev


Re: [ovs-dev] [PATCH ovn v2 branch-22.12] CI: Remove .cirrus.yml

2022-12-11 Thread Ales Musil
On Fri, Dec 9, 2022 at 3:48 PM Dumitru Ceara  wrote:

> On 12/9/22 10:09, Ales Musil wrote:
> > Because we have enabled Cirrus CI remove the
> > old config from branches to prevent it from failing.
> >
> > Signed-off-by: Ales Musil 
> > ---
> > It should be backported all the way down to 21.12.
> >
> > v2: Add forgotten Signed-off-by.
> > ---
> >  .cirrus.yml | 31 ---
> >  1 file changed, 31 deletions(-)
> >  delete mode 100644 .cirrus.yml
>
> Thanks for the patch but this breaks the build.  This is from the
> ovsrobot run:
>
>
> https://github.com/ovsrobot/ovn/actions/runs/3656074515/jobs/6178100493#step:13:3206
>
> make[1]: *** No rule to make target '.cirrus.yml', needed by 'all-am'.
> Stop.
>
> Regards,
> Dumitru
>

Yeah I managed to post this patch in broken state twice. It should be fixed
in v3.

Thanks,
Ales.


>
> >
> > diff --git a/.cirrus.yml b/.cirrus.yml
> > deleted file mode 100644
> > index c71a0a249..0
> > --- a/.cirrus.yml
> > +++ /dev/null
> > @@ -1,31 +0,0 @@
> > -freebsd_build_task:
> > -
> > -  freebsd_instance:
> > -matrix:
> > -  image: freebsd-12-0-release-amd64
> > -  image: freebsd-11-2-release-amd64
> > -cpu: 4
> > -memory: 8G
> > -
> > -  env:
> > -DEPENDENCIES: automake libtool gmake gcc wget openssl
> > -  python py27-six py27-sphinx py27-openssl
> > -  python3 py36-six py36-openssl
> > -matrix:
> > -  COMPILER: gcc
> > -  COMPILER: clang
> > -
> > -  prepare_script:
> > -- pkg install -y ${DEPENDENCIES}
> > -
> > -  configure_script:
> > -- ./boot.sh
> > -- ./configure CC=${COMPILER} MAKE=gmake OVS_CFLAGS='-Wall'
> --enable-Werror
> > -  || { cat config.log; exit 1; }
> > -
> > -  build_script:
> > -- gmake -j8
> > -
> > -  check_script:
> > -- gmake -j8 check TESTSUITEFLAGS=-j8 RECHECK=yes
> > -|| { cat ./tests/testsuite.log; exit 1; }
>
>

-- 

Ales Musil

Senior Software Engineer - OVN Core

Red Hat EMEA 

amu...@redhat.comIM: amusil

___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev


Re: [ovs-dev] [PATCH ovn v2 branch-22.12] CI: Remove .cirrus.yml

2022-12-09 Thread Dumitru Ceara
On 12/9/22 10:09, Ales Musil wrote:
> Because we have enabled Cirrus CI remove the
> old config from branches to prevent it from failing.
> 
> Signed-off-by: Ales Musil 
> ---
> It should be backported all the way down to 21.12.
> 
> v2: Add forgotten Signed-off-by.
> ---
>  .cirrus.yml | 31 ---
>  1 file changed, 31 deletions(-)
>  delete mode 100644 .cirrus.yml

Thanks for the patch but this breaks the build.  This is from the
ovsrobot run:

https://github.com/ovsrobot/ovn/actions/runs/3656074515/jobs/6178100493#step:13:3206

make[1]: *** No rule to make target '.cirrus.yml', needed by 'all-am'.
Stop.

Regards,
Dumitru

> 
> diff --git a/.cirrus.yml b/.cirrus.yml
> deleted file mode 100644
> index c71a0a249..0
> --- a/.cirrus.yml
> +++ /dev/null
> @@ -1,31 +0,0 @@
> -freebsd_build_task:
> -
> -  freebsd_instance:
> -matrix:
> -  image: freebsd-12-0-release-amd64
> -  image: freebsd-11-2-release-amd64
> -cpu: 4
> -memory: 8G
> -
> -  env:
> -DEPENDENCIES: automake libtool gmake gcc wget openssl
> -  python py27-six py27-sphinx py27-openssl
> -  python3 py36-six py36-openssl
> -matrix:
> -  COMPILER: gcc
> -  COMPILER: clang
> -
> -  prepare_script:
> -- pkg install -y ${DEPENDENCIES}
> -
> -  configure_script:
> -- ./boot.sh
> -- ./configure CC=${COMPILER} MAKE=gmake OVS_CFLAGS='-Wall' 
> --enable-Werror
> -  || { cat config.log; exit 1; }
> -
> -  build_script:
> -- gmake -j8
> -
> -  check_script:
> -- gmake -j8 check TESTSUITEFLAGS=-j8 RECHECK=yes
> -|| { cat ./tests/testsuite.log; exit 1; }

___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev


[ovs-dev] [PATCH ovn v2 branch-22.12] CI: Remove .cirrus.yml

2022-12-09 Thread Ales Musil
Because we have enabled Cirrus CI remove the
old config from branches to prevent it from failing.

Signed-off-by: Ales Musil 
---
It should be backported all the way down to 21.12.

v2: Add forgotten Signed-off-by.
---
 .cirrus.yml | 31 ---
 1 file changed, 31 deletions(-)
 delete mode 100644 .cirrus.yml

diff --git a/.cirrus.yml b/.cirrus.yml
deleted file mode 100644
index c71a0a249..0
--- a/.cirrus.yml
+++ /dev/null
@@ -1,31 +0,0 @@
-freebsd_build_task:
-
-  freebsd_instance:
-matrix:
-  image: freebsd-12-0-release-amd64
-  image: freebsd-11-2-release-amd64
-cpu: 4
-memory: 8G
-
-  env:
-DEPENDENCIES: automake libtool gmake gcc wget openssl
-  python py27-six py27-sphinx py27-openssl
-  python3 py36-six py36-openssl
-matrix:
-  COMPILER: gcc
-  COMPILER: clang
-
-  prepare_script:
-- pkg install -y ${DEPENDENCIES}
-
-  configure_script:
-- ./boot.sh
-- ./configure CC=${COMPILER} MAKE=gmake OVS_CFLAGS='-Wall' --enable-Werror
-  || { cat config.log; exit 1; }
-
-  build_script:
-- gmake -j8
-
-  check_script:
-- gmake -j8 check TESTSUITEFLAGS=-j8 RECHECK=yes
-|| { cat ./tests/testsuite.log; exit 1; }
-- 
2.38.1

___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev