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
configure traffic policy for svc #190
base: main
Are you sure you want to change the base?
Conversation
This change add option to configure traffic policy for the service. It is backward compatible and will not change the default behavior.
df70fae
to
33f67c0
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.
These new parameters must be documented here:
https://github.com/oauth2-proxy/manifests/blob/main/helm/oauth2-proxy/README.md#configuration
# configure externalTrafficPolicy | ||
externalTrafficPolicy: | ||
# configure internalTrafficPolicy | ||
internalTrafficPolicy: |
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 add Cluster
as default.
@@ -128,6 +128,11 @@ service: | |||
appProtocol: http | |||
annotations: {} | |||
# foo.io/bar: "true" | |||
# configure externalTrafficPolicy | |||
externalTrafficPolicy: |
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 add Cluster
as default.
helm/oauth2-proxy/Chart.yaml
Outdated
@@ -1,5 +1,5 @@ | |||
name: oauth2-proxy | |||
version: 7.2.0 | |||
version: 7.3.0 |
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.
artifacthub.io/changes
needs to be changed.
@tomkukral, could you please take a look and try to move forward with the PR? |
@tomkukral I tried to change the PR, but it's impossible. |
This change add option to configure traffic policy for the service. It is backward compatible and will not change the default behavior.