-
Notifications
You must be signed in to change notification settings - Fork 66
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
Add proposal for CA bundles from secret refs #1984
Add proposal for CA bundles from secret refs #1984
Conversation
73e7797
to
d2c4956
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1984 +/- ##
=======================================
Coverage 30.68% 30.68%
=======================================
Files 365 365
Lines 60617 60617
=======================================
+ Hits 18599 18602 +3
+ Misses 41481 41479 -2
+ Partials 537 536 -1 ☔ View full report in Codecov by Sentry. |
f2cf526
to
259a7f3
Compare
259a7f3
to
3e55ecf
Compare
1606600
to
14911ea
Compare
14911ea
to
cbd04c4
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
--- | ||
title: "Source Certificate Authority Data from Secrets and ConfigMaps into Pinniped Custom Resources" | ||
authors: [ "@cfryanr", "@joshuatcasey" , "@ashish-amarnath"] | ||
status: "proposed" |
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.
Please feel free to change the status to "approved" before merging this PR.
Feel free to merge without CI passing because this PR only adds a proposal markdown document, which is not tested by CI in any way. |
Signed-off-by: Ashish Amarnath <[email protected]>
cbd04c4
to
e38f082
Compare
Add proposal for CA bundles from secret refs
rendered markdown