morhidi opened a new pull request, #254:
URL: https://github.com/apache/flink-kubernetes-operator/pull/254
This PR enables handling namespace changes via Dynamically Operator
Configuration through the config option:
`kubernetes.operator.watched.namespaces`. Watched namespaces originally were
defined using environment variables, thus namespaces changes required a POD
restart always. With the current feature Helm upgrades won't trigger an
automatic POD restart on namespace changes anymore.
Suggestions for testing:
```
helm install flink-kubernetes-operator helm/flink-kubernetes-operator --set
"watchNamespaces={default}"
helm upgrade flink-kubernetes-operator helm/flink-kubernetes-operator --set
"watchNamespaces={default,flink}"
```
The upgrade will update the config map, but it takes some time until the new
configs are propagated in the operator. Worth setting the
`kubernetes.operator.dynamic.config.check.interval: 10 s` for local testing.
I would be happy to get your feedback on this @wangyang0918 @Aitozi @gyfora
@tweise
--
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-unsubscr...@flink.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
