Merge pull request #9987 from jfrazelle/fix-filter-status-exited-not-require-all

`docker ps --filter status=exited should not require passing -a`
This commit is contained in:
Vincent Batts 2015-01-12 15:05:58 -05:00
commit e90e754682
2 changed files with 9 additions and 1 deletions

View file

@ -45,6 +45,14 @@ func (daemon *Daemon) Containers(job *engine.Job) engine.Status {
}
}
if i, ok := psFilters["status"]; ok {
for _, value := range i {
if value == "exited" {
all = true
}
}
}
names := map[string][]string{}
daemon.ContainerGraph().Walk("/", func(p string, e *graphdb.Entity) error {
names[e.ID()] = append(names[e.ID()], p)

View file

@ -312,7 +312,7 @@ func TestPsListContainersFilterStatus(t *testing.T) {
secondID := stripTrailingCharacters(out)
// filter containers by exited
runCmd = exec.Command(dockerBinary, "ps", "-a", "-q", "--filter=status=exited")
runCmd = exec.Command(dockerBinary, "ps", "-q", "--filter=status=exited")
out, _, err = runCommandWithOutput(runCmd)
if err != nil {
t.Fatal(out, err)