code review

This commit is contained in:
Alex Dadgar
2018-06-06 14:52:26 -07:00
parent 3c9ad243a4
commit 3402f2539f
2 changed files with 1 additions and 3 deletions

View File

@@ -179,7 +179,7 @@ func (n *Nodes) monitorDrainMultiplex(ctx context.Context, cancel func(),
case <-ctx.Done():
// If we are exiting but we have a message, attempt to send it
// so we don't loose a message but do not block.
// so we don't lose a message but do not block.
select {
case outCh <- msg:
default:

View File

@@ -267,7 +267,6 @@ func TestNodeDrainCommand_Monitor_NoDrainStrategy(t *testing.T) {
defer server.Shutdown()
// Wait for a node to appear
var nodeID string
testutil.WaitForResult(func() (bool, error) {
nodes, _, err := client.Nodes().List(nil)
if err != nil {
@@ -276,7 +275,6 @@ func TestNodeDrainCommand_Monitor_NoDrainStrategy(t *testing.T) {
if len(nodes) == 0 {
return false, fmt.Errorf("missing node")
}
nodeID = nodes[0].ID
return true, nil
}, func(err error) {
t.Fatalf("err: %s", err)