Skip to content

Conversation

triviajon
Copy link

@triviajon triviajon commented Oct 7, 2025

What does this PR do? What is the motivation?

This PR updates the public control plane monitoring documentation with new instructions about an automatic way to do this using the Operator (on EKS and Openshift 4). This is largely based on our internal documentation which can be found here.

Currently, this PR frames this as a "Using the Operator" vs "Legacy" methods, but alternative framings are also welcome.

Motivation: https://datadoghq.atlassian.net/browse/CONTINT-4834

Merge readiness:

  • Ready for merge

@triviajon triviajon requested a review from a team as a code owner October 7, 2025 21:25
Copy link
Contributor

github-actions bot commented Oct 7, 2025

Preview links (active after the build_preview check completes)

Modified Files

@buraizu buraizu added the editorial review Waiting on a more in-depth review label Oct 7, 2025
@buraizu
Copy link
Contributor

buraizu commented Oct 7, 2025

Created DOCS-12268 to review

#### Validation
Check that checks are running:
```shell
kubectl exec -it <cluster-agent-pod> -- agent clusterchecks

Choose a reason for hiding this comment

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

I suggest we remove the -it as we run a single command and its output will print to the console.
adding the -it will increase chances of i/o errors when piping the stream to some toole like grep or less

#### Validation
Check that checks are running:
```shell
kubectl exec -it <cluster-agent-pod> -- agent clusterchecks

Choose a reason for hiding this comment

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

same here suggest we remove the -it to prevent issue when piping output.

Comment on lines +355 to +363
Look for:
- `kube_apiserver_metrics`
- `kube_controller_manager`
- `kube_scheduler`

You should see control plane metrics in Datadog like:
- `kube_apiserver.*`
- `kube_controller_manager.*`
- `kube_scheduler.*`

Choose a reason for hiding this comment

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

you omit etcd here is it expected for some reasons ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
editorial review Waiting on a more in-depth review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants