[GitHub] [flink-kubernetes-operator] jbusche commented on pull request #420: FLINK-29536 - Add WATCH_NAMESPACE env var to operator

[GitBox <gi...@apache.org>]

jbusche commented on PR #420:
URL: https://github.com/apache/flink-kubernetes-operator/pull/420#issuecomment-1334199315

   Hi @tagarr @morhidi @tedhtchang 
   I've tested Tony's PR.  With a helm install, it acts as before, no change in behavior.
   ```
   oc logs -f flink-kubernetes-operator-fdc8c59b9-dzpzr |grep watch
   2022-12-01 18:31:07,701 o.a.f.k.o.FlinkOperator        [INFO ] Configuring operator to watch the following namespaces: [JOSDK_ALL_NAMESPACES].
   ```
   But when I create an OLM bundle and install it in a particular namespace, for example default, then it properly only watches flinkdeps in that particular namespace:
   ```
   oc rsh flink-kubernetes-operator-58f4967899-jz4w2 env |grep -i watch
   WATCH_NAMESPACE=default
   
   oc logs -f flink-kubernetes-operator-55585bc6bc-6lltj |grep watch
   2022-12-01 18:12:54,931 o.a.f.k.o.FlinkOperator        [INFO ] Configuring operator to watch the following namespaces: [default].
   ```
   Is it possible to get this merged in prior to the release 1.3.0 cutoff?  Thanks!


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org