fix #258: fix race condition for stopping killed daemons

src/app/daemon.go

@@ -19,7 +19,7 @@ loop:
 }
 
 func (p *Process) notifyDaemonStopped() {
-	if p.isDaemonLaunched() {
+	if p.procConf.IsDaemon {
 		p.procStateChan <- types.ProcessStateCompleted
 	}
 }

src/app/process.go

@@ -337,24 +337,25 @@ func (p *Process) waitForCompletion() int {
 }
 
 func (p *Process) waitUntilReady() bool {
-	for {
-		select {
-		case <-p.procReadyCtx.Done():
-			if p.procState.Health == types.ProcessHealthReady {
-				return true
-			}
-			log.Error().Msgf("Process %s was aborted and won't become ready", p.getName())
-			p.setExitCode(1)
-			return false
-		case <-p.procLogReadyCtx.Done():
-			err := context.Cause(p.procLogReadyCtx)
-			if errors.Is(err, context.Canceled) {
-				return true
-			}
-			log.Error().Err(err).Msgf("Process %s was aborted and won't become log ready", p.getName())
-			return false
-		}
+	<-p.procReadyCtx.Done()
+	if p.procState.Health == types.ProcessHealthReady {
+		return true
+	}
+	log.Error().Msgf("Process %s was aborted and won't become ready", p.getName())
+	p.setExitCode(1)
+	return false
+
+}
+
+func (p *Process) waitUntilLogReady() bool {
+	<-p.procLogReadyCtx.Done()
+	err := context.Cause(p.procLogReadyCtx)
+	if errors.Is(err, context.Canceled) {
+		return true
 	}
+	log.Error().Err(err).Msgf("Process %s was aborted and won't become log ready", p.getName())
+	return false
+
 }
 
 func (p *Process) wontRun() {

src/app/project_runner.go

@@ -168,7 +168,7 @@ func (p *ProjectRunner) waitIfNeeded(process *types.ProcessConfig) error {
 				}
 			case types.ProcessConditionLogReady:
 				log.Info().Msgf("%s is waiting for %s log line %s", process.ReplicaName, k, runningProc.procConf.ReadyLogLine)
-				ready := runningProc.waitUntilReady()
+				ready := runningProc.waitUntilLogReady()
 				if !ready {
 					return fmt.Errorf("process %s depended on %s to become ready, but it was terminated", process.ReplicaName, k)
 				}