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

Posted to reviews@yunikorn.apache.org by GitBox <gi...@apache.org> on 2022/01/24 13:03:47 UTC

[GitHub] [incubator-yunikorn-core] anuraagnalluri edited a comment on pull request #365: [YUNIKORN-949] Location of the state dump file should be configurable

anuraagnalluri edited a comment on pull request #365:
URL: https://github.com/apache/incubator-yunikorn-core/pull/365#issuecomment-1020075314


   @yangwwei The added unit test works, and I've verified the environment variable to set the dump file path is properly sourced from the configmap. Relevant PRs [for k8sshim](https://github.com/apache/incubator-yunikorn-k8shim/pull/358) and [for release](https://github.com/apache/incubator-yunikorn-release/pull/70).
   
   However, when I edit the configmap, I don't see the environment variable change in the scheduler pod even after 30 min. I read about config hot refresh [here](https://yunikorn.apache.org/docs/0.8.0/setup/configure_scheduler/#configuration-hot-refresh), but am missing a few details. Is the hot refresh enabled by default, and if so, how long does it typically take? 


-- 
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: reviews-unsubscribe@yunikorn.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org