Fix golint errors for casing in api/client package

Signed-off-by: Peggy Li <peggyli.224@gmail.com>
This commit is contained in:
Peggy Li 2015-03-25 19:31:29 -07:00
parent f84086be2a
commit a465e26bb0
8 changed files with 35 additions and 35 deletions

View File

@ -47,8 +47,8 @@ func (cli *DockerCli) CmdBuild(args ...string) error {
dockerfileName := cmd.String([]string{"f", "-file"}, "", "Name of the Dockerfile (Default is 'PATH/Dockerfile')") dockerfileName := cmd.String([]string{"f", "-file"}, "", "Name of the Dockerfile (Default is 'PATH/Dockerfile')")
flMemoryString := cmd.String([]string{"m", "-memory"}, "", "Memory limit") flMemoryString := cmd.String([]string{"m", "-memory"}, "", "Memory limit")
flMemorySwap := cmd.String([]string{"-memory-swap"}, "", "Total memory (memory + swap), '-1' to disable swap") flMemorySwap := cmd.String([]string{"-memory-swap"}, "", "Total memory (memory + swap), '-1' to disable swap")
flCpuShares := cmd.Int64([]string{"c", "-cpu-shares"}, 0, "CPU shares (relative weight)") flCPUShares := cmd.Int64([]string{"c", "-cpu-shares"}, 0, "CPU shares (relative weight)")
flCpuSetCpus := cmd.String([]string{"-cpuset-cpus"}, "", "CPUs in which to allow execution (0-3, 0,1)") flCPUSetCpus := cmd.String([]string{"-cpuset-cpus"}, "", "CPUs in which to allow execution (0-3, 0,1)")
cmd.Require(flag.Exact, 1) cmd.Require(flag.Exact, 1)
@ -271,8 +271,8 @@ func (cli *DockerCli) CmdBuild(args ...string) error {
v.Set("pull", "1") v.Set("pull", "1")
} }
v.Set("cpusetcpus", *flCpuSetCpus) v.Set("cpusetcpus", *flCPUSetCpus)
v.Set("cpushares", strconv.FormatInt(*flCpuShares, 10)) v.Set("cpushares", strconv.FormatInt(*flCPUShares, 10))
v.Set("memory", strconv.FormatInt(memory, 10)) v.Set("memory", strconv.FormatInt(memory, 10))
v.Set("memswap", strconv.FormatInt(memorySwap, 10)) v.Set("memswap", strconv.FormatInt(memorySwap, 10))

View File

@ -55,11 +55,11 @@ func (cli *DockerCli) CmdEvents(args ...string) error {
setTime("until", *until) setTime("until", *until)
} }
if len(eventFilterArgs) > 0 { if len(eventFilterArgs) > 0 {
filterJson, err := filters.ToParam(eventFilterArgs) filterJSON, err := filters.ToParam(eventFilterArgs)
if err != nil { if err != nil {
return err return err
} }
v.Set("filters", filterJson) v.Set("filters", filterJSON)
} }
if err := cli.stream("GET", "/events?"+v.Encode(), nil, cli.out, nil); err != nil { if err := cli.stream("GET", "/events?"+v.Encode(), nil, cli.out, nil); err != nil {
return err return err

View File

@ -119,11 +119,11 @@ func (cli *DockerCli) CmdImages(args ...string) error {
"all": []string{"1"}, "all": []string{"1"},
} }
if len(imageFilterArgs) > 0 { if len(imageFilterArgs) > 0 {
filterJson, err := filters.ToParam(imageFilterArgs) filterJSON, err := filters.ToParam(imageFilterArgs)
if err != nil { if err != nil {
return err return err
} }
v.Set("filters", filterJson) v.Set("filters", filterJSON)
} }
body, _, err := readBody(cli.call("GET", "/images/json?"+v.Encode(), nil, false)) body, _, err := readBody(cli.call("GET", "/images/json?"+v.Encode(), nil, false))
@ -189,11 +189,11 @@ func (cli *DockerCli) CmdImages(args ...string) error {
} else { } else {
v := url.Values{} v := url.Values{}
if len(imageFilterArgs) > 0 { if len(imageFilterArgs) > 0 {
filterJson, err := filters.ToParam(imageFilterArgs) filterJSON, err := filters.ToParam(imageFilterArgs)
if err != nil { if err != nil {
return err return err
} }
v.Set("filters", filterJson) v.Set("filters", filterJSON)
} }
if cmd.NArg() == 1 { if cmd.NArg() == 1 {

View File

@ -74,12 +74,12 @@ func (cli *DockerCli) CmdPs(args ...string) error {
} }
if len(psFilterArgs) > 0 { if len(psFilterArgs) > 0 {
filterJson, err := filters.ToParam(psFilterArgs) filterJSON, err := filters.ToParam(psFilterArgs)
if err != nil { if err != nil {
return err return err
} }
v.Set("filters", filterJson) v.Set("filters", filterJSON)
} }
body, _, err := readBody(cli.call("GET", "/containers/json?"+v.Encode(), nil, false)) body, _, err := readBody(cli.call("GET", "/containers/json?"+v.Encode(), nil, false))

View File

@ -12,12 +12,12 @@ func (cli *DockerCli) CmdRename(args ...string) error {
cmd.Usage() cmd.Usage()
return nil return nil
} }
old_name := cmd.Arg(0) oldName := cmd.Arg(0)
new_name := cmd.Arg(1) newName := cmd.Arg(1)
if _, _, err := readBody(cli.call("POST", fmt.Sprintf("/containers/%s/rename?name=%s", old_name, new_name), nil, false)); err != nil { if _, _, err := readBody(cli.call("POST", fmt.Sprintf("/containers/%s/rename?name=%s", oldName, newName), nil, false)); err != nil {
fmt.Fprintf(cli.err, "%s\n", err) fmt.Fprintf(cli.err, "%s\n", err)
return fmt.Errorf("Error: failed to rename container named %s", old_name) return fmt.Errorf("Error: failed to rename container named %s", oldName)
} }
return nil return nil
} }

