@WeiZhang555 I wondet about the move of events in the container package as events are not only for containers so the command is not only for containers 😇

@vdemeester Oh no, you're definitely right.

How about moving events to a new package named "system", this will make it consistent with api/server/router/system where events API inhabits in.

stats can still keep in package container. WDYT?

@WeiZhang555 yeah make sense I think 😉
stats is a container thing so yeah, container package is about right 👼

@vdemeester Updated. PTAL.

@WeiZhang555 CI failures look related;

5:15:59 ---> Making bundle: validate-vet (in bundles/1.12.0-dev/validate-vet)
15:15:59 api/client/system/events_utils.go:29: Handle passes lock by value: system.eventHandler contains sync.Mutex
15:15:59 api/client/system/events_utils.go:38: Watch passes lock by value: system.eventHandler contains sync.Mutex
15:15:59 exit status 1
15:16:00 Unable to find image 'busybox:latest' locally

@thaJeztah It passed now :-)

ping @vdemeester @thaJeztah @cpuguy83 @runcom

LGTM

LGTM 🐮