diff mbox series

[ovs-dev] cirrus: Reduce memory requirements for FreeBSD VMs.

Message ID 20210916114810.2936394-1-i.maximets@ovn.org
State Accepted
Headers show
Series [ovs-dev] cirrus: Reduce memory requirements for FreeBSD VMs. | expand

Checks

Context Check Description
ovsrobot/apply-robot success apply and check: success
ovsrobot/github-robot-_Build_and_Test success github build: passed

Commit Message

Ilya Maximets Sept. 16, 2021, 11:48 a.m. UTC
According to memory usage graphs, our builds are using 3GB at most.
Reducing memory requirements to 4GB to have some room.  This change
doesn't affect time needed to finish the build, but should have a
slight positive effect on scheduling time on a community cluster.
And it's also not cool from our side to reserve shared resources that
we're not using, while they could be used by some other project.

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

Comments

Aaron Conole Sept. 16, 2021, 12:42 p.m. UTC | #1
Ilya Maximets <i.maximets@ovn.org> writes:

> According to memory usage graphs, our builds are using 3GB at most.
> Reducing memory requirements to 4GB to have some room.  This change
> doesn't affect time needed to finish the build, but should have a
> slight positive effect on scheduling time on a community cluster.
> And it's also not cool from our side to reserve shared resources that
> we're not using, while they could be used by some other project.
>
> Signed-off-by: Ilya Maximets <i.maximets@ovn.org>
> ---

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

>  .cirrus.yml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/.cirrus.yml b/.cirrus.yml
> index 358f2ba25..480fea242 100644
> --- a/.cirrus.yml
> +++ b/.cirrus.yml
> @@ -5,7 +5,7 @@ freebsd_build_task:
>        image_family: freebsd-12-2-snap
>        image_family: freebsd-11-4-snap
>      cpu: 4
> -    memory: 8G
> +    memory: 4G
>  
>    env:
>      DEPENDENCIES: automake libtool gmake gcc wget openssl python3
Ilya Maximets Sept. 22, 2021, 10:53 p.m. UTC | #2
On 9/16/21 14:42, Aaron Conole wrote:
> Ilya Maximets <i.maximets@ovn.org> writes:
> 
>> According to memory usage graphs, our builds are using 3GB at most.
>> Reducing memory requirements to 4GB to have some room.  This change
>> doesn't affect time needed to finish the build, but should have a
>> slight positive effect on scheduling time on a community cluster.
>> And it's also not cool from our side to reserve shared resources that
>> we're not using, while they could be used by some other project.
>>
>> Signed-off-by: Ilya Maximets <i.maximets@ovn.org>
>> ---
> 
> Acked-by: Aaron Conole <aconole@redhat.com>

Thanks!  Applied to all active branches.

Best regards, Ilya Maximets.

> 
>>  .cirrus.yml | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/.cirrus.yml b/.cirrus.yml
>> index 358f2ba25..480fea242 100644
>> --- a/.cirrus.yml
>> +++ b/.cirrus.yml
>> @@ -5,7 +5,7 @@ freebsd_build_task:
>>        image_family: freebsd-12-2-snap
>>        image_family: freebsd-11-4-snap
>>      cpu: 4
>> -    memory: 8G
>> +    memory: 4G
>>  
>>    env:
>>      DEPENDENCIES: automake libtool gmake gcc wget openssl python3
>
diff mbox series

Patch

diff --git a/.cirrus.yml b/.cirrus.yml
index 358f2ba25..480fea242 100644
--- a/.cirrus.yml
+++ b/.cirrus.yml
@@ -5,7 +5,7 @@  freebsd_build_task:
       image_family: freebsd-12-2-snap
       image_family: freebsd-11-4-snap
     cpu: 4
-    memory: 8G
+    memory: 4G
 
   env:
     DEPENDENCIES: automake libtool gmake gcc wget openssl python3