node_lease.go 8.9 KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214
  1. /*
  2. Copyright 2018 The Kubernetes Authors.
  3. Licensed under the Apache License, Version 2.0 (the "License");
  4. you may not use this file except in compliance with the License.
  5. You may obtain a copy of the License at
  6. http://www.apache.org/licenses/LICENSE-2.0
  7. Unless required by applicable law or agreed to in writing, software
  8. distributed under the License is distributed on an "AS IS" BASIS,
  9. WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
  10. See the License for the specific language governing permissions and
  11. limitations under the License.
  12. */
  13. package common
  14. import (
  15. "context"
  16. "fmt"
  17. "time"
  18. coordinationv1 "k8s.io/api/coordination/v1"
  19. v1 "k8s.io/api/core/v1"
  20. apiequality "k8s.io/apimachinery/pkg/api/equality"
  21. metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
  22. "k8s.io/apimachinery/pkg/util/diff"
  23. "k8s.io/apimachinery/pkg/util/wait"
  24. clientset "k8s.io/client-go/kubernetes"
  25. "k8s.io/kubernetes/test/e2e/framework"
  26. e2enode "k8s.io/kubernetes/test/e2e/framework/node"
  27. testutils "k8s.io/kubernetes/test/utils"
  28. "github.com/onsi/ginkgo"
  29. "github.com/onsi/gomega"
  30. )
  31. var _ = framework.KubeDescribe("NodeLease", func() {
  32. var nodeName string
  33. f := framework.NewDefaultFramework("node-lease-test")
  34. ginkgo.BeforeEach(func() {
  35. node, err := e2enode.GetRandomReadySchedulableNode(f.ClientSet)
  36. framework.ExpectNoError(err)
  37. nodeName = node.Name
  38. })
  39. ginkgo.Context("when the NodeLease feature is enabled", func() {
  40. ginkgo.It("the kubelet should create and update a lease in the kube-node-lease namespace", func() {
  41. leaseClient := f.ClientSet.CoordinationV1().Leases(v1.NamespaceNodeLease)
  42. var (
  43. err error
  44. lease *coordinationv1.Lease
  45. )
  46. ginkgo.By("check that lease for this Kubelet exists in the kube-node-lease namespace")
  47. gomega.Eventually(func() error {
  48. lease, err = leaseClient.Get(context.TODO(), nodeName, metav1.GetOptions{})
  49. if err != nil {
  50. return err
  51. }
  52. return nil
  53. }, 5*time.Minute, 5*time.Second).Should(gomega.BeNil())
  54. // check basic expectations for the lease
  55. gomega.Expect(expectLease(lease, nodeName)).To(gomega.BeNil())
  56. ginkgo.By("check that node lease is updated at least once within the lease duration")
  57. gomega.Eventually(func() error {
  58. newLease, err := leaseClient.Get(context.TODO(), nodeName, metav1.GetOptions{})
  59. if err != nil {
  60. return err
  61. }
  62. // check basic expectations for the latest lease
  63. if err := expectLease(newLease, nodeName); err != nil {
  64. return err
  65. }
  66. // check that RenewTime has been updated on the latest lease
  67. newTime := (*newLease.Spec.RenewTime).Time
  68. oldTime := (*lease.Spec.RenewTime).Time
  69. if !newTime.After(oldTime) {
  70. return fmt.Errorf("new lease has time %v, which is not after old lease time %v", newTime, oldTime)
  71. }
  72. return nil
  73. }, time.Duration(*lease.Spec.LeaseDurationSeconds)*time.Second,
  74. time.Duration(*lease.Spec.LeaseDurationSeconds/4)*time.Second)
  75. })
  76. ginkgo.It("should have OwnerReferences set", func() {
  77. leaseClient := f.ClientSet.CoordinationV1().Leases(v1.NamespaceNodeLease)
  78. var (
  79. err error
  80. leaseList *coordinationv1.LeaseList
  81. )
  82. gomega.Eventually(func() error {
  83. leaseList, err = leaseClient.List(context.TODO(), metav1.ListOptions{})
  84. if err != nil {
  85. return err
  86. }
  87. return nil
  88. }, 5*time.Minute, 5*time.Second).Should(gomega.BeNil())
  89. // All the leases should have OwnerReferences set to their corresponding
  90. // Node object.
  91. for i := range leaseList.Items {
  92. lease := &leaseList.Items[i]
  93. ownerRefs := lease.ObjectMeta.OwnerReferences
  94. framework.ExpectEqual(len(ownerRefs), 1)
  95. framework.ExpectEqual(ownerRefs[0].Kind, v1.SchemeGroupVersion.WithKind("Node").Kind)
  96. framework.ExpectEqual(ownerRefs[0].APIVersion, v1.SchemeGroupVersion.WithKind("Node").Version)
  97. }
  98. })
  99. ginkgo.It("the kubelet should report node status infrequently", func() {
  100. ginkgo.By("wait until node is ready")
  101. e2enode.WaitForNodeToBeReady(f.ClientSet, nodeName, 5*time.Minute)
  102. ginkgo.By("wait until there is node lease")
  103. var err error
  104. var lease *coordinationv1.Lease
  105. gomega.Eventually(func() error {
  106. lease, err = f.ClientSet.CoordinationV1().Leases(v1.NamespaceNodeLease).Get(context.TODO(), nodeName, metav1.GetOptions{})
  107. if err != nil {
  108. return err
  109. }
  110. return nil
  111. }, 5*time.Minute, 5*time.Second).Should(gomega.BeNil())
  112. // check basic expectations for the lease
  113. gomega.Expect(expectLease(lease, nodeName)).To(gomega.BeNil())
  114. leaseDuration := time.Duration(*lease.Spec.LeaseDurationSeconds) * time.Second
  115. ginkgo.By("verify NodeStatus report period is longer than lease duration")
  116. // NodeStatus is reported from node to master when there is some change or
  117. // enough time has passed. So for here, keep checking the time diff
  118. // between 2 NodeStatus report, until it is longer than lease duration
  119. // (the same as nodeMonitorGracePeriod), or it doesn't change for at least leaseDuration
  120. lastHeartbeatTime, lastStatus := getHeartbeatTimeAndStatus(f.ClientSet, nodeName)
  121. lastObserved := time.Now()
  122. err = wait.Poll(time.Second, 5*time.Minute, func() (bool, error) {
  123. currentHeartbeatTime, currentStatus := getHeartbeatTimeAndStatus(f.ClientSet, nodeName)
  124. currentObserved := time.Now()
  125. if currentHeartbeatTime == lastHeartbeatTime {
  126. if currentObserved.Sub(lastObserved) > 2*leaseDuration {
  127. // heartbeat hasn't changed while watching for at least 2*leaseDuration, success!
  128. framework.Logf("node status heartbeat is unchanged for %s, was waiting for at least %s, success!", currentObserved.Sub(lastObserved), 2*leaseDuration)
  129. return true, nil
  130. }
  131. framework.Logf("node status heartbeat is unchanged for %s, waiting for %s", currentObserved.Sub(lastObserved), 2*leaseDuration)
  132. return false, nil
  133. }
  134. if currentHeartbeatTime.Sub(lastHeartbeatTime) >= leaseDuration {
  135. // heartbeat time changed, but the diff was greater than leaseDuration, success!
  136. framework.Logf("node status heartbeat changed in %s, was waiting for at least %s, success!", currentHeartbeatTime.Sub(lastHeartbeatTime), leaseDuration)
  137. return true, nil
  138. }
  139. if !apiequality.Semantic.DeepEqual(lastStatus, currentStatus) {
  140. // heartbeat time changed, but there were relevant changes in the status, keep waiting
  141. framework.Logf("node status heartbeat changed in %s (with other status changes), waiting for %s", currentHeartbeatTime.Sub(lastHeartbeatTime), leaseDuration)
  142. framework.Logf("%s", diff.ObjectReflectDiff(lastStatus, currentStatus))
  143. lastHeartbeatTime = currentHeartbeatTime
  144. lastObserved = currentObserved
  145. lastStatus = currentStatus
  146. return false, nil
  147. }
  148. // heartbeat time changed, with no other status changes, in less time than we expected, so fail.
  149. return false, fmt.Errorf("node status heartbeat changed in %s (with no other status changes), was waiting for %s", currentHeartbeatTime.Sub(lastHeartbeatTime), leaseDuration)
  150. })
  151. // a timeout is acceptable, since it means we waited 5 minutes and didn't see any unwarranted node status updates
  152. if err != nil && err != wait.ErrWaitTimeout {
  153. framework.ExpectNoError(err, "error waiting for infrequent nodestatus update")
  154. }
  155. ginkgo.By("verify node is still in ready status even though node status report is infrequent")
  156. // This check on node status is only meaningful when this e2e test is
  157. // running as cluster e2e test, because node e2e test does not create and
  158. // run controller manager, i.e., no node lifecycle controller.
  159. node, err := f.ClientSet.CoreV1().Nodes().Get(context.TODO(), nodeName, metav1.GetOptions{})
  160. gomega.Expect(err).To(gomega.BeNil())
  161. _, readyCondition := testutils.GetNodeCondition(&node.Status, v1.NodeReady)
  162. framework.ExpectEqual(readyCondition.Status, v1.ConditionTrue)
  163. })
  164. })
  165. })
  166. func getHeartbeatTimeAndStatus(clientSet clientset.Interface, nodeName string) (time.Time, v1.NodeStatus) {
  167. node, err := clientSet.CoreV1().Nodes().Get(context.TODO(), nodeName, metav1.GetOptions{})
  168. gomega.Expect(err).To(gomega.BeNil())
  169. _, readyCondition := testutils.GetNodeCondition(&node.Status, v1.NodeReady)
  170. framework.ExpectEqual(readyCondition.Status, v1.ConditionTrue)
  171. heartbeatTime := readyCondition.LastHeartbeatTime.Time
  172. readyCondition.LastHeartbeatTime = metav1.Time{}
  173. return heartbeatTime, node.Status
  174. }
  175. func expectLease(lease *coordinationv1.Lease, nodeName string) error {
  176. // expect values for HolderIdentity, LeaseDurationSeconds, and RenewTime
  177. if lease.Spec.HolderIdentity == nil {
  178. return fmt.Errorf("Spec.HolderIdentity should not be nil")
  179. }
  180. if lease.Spec.LeaseDurationSeconds == nil {
  181. return fmt.Errorf("Spec.LeaseDurationSeconds should not be nil")
  182. }
  183. if lease.Spec.RenewTime == nil {
  184. return fmt.Errorf("Spec.RenewTime should not be nil")
  185. }
  186. // ensure that the HolderIdentity matches the node name
  187. if *lease.Spec.HolderIdentity != nodeName {
  188. return fmt.Errorf("Spec.HolderIdentity (%v) should match the node name (%v)", *lease.Spec.HolderIdentity, nodeName)
  189. }
  190. return nil
  191. }