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
Eventing TLS: add tests for dynamically added CA trust bundles #7726
base: main
Are you sure you want to change the base?
Conversation
5073bbb
to
488bde0
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #7726 +/- ##
==========================================
+ Coverage 69.22% 69.45% +0.23%
==========================================
Files 339 344 +5
Lines 19494 16019 -3475
==========================================
- Hits 13494 11126 -2368
+ Misses 5337 4205 -1132
- Partials 663 688 +25 ☔ View full report in Codecov by Sentry. |
488bde0
to
be92f5e
Compare
Signed-off-by: Pierangelo Di Pilato <pierdipi@redhat.com>
be92f5e
to
023a7ce
Compare
/lgtm |
023a7ce
to
754a320
Compare
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
I rebased this PR 🙏 |
/cherry-pick release-1.14 |
@matzew: once the present PR merges, I will cherry-pick it on top of release-1.14 in a new PR and assign it to you. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/cherry-pick release-1.13 |
@matzew: once the present PR merges, I will cherry-pick it on top of release-1.13 in a new PR and assign it to you. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/test reconciler-tests |
2 similar comments
/test reconciler-tests |
/test reconciler-tests |
754a320
to
8109afd
Compare
/lgtm |
/hold |
New changes are detected. LGTM label has been removed. |
/test reconciler-tests |
Fixes in #7924 |
/retest Tests worked locally |
|
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: matzew, 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 |
9b9892c
to
e4eba93
Compare
Signed-off-by: Pierangelo Di Pilato <pierdipi@redhat.com>
e4eba93
to
0c5a8a1
Compare
Signed-off-by: Pierangelo Di Pilato <pierdipi@redhat.com>
Signed-off-by: Pierangelo Di Pilato <pierdipi@redhat.com>
a513f14
to
c592d8e
Compare
Signed-off-by: Pierangelo Di Pilato <pierdipi@redhat.com>
Signed-off-by: Pierangelo Di Pilato <pierdipi@redhat.com>
@pierDipi: The following tests failed, say
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
This adds new tests for testing when CA trust bundles are dynamically added to the clients trust stores