consul: fixing overly redundant logging

This commit is contained in:
Armon Dadgar 2015-05-05 13:00:47 -07:00
parent a86f31517b
commit 8b8abe2268
1 changed files with 2 additions and 1 deletions

View File

@ -479,7 +479,6 @@ func (s *Server) handleDeregisterMember(reason string, member serf.Member) error
// Remove from Raft peers if this was a server // Remove from Raft peers if this was a server
if valid, parts := isConsulServer(member); valid { if valid, parts := isConsulServer(member); valid {
s.logger.Printf("[INFO] consul: server '%s' %s, removing as peer", member.Name, reason)
if err := s.removeConsulServer(member, parts.Port); err != nil { if err := s.removeConsulServer(member, parts.Port); err != nil {
return err return err
} }
@ -539,6 +538,8 @@ func (s *Server) removeConsulServer(m serf.Member, port int) error {
s.logger.Printf("[ERR] consul: failed to remove raft peer '%v': %v", s.logger.Printf("[ERR] consul: failed to remove raft peer '%v': %v",
peer, err) peer, err)
return err return err
} else if err == nil {
s.logger.Printf("[INFO] consul: removed server '%s' as peer", m.Name)
} }
return nil return nil
} }