diff mbox series

[ovs-dev,ovn,v1] vagrant: Increase memory sizes

Message ID 20200131022542.17657-1-flavio@flaviof.com
State Accepted
Headers show
Series [ovs-dev,ovn,v1] vagrant: Increase memory sizes | expand

Commit Message

Flaviof Jan. 31, 2020, 2:25 a.m. UTC
Using default memory sizes is not enough to run some of the tests
invoked by make check. Being so, This change bumps the default
memory size of boxes on Virtualbox and Libvirt providers
to 1Gb and 4 CPUs. This value can be changed by setting the
environment variables VM_MEMORY and VM_CPUS, respectively.

   debian-10: OVN components
   debian-10:
   debian-10:  15: ovn -- 4-term numeric expression normalization
   debian-10:  FAILED (ovn.at:497)
   debian-10:  18: ovn -- 5-term numeric expression normalization
   debian-10:  FAILED (ovn.at:515)
   debian-10:  19: ovn -- 5-term string expression normalization
   debian-10:  FAILED (ovn.at:521)
   debian-10:
   debian-10: OVN end-to-end tests
   debian-10:
   debian-10: 126: ovn -- interconnection
   debian-10:  FAILED (ovn.at:18133)

dmesg:

[ 2303.348732] Out of memory: Kill process 10068 (ovstest) score 821 or sacrifice child
[ 2303.350508] Killed process 10068 (ovstest) total-vm:419844kB, anon-rss:405848kB, file-rss:0kB, shmem-rss:0kB
[ 2303.369933] oom_reaper: reaped process 10068 (ovstest), now anon-rss:0kB, file-rss:0kB, shmem-rss:0kB
root@debian-10:/vagrant/ovn/tests/testsuite.dir/015#

Signed-off-by: Flavio Fernandes <flavio@flaviof.com>
---
 Vagrantfile | 11 +++++++++++
 1 file changed, 11 insertions(+)

Comments

Numan Siddique Jan. 31, 2020, 6:36 a.m. UTC | #1
On Fri, Jan 31, 2020 at 7:56 AM Flavio Fernandes <flavio@flaviof.com> wrote:
>
> Using default memory sizes is not enough to run some of the tests
> invoked by make check. Being so, This change bumps the default
> memory size of boxes on Virtualbox and Libvirt providers
> to 1Gb and 4 CPUs. This value can be changed by setting the
> environment variables VM_MEMORY and VM_CPUS, respectively.
>
>    debian-10: OVN components
>    debian-10:
>    debian-10:  15: ovn -- 4-term numeric expression normalization
>    debian-10:  FAILED (ovn.at:497)
>    debian-10:  18: ovn -- 5-term numeric expression normalization
>    debian-10:  FAILED (ovn.at:515)
>    debian-10:  19: ovn -- 5-term string expression normalization
>    debian-10:  FAILED (ovn.at:521)
>    debian-10:
>    debian-10: OVN end-to-end tests
>    debian-10:
>    debian-10: 126: ovn -- interconnection
>    debian-10:  FAILED (ovn.at:18133)
>
> dmesg:
>
> [ 2303.348732] Out of memory: Kill process 10068 (ovstest) score 821 or sacrifice child
> [ 2303.350508] Killed process 10068 (ovstest) total-vm:419844kB, anon-rss:405848kB, file-rss:0kB, shmem-rss:0kB
> [ 2303.369933] oom_reaper: reaped process 10068 (ovstest), now anon-rss:0kB, file-rss:0kB, shmem-rss:0kB
> root@debian-10:/vagrant/ovn/tests/testsuite.dir/015#
>
> Signed-off-by: Flavio Fernandes <flavio@flaviof.com>

Thanks. I applied this to master.

Numan

> ---
>  Vagrantfile | 11 +++++++++++
>  1 file changed, 11 insertions(+)
>
> diff --git a/Vagrantfile b/Vagrantfile
> index f0526ec48..6a3f32010 100644
> --- a/Vagrantfile
> +++ b/Vagrantfile
> @@ -102,6 +102,17 @@ make check RECHECK=yes || {
>  SCRIPT
>
>  Vagrant.configure(VAGRANTFILE_API_VERSION) do |config|
> +  vm_memory = ENV['VM_MEMORY'] || '1024'
> +  vm_cpus = ENV['VM_CPUS'] || '4'
> +  config.vm.provider 'libvirt' do |lb|
> +    lb.memory = vm_memory
> +    lb.cpus = vm_cpus
> +  end
> +  config.vm.provider "virtualbox" do |vb|
> +    vb.memory = vm_memory
> +    vb.cpus = vm_cpus
> +  end
> +
>    config.vm.define "debian-10" do |debian|
>         debian.vm.hostname = "debian-10"
>         debian.vm.box = "debian/buster64"
> --
> 2.17.1
>
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
diff mbox series

Patch

diff --git a/Vagrantfile b/Vagrantfile
index f0526ec48..6a3f32010 100644
--- a/Vagrantfile
+++ b/Vagrantfile
@@ -102,6 +102,17 @@  make check RECHECK=yes || {
 SCRIPT
 
 Vagrant.configure(VAGRANTFILE_API_VERSION) do |config|
+  vm_memory = ENV['VM_MEMORY'] || '1024'
+  vm_cpus = ENV['VM_CPUS'] || '4'
+  config.vm.provider 'libvirt' do |lb|
+    lb.memory = vm_memory
+    lb.cpus = vm_cpus
+  end
+  config.vm.provider "virtualbox" do |vb|
+    vb.memory = vm_memory
+    vb.cpus = vm_cpus
+  end
+
   config.vm.define "debian-10" do |debian|
        debian.vm.hostname = "debian-10"
        debian.vm.box = "debian/buster64"