Skip to content

docs: clarify private link traffic imbalance for all node types including APM#5801

Merged
kunisen merged 7 commits into
mainfrom
kunisen-docpr-stl1773
Apr 8, 2026
Merged

docs: clarify private link traffic imbalance for all node types including APM#5801
kunisen merged 7 commits into
mainfrom
kunisen-docpr-stl1773

Conversation

@kunisen

@kunisen kunisen commented Apr 7, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Summary

Clarify the importance of ensuring user's VPC endpoint is in all availability zones supported by Elastic Cloud

Motivation

Per this internal ticket and this ticket

🤖 Partially Generated with Claude Code

@kunisen
docs: clarify private link traffic imbalance for all node types inclu…
c6cf2a4 
…ding APM

Expands the 'Before you begin' section to:
- Explain that traffic imbalance affects all node types (ES coordinating nodes and APM/Integration servers), not just coordinating nodes
- Describe the AWS NLB flow-hash routing behavior as the root cause
- Call out the common symptom of scaling nodes into an AZ not covered by the VPC endpoint

Related: elastic/support-tech-lead#1773
SDH ref: elastic/sdh-control-plane#11613
@github-actions

github-actions Bot commented Apr 7, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

🔍 Preview links for changed docs

@github-actions

github-actions Bot commented Apr 7, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Vale Linting Results

Summary: 1 warning found

⚠️ Warnings (1)
File Line Rule Message
deploy-manage/security/private-connectivity-aws.md 147 Elastic.Spelling 'underutilize' is a possible misspelling.

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.

@kunisen kunisen marked this pull request as ready for review April 7, 2026 05:07
@kunisen kunisen requested a review from a team as a code owner April 7, 2026 05:07
@kunisen kunisen requested a review from dnraitzyk April 7, 2026 05:08
cargious
cargious requested changes Apr 7, 2026
View reviewed changes

@cargious cargious left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@alxchalkias please review this as I remember we have produced a PM-like doc with the PrivateLink AZ-overlapping aspects.

@dnraitzyk @kunisen for a deep dive on PrivateLink flows you can read here. I believe the doc that Alex will provide is covering in high level what we need.

I agree with adding some Important flags and possibly add some more cases/examples, but we should not expose internal info for the NLB -> Elastic proxies hop.

Comment thread deploy-manage/security/private-connectivity-aws.md Outdated
Comment thread deploy-manage/security/private-connectivity-aws.md Outdated
Comment thread deploy-manage/security/private-connectivity-aws.md Outdated
@cargious cargious requested a review from alxchalkias April 7, 2026 06:54
@kunisen

kunisen commented Apr 7, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Thank you @cargious for the review!

I made an update based on your comment.
b76358e

My bad for the confusion, as well as the back and forth.
Based on the context, I think we could point out below things in simple:

  • Make it an "important" block
  • Include the fact that not only ES coordinating nodes, but also other nodes, such as APM / Integrations Server could have such imbalanced traffic (this is one of the key point from our internal ticket)
  • Remove the unnecessary descriptions

Could you please have a second check? 🙏

Preview link:
https://docs-v3-preview.elastic.dev/elastic/docs-content/pull/5801/deploy-manage/security/private-connectivity-aws#ec-aws-vpc-overlapping-azs

Screenshot after update:

image

@kunisen kunisen self-assigned this Apr 7, 2026
@kunisen kunisen added supportability ability enable self-service or support of product Team:Admin Issues owned by the Admin Docs Team labels Apr 7, 2026
@alxchalkias

alxchalkias commented Apr 7, 2026

Copy link
Copy Markdown
Contributor

Fine to make this more visible, I've only added one other minor edit. @shainaraskas, maybe you want to have a look at this yourself. While we have made the initial docs improvements many months ago, it seems like the information is not clear/visible enough to all customers, and Support still gets cases because of the traffic imbalance.

alxchalkias
alxchalkias approved these changes Apr 7, 2026
View reviewed changes
Comment thread deploy-manage/security/private-connectivity-aws.md Outdated
@kunisen @alxchalkias
Update deploy-manage/security/private-connectivity-aws.md
c424394 
Co-authored-by: Alex Chalkias <34575586+alxchalkias@users.noreply.github.com>
@kunisen

kunisen commented Apr 7, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Thank you @alxchalkias !!
I accepted your suggestion.

One more thing - out side of this PR - while I was looking at the page, I found there are some discrepancies in between the lines

[1]

image

[2]

image

I guess this happened during our doc internal (v3) migration. I will leave it to doc team's capable hands to decide how we handle this.

cargious
cargious approved these changes Apr 7, 2026
View reviewed changes

@cargious cargious left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I am approving the PR, though there are some docs-build suggestions that you might want to take into account.

Thank you again for making our docs better, it is really important for the customers and the load Elastic teams have when supporting the customers!

shainaraskas
shainaraskas approved these changes Apr 7, 2026
View reviewed changes

@shainaraskas shainaraskas left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

small style edits. will open an issue for the other things you highlighted. this is not a result of the migration, but instead a long page being updated over time.

Comment thread deploy-manage/security/private-connectivity-aws.md Outdated
@shainaraskas shainaraskas mentioned this pull request Apr 7, 2026
Closed
@dnraitzyk

dnraitzyk commented Apr 7, 2026

Copy link
Copy Markdown
Contributor

Approved. Sorry, I had not seen the notification

kunisen and others added 2 commits April 8, 2026 09:10
@kunisen @shainaraskas
Update deploy-manage/security/private-connectivity-aws.md
f3139fc 
Co-authored-by: shainaraskas <58563081+shainaraskas@users.noreply.github.com>
@kunisen
Merge branch 'main' into kunisen-docpr-stl1773
5fcb778
@kunisen

kunisen commented Apr 8, 2026

Copy link
Copy Markdown
Contributor Author

Thank you all for the help and review! I will merge.

@kunisen kunisen enabled auto-merge (squash) April 8, 2026 00:11
@kunisen kunisen merged commit 9bfb87d into main Apr 8, 2026
6 of 7 checks passed
@kunisen kunisen deleted the kunisen-docpr-stl1773 branch April 8, 2026 00:13
@kunisen kunisen added Serverless Improvements and changes to the Serverless Docs ech Elastic Cloud Hosted labels Jun 3, 2026
@kunisen

kunisen commented Jun 3, 2026

Copy link
Copy Markdown
Contributor Author

operational (ops) update - adding ech and serverless label (this applies to both ECH and serverless)

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

Reviewers

@cargious cargious cargious approved these changes

@alxchalkias alxchalkias alxchalkias approved these changes

@dnraitzyk dnraitzyk dnraitzyk approved these changes

@shainaraskas shainaraskas shainaraskas approved these changes

Assignees

@kunisen kunisen

Labels

ech Elastic Cloud Hosted Serverless Improvements and changes to the Serverless Docs supportability ability enable self-service or support of product Team:Admin Issues owned by the Admin Docs Team

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@kunisen @alxchalkias @dnraitzyk @cargious @shainaraskas