diff mbox

[ovs-dev,2/2] vagrant: use RECHECK=yes for system checks

Message ID 1474313464-30598-2-git-send-email-cascardo@redhat.com
State Accepted
Delegated to: Daniele Di Proietto
Headers show

Commit Message

Thadeu Lima de Souza Cascardo Sept. 19, 2016, 7:31 p.m. UTC 
Use RECHECK=yes for both kernel and userspace datapath tests.

Signed-off-by: Thadeu Lima de Souza Cascardo <cascardo@redhat.com>
---
 Vagrantfile | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Ben Pfaff Oct. 4, 2016, 7:02 p.m. UTC | #1 
On Mon, Sep 19, 2016 at 04:31:04PM -0300, Thadeu Lima de Souza Cascardo wrote:
> Use RECHECK=yes for both kernel and userspace datapath tests.
> 
> Signed-off-by: Thadeu Lima de Souza Cascardo <cascardo@redhat.com>

Looks good to me but Daniele knows the system testsuite (much!) better
than me so I'll leave this to him.
Daniele Di Proietto Oct. 7, 2016, 10:54 p.m. UTC | #2 
On 04/10/2016 12:02, "Ben Pfaff" <blp@ovn.org> wrote:

>On Mon, Sep 19, 2016 at 04:31:04PM -0300, Thadeu Lima de Souza Cascardo wrote:

>> Use RECHECK=yes for both kernel and userspace datapath tests.

>> 

>> Signed-off-by: Thadeu Lima de Souza Cascardo <cascardo@redhat.com>

>

>Looks good to me but Daniele knows the system testsuite (much!) better

>than me so I'll leave this to him.


Looks good to me, applied to master, thanks!
diff mbox

Patch

diff --git a/Vagrantfile b/Vagrantfile
index 11bd048..91f943d 100644
--- a/Vagrantfile
+++ b/Vagrantfile
@@ -47,7 +47,7 @@  SCRIPT
 
 $test_kmod = <<SCRIPT
 cd ~/build
-make check-kmod
+make check-kmod RECHECK=yes
 SCRIPT
 
 $install_rpm = <<SCRIPT
@@ -83,7 +83,7 @@  SCRIPT
 
 $test_ovs_system_userspace = <<SCRIPT
 cd ~/build
-make check-system-userspace
+make check-system-userspace RECHECK=yes
 SCRIPT
 
 Vagrant.configure(VAGRANTFILE_API_VERSION) do |config|