Closed khrm closed 1 year ago
Fixes #1599
The following is the coverage report on the affected files.
Say /test pull-tekton-triggers-go-coverage
to re-run this coverage report
File | Old Coverage | New Coverage | Delta |
---|---|---|---|
pkg/sink/sink.go | 70.8% | 71.1% | 0.2 |
It looks like the events.TriggerProcessingDoneV1
event may be also produced asynchronously if CloudEventURI
is set.
The event produced asynchronously and the one produced via HTTP response should be identical (at least id and source should be the same).
Another idea: when CloudEventURI is set couldn't we just return empty body with 202 since we know the even will be emitted asynchronously?
The following is the coverage report on the affected files.
Say /test pull-tekton-triggers-go-coverage
to re-run this coverage report
File | Old Coverage | New Coverage | Delta |
---|---|---|---|
pkg/sink/sink.go | 70.8% | 71.1% | 0.2 |
/lgtm
The following is the coverage report on the affected files.
Say /test pull-tekton-triggers-go-coverage
to re-run this coverage report
File | Old Coverage | New Coverage | Delta |
---|---|---|---|
pkg/sink/sink.go | 70.8% | 71.1% | 0.2 |
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: savitaashture
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/cherry-pick release-v0.24.x
@khrm: new pull request created: #1605
Changes
Submitter Checklist
As the author of this PR, please check off the items in this checklist:
/kind <type>
. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tepRelease Notes