-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
Config-entry: Support proxy config in service-defaults #14395
Conversation
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 reviewed the changes to service-defaults.mdx on behalf of the consul-docs team. One minor suggestion to review.
Please let me know if you have any questions or require additional review.
Granting approval on behalf of consul-docs.
website/content/docs/connect/config-entries/service-defaults.mdx
Outdated
Show resolved
Hide resolved
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.
This will need a rebase but overall it LGTM. I was a little surprised how little code it took to make this happen.
Co-authored-by: Jeff Boruszak <104028618+boruszak@users.noreply.github.com>
89e9757
to
e7c190b
Compare
* Config-entry: Support proxy config in service-defaults * Update website/content/docs/connect/config-entries/service-defaults.mdx Co-authored-by: Jeff Boruszak <104028618+boruszak@users.noreply.github.com>
* Config-entry: Support proxy config in service-defaults * Update website/content/docs/connect/config-entries/service-defaults.mdx Co-authored-by: Jeff Boruszak <104028618+boruszak@users.noreply.github.com>
Description
In current implementation,
local_request_timeout_ms
andlocal_connect_timeout_ms
can only be configured either in proxy.config of individual service instances or the globalproxy-defaults
. Sometimes, users want to use theservice-defaults
config-entry to propagate the values only to the proxies of the selected service.This PR enables this feature by adding
local_request_timeout_ms
andlocal_connect_timeout_ms
to theservice-defaults
config-entry.Testing & Reproduction steps
Following is an example of the service-defaults entry with the added fields.
Links
Address #12553
PR Checklist