-
Notifications
You must be signed in to change notification settings - Fork 89
add remote cluster traffic filter info in remote-cluster connection page #1478
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
Conversation
Thanks @bobbybho !! 🙏 Please allow me to add some notes about background: Excerpt:
The above is an internal slack discussion. Should we have any further internal discussions, let's use another repo (https://github.com/elastic/docs-projects/issues/) as we synced earlier. Thanks! |
thanks @bobbybho and @kunisen for the PR and background info. @bobbybho , I'm totally ok with the change but I would include the note (or the text of the note) much closer to the beginning of the page, in the It feels a bit hidden in the bottom, and it's not clear that the note is applicable only when TLS certificates is the method used for the remote cluster authentication and authorization. Do you want us (me and @shainaraskas ) to take over and edit directly your PR? PS - this is the preview of the PR: |
Thank you for reviewing this PR and sharing your feedback, @eedugon. Please feel free to make any additional changes directly in this PR, or alternatively, you can open a new PR if you prefer. |
@shainaraskas , yes, the linked issue from Florent is clearly related, but I don't think we are solving it completely in this PR. At the moment I have only moved upwards the note added by @bobbybho , as it was added at the bottom in the heading 3 section called I've also added a line in the parent page when it talks about remote clusters and traffic filters to clarify that API key based authentication cannot be used in conjunction with traffic filters. While reviewing all this I've detected other issues in the way we are presenting the docs for remote clusters in ECE and ECH (we have a lot of duplicated content without snippets and there are a lot of non clear areas, some of them related with traffic filters). I'd like to discuss that in private. @bobbybho , @kunisen , let us know if you are ok with this change, and we will plan for extra improvements in other PRs. btw, @kunisen , in your original analysis, when you said:
The conclusion ( We definitely have an issue if we are deprecating a feature which is the only way to make remote clusters with traffic filters to work on ECH, but that's a different story. |
This pull request updates the documentation for Elastic Cloud remote clusters to include a note about traffic filtering requirements when it is enabled. The note provides guidance for administrators on configuring a traffic filter for remote clusters.
Documentation updates for traffic filtering:
deploy-manage/remote-clusters/ec-remote-cluster-other-ess.md
: Added a note explaining that when traffic filtering is enabled, the remote cluster administrator must configure a traffic filter of type remote cluster using the organization ID or Elasticsearch cluster ID. A link to detailed instructions is also provided.deploy-manage/remote-clusters/ec-remote-cluster-same-ess.md
: Added the same note about traffic filtering configuration as in the previous file, ensuring consistent guidance across documentation for remote clusters.https://elasticco.atlassian.net/browse/ECPTRAFFIC-546