-
Notifications
You must be signed in to change notification settings - Fork 361
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
user doc for oidc #2256
user doc for oidc #2256
Conversation
dfa7473
to
78737af
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2256 +/- ##
==========================================
- Coverage 64.29% 64.28% -0.02%
==========================================
Files 112 112
Lines 15728 15728
==========================================
- Hits 10112 10110 -2
- Misses 4976 4978 +2
Partials 640 640 ☔ View full report in Codecov by Sentry. |
cfcfcd7
to
78737af
Compare
449b109
to
8111b29
Compare
Signed-off-by: huabing zhao <[email protected]>
8111b29
to
c70dba4
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.
thanks, just some nits
```shell | ||
export ENVOY_SERVICE=$(kubectl get svc -n envoy-gateway-system --selector=gateway.envoyproxy.io/owning-gateway-namespace=default,gateway.envoyproxy.io/owning-gateway-name=eg -o jsonpath='{.items[0].metadata.name}') | ||
|
||
sudo kubectl -n envoy-gateway-system port-forward service/${ENVOY_SERVICE} 443:443 |
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.
i don't think we need to bring sudo
in
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.
It's because of the 443 port.
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.
got it
Co-authored-by: sh2 <[email protected]> Signed-off-by: Huabing Zhao <[email protected]>
Co-authored-by: sh2 <[email protected]> Signed-off-by: Huabing Zhao <[email protected]>
Co-authored-by: sh2 <[email protected]> Signed-off-by: Huabing Zhao <[email protected]>
/retest |
close #2208