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
Get sidecar weights from node service if not specified #15112
base: main
Are you sure you want to change the base?
Conversation
53b9853
to
dd5bd9f
Compare
@@ -131,6 +131,7 @@ proxy. | |||
- `port` - Defaults to being auto-assigned from a configurable | |||
range specified by [`sidecar_min_port`](/docs/agent/config/config-files#sidecar_min_port) | |||
and [`sidecar_max_port`](/docs/agent/config/config-files#sidecar_max_port). | |||
- `weights` - Defaults to the parent weights. |
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.
- `weights` - Defaults to the parent weights. | |
- `weights` - Defaults to the weights of the parent service. |
Slight variation to the wording to match the phrasing of the other items in the list.
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.
Updated
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 edits to sidecar-service.mdx on behalf of consul-docs.
LGTM, approving on behalf of consul-docs.
The sidecar service weight was set to default values if not specified by use in connect block of service definition. This patch aims of reusing the node service weights if no weights is specified from connect definition block or if defined to default one.
dd5bd9f
to
74c3a81
Compare
This pull request has been automatically flagged for inactivity because it has not been acted upon in the last 60 days. It will be closed if no new activity occurs in the next 30 days. Please feel free to re-open to resurrect the change if you feel this has happened by mistake. Thank you for your contributions. |
Description
The sidecar service weight was set to default values if not specified by use in connect block of service definition. This patch aims of reusing the node service weights if no weights is specified from connect definition block or if defined to default one.
Testing & Reproduction steps
Test has been updated
Links
This PR is a refresh of #9335
And implements #9334
PR Checklist