Closed gabemontero closed 4 months ago
The following is the coverage report on the affected files.
Say /test pull-tekton-results-go-coverage
to re-run this coverage report
File | Old Coverage | New Coverage | Delta |
---|---|---|---|
pkg/api/server/v1alpha2/logs.go | 68.4% | 57.1% | -11.3 |
It feels to me that we are masking the "real" error which got us to handleReturn
in the first place. I mean when we try to flush the log and fail. Maybe at least concatenate the error messages like "got flushErr while handling otherError"?
It feels to me that we are masking the "real" error which got us to
handleReturn
in the first place. I mean when we try to flush the log and fail. Maybe at least concatenate the error messages like "got flushErr while handling otherError"?
yeah I was wondering about that as well after I submitted the PR @enarha .... I'll look into doing this. Will post a comment when I 've updated it.
It feels to me that we are masking the "real" error which got us to
handleReturn
in the first place. I mean when we try to flush the log and fail. Maybe at least concatenate the error messages like "got flushErr while handling otherError"?yeah I was wondering about that as well after I submitted the PR @enarha .... I'll look into doing this. Will post a comment when I 've updated it.
I've updated the PR @enarha to aggregate the errors ... PTAL
The following is the coverage report on the affected files.
Say /test pull-tekton-results-go-coverage
to re-run this coverage report
File | Old Coverage | New Coverage | Delta |
---|---|---|---|
pkg/api/server/v1alpha2/logs.go | 68.4% | 63.4% | -5.0 |
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: enarha
The full list of commands accepted by this bot can be found here.
The pull request process is described here
Changes
/kind bug
An intermittent server side update log error I discussed with @sayan-biswas @avinal @khrm @enarha last month:
If we get an error on the flush let's return it to the client in case retry is possible
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you review them:
/kind <type>
. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tepRelease Notes