Skip to content

Commit

Permalink
Merge pull request #16912 from vrothberg/fix-16875
Browse files Browse the repository at this point in the history
events: support "die" filter
  • Loading branch information
openshift-merge-robot authored Dec 22, 2022
2 parents 5211446 + 45b180c commit 2a46b5e
Show file tree
Hide file tree
Showing 3 changed files with 17 additions and 1 deletion.
3 changes: 2 additions & 1 deletion docs/source/markdown/podman-events.1.md
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,7 @@ The *container* event type will report the follow statuses:
* commit
* connect
* create
* died
* disconnect
* exec
* exec_died
Expand Down Expand Up @@ -91,7 +92,7 @@ filters are supported:
* volume=name_or_id
* type=event_type (described above)

In the case where an ID is used, the ID may be in its full or shortened form.
In the case where an ID is used, the ID may be in its full or shortened form. The "die" event is mapped to "died" for Docker compatibility.

#### **--format**

Expand Down
3 changes: 3 additions & 0 deletions libpod/events/filters.go
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,9 @@ func generateEventFilter(filter, filterValue string) (func(e *Event) bool, error
return strings.HasPrefix(e.ID, filterValue)
}, nil
case "EVENT", "STATUS":
if filterValue == "die" { // Docker compat
filterValue = "died"
}
return func(e *Event) bool {
return string(e.Status) == filterValue
}, nil
Expand Down
12 changes: 12 additions & 0 deletions test/system/090-events.bats
Original file line number Diff line number Diff line change
Expand Up @@ -291,3 +291,15 @@ EOF
_events_container_create_inspect_data journald
_events_container_create_inspect_data file
}

@test "events - docker compat" {
local cname=c$(random_string 15)
t0=$(date --iso-8601=seconds)
run_podman run --name=$cname --rm $IMAGE true
run_podman events \
--since="$t0" \
--filter=status=$cname \
--filter=status=die \
--stream=false
is "${lines[0]}" ".* container died .* (image=$IMAGE, name=$cname, .*)"
}

0 comments on commit 2a46b5e

Please sign in to comment.