mateczagany commented on PR #821: URL: https://github.com/apache/flink-kubernetes-operator/pull/821#issuecomment-2247282685
Thanks for your feedback @anupamaggarwal, really appreciate it! 1. I agree, and this is what I expected to happen. Except it does not, because I forgot to add the FlinkStateSnapshot resource to the `ValidatingWebhookConfiguration` resource in the Helm chart, I will push a commit soon. 2. This PR is already quite huge, so I have decided to do the examples/docs in a separate PR, there is currently a draft for it: https://github.com/apache/flink-kubernetes-operator/pull/854 3. You are right, adding the config there would improve its visibility to the end-users, so I will do that in a future commit, thank you! -- 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
