simplify retry logic

hashicorp · Mar 1, 2018 · bf72619 · bf72619
1 parent d635323
commit bf72619
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/client/client.go b/client/client.go
@@ -1544,9 +1544,6 @@ func (c *Client) watchNodeUpdates() {
 	for {
 		select {
 		case <-timer.C:
-			if !hasChanged {
-				continue
-			}
 			c.logger.Printf("[DEBUG] client: state changed, updating node and re-registering.")
 
 			// Update the config copy.
@@ -1558,9 +1555,12 @@ func (c *Client) watchNodeUpdates() {
 			c.retryRegisterNode()
 
 			hasChanged = false
-			timer.Reset(c.retryIntv(nodeUpdateRetryIntv))
 		case <-c.triggerNodeUpdate:
+			if hasChanged {
+				continue
+			}
 			hasChanged = true
+			timer.Reset(c.retryIntv(nodeUpdateRetryIntv))
 		case <-c.shutdownCh:
 			return
 		}