You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@flink.apache.org by GitBox <gi...@apache.org> on 2022/05/15 09:32:28 UTC

[GitHub] [flink-kubernetes-operator] wangyang0918 commented on pull request #215: [FLINK-27163] Add label for the session job to help list the session …

wangyang0918 commented on PR #215:
URL: https://github.com/apache/flink-kubernetes-operator/pull/215#issuecomment-1126894901

   TBH, I do not like to do the CR `.metadata` or `.spec` updates in the reconciler. Especially, this is only for easier filter, not for operator internal use case. In the production environment, I believe the users could do this manually or covered by external job management tools.
   
   I also have another feeling this is the responsibility of [mutating admission webhook](https://kubernetes.io/docs/reference/access-authn-authz/admission-controllers/#mutatingadmissionwebhook).


-- 
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