Browse Source

fix #55 : now that we have split the code for external shutdown (service) and one-shot, we don't need the 5s sleep anymore

Thibault bui Koechlin 5 years ago
parent
commit
fa2fd471f9
1 changed files with 0 additions and 2 deletions
  1. 0 2
      cmd/crowdsec/serve.go

+ 0 - 2
cmd/crowdsec/serve.go

@@ -26,8 +26,6 @@ func termHandler(sig os.Signal) error {
 		log.Warningf("Acquisition returned error : %s", err)
 		log.Warningf("Acquisition returned error : %s", err)
 	}
 	}
 	log.Infof("acquisition is finished, wait for parser/bucket/ouputs.")
 	log.Infof("acquisition is finished, wait for parser/bucket/ouputs.")
-	//let's wait more than enough for in-flight events to be parsed.
-	time.Sleep(5 * time.Second)
 	parsersTomb.Kill(nil)
 	parsersTomb.Kill(nil)
 	if err := parsersTomb.Wait(); err != nil {
 	if err := parsersTomb.Wait(); err != nil {
 		log.Warningf("Parsers returned error : %s", err)
 		log.Warningf("Parsers returned error : %s", err)