search

aws-ia / cfn-ps-clickhouse-cluster

Apache License 2.0
10 stars 6 forks source link

Zookeepers not configured in clickhouse config #44

Closed mohammadQBNL closed 2 months ago

mohammadQBNL commented 2 months ago

I was taking a look at one of my instance's clickhouse config, and noticed zookeepers and not configured there at all. Shouldn't this be part of the stack setup process?

In /etc/clickhouse-server/config.xml:

 <!--</remote_url_allow_hosts>--><include_from>/etc/clickhouse-server/metrika.xml</include_from><remote_servers incl="clickhouse_remote_servers" /><zookeeper incl="zookeeper-servers" optional="true" />

    <!-- If element has 'incl' attribute, then for it's value will be used corresponding substitution from another file.
         By default, path to file with substitutions is /etc/metrika.xml. It could be changed in config in 'include_from' element.
         Values for substitutions are specified in /clickhouse/name_of_substitution elements in that file.
      -->

    <!-- ZooKeeper is used to store metadata about replicas, when using Replicated tables.
         Optional. If you don't use replicated tables, you could omit that.

         See https://clickhouse.com/docs/en/engines/table-engines/mergetree-family/replication/
      -->

    <!--
    <zookeeper>
        <node>
            <host>example1</host>
            <port>2181</port>
        </node>
        <node>
            <host>example2</host>
            <port>2181</port>
        </node>
        <node>
            <host>example3</host>
            <port>2181</port>
        </node>
    </zookeeper>
    -->

Should I add them myself or was there something wrong with the set up?