Skip to content

Comments

Improve loadbalancer docs based on SDH#5224

Open
yetanothertw wants to merge 3 commits intomainfrom
781-lb-ece
Open

Improve loadbalancer docs based on SDH#5224
yetanothertw wants to merge 3 commits intomainfrom
781-lb-ece

Conversation

@yetanothertw
Copy link
Contributor

@yetanothertw yetanothertw commented Feb 19, 2026

Summary

Fixes #781

To add the recommendation mentioned here I've restructured the page (PREVIEW of changes):

  • the port/mode-specific bullets are removed from the unordered list (it's pretty hard to parse the info in the list) and added into a new section in the page Port and mode configuration. The new section includes a table (easier to scan) and replaces the old scattered list items. The table clearly shows that ports 9200/9243 support both HTTP (L7) and TCP (L4) modes, with the corresponding client IP mechanism for each.
  • added a new Client IP preservation section to explain the three different mechanisms that can be used to preserve the IP. This way other pages can link to this specific section.
  • Updated the prerequisites section in the Manage IP filters in ECE.
  • Added links between the two pages (ip filtering and load balancer recommendations).

This will require some Eng Team 👀 to review.

Generative AI disclosure

  1. Did you use a generative AI (GenAI) tool to assist in creating this contribution?
  • Yes
  • No

Tool(s) and model(s) used: Used Claude 4.6 sonnet to validate some assumptions when building the table.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 19, 2026

✅ Vale Linting Results

No issues found on modified lines!


The Vale linter checks documentation changes against the Elastic Docs style guide.

To use Vale locally or report issues, refer to Elastic style guide for Vale.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 19, 2026

@yetanothertw yetanothertw self-assigned this Feb 20, 2026
Copy link
Member

@alstolten alstolten left a comment

Choose a reason for hiding this comment

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

LGTM

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants