Skip to content

Commit

Permalink
Zone aware replication is stable (#150)
Browse files Browse the repository at this point in the history
Signed-off-by: Marco Pracucci <[email protected]>
  • Loading branch information
pracucci authored Mar 30, 2022
1 parent 8de36ff commit de9a686
Showing 1 changed file with 1 addition and 6 deletions.
7 changes: 1 addition & 6 deletions ring/lifecycler.go
Original file line number Diff line number Diff line change
Expand Up @@ -153,11 +153,6 @@ func NewLifecycler(cfg LifecyclerConfig, flushTransferer FlushTransferer, ringNa
return nil, err
}

zone := cfg.Zone
if zone != "" {
level.Warn(logger).Log("msg", "experimental feature in use", "feature", "Zone aware replication")
}

// We do allow a nil FlushTransferer, but to keep the ring logic easier we assume
// it's always set, so we use a noop FlushTransferer
if flushTransferer == nil {
Expand All @@ -174,7 +169,7 @@ func NewLifecycler(cfg LifecyclerConfig, flushTransferer FlushTransferer, ringNa
RingKey: ringKey,
flushOnShutdown: atomic.NewBool(flushOnShutdown),
unregisterOnShutdown: atomic.NewBool(cfg.UnregisterOnShutdown),
Zone: zone,
Zone: cfg.Zone,
actorChan: make(chan func()),
state: PENDING,
lifecyclerMetrics: NewLifecyclerMetrics(ringName, reg),
Expand Down

0 comments on commit de9a686

Please sign in to comment.