grafana/mimir

Allow to configure hash ring via common config block

Open

#3,031 opened on Sep 23, 2022

View on GitHub
 (0 comments) (1 reaction) (0 assignees)Go (808 forks)auto 404
ease-of-usehacktoberfest

Repository metrics

Stars
 (5,181 stars)
PR merge metrics
 (PR metrics pending)

Description

I proposed to deprecated consul and etcd for the KV store but apparently we're not ready to move everyone to memberlist yet. For the people still using etcd or consul backend, we could simplify their config adding support to configure the hash ring KV store via the common config block.

I propose the following config:

common:
  ring:
    kvstore:
      store: <string>
      consul: <consul>
      etcd: <etcd>
    instance_id: <string>
    instance_interface_names: <list of strings>
    instance_port: <int>
    instance_addr: <string>
    heartbeat_period: <duration>
    heartbeat_timeout: <duration>

Other fields I intentionally excluded because it's not safe to set the same value everywhere:

  • ring.kvstore.prefix
  • Replication factor

Notes:

Contributor guide