New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Set DeadLetterSinkURI in the Broker and Trigger status #1349
Set DeadLetterSinkURI in the Broker and Trigger status #1349
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1349 +/- ##
============================================
+ Coverage 75.82% 75.85% +0.03%
Complexity 540 540
============================================
Files 95 95
Lines 3375 3380 +5
Branches 154 154
============================================
+ Hits 2559 2564 +5
Misses 628 628
Partials 188 188
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
This needs the new Broker and Trigger CRDs in a PR like #1346 |
@pierDipi what PR did change the CRDs of broker/trigger? |
PRs with a title like "[Automated] Update eventing-kafka-broker-eventing nightly" (example #1346) |
bumping knative.dev/eventing a6a819d...a34aaa0: > a34aaa0 💄 It's a deadLetterSink that we define. It is not a queue (# 5824) > eb87183 fixing COC doc (# 5774) > d98ae8c Fix broker status dlquri (# 5793) > 7ab2c01 upgrade to latest dependencies (# 5822) > 5f69768 upgrade to latest dependencies (# 5817) bumping knative.dev/hack 815cd31...47ac3b0: > 47ac3b0 Split build tags that are in a comma separated list (# 83) > 8687668 [# 50] Add error message when Bash < 4 (# 79) > 8d79600 Fixes # 81 (# 82) > 51582ce fix misspelling functionallity -> functionality (# 78) > 675e1fd Correctly specify the user when creating clusterrolebinding for kind (# 80) bumping knative.dev/pkg fa2f8f1...a34efd6: > a34efd6 upgrade to latest dependencies (# 2317) > ecdde77 Update actions (# 2316) > 3463dd7 upgrade to latest dependencies (# 2315) > a5bb759 upgrade to latest dependencies (# 2314) > e61a20d Update actions (# 2313) > 0e24b49 Format go code (# 2312) bumping knative.dev/reconciler-test 45904ca...816dbd1: > 816dbd1 Update actions (# 237) Signed-off-by: Knative Automation <automation@knative.team>
Signed-off-by: Pierangelo Di Pilato <pdipilat@redhat.com>
067a605
to
cd87fd7
Compare
The following is the coverage report on the affected files.
|
/unhold |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: aliok, pierDipi The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Fixes #1334
Proposed Changes
Release Note
Docs