Skip to content

Conversation

@gheorghepucea
Copy link
Contributor

@gheorghepucea gheorghepucea commented Nov 24, 2025

Summary

This PR adds documentation for additional ports that must be open to enable proper communication between containers on the same host in an ECE setup.

Generative AI disclosure

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

@gheorghepucea gheorghepucea requested a review from a team as a code owner November 24, 2025 10:06
@github-actions
Copy link

github-actions bot commented Nov 24, 2025

✅ Vale Linting Results

No issues found on modified lines!

@github-actions
Copy link

github-actions bot commented Nov 24, 2025

🔍 Preview links for changed docs

| 8080-8084 | Health/monitoring ports | All roles |
| 9000, 9043 | Internal proxy use | Proxy |
| 9244 | Internal proxy port | All roles |

Copy link
Contributor

@eedugon eedugon Nov 24, 2025

Choose a reason for hiding this comment

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

What about 22191-22199? Aren't they the ports used for the stunnels to zk servers? I think we have them mentioned in some KBs for connectivity verifications (similar to 2191-2199).

Copy link
Contributor Author

@gheorghepucea gheorghepucea Nov 24, 2025

Choose a reason for hiding this comment

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

good question @eedugon , they are already mentioned in the section "Inbound traffic from other ECE hosts", so I believe mentioning them twice would just confuse people.

Copy link
Contributor

Choose a reason for hiding this comment

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

Sorry I hadn't seen it! Thanks!

Copy link
Contributor

@yetanothertw yetanothertw left a comment

Choose a reason for hiding this comment

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

Thank you for adding that! 🪴

Copy link

@AlexP-Elastic AlexP-Elastic left a comment

Choose a reason for hiding this comment

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

LGTM

@gheorghepucea gheorghepucea enabled auto-merge (squash) November 24, 2025 14:07
@gheorghepucea
Copy link
Contributor Author

Thank you all for the reviews. @yetanothertw @eedugon how can we get this merged?

@eedugon
Copy link
Contributor

eedugon commented Nov 25, 2025

@gheorghepucea , we just needed to update the branch with the latest changes from main. I think auto-merge will merge it in a few minutes.

@gheorghepucea gheorghepucea merged commit a83331d into elastic:main Nov 25, 2025
7 of 8 checks passed
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.

4 participants