DNM: Run the e2e regression at mutiple nodes in parallel to check sta…

…bility Signed-off-by: Quique Llorente <[email protected]>
nmstate · Sep 26, 2019 · 7d56f18 · 7d56f18
1 parent 6842b13
commit 7d56f18
Show file tree

Hide file tree

Showing 20 changed files with 29 additions and 3 deletions.
diff --git a/Makefile b/Makefile
@@ -17,7 +17,7 @@ ifdef UNIT_TEST_EXTRA_ARGS
 	UNIT_TEST_ARGS += $(UNIT_TEST_ARGS)
 endif
 
-E2E_TEST_ARGS ?= -test.v -test.timeout=20m -ginkgo.v -ginkgo.slowSpecThreshold=60
+E2E_TEST_ARGS ?= -test.v -test.timeout=40m -ginkgo.v -ginkgo.slowSpecThreshold=60
 ifdef E2E_TEST_FOCUS
 	E2E_TEST_ARGS +=  -ginkgo.focus $(E2E_TEST_FOCUS)
 endif

diff --git a/automation/check-patch.k8s-1.13.3-1.mounts b/automation/check-patch.k8s-1.13.3-1.mounts
@@ -0,0 +1 @@
+check-patch.mounts
diff --git a/automation/check-patch.k8s-1.13.3-1.packages b/automation/check-patch.k8s-1.13.3-1.packages
@@ -0,0 +1 @@
+check-patch.packages
diff --git a/automation/check-patch.k8s-1.13.3-1.sh b/automation/check-patch.k8s-1.13.3-1.sh
@@ -0,0 +1 @@
+check-patch.sh
diff --git a/automation/check-patch.k8s-1.13.3-2.mounts b/automation/check-patch.k8s-1.13.3-2.mounts
@@ -0,0 +1 @@
+check-patch.mounts
diff --git a/automation/check-patch.k8s-1.13.3-2.packages b/automation/check-patch.k8s-1.13.3-2.packages
@@ -0,0 +1 @@
+check-patch.packages
diff --git a/automation/check-patch.k8s-1.13.3-2.sh b/automation/check-patch.k8s-1.13.3-2.sh
@@ -0,0 +1 @@
+check-patch.sh
diff --git a/automation/check-patch.k8s-1.13.3-3.mounts b/automation/check-patch.k8s-1.13.3-3.mounts
@@ -0,0 +1 @@
+check-patch.mounts
diff --git a/automation/check-patch.k8s-1.13.3-3.packages b/automation/check-patch.k8s-1.13.3-3.packages
@@ -0,0 +1 @@
+check-patch.packages
diff --git a/automation/check-patch.k8s-1.13.3-3.sh b/automation/check-patch.k8s-1.13.3-3.sh
@@ -0,0 +1 @@
+check-patch.sh
diff --git a/automation/check-patch.k8s-1.13.3-4.mounts b/automation/check-patch.k8s-1.13.3-4.mounts
@@ -0,0 +1 @@
+check-patch.mounts
diff --git a/automation/check-patch.k8s-1.13.3-4.packages b/automation/check-patch.k8s-1.13.3-4.packages
@@ -0,0 +1 @@
+check-patch.packages
diff --git a/automation/check-patch.k8s-1.13.3-4.sh b/automation/check-patch.k8s-1.13.3-4.sh
@@ -0,0 +1 @@
+check-patch.sh
diff --git a/automation/check-patch.k8s-1.13.3-5.mounts b/automation/check-patch.k8s-1.13.3-5.mounts
@@ -0,0 +1 @@
+check-patch.mounts
diff --git a/automation/check-patch.k8s-1.13.3-5.packages b/automation/check-patch.k8s-1.13.3-5.packages
@@ -0,0 +1 @@
+check-patch.packages
diff --git a/automation/check-patch.k8s-1.13.3-5.sh b/automation/check-patch.k8s-1.13.3-5.sh
@@ -0,0 +1 @@
+check-patch.sh
diff --git a/automation/test.sh b/automation/test.sh
@@ -8,7 +8,7 @@ teardown() {
     make cluster-down
 }
 
-export KUBEVIRT_PROVIDER=$TARGET
+export KUBEVIRT_PROVIDER=$(echo $TARGET| sed "s/3-.*/3/")
 
 # Make sure that the VM is properly shut down on exit
 trap teardown EXIT SIGINT SIGTERM SIGSTOP

diff --git a/stdci.yaml b/stdci.yaml
@@ -1,5 +1,10 @@
 sub-stages:
   - k8s-1.13.3
+  - k8s-1.13.3-1
+  - k8s-1.13.3-2
+  - k8s-1.13.3-3
+  - k8s-1.13.3-4
+  - k8s-1.13.3-5
   - os-3.11.0
 
 runtime_requirements:

diff --git a/test/e2e/default_bridged_network_test.go b/test/e2e/default_bridged_network_test.go
@@ -159,6 +159,6 @@ func waitForNodesReady() {
 	for _, node := range nodes {
 		Eventually(func() (corev1.ConditionStatus, error) {
 			return nodeReadyConditionStatus(node)
-		}, 60*time.Second, 1*time.Second).Should(Equal(corev1.ConditionTrue))
+		}, 180*time.Second, 1*time.Second).Should(Equal(corev1.ConditionTrue))
 	}
 }
diff --git a/test/e2e/nns_conditions_test.go b/test/e2e/nns_conditions_test.go
@@ -56,6 +56,12 @@ var _ = Describe("NodeNetworkStateCondition", func() {
 		BeforeEach(func() {
 			updateDesiredState(invalidConfig)
 		})
+		AfterEach(func() {
+			updateDesiredState(br1Absent)
+			for _, node := range nodes {
+				interfacesNameForNode(node).ShouldNot(ContainElement("br1"))
+			}
+		})
 		It("should have Failing ConditionType set to true", func() {
 			for _, node := range nodes {
 				checkCondition(node, nmstatev1alpha1.NodeNetworkStateConditionFailing).Should(