vbatts/containerd
1
0
Fork 0
Code Pull requests Releases Activity

Merge pull request #238 from duglin/fixErrMsg

Browse source 
Add a better error msg
This commit is contained in:
Kenfe-Mickaël Laventure 2016-05-18 14:04:19 -07:00
commit b5981e004f
1 changed files with 7 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
runtime/container.go
View file

@ -294,7 +294,7 @@ func (c *container) UpdateResources(r *Resource) error {
cmd.Stdin = srStr cmd.Stdin = srStr
b, err := cmd.CombinedOutput() b, err := cmd.CombinedOutput()
if err != nil { if err != nil {
return fmt.Errorf(string(b)) return fmt.Errorf("%s: %q", err.Error(), string(b))
} }
return nil return nil
} }
@ -335,7 +335,7 @@ func (c *container) Pause() error {
args = append(args, "pause", c.id) args = append(args, "pause", c.id)
b, err := exec.Command(c.runtime, args...).CombinedOutput() b, err := exec.Command(c.runtime, args...).CombinedOutput()
if err != nil { if err != nil {
return fmt.Errorf(string(b)) return fmt.Errorf("%s: %q", err.Error(), string(b))
} }
return nil return nil
} }
@ -345,7 +345,7 @@ func (c *container) Resume() error {
args = append(args, "resume", c.id) args = append(args, "resume", c.id)
b, err := exec.Command(c.runtime, args...).CombinedOutput() b, err := exec.Command(c.runtime, args...).CombinedOutput()
if err != nil { if err != nil {
return fmt.Errorf(string(b)) return fmt.Errorf("%s: %q", err.Error(), string(b))
} }
return nil return nil
} }
@ -415,7 +415,7 @@ func (c *container) Checkpoint(cpt Checkpoint) error {
add(c.id) add(c.id)
out, err := exec.Command(c.runtime, args...).CombinedOutput() out, err := exec.Command(c.runtime, args...).CombinedOutput()
if err != nil { if err != nil {
return fmt.Errorf("%s: %s", err.Error(), string(out)) return fmt.Errorf("%s: %q", err.Error(), string(out))
} }
return err return err
} }
@ -548,7 +548,7 @@ func (c *container) Pids() ([]int, error) {
args = append(args, "ps", "--format=json", c.id) args = append(args, "ps", "--format=json", c.id)
out, err := exec.Command(c.runtime, args...).CombinedOutput() out, err := exec.Command(c.runtime, args...).CombinedOutput()
if err != nil { if err != nil {
return nil, fmt.Errorf("%s", out) return nil, fmt.Errorf("%s: %q", err.Error(), out)
} }
var pids []int var pids []int
if err := json.Unmarshal(out, &pids); err != nil { if err := json.Unmarshal(out, &pids); err != nil {
@ -563,7 +563,7 @@ func (c *container) Stats() (*Stat, error) {
args = append(args, "events", "--stats", c.id) args = append(args, "events", "--stats", c.id)
out, err := exec.Command(c.runtime, args...).CombinedOutput() out, err := exec.Command(c.runtime, args...).CombinedOutput()
if err != nil { if err != nil {
return nil, fmt.Errorf("%s", out) return nil, fmt.Errorf("%s: %q", err.Error(), out)
} }
s := struct { s := struct {
Data *Stat `json:"data"` Data *Stat `json:"data"`
@ -602,7 +602,7 @@ func (c *container) Status() (State, error) {
out, err := exec.Command(c.runtime, args...).CombinedOutput() out, err := exec.Command(c.runtime, args...).CombinedOutput()
if err != nil { if err != nil {
return "", fmt.Errorf("%s", out) return "", fmt.Errorf("%s: %q", err.Error(), out)
} }
// We only require the runtime json output to have a top level Status field. // We only require the runtime json output to have a top level Status field.