Skip to content
This repository was archived by the owner on Jul 30, 2021. It is now read-only.

Commit 3ce7d09

Browse files
author
Yifan Gu
authored
Merge pull request #560 from yifan-gu/fix_test
e2e: Fix GetCluster().
2 parents b0be7a5 + d8348ec commit 3ce7d09

File tree

1 file changed

+17
-3
lines changed

1 file changed

+17
-3
lines changed

e2e/node.go

Lines changed: 17 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,9 @@
11
package e2e
22

33
import (
4+
"bytes"
45
"fmt"
6+
"time"
57

68
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
79
"k8s.io/client-go/pkg/api/v1"
@@ -50,7 +52,19 @@ func (n *Node) Reboot() error {
5052
if err != nil {
5153
return fmt.Errorf("issuing reboot command failed\nstdout:%s\nstderr:%s", stdout, stderr)
5254
}
53-
return nil
55+
56+
checker := func() error {
57+
stdout, stderr, err = n.SSH("systemctl is-system-running")
58+
if err != nil {
59+
return err
60+
}
61+
if !bytes.Contains(stdout, []byte("running")) {
62+
return fmt.Errorf("system is not running yet")
63+
}
64+
return nil
65+
}
66+
67+
return retry(20, 10*time.Second, checker)
5468
}
5569

5670
// IsMaster returns true if the node's labels contains "node-role.kubernetes.io/master".
@@ -74,8 +88,8 @@ func GetCluster() (*Cluster, error) {
7488
return nil, err
7589
}
7690

77-
for _, n := range nodelist.Items {
78-
nn := newNode(&n)
91+
for i := range nodelist.Items {
92+
nn := newNode(&nodelist.Items[i])
7993
if nn.IsMaster() {
8094
c.Masters = append(c.Masters, nn)
8195
} else {

0 commit comments

Comments
 (0)