Skip to content

Make the default configuration for watched namespaces more explicit #1176

Closed
@metacosm

Description

@metacosm

The default value (if no value is provided via the @ControllerConfiguration annotation) is currently considered as a request to watch all namespaces. While the default behavior shouldn't change (i.e., if no value is provided all namespaces should still be watched by default), the value should be more explicit and have a clearer semantics.

Similarly, if a user currently specifies a list of namespaces to watch but if that list also includes the Constants.WATCH_CURRENT_NAMESPACE value, then the behavior will be changed to only watching the namespace in which the operator is deployed. This is potential source of error so this should actually be an error to use Constants.WATCH_CURRENT_NAMESPACE in combination with any other value. We will also match this semantics for the new explicit constant to watch all namespaces.

Metadata

Metadata

Assignees

Labels

kind/featureCategorizes issue or PR as related to a new feature.

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions