summaryrefslogtreecommitdiff
path: root/pkg/domain/infra/abi
diff options
context:
space:
mode:
authorDaniel J Walsh <dwalsh@redhat.com>2021-02-16 07:01:14 -0500
committerDaniel J Walsh <dwalsh@redhat.com>2021-02-16 09:47:38 -0500
commit958f90143199bbc4b5dcd7ad4fffdd56d3f6fbc3 (patch)
tree4eaaf18c511d9e3e05a58ca47039333bd4c86ee3 /pkg/domain/infra/abi
parent58a4793bec30058f648dcd1248da333a7bb6d47c (diff)
downloadpodman-958f90143199bbc4b5dcd7ad4fffdd56d3f6fbc3.tar.gz
podman-958f90143199bbc4b5dcd7ad4fffdd56d3f6fbc3.tar.bz2
podman-958f90143199bbc4b5dcd7ad4fffdd56d3f6fbc3.zip
podman kill should report rawInput not container id
Docker always reports back the users input, not the full id, we should do the same. Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
Diffstat (limited to 'pkg/domain/infra/abi')
-rw-r--r--pkg/domain/infra/abi/containers.go13
1 files changed, 10 insertions, 3 deletions
diff --git a/pkg/domain/infra/abi/containers.go b/pkg/domain/infra/abi/containers.go
index 9bef97a5c..0ee4048d1 100644
--- a/pkg/domain/infra/abi/containers.go
+++ b/pkg/domain/infra/abi/containers.go
@@ -208,15 +208,22 @@ func (ic *ContainerEngine) ContainerKill(ctx context.Context, namesOrIds []strin
if err != nil {
return nil, err
}
- ctrs, err := getContainersByContext(options.All, options.Latest, namesOrIds, ic.Libpod)
+ ctrs, rawInputs, err := getContainersAndInputByContext(options.All, options.Latest, namesOrIds, ic.Libpod)
if err != nil {
return nil, err
}
+ ctrMap := map[string]string{}
+ if len(rawInputs) == len(ctrs) {
+ for i := range ctrs {
+ ctrMap[ctrs[i].ID()] = rawInputs[i]
+ }
+ }
reports := make([]*entities.KillReport, 0, len(ctrs))
for _, con := range ctrs {
reports = append(reports, &entities.KillReport{
- Id: con.ID(),
- Err: con.Kill(uint(sig)),
+ Id: con.ID(),
+ Err: con.Kill(uint(sig)),
+ RawInput: ctrMap[con.ID()],
})
}
return reports, nil