gyfora commented on PR #1126: URL: https://github.com/apache/flink-kubernetes-operator/pull/1126#issuecomment-5240434226
@gerkElznik one last question before we merge this. How will this affect the behaviour of the "ENV_CONF_OVERRIDE_DIR" feature that allows users to mount a new configmap with some extra config overrides. That extra configmap may use the old `flink-conf.yaml` format. Code is at `FlinkConfigManager # 448-462` We have to make sure that even though the operator config follows the new format (`config.yaml`) we can still load dynamic overrides from an external configmap in the old format. This is essential for operational continuity and we have to explicitly test this. Could you please add a unit test for this? -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
