Skip to content

Commit

Permalink
dont try to use the scheduler ring when a downstream url is configured (
Browse files Browse the repository at this point in the history
  • Loading branch information
owen-d authored Nov 3, 2021
1 parent d53aef6 commit 0e0d0b7
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion pkg/loki/config_wrapper.go
Original file line number Diff line number Diff line change
Expand Up @@ -69,10 +69,11 @@ func (c *ConfigWrapper) ApplyDynamicConfig() cfg.Source {
return errors.New("dst is not a Loki ConfigWrapper")
}

// If nobody has defined any frontend address or scheduler address
// If nobody has defined any frontend address, scheduler address, or downstream url
// we can default to using the query scheduler ring for scheduler discovery.
if r.Worker.FrontendAddress == "" &&
r.Worker.SchedulerAddress == "" &&
r.Frontend.DownstreamURL == "" &&
r.Frontend.FrontendV2.SchedulerAddress == "" {
r.QueryScheduler.UseSchedulerRing = true
}
Expand Down

0 comments on commit 0e0d0b7

Please sign in to comment.