Skip to content

Commit

Permalink
Set default value for BatchWait as ticker does not accept 0
Browse files Browse the repository at this point in the history
  • Loading branch information
ctrox authored and Ed Welch committed Sep 5, 2019
1 parent 5cd384a commit 6b51805
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions pkg/promtail/client/multi_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,8 @@ func TestNewMulti(t *testing.T) {
}
host1, _ := url.Parse("http://localhost:3100")
host2, _ := url.Parse("https://grafana.com")
expectedCfg1 := Config{BatchSize: 20, URL: flagext.URLValue{URL: host1}}
expectedCfg2 := Config{BatchSize: 10, URL: flagext.URLValue{URL: host2}}
expectedCfg1 := Config{BatchSize: 20, BatchWait: 1 * time.Second, URL: flagext.URLValue{URL: host1}}
expectedCfg2 := Config{BatchSize: 10, BatchWait: 1 * time.Second, URL: flagext.URLValue{URL: host2}}

clients, err := NewMulti(util.Logger, expectedCfg1, expectedCfg2)
if err != nil {
Expand Down

0 comments on commit 6b51805

Please sign in to comment.