Skip to content

Conversation

@deepankarrelativity002
Copy link
Contributor

@deepankarrelativity002 deepankarrelativity002 commented Aug 7, 2025

A new ./server-bundle-release/docs/environment-watch/port-diagram.md page created
Renamed the "elasticsearch_setup.md" to "elasticsearch_pre_installation_overview.md"
In "elasticsearch_pre-installation-overview.md" added a Port diagram under the "Before You Start" section, a new bullet added as the second bullet with a link to the portdiagram.md page.
For the existing Kibana/elasticsearch/apm-server setup instructions added hyperlink of Port diagram as a first step :
Added a Banner before Installation steps - "Ensure inbound/outbound ports are open for this elastic stack component"
Added System Requirements Section that detail following
Relativity Version
Elasticsearch Version for Environment Watch : updated 1 singular version which is supported.

@deepankarrelativity002 deepankarrelativity002 requested a review from a team as a code owner August 7, 2025 12:02
Copy link
Collaborator

@scott-parillo scott-parillo left a comment

Choose a reason for hiding this comment

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

The updates to the diagram are nearly correct. The general idea is to modify the diagram so that it only shows Elasticsearch ports. It shouldn't be called "Datagrid" or "EW" or "Environment Watch" etc. Refer back to the Acceptance Criteria...

image

The AC explicitly wants you to completely eliminate DG from the table/diagram and replace it with these 3:

  • Elastic Stack - Elasticsearch - (9200)
  • Elastic Stack - Kibana - (5601)
  • Elastic Stack - Apm-Server - (8200)
Image

On the subject of public documentation and acronyms, you must avoid using "EW" altogether. The diagram should ONLY include Elasticsearch.

Copy link
Collaborator

@scott-parillo scott-parillo left a comment

Choose a reason for hiding this comment

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

I'm going to approve this PR with the understanding you will do your best to address the comment on having only 3 entries within the table and diagram indicating the Elastic Stack (e.g. Elasticsearch/Kibana/APM Server). When @KarunaDhawan returns next week, there will likely be a single PR open to address key edits and/or final structure changes.

@deepankarrelativity002
Copy link
Contributor Author

The updates to the diagram are nearly correct. The general idea is to modify the diagram so that it only shows Elasticsearch ports. It shouldn't be called "Datagrid" or "EW" or "Environment Watch" etc. Refer back to the Acceptance Criteria...

image The AC explicitly wants you to completely eliminate DG from the table/diagram and replace it with these 3:
  • Elastic Stack - Elasticsearch - (9200)
  • Elastic Stack - Kibana - (5601)
  • Elastic Stack - Apm-Server - (8200)
Image **On the subject of public documentation and acronyms, you must avoid using "EW" altogether. The diagram should ONLY include Elasticsearch.**

@deepankarrelativity002
Copy link
Contributor Author

The updates to the diagram are nearly correct. The general idea is to modify the diagram so that it only shows Elasticsearch ports. It shouldn't be called "Datagrid" or "EW" or "Environment Watch" etc. Refer back to the Acceptance Criteria...
image
The AC explicitly wants you to completely eliminate DG from the table/diagram and replace it with these 3:

  • Elastic Stack - Elasticsearch - (9200)
  • Elastic Stack - Kibana - (5601)
  • Elastic Stack - Apm-Server - (8200)
Image **On the subject of public documentation and acronyms, you must avoid using "EW" altogether. The diagram should ONLY include Elasticsearch.**

Hi @scott-parillo, Earlier I have done as per AC, later was suggested to have additional 3 and verify with you once. Now, updated as per AC. Additional Port number 9300-9400 for elasticsearch being added as intercommunication of ElasticSearch Nodes

Thanks

@deepankarrelativity002
Copy link
Contributor Author

In addition to REL-1143044, few changes are being added as suggested in #24

Copy link

@karthikanumula karthikanumula left a comment

Choose a reason for hiding this comment

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

Have reviewed the port diagram and environment size references with Engineer for the https://github.com/relativitydev/server-bundle-release/blob/9b4e56d379d76047d4783103838c237e7d1a9769/docs/environment-watch/port-diagram.md
Approving in the absence of Karuna for this sub task.

@Chaitan2681 Chaitan2681 requested a review from amithshet August 8, 2025 15:26
@Chaitan2681 Chaitan2681 merged commit a16306b into main Aug 8, 2025
1 check 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.

8 participants