status-im/consul
2
0
Fork 0
mirror of https://github.com/status-im/consul.git synced 2025-01-26 21:51:39 +00:00
Code Issues Projects Releases Wiki Activity

Makes empty checkServiceNode return a nil.

Browse Source 
The change in #2308 had an inadvertent interface change, so we fix that with
a special case in this fix.
This commit is contained in:
James Phillips 2016-08-29 19:12:07 -07:00
parent a7cfcc3634
commit 53149bd2f9
No known key found for this signature in database
GPG Key ID: 77183E682AC5FC11
1 changed files with 6 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
consul/state/state_store.go
View File

@ -1248,6 +1248,12 @@ func (s *StateStore) parseCheckServiceNodes(
return 0, nil, err return 0, nil, err
} }
// Special-case the zero return value to nil, since this ends up in
// external APIs.
if len(services) == 0 {
return idx, nil, nil
}
results := make(structs.CheckServiceNodes, 0, len(services)) results := make(structs.CheckServiceNodes, 0, len(services))
for _, sn := range services { for _, sn := range services {
// Retrieve the node. // Retrieve the node.