Skip to content

Conversation

eedugon
Copy link
Contributor

@eedugon eedugon commented Feb 20, 2025

Related with https://github.com/elastic/docs-projects/issues/339

Docs to validate:
https://docs-v3-preview.elastic.dev/elastic/docs-content/pull/529/deploy-manage/deploy/cloud-enterprise/find-cloud-id
https://docs-v3-preview.elastic.dev/elastic/docs-content/pull/529/deploy-manage/deploy/cloud-enterprise/find-endpoint-url

We could consider merging both docs into one doc (showing both as methods to connect clients to Elasticsearch). At the moment I've left all original content and scope, just changed the navigation titles.

I don't like much the docs but I'm not sure if we have time to improve them during this migration.

: Used by the `curl` command and most programming languages that aren’t Java. To interact with your cluster, use your Elasticsearch cluster endpoint information from the deployment overview page in the Cloud UI. Port 9200 is used for plain text, insecure HTTP connections, while port 9243 is used for HTTPS. Using HTTPS is generally recommended, as it is more secure.

If this is your first time using Elasticsearch, you can try out some `curl` commands to become familiar with the basics. If you’re on an operating system like macOS or Linux, you probably already have the `curl` command installed. For example, to connect to your cluster from the command line over HTTPS with the `curl` command:

Copy link
Collaborator

Choose a reason for hiding this comment

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

can you either delete the ingest methods section (line 71) or give the title heading formatting?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've added a heading, but I wouldn't mind deleting it, whatever you prefer :)

@eedugon eedugon requested a review from shainaraskas February 20, 2025 16:09
shainaraskas
shainaraskas previously approved these changes Feb 20, 2025
Copy link
Collaborator

@shainaraskas shainaraskas left a comment

Choose a reason for hiding this comment

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

good enough!

Copy link
Collaborator

@shainaraskas shainaraskas left a comment

Choose a reason for hiding this comment

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

still good enough!

@eedugon eedugon merged commit afe0b53 into main Feb 20, 2025
4 checks passed
@eedugon eedugon deleted the ece_cloud_id_endpoint branch February 20, 2025 19:57
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