do not surface left servers in catalog

hashicorp · Aug 29, 2019 · f0cb17f · f0cb17f
1 parent 2f37d68
commit f0cb17f
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/agent/metadata/server.go b/agent/metadata/server.go
@@ -64,6 +64,10 @@ func (s *Server) String() string {
 	return fmt.Sprintf("%s (Addr: %s/%s) (DC: %s)", s.Name, networkStr, addrStr, s.Datacenter)
 }
 
+func (s *Server) HasLeft() bool {
+	return s.Status == serf.StatusLeft
+}
+
 var versionFormat = regexp.MustCompile(`\d+\.\d+\.\d+`)
 
 // IsConsulServer returns true if a serf member is a consul server

diff --git a/agent/router/router.go b/agent/router/router.go
@@ -406,6 +406,12 @@ func (r *Router) GetDatacentersByDistance() ([]string, error) {
 				continue
 			}
 
+			if parts.HasLeft() {
+				r.logger.Printf("[DEBUG]: consul: server %q in area %q left, skipping",
+					m.Name, areaID)
+				continue
+			}
+
 			existing := index[parts.Datacenter]
 			if parts.Datacenter == r.localDatacenter {
 				// Everything in the local datacenter looks like zero RTT.