Avoid duplicate default options if present

mesosphere / chronos-pkg

Apache License 2.0

5 stars 16 forks source link

Avoid duplicate default options if present #19

Closed ninthnails closed 8 years ago

ninthnails commented 8 years ago

Avoid duplicating default ZK and master options if they already exist in cmd array. Also, loading config from /etc/mesos/zk regardless of provided configuration (through environment variables or config in /etc/chronos/conf) make it difficult to control what Zookeeper to use. Zookeeper may be installed somewhere else.

bradhe commented 8 years ago

:+1: I just ran in to this issue as well. Glad to see a PR is open, but would be useful to get it in!

ninthnails commented 8 years ago

Issue has been fixed in another PR #21.