Skip to content

Conversation

shainaraskas
Copy link
Collaborator

sorry to tag the world on this. made some H1 changes and cleaned up some xrefs to said H1s.

@shainaraskas shainaraskas requested review from a team as code owners September 15, 2025 16:00
Copy link

github-actions bot commented Sep 15, 2025

🔍 Preview links for changed docs

More links …

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.

🚀!

Copy link
Contributor

@florent-leborgne florent-leborgne left a comment

Choose a reason for hiding this comment

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

LGTM for experience-docs files

# Quickstart [k8s-elastic-agent-quickstart]

1. Apply the following specification to deploy Elastic Agent with the System metrics integration to harvest CPU metrics from the Agent Pods. ECK automatically configures the secured connection to an {{es}} cluster named `quickstart`, created in the [{{es}} quickstart](deploy-an-orchestrator.md).
1. Apply the following specification to deploy Elastic Agent with the System metrics integration to harvest CPU metrics from the Agent Pods. ECK automatically configures the secured connection to an {{es}} cluster named `quickstart`, created in [](/deploy-manage/deploy/cloud-on-k8s/elasticsearch-deployment-quickstart.md).
Copy link
Contributor

@vishaangelova vishaangelova Sep 15, 2025

Choose a reason for hiding this comment

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

Suggested change
1. Apply the following specification to deploy Elastic Agent with the System metrics integration to harvest CPU metrics from the Agent Pods. ECK automatically configures the secured connection to an {{es}} cluster named `quickstart`, created in [](/deploy-manage/deploy/cloud-on-k8s/elasticsearch-deployment-quickstart.md).
1. Apply the following specification to deploy Elastic Agent with the System metrics integration to harvest CPU metrics from the Agent Pods. ECK automatically configures the secured connection to an {{es}} cluster named `quickstart`, created in [](/deploy-manage/deploy/cloud-on-k8s/deploy-an-orchestrator.md).

Copy link
Contributor

@vishaangelova vishaangelova Sep 15, 2025

Choose a reason for hiding this comment

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

This is a recurrent thing across the PR, could you please update all instances? Or rename the file. (but then check for /deploy-manage/deploy/cloud-on-k8s/deploy-an-orchestrator.md or deploy-an-orchestrator.md because I think there’s at least one instance of that)

Copy link
Collaborator Author

@shainaraskas shainaraskas Sep 15, 2025

Choose a reason for hiding this comment

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

this change was intentional - I have updated the link to make it better reflect the context. the old link was not very helpful and this page is the one in which we actually create the quickstart cluster.

Copy link
Contributor

Choose a reason for hiding this comment

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

Oh I see... I assumed this was meant to be a link to the same file. 🤦 Sorry!

Copy link
Contributor

@leemthompo leemthompo left a comment

Choose a reason for hiding this comment

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

Don't see anything dev docs specific here, but spot check LGTM

Copy link
Contributor

@vishaangelova vishaangelova left a comment

Choose a reason for hiding this comment

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

LGTM

@shainaraskas shainaraskas enabled auto-merge (squash) September 15, 2025 18:55
@shainaraskas shainaraskas merged commit 71b217a into main Sep 15, 2025
5 checks passed
@shainaraskas shainaraskas deleted the deploy-manage-h1-bash-shaina branch September 15, 2025 18: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.

5 participants