summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorhaircommander <pehunt@redhat.com>2018-08-15 11:32:23 -0400
committerAtomic Bot <atomic-devel@projectatomic.io>2018-08-16 20:31:50 +0000
commita51eb1e70f3647fb5e3126a50a3458fcf486af16 (patch)
treeb21822ae0883ad831bf6cb6f5a460f3bc543cb78
parent8d5e0108d7c8b69abb9821bfe55475ae5d663b3a (diff)
downloadpodman-a51eb1e70f3647fb5e3126a50a3458fcf486af16.tar.gz
podman-a51eb1e70f3647fb5e3126a50a3458fcf486af16.tar.bz2
podman-a51eb1e70f3647fb5e3126a50a3458fcf486af16.zip
Added reason to PodContainerError
Signed-off-by: haircommander <pehunt@redhat.com> Closes: #1275 Approved by: mheon
-rw-r--r--cmd/podman/shared/.pod.go.swpbin12288 -> 0 bytes
-rw-r--r--cmd/podman/varlink/io.podman.varlink7
-rw-r--r--pkg/varlinkapi/pods.go42
3 files changed, 30 insertions, 19 deletions
diff --git a/cmd/podman/shared/.pod.go.swp b/cmd/podman/shared/.pod.go.swp
deleted file mode 100644
index a8565e752..000000000
--- a/cmd/podman/shared/.pod.go.swp
+++ /dev/null
Binary files differ
diff --git a/cmd/podman/varlink/io.podman.varlink b/cmd/podman/varlink/io.podman.varlink
index 0e6f0f054..abbaf1b07 100644
--- a/cmd/podman/varlink/io.podman.varlink
+++ b/cmd/podman/varlink/io.podman.varlink
@@ -346,6 +346,11 @@ type ListPodData (
containersinfo: []ListPodContainerInfo
)
+type PodContainerErrorData (
+ containerid: string,
+ reason: string
+)
+
# Ping provides a response for developers to ensure their varlink setup is working.
# #### Example
# ~~~
@@ -758,7 +763,7 @@ error PodNotFound (name: string)
# PodContainerError means a container associated with a pod failed to preform an operation. It contains
# a container ID of the container that failed.
-error PodContainerError (podname: string, ctrnames: []string)
+error PodContainerError (podname: string, errors: []PodContainerErrorData)
# ErrorOccurred is a generic error for an error that occurs during the execution. The actual error message
# is includes as part of the error's text.
diff --git a/pkg/varlinkapi/pods.go b/pkg/varlinkapi/pods.go
index 98a622d8a..be5e3614c 100644
--- a/pkg/varlinkapi/pods.go
+++ b/pkg/varlinkapi/pods.go
@@ -95,9 +95,10 @@ func (i *LibpodAPI) StartPod(call iopodman.VarlinkCall, name string) error {
return call.ReplyErrorOccurred(err.Error())
}
if ctrErrs != nil {
- containerErrs := make([]string, len(ctrErrs))
- for ctr := range ctrErrs {
- containerErrs = append(containerErrs, ctr)
+ containerErrs := make([]iopodman.PodContainerErrorData, len(ctrErrs))
+ for ctr, reason := range ctrErrs {
+ ctrErr := iopodman.PodContainerErrorData{Containerid: ctr, Reason: reason.Error()}
+ containerErrs = append(containerErrs, ctrErr)
}
return call.ReplyPodContainerError(pod.ID(), containerErrs)
}
@@ -116,9 +117,10 @@ func (i *LibpodAPI) StopPod(call iopodman.VarlinkCall, name string) error {
return call.ReplyErrorOccurred(err.Error())
}
if ctrErrs != nil {
- containerErrs := make([]string, len(ctrErrs))
- for ctr := range ctrErrs {
- containerErrs = append(containerErrs, ctr)
+ containerErrs := make([]iopodman.PodContainerErrorData, len(ctrErrs))
+ for ctr, reason := range ctrErrs {
+ ctrErr := iopodman.PodContainerErrorData{Containerid: ctr, Reason: reason.Error()}
+ containerErrs = append(containerErrs, ctrErr)
}
return call.ReplyPodContainerError(pod.ID(), containerErrs)
}
@@ -137,9 +139,10 @@ func (i *LibpodAPI) RestartPod(call iopodman.VarlinkCall, name string) error {
return call.ReplyErrorOccurred(err.Error())
}
if ctrErrs != nil {
- containerErrs := make([]string, len(ctrErrs))
- for ctr := range ctrErrs {
- containerErrs = append(containerErrs, ctr)
+ containerErrs := make([]iopodman.PodContainerErrorData, len(ctrErrs))
+ for ctr, reason := range ctrErrs {
+ ctrErr := iopodman.PodContainerErrorData{Containerid: ctr, Reason: reason.Error()}
+ containerErrs = append(containerErrs, ctrErr)
}
return call.ReplyPodContainerError(pod.ID(), containerErrs)
}
@@ -164,9 +167,10 @@ func (i *LibpodAPI) KillPod(call iopodman.VarlinkCall, name string, signal int64
return call.ReplyErrorOccurred(err.Error())
}
if ctrErrs != nil {
- containerErrs := make([]string, len(ctrErrs))
- for ctr := range ctrErrs {
- containerErrs = append(containerErrs, ctr)
+ containerErrs := make([]iopodman.PodContainerErrorData, len(ctrErrs))
+ for ctr, reason := range ctrErrs {
+ ctrErr := iopodman.PodContainerErrorData{Containerid: ctr, Reason: reason.Error()}
+ containerErrs = append(containerErrs, ctrErr)
}
return call.ReplyPodContainerError(pod.ID(), containerErrs)
}
@@ -185,9 +189,10 @@ func (i *LibpodAPI) PausePod(call iopodman.VarlinkCall, name string) error {
return call.ReplyErrorOccurred(err.Error())
}
if ctrErrs != nil {
- containerErrs := make([]string, len(ctrErrs))
- for ctr := range ctrErrs {
- containerErrs = append(containerErrs, ctr)
+ containerErrs := make([]iopodman.PodContainerErrorData, len(ctrErrs))
+ for ctr, reason := range ctrErrs {
+ ctrErr := iopodman.PodContainerErrorData{Containerid: ctr, Reason: reason.Error()}
+ containerErrs = append(containerErrs, ctrErr)
}
return call.ReplyPodContainerError(pod.ID(), containerErrs)
}
@@ -206,9 +211,10 @@ func (i *LibpodAPI) UnpausePod(call iopodman.VarlinkCall, name string) error {
return call.ReplyErrorOccurred(err.Error())
}
if ctrErrs != nil {
- containerErrs := make([]string, len(ctrErrs))
- for ctr := range ctrErrs {
- containerErrs = append(containerErrs, ctr)
+ containerErrs := make([]iopodman.PodContainerErrorData, len(ctrErrs))
+ for ctr, reason := range ctrErrs {
+ ctrErr := iopodman.PodContainerErrorData{Containerid: ctr, Reason: reason.Error()}
+ containerErrs = append(containerErrs, ctrErr)
}
return call.ReplyPodContainerError(pod.ID(), containerErrs)
}