diff options
author | OpenShift Merge Robot <openshift-merge-robot@users.noreply.github.com> | 2020-10-06 13:17:15 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-10-06 13:17:15 +0200 |
commit | defb754945b3f99c1d786dac95d9b17b24f55e59 (patch) | |
tree | 3ec5cd697deb7babf9db415ebb0081fbfa1569ab | |
parent | 80a2317ca20b0e5e1cd064a8962beed642be3a36 (diff) | |
parent | adde67c74f265c2f4c9245b975d531dd635dce8e (diff) | |
download | podman-defb754945b3f99c1d786dac95d9b17b24f55e59.tar.gz podman-defb754945b3f99c1d786dac95d9b17b24f55e59.tar.bz2 podman-defb754945b3f99c1d786dac95d9b17b24f55e59.zip |
Merge pull request #7918 from zhangguanzhang/apiv2-wrong-StopSignal
[apiv2] /containers/$name/json return wrong value in `.Config.StopSignal`
-rw-r--r-- | pkg/api/handlers/compat/containers.go | 3 | ||||
-rw-r--r-- | test/apiv2/20-containers.at | 8 |
2 files changed, 7 insertions, 4 deletions
diff --git a/pkg/api/handlers/compat/containers.go b/pkg/api/handlers/compat/containers.go index 95499c6a2..48ecfff5d 100644 --- a/pkg/api/handlers/compat/containers.go +++ b/pkg/api/handlers/compat/containers.go @@ -4,6 +4,7 @@ import ( "encoding/json" "fmt" "net/http" + "strconv" "strings" "syscall" @@ -385,7 +386,7 @@ func LibpodToContainerJSON(l *libpod.Container, sz bool) (*types.ContainerJSON, MacAddress: "", OnBuild: nil, Labels: l.Labels(), - StopSignal: string(l.StopSignal()), + StopSignal: strconv.Itoa(int(l.StopSignal())), StopTimeout: &stopTimeout, Shell: nil, } diff --git a/test/apiv2/20-containers.at b/test/apiv2/20-containers.at index 15b5dc4be..28289955a 100644 --- a/test/apiv2/20-containers.at +++ b/test/apiv2/20-containers.at @@ -216,12 +216,14 @@ t GET containers/$cid/json 200 \ .Config.WorkingDir="/data" # default is /data t DELETE containers/$cid 204 -# test the WORKDIR -t POST containers/create '"Image":"'$ENV_WORKDIR_IMG'","WorkingDir":"/dataDir"' 201 \ +# test the WORKDIR and StopSignal +t POST containers/create '"Image":"'$ENV_WORKDIR_IMG'","WorkingDir":"/dataDir","StopSignal":"9"' 201 \ .Id~[0-9a-f]\\{64\\} cid=$(jq -r '.Id' <<<"$output") t GET containers/$cid/json 200 \ - .Config.WorkingDir="/dataDir" + .Config.WorkingDir="/dataDir" \ + .Config.StopSignal="9" + t DELETE containers/$cid 204 # vim: filetype=sh |