Skip to content

Commit

Permalink
lack of name does not an invalide member make
Browse files Browse the repository at this point in the history
  • Loading branch information
matt-deboer committed Mar 23, 2017
1 parent 4106edc commit 0d503ff
Showing 1 changed file with 3 additions and 16 deletions.
19 changes: 3 additions & 16 deletions pkg/discovery/discovery.go
Original file line number Diff line number Diff line change
Expand Up @@ -304,23 +304,10 @@ func (d *Discovery) resolveMembersAndAPI(expectedMembers []etcd.Member,
log.Debugf("Member %s reported current leader uptime is %s", member.Name, uptime)
}

// sanity-check the returned members; it may be partial in case of a yet-forming cluster
hasInvalidMembers := false
for _, m := range currentMembers {
if len(m.Name) == 0 || len(m.PeerURLs) == 0 {
if log.GetLevel() >= log.DebugLevel {
log.Debugf("Returned actual member list contains invalid member: %#v", m)
}
hasInvalidMembers = true
break
}
}
if !hasInvalidMembers {
if log.GetLevel() >= log.DebugLevel {
log.Debugf("Actual cluster members: %#v", currentMembers)
}
return membersAPI, currentMembers, uptime, nil
if log.GetLevel() >= log.DebugLevel {
log.Debugf("Actual cluster members: %#v", currentMembers)
}
return membersAPI, currentMembers, uptime, nil
}
}
if len(currentMembers) == 0 {
Expand Down

0 comments on commit 0d503ff

Please sign in to comment.