-
Notifications
You must be signed in to change notification settings - Fork 360
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
ext_auth support backendRefs #3469
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #3469 +/- ##
==========================================
+ Coverage 67.22% 67.24% +0.02%
==========================================
Files 166 167 +1
Lines 19722 19744 +22
==========================================
+ Hits 13258 13277 +19
- Misses 5507 5512 +5
+ Partials 957 955 -2 ☔ View full report in Codecov by Sentry. |
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 thanks !
just realized, this is the Api PR :) |
everything is here.
is there any user case for this? |
7e2f577
to
4329b2a
Compare
Signed-off-by: zirain <zirain2009@gmail.com>
Signed-off-by: zirain <zirain2009@gmail.com>
Signed-off-by: zirain <zirain2009@gmail.com>
1287a27
to
1aa1ef8
Compare
wait #3478 |
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! Just some nits.
|
||
// BackendRefs references a Kubernetes object that represents the | ||
// backend server to which the authorization request will be sent. | ||
// Only service Kind is supported for now. |
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.
// Only service Kind is supported for now. | |
// Only Service kind is supported for now. |
xref: #3091