2017-03-18 01:42:28 +00:00
|
|
|
package consul
|
|
|
|
|
|
|
|
import (
|
2017-03-20 03:48:42 +00:00
|
|
|
"context"
|
|
|
|
"log"
|
2017-03-18 01:42:28 +00:00
|
|
|
"sync"
|
|
|
|
|
pkg refactor
command/agent/* -> agent/*
command/consul/* -> agent/consul/*
command/agent/command{,_test}.go -> command/agent{,_test}.go
command/base/command.go -> command/base.go
command/base/* -> command/*
commands.go -> command/commands.go
The script which did the refactor is:
(
cd $GOPATH/src/github.com/hashicorp/consul
git mv command/agent/command.go command/agent.go
git mv command/agent/command_test.go command/agent_test.go
git mv command/agent/flag_slice_value{,_test}.go command/
git mv command/agent .
git mv command/base/command.go command/base.go
git mv command/base/config_util{,_test}.go command/
git mv commands.go command/
git mv consul agent
rmdir command/base/
gsed -i -e 's|package agent|package command|' command/agent{,_test}.go
gsed -i -e 's|package agent|package command|' command/flag_slice_value{,_test}.go
gsed -i -e 's|package base|package command|' command/base.go command/config_util{,_test}.go
gsed -i -e 's|package main|package command|' command/commands.go
gsed -i -e 's|base.Command|BaseCommand|' command/commands.go
gsed -i -e 's|agent.Command|AgentCommand|' command/commands.go
gsed -i -e 's|\tCommand:|\tBaseCommand:|' command/commands.go
gsed -i -e 's|base\.||' command/commands.go
gsed -i -e 's|command\.||' command/commands.go
gsed -i -e 's|command|c|' main.go
gsed -i -e 's|range Commands|range command.Commands|' main.go
gsed -i -e 's|Commands: Commands|Commands: command.Commands|' main.go
gsed -i -e 's|base\.BoolValue|BoolValue|' command/operator_autopilot_set.go
gsed -i -e 's|base\.DurationValue|DurationValue|' command/operator_autopilot_set.go
gsed -i -e 's|base\.StringValue|StringValue|' command/operator_autopilot_set.go
gsed -i -e 's|base\.UintValue|UintValue|' command/operator_autopilot_set.go
gsed -i -e 's|\bCommand\b|BaseCommand|' command/base.go
gsed -i -e 's|BaseCommand Options|Command Options|' command/base.go
gsed -i -e 's|base.Command|BaseCommand|' command/*.go
gsed -i -e 's|c\.Command|c.BaseCommand|g' command/*.go
gsed -i -e 's|\tCommand:|\tBaseCommand:|' command/*_test.go
gsed -i -e 's|base\.||' command/*_test.go
gsed -i -e 's|\bCommand\b|AgentCommand|' command/agent{,_test}.go
gsed -i -e 's|cmd.AgentCommand|cmd.BaseCommand|' command/agent.go
gsed -i -e 's|cli.AgentCommand = new(Command)|cli.Command = new(AgentCommand)|' command/agent_test.go
gsed -i -e 's|exec.AgentCommand|exec.Command|' command/agent_test.go
gsed -i -e 's|exec.BaseCommand|exec.Command|' command/agent_test.go
gsed -i -e 's|NewTestAgent|agent.NewTestAgent|' command/agent_test.go
gsed -i -e 's|= TestConfig|= agent.TestConfig|' command/agent_test.go
gsed -i -e 's|: RetryJoin|: agent.RetryJoin|' command/agent_test.go
gsed -i -e 's|\.\./\.\./|../|' command/config_util_test.go
gsed -i -e 's|\bverifyUniqueListeners|VerifyUniqueListeners|' agent/config{,_test}.go command/agent.go
gsed -i -e 's|\bserfLANKeyring\b|SerfLANKeyring|g' agent/{agent,keyring,testagent}.go command/agent.go
gsed -i -e 's|\bserfWANKeyring\b|SerfWANKeyring|g' agent/{agent,keyring,testagent}.go command/agent.go
gsed -i -e 's|\bNewAgent\b|agent.New|g' command/agent{,_test}.go
gsed -i -e 's|\bNewAgent|New|' agent/{acl_test,agent,testagent}.go
gsed -i -e 's|\bAgent\b|agent.&|g' command/agent{,_test}.go
gsed -i -e 's|\bBool\b|agent.&|g' command/agent{,_test}.go
gsed -i -e 's|\bConfig\b|agent.&|g' command/agent{,_test}.go
gsed -i -e 's|\bDefaultConfig\b|agent.&|g' command/agent{,_test}.go
gsed -i -e 's|\bDevConfig\b|agent.&|g' command/agent{,_test}.go
gsed -i -e 's|\bMergeConfig\b|agent.&|g' command/agent{,_test}.go
gsed -i -e 's|\bReadConfigPaths\b|agent.&|g' command/agent{,_test}.go
gsed -i -e 's|\bParseMetaPair\b|agent.&|g' command/agent{,_test}.go
gsed -i -e 's|\bSerfLANKeyring\b|agent.&|g' command/agent{,_test}.go
gsed -i -e 's|\bSerfWANKeyring\b|agent.&|g' command/agent{,_test}.go
gsed -i -e 's|circonus\.agent|circonus|g' command/agent{,_test}.go
gsed -i -e 's|logger\.agent|logger|g' command/agent{,_test}.go
gsed -i -e 's|metrics\.agent|metrics|g' command/agent{,_test}.go
gsed -i -e 's|// agent.Agent|// agent|' command/agent{,_test}.go
gsed -i -e 's|a\.agent\.Config|a.Config|' command/agent{,_test}.go
gsed -i -e 's|agent\.AppendSliceValue|AppendSliceValue|' command/{configtest,validate}.go
gsed -i -e 's|consul/consul|agent/consul|' GNUmakefile
gsed -i -e 's|\.\./test|../../test|' agent/consul/server_test.go
# fix imports
f=$(grep -rl 'github.com/hashicorp/consul/command/agent' * | grep '\.go')
gsed -i -e 's|github.com/hashicorp/consul/command/agent|github.com/hashicorp/consul/agent|' $f
goimports -w $f
f=$(grep -rl 'github.com/hashicorp/consul/consul' * | grep '\.go')
gsed -i -e 's|github.com/hashicorp/consul/consul|github.com/hashicorp/consul/agent/consul|' $f
goimports -w $f
goimports -w command/*.go main.go
)
2017-06-09 22:28:28 +00:00
|
|
|
"github.com/hashicorp/consul/agent/consul/agent"
|
|
|
|
"github.com/hashicorp/consul/agent/consul/structs"
|
2017-06-15 13:16:16 +00:00
|
|
|
"github.com/hashicorp/consul/agent/pool"
|
2017-03-18 01:42:28 +00:00
|
|
|
)
|
|
|
|
|
2017-03-20 03:48:42 +00:00
|
|
|
// StatsFetcher has two functions for autopilot. First, lets us fetch all the
|
|
|
|
// stats in parallel so we are taking a sample as close to the same time as
|
|
|
|
// possible, since we are comparing time-sensitive info for the health check.
|
|
|
|
// Second, it bounds the time so that one slow RPC can't hold up the health
|
|
|
|
// check loop; as a side effect of how it implements this, it also limits to
|
|
|
|
// a single in-flight RPC to any given server, so goroutines don't accumulate
|
|
|
|
// as we run the health check fairly frequently.
|
2017-03-18 01:42:28 +00:00
|
|
|
type StatsFetcher struct {
|
2017-03-20 03:48:42 +00:00
|
|
|
logger *log.Logger
|
2017-06-15 13:16:16 +00:00
|
|
|
pool *pool.ConnPool
|
2017-03-18 01:42:28 +00:00
|
|
|
datacenter string
|
|
|
|
inflight map[string]struct{}
|
|
|
|
inflightLock sync.Mutex
|
|
|
|
}
|
|
|
|
|
|
|
|
// NewStatsFetcher returns a stats fetcher.
|
2017-06-15 13:16:16 +00:00
|
|
|
func NewStatsFetcher(logger *log.Logger, pool *pool.ConnPool, datacenter string) *StatsFetcher {
|
2017-03-18 01:42:28 +00:00
|
|
|
return &StatsFetcher{
|
2017-03-20 03:48:42 +00:00
|
|
|
logger: logger,
|
2017-03-18 01:42:28 +00:00
|
|
|
pool: pool,
|
|
|
|
datacenter: datacenter,
|
|
|
|
inflight: make(map[string]struct{}),
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2017-03-20 03:48:42 +00:00
|
|
|
// fetch does the RPC to fetch the server stats from a single server. We don't
|
|
|
|
// cancel this when the context is canceled because we only want one in-flight
|
|
|
|
// RPC to each server, so we let it finish and then clean up the in-flight
|
|
|
|
// tracking.
|
|
|
|
func (f *StatsFetcher) fetch(server *agent.Server, replyCh chan *structs.ServerStats) {
|
|
|
|
var args struct{}
|
2017-03-18 01:42:28 +00:00
|
|
|
var reply structs.ServerStats
|
2017-05-10 21:25:48 +00:00
|
|
|
err := f.pool.RPC(f.datacenter, server.Addr, server.Version, "Status.RaftStats", server.UseTLS, &args, &reply)
|
2017-03-20 03:48:42 +00:00
|
|
|
if err != nil {
|
2017-03-20 16:27:28 +00:00
|
|
|
f.logger.Printf("[WARN] consul: error getting server health from %q: %v",
|
|
|
|
server.Name, err)
|
2017-03-20 03:48:42 +00:00
|
|
|
} else {
|
|
|
|
replyCh <- &reply
|
|
|
|
}
|
2017-03-18 01:42:28 +00:00
|
|
|
|
2017-03-20 03:48:42 +00:00
|
|
|
f.inflightLock.Lock()
|
|
|
|
delete(f.inflight, server.ID)
|
|
|
|
f.inflightLock.Unlock()
|
|
|
|
}
|
2017-03-18 01:42:28 +00:00
|
|
|
|
2017-03-20 03:48:42 +00:00
|
|
|
// Fetch will attempt to query all the servers in parallel.
|
|
|
|
func (f *StatsFetcher) Fetch(ctx context.Context, servers []*agent.Server) map[string]*structs.ServerStats {
|
|
|
|
type workItem struct {
|
|
|
|
server *agent.Server
|
|
|
|
replyCh chan *structs.ServerStats
|
|
|
|
}
|
|
|
|
var work []*workItem
|
2017-03-18 01:42:28 +00:00
|
|
|
|
2017-03-20 03:48:42 +00:00
|
|
|
// Skip any servers that have inflight requests.
|
|
|
|
f.inflightLock.Lock()
|
|
|
|
for _, server := range servers {
|
|
|
|
if _, ok := f.inflight[server.ID]; ok {
|
2017-03-20 16:27:28 +00:00
|
|
|
f.logger.Printf("[WARN] consul: error getting server health from %q: last request still outstanding",
|
|
|
|
server.Name)
|
2017-03-18 01:42:28 +00:00
|
|
|
} else {
|
2017-03-20 03:48:42 +00:00
|
|
|
workItem := &workItem{
|
|
|
|
server: server,
|
|
|
|
replyCh: make(chan *structs.ServerStats, 1),
|
|
|
|
}
|
|
|
|
work = append(work, workItem)
|
|
|
|
f.inflight[server.ID] = struct{}{}
|
|
|
|
go f.fetch(workItem.server, workItem.replyCh)
|
2017-03-18 01:42:28 +00:00
|
|
|
}
|
2017-03-20 03:48:42 +00:00
|
|
|
}
|
|
|
|
f.inflightLock.Unlock()
|
|
|
|
|
|
|
|
// Now wait for the results to come in, or for the context to be
|
|
|
|
// canceled.
|
|
|
|
replies := make(map[string]*structs.ServerStats)
|
|
|
|
for _, workItem := range work {
|
|
|
|
select {
|
|
|
|
case reply := <-workItem.replyCh:
|
|
|
|
replies[workItem.server.ID] = reply
|
2017-03-18 01:42:28 +00:00
|
|
|
|
2017-03-20 03:48:42 +00:00
|
|
|
case <-ctx.Done():
|
2017-03-20 16:27:28 +00:00
|
|
|
f.logger.Printf("[WARN] consul: error getting server health from %q: %v",
|
|
|
|
workItem.server.Name, ctx.Err())
|
2017-03-20 03:48:42 +00:00
|
|
|
}
|
2017-03-18 01:42:28 +00:00
|
|
|
}
|
2017-03-20 03:48:42 +00:00
|
|
|
return replies
|
2017-03-18 01:42:28 +00:00
|
|
|
}
|