Ignore events that are outside of namespaces that the controller manages#5058

Closed

naemono wants to merge 29 commits intoelastic:masterfrom naemono:4168-ignore-unmanaged-namespace-events

+281-35

Commits on Dec 13, 2021

ignore events that are outside of namespaces that the controller manages.
naemono
committed
Add a test for ignoring unmanaged namespaces
naemono
committed
Fix newline issues
naemono
committed
Add helper method to cleanup the code a bit.
naemono
committed
Better name function
naemono
committed
Add predicate to additional missing CRD types.
naemono
committed
Ignore other objects outside managed namespaces in beats controller.
naemono
committed
Move predicates to it's own package to avoid import cycles
naemono
committed
Clean up the code a bit by providing the predicates as a variadic argument to the 'addWatches' functions in most of the packages.
naemono
committed
Adjust how garbage collection of secrets is handled on startup, taking into account managed namespaces
naemono
committed
Prevent empty namespace "Orphan secrets garbage collection failed" error message when
naemono
committed
Adding some comments around corev1.Namespaceall
naemono
committed
Making license controller consistent with other packages
naemono
committed
Fix garbage collection test
naemono
committed
Also ignore generic events in unmanaged namespaces

naemono
and
barkbay
committed
Cleanup predicate calling code per @barkbay suggestion.
naemono
committed
Use IsNamespaceManaged in suggestion
naemono
committed
remove predicate on associationinfo struct
naemono
committed
remove redundant comments
naemono
committed
simplify GarbageCollectAllSoftOwnedOrphanSecrets per @barkbay comments
naemono
committed
revert invalid change to Makefile
naemono
committed
ensure managed namespace predicate is initialized prior to license integration test
naemono
committed
Per Suggestion, simplify how predicates get used in Watches.
naemono
committed
remove the exported predicates.ManagedNamespacePredicate as it's unused atm.
naemono
committed
Add some comments for newNamespaceAwareWatchersController
naemono
committed
Introduce a new function NewControllerWithOptions that allows setting of controller options on create.
naemono
committed
Actually use watchedNamespaces, not managedNamespaces
naemono
committed
Do not append 'namespaceAll' to managed namespaces on initialization.
naemono
committed
ensure we do not append anything to the managed namespaces when it's empty (managing all namespaces)
naemono
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Ignore events that are outside of namespaces that the controller manages#5058

Commits on Dec 13, 2021

ignore events that are outside of namespaces that the controller manages.

Add a test for ignoring unmanaged namespaces

Fix newline issues

Add helper method to cleanup the code a bit.

Better name function

Add predicate to additional missing CRD types.

Ignore other objects outside managed namespaces in beats controller.

Move predicates to it's own package to avoid import cycles

Clean up the code a bit by providing the predicates as a variadic argument to the 'addWatches' functions in most of the packages.

Adjust how garbage collection of secrets is handled on startup, taking into account managed namespaces

Prevent empty namespace "Orphan secrets garbage collection failed" error message when

Adding some comments around corev1.Namespaceall

Making license controller consistent with other packages

Fix garbage collection test

Also ignore generic events in unmanaged namespaces

Cleanup predicate calling code per @barkbay suggestion.

Use IsNamespaceManaged in suggestion

remove predicate on associationinfo struct

remove redundant comments

simplify GarbageCollectAllSoftOwnedOrphanSecrets per @barkbay comments

revert invalid change to Makefile

ensure managed namespace predicate is initialized prior to license integration test

Per Suggestion, simplify how predicates get used in Watches.

remove the exported predicates.ManagedNamespacePredicate as it's unused atm.

Add some comments for newNamespaceAwareWatchersController

Introduce a new function NewControllerWithOptions that allows setting of controller options on create.

Actually use watchedNamespaces, not managedNamespaces

Do not append 'namespaceAll' to managed namespaces on initialization.

ensure we do not append anything to the managed namespaces when it's empty (managing all namespaces)