View File

@ -110,14 +110,14 @@ func (cli *DockerCli) CmdRun(args ...string) error {
defer signal.StopCatch(sigc) defer signal.StopCatch(sigc)
} }
var ( var (
waitDisplayId chan struct{} waitDisplayID chan struct{}
errCh chan error errCh chan error
) )
if !config.AttachStdout && !config.AttachStderr { if !config.AttachStdout && !config.AttachStderr {
// Make this asynchronous to allow the client to write to stdin before having to read the ID // Make this asynchronous to allow the client to write to stdin before having to read the ID
waitDisplayId = make(chan struct{}) waitDisplayID = make(chan struct{})
go func() { go func() {
defer close(waitDisplayId) defer close(waitDisplayID)
fmt.Fprintf(cli.out, "%s\n", createResponse.ID) fmt.Fprintf(cli.out, "%s\n", createResponse.ID)
}() }()
} }
@ -207,7 +207,7 @@ func (cli *DockerCli) CmdRun(args ...string) error {
// Detached mode: wait for the id to be displayed and return. // Detached mode: wait for the id to be displayed and return.
if !config.AttachStdout && !config.AttachStderr { if !config.AttachStdout && !config.AttachStderr {
// Detached mode // Detached mode
<-waitDisplayId <-waitDisplayID
return nil return nil
} }

View File

@ -18,7 +18,7 @@ import (
type containerStats struct { type containerStats struct {
Name string Name string
CpuPercentage float64 CPUPercentage float64
Memory float64 Memory float64
MemoryLimit float64 MemoryLimit float64
MemoryPercentage float64 MemoryPercentage float64
@ -36,7 +36,7 @@ func (s *containerStats) Collect(cli *DockerCli) {
} }
defer stream.Close() defer stream.Close()
var ( var (
previousCpu uint64 previousCPU uint64
previousSystem uint64 previousSystem uint64
start = true start = true
dec = json.NewDecoder(stream) dec = json.NewDecoder(stream)
@ -54,18 +54,18 @@ func (s *containerStats) Collect(cli *DockerCli) {
cpuPercent = 0.0 cpuPercent = 0.0
) )
if !start { if !start {
cpuPercent = calculateCpuPercent(previousCpu, previousSystem, v) cpuPercent = calculateCPUPercent(previousCPU, previousSystem, v)
} }
start = false start = false
s.mu.Lock() s.mu.Lock()
s.CpuPercentage = cpuPercent s.CPUPercentage = cpuPercent
s.Memory = float64(v.MemoryStats.Usage) s.Memory = float64(v.MemoryStats.Usage)
s.MemoryLimit = float64(v.MemoryStats.Limit) s.MemoryLimit = float64(v.MemoryStats.Limit)
s.MemoryPercentage = memPercent s.MemoryPercentage = memPercent
s.NetworkRx = float64(v.Network.RxBytes) s.NetworkRx = float64(v.Network.RxBytes)
s.NetworkTx = float64(v.Network.TxBytes) s.NetworkTx = float64(v.Network.TxBytes)
s.mu.Unlock() s.mu.Unlock()
previousCpu = v.CpuStats.CpuUsage.TotalUsage previousCPU = v.CpuStats.CpuUsage.TotalUsage
previousSystem = v.CpuStats.SystemUsage previousSystem = v.CpuStats.SystemUsage
u <- nil u <- nil
} }
@ -76,7 +76,7 @@ func (s *containerStats) Collect(cli *DockerCli) {
// zero out the values if we have not received an update within // zero out the values if we have not received an update within
// the specified duration. // the specified duration.
s.mu.Lock() s.mu.Lock()
s.CpuPercentage = 0 s.CPUPercentage = 0
s.Memory = 0 s.Memory = 0
s.MemoryPercentage = 0 s.MemoryPercentage = 0
s.mu.Unlock() s.mu.Unlock()
@ -99,7 +99,7 @@ func (s *containerStats) Display(w io.Writer) error {
} }
fmt.Fprintf(w, "%s\t%.2f%%\t%s/%s\t%.2f%%\t%s/%s\n", fmt.Fprintf(w, "%s\t%.2f%%\t%s/%s\t%.2f%%\t%s/%s\n",
s.Name, s.Name,
s.CpuPercentage, s.CPUPercentage,
units.BytesSize(s.Memory), units.BytesSize(s.MemoryLimit), units.BytesSize(s.Memory), units.BytesSize(s.MemoryLimit),
s.MemoryPercentage, s.MemoryPercentage,
units.BytesSize(s.NetworkRx), units.BytesSize(s.NetworkTx)) units.BytesSize(s.NetworkRx), units.BytesSize(s.NetworkTx))
@ -161,11 +161,11 @@ func (cli *DockerCli) CmdStats(args ...string) error {
return nil return nil
} }
func calculateCpuPercent(previousCpu, previousSystem uint64, v *types.Stats) float64 { func calculateCPUPercent(previousCPU, previousSystem uint64, v *types.Stats) float64 {
var ( var (
cpuPercent = 0.0 cpuPercent = 0.0
// calculate the change for the cpu usage of the container in between readings // calculate the change for the cpu usage of the container in between readings
cpuDelta = float64(v.CpuStats.CpuUsage.TotalUsage - previousCpu) cpuDelta = float64(v.CpuStats.CpuUsage.TotalUsage - previousCPU)
// calculate the change for the entire system between readings // calculate the change for the entire system between readings
systemDelta = float64(v.CpuStats.SystemUsage - previousSystem) systemDelta = float64(v.CpuStats.SystemUsage - previousSystem)
) )

View File

@ -200,8 +200,8 @@ func (cli *DockerCli) resizeTty(id string, isExec bool) {
} }
} }
func waitForExit(cli *DockerCli, containerId string) (int, error) { func waitForExit(cli *DockerCli, containerID string) (int, error) {
stream, _, err := cli.call("POST", "/containers/"+containerId+"/wait", nil, false) stream, _, err := cli.call("POST", "/containers/"+containerID+"/wait", nil, false)
if err != nil { if err != nil {
return -1, err return -1, err
} }
@ -215,8 +215,8 @@ func waitForExit(cli *DockerCli, containerId string) (int, error) {
// getExitCode perform an inspect on the container. It returns // getExitCode perform an inspect on the container. It returns
// the running state and the exit code. // the running state and the exit code.
func getExitCode(cli *DockerCli, containerId string) (bool, int, error) { func getExitCode(cli *DockerCli, containerID string) (bool, int, error) {
stream, _, err := cli.call("GET", "/containers/"+containerId+"/json", nil, false) stream, _, err := cli.call("GET", "/containers/"+containerID+"/json", nil, false)
if err != nil { if err != nil {
// If we can't connect, then the daemon probably died. // If we can't connect, then the daemon probably died.
if err != ErrConnectionRefused { if err != ErrConnectionRefused {
@ -236,8 +236,8 @@ func getExitCode(cli *DockerCli, containerId string) (bool, int, error) {
// getExecExitCode perform an inspect on the exec command. It returns // getExecExitCode perform an inspect on the exec command. It returns
// the running state and the exit code. // the running state and the exit code.
func getExecExitCode(cli *DockerCli, execId string) (bool, int, error) { func getExecExitCode(cli *DockerCli, execID string) (bool, int, error) {
stream, _, err := cli.call("GET", "/exec/"+execId+"/json", nil, false) stream, _, err := cli.call("GET", "/exec/"+execID+"/json", nil, false)
if err != nil { if err != nil {
// If we can't connect, then the daemon probably died. // If we can't connect, then the daemon probably died.
if err != ErrConnectionRefused { if err != ErrConnectionRefused {