Merge pull request #78825 from pswica/logs-wg

Adding defer wg.Done() Statement To logs.go
This commit is contained in:
Kubernetes Prow Robot 2019-06-14 20:46:49 -07:00 committed by GitHub
commit 14b6dce953
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -324,6 +324,7 @@ func (o LogsOptions) parallelConsumeRequest(requests []rest.ResponseWrapper) err
wg.Add(len(requests))
for _, request := range requests {
go func(request rest.ResponseWrapper) {
defer wg.Done()
if err := o.ConsumeRequestFn(request, writer); err != nil {
if !o.IgnoreLogErrors {
writer.CloseWithError(err)
@ -335,7 +336,6 @@ func (o LogsOptions) parallelConsumeRequest(requests []rest.ResponseWrapper) err
fmt.Fprintf(writer, "error: %v\n", err)
}
wg.Done()
}(request)
}