Skip to content
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

Disable peering queries by default in proxycfg #17731

Merged
merged 1 commit into from Jun 14, 2023

Conversation

hashi-derek
Copy link
Member

This PR disables peering functionality in proxycfg by default, unless peering.enabled = true is explicitly set in the agent config.

This change was necessary to prevent expensive polling watch queries (that were introduced for peer upstreams) from impacting users that do not wish to leverage peering features. While this is technically a breaking change, the 1.13 implementation of peering is considered to be experimental and not production ready. After this PR is merged, any users that wish to use peering in conjunction with mesh will need to enable peering on each agent and server.

@github-actions github-actions bot added the theme/envoy/xds Related to Envoy support label Jun 14, 2023
@hashi-derek hashi-derek force-pushed the derekm/disable-peering-default branch from 805df9f to 36a6513 Compare June 14, 2023 13:50
@hashi-derek hashi-derek marked this pull request as ready for review June 14, 2023 14:10
Copy link
Member

@DanStough DanStough left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. one nit.

.changelog/17731.txt Outdated Show resolved Hide resolved
This PR disables peering functionality in proxycfg by default, unless
`peering.enabled = true` is explicitly set in the agent config.

This change was necessary to prevent expensive polling watch queries (that were
introduced for peer upstreams) from impacting users that do not wish to leverage
peering features. While this is technically a breaking change, the 1.13
implementation of peering is considered to be experimental and not production
ready. After this PR is merged, any users that wish to use peering in
conjunction with mesh will need to enable peering on each agent and server.
@hashi-derek hashi-derek force-pushed the derekm/disable-peering-default branch from 36a6513 to f7ee779 Compare June 14, 2023 14:16
@hashi-derek hashi-derek merged commit ffdfe04 into release/1.13.x Jun 14, 2023
94 checks passed
@hashi-derek hashi-derek deleted the derekm/disable-peering-default branch June 14, 2023 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr/no-backport theme/envoy/xds Related to Envoy support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants