rickchengx commented on PR #11508:
URL: 
https://github.com/apache/dolphinscheduler/pull/11508#issuecomment-1217381393

   > Hi @rickchengx , for those `application.yaml` configuration files, please 
refer to #11402 , I've enable the Spring Cloud Kubernetes configmap and it can 
watch the configmap in realtime and update the configurations, can you remove 
the related part in this PR to only contains those that are not included in 
#11402 ? i.e. `common.properties`
   
   Hi @kezhenxu94 , thanks for the reply. #11402 seems to be related to 
submitting a `kubernetes-type` task, but the scenario of this pr is to use 
`Helm` to deploy DS in a k8s cluster.
   
   The purpose of this pr is to allow users to easily change the configuration 
files of each server before deploying with `Helm`, and mount the user-defined 
configuration files to the directory `/opt/dolphinscheduler/conf` in each 
server's container through `configmap`.
   
   Please correct me if my understanding is wrong.


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