luxolus
/
open-nomad
2
0
Fork 0
Code Issues 1 Pull Requests Packages Releases Wiki Activity

check in stderrFrame is nil before logging stderrFrame.Data (#17815) (#18041) 

Co-authored-by: Kevin Mulvey <kmulvey@linux.com>
This commit is contained in:
James Rasell 2023-07-24 10:32:10 +01:00 committed by GitHub
parent 4f087674f4
commit 40549e1132
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 9 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
.changelog/17815.txt Normal file
View File

@ -0,0 +1,3 @@
```release-note:bug
cli: Fix panic in `alloc logs` command when receiving empty stdout or stderr log frames
```

4
command/alloc_logs.go
View File

@ -397,14 +397,18 @@ func (l *AllocLogsCommand) tailMultipleFiles(client *api.Client, alloc *api.Allo
case stdoutErr := <-stdoutErrCh:
return fmt.Errorf("received an error from stdout log stream: %v", stdoutErr)
case stdoutFrame := <-stdoutFrames:
if stdoutFrame != nil {
logUI.Output(string(stdoutFrame.Data))
}
case stderrErr := <-stderrErrCh:
return fmt.Errorf("received an error from stderr log stream: %v", stderrErr)
case stderrFrame := <-stderrFrames:
if stderrFrame != nil {
logUI.Warn(string(stderrFrame.Data))
}
}
}
}
func lookupAllocTask(alloc *api.Allocation) (string, error) {
tg := alloc.Job.LookupTaskGroup(alloc.TaskGroup)