Skip to content

Commit 3b236c4

Browse files
committed
disable initial events because targets are already executed immediately after reconfigure anyway
1 parent daa31e2 commit 3b236c4

File tree

1 file changed

+2
-16
lines changed

1 file changed

+2
-16
lines changed

service/reconfigure.go

Lines changed: 2 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ func (app *App) watchConfig() (err error) {
7878
app.config.CheckInterval,
7979
app.config.Directory,
8080
app.ssh,
81-
true)
81+
false)
8282
if err != nil {
8383
return errors.Wrap(err, "failed to watch config target")
8484
}
@@ -91,13 +91,6 @@ func (app *App) watchConfig() (err error) {
9191
}()
9292
zap.L().Debug("created new config watcher, awaiting setup")
9393

94-
select {
95-
case <-app.configWatcher.InitialDone:
96-
zap.L().Debug("config initial setup done")
97-
98-
case err = <-app.errors:
99-
}
100-
10194
return
10295
}
10396

@@ -123,7 +116,7 @@ func (app *App) watchTargets() (err error) {
123116
app.config.CheckInterval,
124117
app.config.Directory,
125118
app.ssh,
126-
true)
119+
false)
127120
if err != nil {
128121
return errors.Wrap(err, "failed to watch targets")
129122
}
@@ -136,13 +129,6 @@ func (app *App) watchTargets() (err error) {
136129
}()
137130
zap.L().Debug("created targets watcher, awaiting setup")
138131

139-
select {
140-
case <-app.targetsWatcher.InitialDone:
141-
zap.L().Debug("targets initial setup done")
142-
143-
case err = <-app.errors:
144-
}
145-
146132
return
147133
}
148134

0 commit comments

Comments
 (0)