Skip to content

Commit

Permalink
Merge pull request #273 from Cryptophobia/master
Browse files Browse the repository at this point in the history
fix(reloader): revert to reload endpoint and info
  • Loading branch information
Cryptophobia authored Oct 19, 2021
2 parents 5033e82 + fc8caba commit 6f8b73d
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions config-reloader/fluentd/reloader.go
Original file line number Diff line number Diff line change
Expand Up @@ -29,9 +29,9 @@ func (r *Reloader) ReloadConfiguration() {
logrus.Infof("Not reloading fluentd (fake or filesystem datasource used)")
return
}
_, err := http.Post(fmt.Sprintf("http://127.0.0.1:%d/api/config.gracefulReload", r.port), "application/json", nil)
_, err := http.Post(fmt.Sprintf("http://127.0.0.1:%d/api/config.reload", r.port), "application/json", nil)

if err != nil {
logrus.Errorf("cannot notify fluentd: %+v", err)
logrus.Infof("cannot notify fluentd: %+v", err)
}
}
2 changes: 1 addition & 1 deletion config-reloader/fluentd/reloader_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ func TestReloaderCalls(t *testing.T) {

handler := func(w http.ResponseWriter, r *http.Request) {
fmt.Printf("req %+v", r)
if r.Method == "POST" && r.RequestURI == "/api/config.gracefulReload" {
if r.Method == "POST" && r.RequestURI == "/api/config.reload" {
counter++
}
}
Expand Down

0 comments on commit 6f8b73d

Please sign in to comment.