WATCH_NAMESPACE doesn't work for a single namespace

[swiatekm]

Component(s)

operator

What happened?

Description

Setting WATCH_NAMESPACE to a single value has no effect. Multiple values delimited by commas work.

Kubernetes Version

1.28

Operator version

0.92.0, most likely present since 0.84.0

Collector version

0.92.0

Environment information

No response

Log output

No response

Additional context

Looks like #2046 accidentally removed the second branch of a conditional instruction, and now we do nothing if the value of WATCH_NAMESPACE contains no commas. See:

opentelemetry-operator/main.go

Line 215 in 1573cd9

if strings.Contains(watchNamespace, ",") {

[davidhaja]

I would like to work on this one.
My idea is to move the namespaces map filling, the watchNamespace splitting, and cache.Config{} creation under the found condition.

opentelemetry-operator/main.go

Lines 200 to 202 in 1573cd9

	if found {
	setupLog.Info("watching namespace(s)", "namespaces", watchNamespace)
	} else {

[jaronoff97]

closed by #2606