mbalassi commented on pull request #17582:
URL: https://github.com/apache/flink/pull/17582#issuecomment-996168516


   Hey @viirya,
   
   The update to the keytab location from the local environment running the 
test to the taskmanager pod happens 
[here](https://github.com/apache/flink/blob/master/flink-kubernetes/src/main/java/org/apache/flink/kubernetes/kubeclient/decorators/KerberosMountDecorator.java#L160-L166),
 and we run into this because the `testHadoopConfConfigMap()` test you 
introduce in this PR ultimately calls the 
[`buildKubernetesTaskManagerSpecification`](https://github.com/apache/flink/blob/master/flink-kubernetes/src/main/java/org/apache/flink/kubernetes/kubeclient/factory/KubernetesTaskManagerFactory.java#L41)
 twice. Once in the `onSetup` call and in the test function body itself, both 
calls are introduced in this PR.
   
   The equivalent taskmanager test only makes this call once, it seems 
straight-forward from here to refactor your change to fix 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]


Reply via email to