[ovs-dev] cirrus: Use latest stable FreeBSD images.
diff mbox series

Message ID 20191010130106.9475-1-i.maximets@ovn.org
State Accepted
Headers show
Series
  • [ovs-dev] cirrus: Use latest stable FreeBSD images.
Related show

Commit Message

Ilya Maximets Oct. 10, 2019, 1:01 p.m. UTC
CirrusCI recently introduced [1] new feature to use image families
instead of bare image names for gCloud based instances.
This allows us to use most recent stable builds. All the stable builds
are in the same image family in gCloud and it will run instances using
the most recent one.
This also allows us to simply use 11.3 image instead of 11.2.  There
was no such ability previously, because base freebsd-11-3-release-amd64
image has issues[2] that doesn't allow CirrusCI to use it.  However,
later stable 11.3 images from freebsd-11-3-snap family works fine.

[1] https://github.com/cirruslabs/cirrus-ci-docs/issues/422
[2] https://github.com/cirruslabs/cirrus-ci-docs/issues/359

Signed-off-by: Ilya Maximets <i.maximets@ovn.org>
---
 .cirrus.yml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Aaron Conole Oct. 10, 2019, 2:31 p.m. UTC | #1
Ilya Maximets <i.maximets@ovn.org> writes:

> CirrusCI recently introduced [1] new feature to use image families
> instead of bare image names for gCloud based instances.
> This allows us to use most recent stable builds. All the stable builds
> are in the same image family in gCloud and it will run instances using
> the most recent one.
> This also allows us to simply use 11.3 image instead of 11.2.  There
> was no such ability previously, because base freebsd-11-3-release-amd64
> image has issues[2] that doesn't allow CirrusCI to use it.  However,
> later stable 11.3 images from freebsd-11-3-snap family works fine.
>
> [1] https://github.com/cirruslabs/cirrus-ci-docs/issues/422
> [2] https://github.com/cirruslabs/cirrus-ci-docs/issues/359
>
> Signed-off-by: Ilya Maximets <i.maximets@ovn.org>
> ---

Acked-by: Aaron Conole <aconole@redhat.com>

Doesn't look like the build time changed much.  I wasn't sure whether it
could possibly get longer / shorter with this change.
Ilya Maximets Oct. 10, 2019, 5:12 p.m. UTC | #2
On 10.10.2019 16:31, Aaron Conole wrote:
> Ilya Maximets <i.maximets@ovn.org> writes:
> 
>> CirrusCI recently introduced [1] new feature to use image families
>> instead of bare image names for gCloud based instances.
>> This allows us to use most recent stable builds. All the stable builds
>> are in the same image family in gCloud and it will run instances using
>> the most recent one.
>> This also allows us to simply use 11.3 image instead of 11.2.  There
>> was no such ability previously, because base freebsd-11-3-release-amd64
>> image has issues[2] that doesn't allow CirrusCI to use it.  However,
>> later stable 11.3 images from freebsd-11-3-snap family works fine.
>>
>> [1] https://github.com/cirruslabs/cirrus-ci-docs/issues/422
>> [2] https://github.com/cirruslabs/cirrus-ci-docs/issues/359
>>
>> Signed-off-by: Ilya Maximets <i.maximets@ovn.org>
>> ---
> 
> Acked-by: Aaron Conole <aconole@redhat.com>

Thanks.

> Doesn't look like the build time changed much.  I wasn't sure whether it
> could possibly get longer / shorter with this change.

 From my testing I don't see any valuable changes in the build time.
This mostly functional change. e.g. it allows us to test with freebsd
11.3 without thinking about which stable image we should use.

I applied this to master.

Best regards, Ilya Maximets.

Patch
diff mbox series

diff --git a/.cirrus.yml b/.cirrus.yml
index 251f6734d..281bd75de 100644
--- a/.cirrus.yml
+++ b/.cirrus.yml
@@ -2,8 +2,8 @@  freebsd_build_task:
 
   freebsd_instance:
     matrix:
-      image: freebsd-12-0-release-amd64
-      image: freebsd-11-2-release-amd64
+      image_family: freebsd-12-0-snap
+      image_family: freebsd-11-3-snap
     cpu: 4
     memory: 8G