Skip to content

Commit

Permalink
fix: return after stream err (#1301)
Browse files Browse the repository at this point in the history
## Description:
Minor fix after follow logs pr.

## Is this change user facing?
NO

## References:
#1298
#1293
  • Loading branch information
tedim52 committed Sep 14, 2023
1 parent 9b0bcb7 commit f40559b
Showing 1 changed file with 2 additions and 0 deletions.
Expand Up @@ -132,12 +132,14 @@ func (strategy *PerWeekStreamLogsStrategy) StreamLogs(

if err = strategy.sendJsonLogLine(jsonLogStr, logsByKurtosisUserServiceUuidChan, serviceUuid, conjunctiveLogLinesFiltersWithRegex); err != nil {
streamErrChan <- stacktrace.Propagate(err, "An error occurred sending log line for service '%v' in enclave '%v'.", serviceUuid, enclaveUuid)
return
}

if isLastLogLine {
if shouldFollowLogs {
if err = strategy.tailLogs(latestLogFile, logsByKurtosisUserServiceUuidChan, serviceUuid, conjunctiveLogLinesFiltersWithRegex); err != nil {
streamErrChan <- stacktrace.Propagate(err, "An error occurred following logs for service '%v' in enclave '%v'.", serviceUuid, enclaveUuid)
return
}
} else {
return
Expand Down

0 comments on commit f40559b

Please sign in to comment.