filter namespace
Signed-off-by: Mike Brown <brownwm@us.ibm.com>
This commit is contained in:
parent
1dc6ea19dc
commit
b59dd55966
@ -31,6 +31,7 @@ import (
|
|||||||
"golang.org/x/net/context"
|
"golang.org/x/net/context"
|
||||||
"k8s.io/apimachinery/pkg/util/clock"
|
"k8s.io/apimachinery/pkg/util/clock"
|
||||||
|
|
||||||
|
"github.com/containerd/cri/pkg/constants"
|
||||||
ctrdutil "github.com/containerd/cri/pkg/containerd/util"
|
ctrdutil "github.com/containerd/cri/pkg/containerd/util"
|
||||||
"github.com/containerd/cri/pkg/store"
|
"github.com/containerd/cri/pkg/store"
|
||||||
containerstore "github.com/containerd/cri/pkg/store/container"
|
containerstore "github.com/containerd/cri/pkg/store/container"
|
||||||
@ -82,7 +83,6 @@ type backOffQueue struct {
|
|||||||
// Create new event monitor. New event monitor will start subscribing containerd event. All events
|
// Create new event monitor. New event monitor will start subscribing containerd event. All events
|
||||||
// happen after it should be monitored.
|
// happen after it should be monitored.
|
||||||
func newEventMonitor(c *criService) *eventMonitor {
|
func newEventMonitor(c *criService) *eventMonitor {
|
||||||
// event subscribe doesn't need namespace.
|
|
||||||
ctx, cancel := context.WithCancel(context.Background())
|
ctx, cancel := context.WithCancel(context.Background())
|
||||||
return &eventMonitor{
|
return &eventMonitor{
|
||||||
c: c,
|
c: c,
|
||||||
@ -94,6 +94,8 @@ func newEventMonitor(c *criService) *eventMonitor {
|
|||||||
|
|
||||||
// subscribe starts to subscribe containerd events.
|
// subscribe starts to subscribe containerd events.
|
||||||
func (em *eventMonitor) subscribe(subscriber events.Subscriber) {
|
func (em *eventMonitor) subscribe(subscriber events.Subscriber) {
|
||||||
|
// note: filters are any match, if you want any match but not in namespace foo
|
||||||
|
// then you have to manually filter namespace foo
|
||||||
filters := []string{
|
filters := []string{
|
||||||
`topic=="/tasks/exit"`,
|
`topic=="/tasks/exit"`,
|
||||||
`topic=="/tasks/oom"`,
|
`topic=="/tasks/oom"`,
|
||||||
@ -141,6 +143,10 @@ func (em *eventMonitor) start() <-chan error {
|
|||||||
select {
|
select {
|
||||||
case e := <-em.ch:
|
case e := <-em.ch:
|
||||||
logrus.Debugf("Received containerd event timestamp - %v, namespace - %q, topic - %q", e.Timestamp, e.Namespace, e.Topic)
|
logrus.Debugf("Received containerd event timestamp - %v, namespace - %q, topic - %q", e.Timestamp, e.Namespace, e.Topic)
|
||||||
|
if e.Namespace != constants.K8sContainerdNamespace {
|
||||||
|
logrus.Debugf("Ignoring events in namespace - %q", e.Namespace)
|
||||||
|
break
|
||||||
|
}
|
||||||
id, evt, err := convertEvent(e.Event)
|
id, evt, err := convertEvent(e.Event)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
logrus.WithError(err).Errorf("Failed to convert event %+v", e)
|
logrus.WithError(err).Errorf("Failed to convert event %+v", e)
|
||||||
|
Loading…
Reference in New Issue
Block a user