Skip to content

docs: note that OutlierDetection implicitly activates locality LB failover#3693

Open
yasaswyk wants to merge 1 commit intoistio:masterfrom
yasaswyk:docs/outlier-detection-locality-lb
Open

docs: note that OutlierDetection implicitly activates locality LB failover#3693
yasaswyk wants to merge 1 commit intoistio:masterfrom
yasaswyk:docs/outlier-detection-locality-lb

Conversation

@yasaswyk
Copy link
Copy Markdown

Adds a note to the OutlierDetection message doc comment in destination_rule.proto explaining that because Istio's default mesh config enables LocalityLbSetting (Enabled:true), configuring OutlierDetection in a DestinationRule automatically activates locality-aware failover — even without an explicit localityLbSetting in the rule. Documents the opt-out: set localityLbSetting.enabled: false in the DR's load balancer settings.

…lover

Affirm-Skill: acli-jira
Affirm-Skill: att-deploy-check
Affirm-Skill: att-test
Affirm-Skill: buildkite-debug
Affirm-Skill: capture-context
Affirm-Skill: cmt-writer
Affirm-Skill: commit-and-push
Affirm-Skill: create-adr
Affirm-Skill: create-cmt-ticket
Affirm-Skill: create-tech-spec
Affirm-Skill: export-conversation-record
Affirm-Skill: gather-best-practices
Affirm-Skill: mcp-debug
Affirm-Skill: multi-thor-provisioner
Affirm-Skill: resolve-pr-comments
Affirm-Skill: save-output
Affirm-Skill: security-scanner-for-skill-md
Affirm-Skill: skill-evaluator
Affirm-Skill: thor-control
Affirm-Skill: tickets-from-plan
Affirm-Skill: write-implementation-plan
@istio-testing istio-testing added the do-not-merge/work-in-progress Block merging of a PR because it isn't ready yet. label Apr 15, 2026
@istio-policy-bot
Copy link
Copy Markdown

😊 Welcome @yasaswyk! This is either your first contribution to the Istio api repo, or it's been
a while since you've been here.

You can learn more about the Istio working groups, Code of Conduct, and contribution guidelines
by referring to Contributing to Istio.

Thanks for contributing!

Courtesy of your friendly welcome wagon.

@linux-foundation-easycla
Copy link
Copy Markdown

linux-foundation-easycla bot commented Apr 15, 2026

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: yasaswyk / name: Yasaswy Kota (534e7c3)

@istio-testing istio-testing added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Apr 15, 2026
@istio-testing
Copy link
Copy Markdown
Collaborator

Hi @yasaswyk. Thanks for your PR.

I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work.

Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@yasaswyk yasaswyk marked this pull request as ready for review April 15, 2026 15:26
@yasaswyk yasaswyk requested a review from a team as a code owner April 15, 2026 15:26
@istio-testing istio-testing removed the do-not-merge/work-in-progress Block merging of a PR because it isn't ready yet. label Apr 15, 2026
@yasaswyk
Copy link
Copy Markdown
Author

/easycla

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-ok-to-test size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants