Skip to content

CONTINT-4831 - add kubernetes control-plane monitoring dashboard #21691

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

CONTINT-4831 - add kubernetes control-plane monitoring dashboard #21691

wants to merge 1 commit into from
+3,047 −2

Conversation

@lavigne958
Copy link
Contributor

What does this PR do?

Add a new dashboard to monitor the kubernetes control-plane.

Motivation

https://datadoghq.atlassian.net/browse/CONTINT-4831

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Add the qa/skip-qa label if the PR doesn't need to be tested during QA.
  • If you need to backport this PR to another branch, you can add the backport/<branch-name> label to the PR and it will automatically open a backport PR once this one is merged

Result

Dashbaord can be viewed here for test purposes:
https://dddev.datadoghq.com/dashboard/zs7-a89-tpz/kubernetes-control-plane-dev

@lavigne958 lavigne958 self-assigned this Oct 17, 2025
@lavigne958 lavigne958 requested review from a team as code owners October 17, 2025 15:56
@lavigne958 lavigne958 requested a review from a team as a code owner October 20, 2025 07:48
@buraizu buraizu self-assigned this Oct 20, 2025
buraizu
buraizu previously approved these changes Oct 20, 2025
View reviewed changes
Copy link
Contributor

@buraizu buraizu left a comment

Choose a reason for hiding this comment

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

Approving with a small update requested for consistency

@buraizu buraizu removed their assignment Oct 20, 2025
frank-spano
frank-spano previously approved these changes Oct 22, 2025
View reviewed changes
sblumenthal
sblumenthal reviewed Oct 22, 2025
View reviewed changes
sblumenthal
sblumenthal reviewed Oct 22, 2025
View reviewed changes
sblumenthal
sblumenthal reviewed Oct 22, 2025
View reviewed changes
@temporal-github-worker-1 temporal-github-worker-1 bot dismissed stale reviews from buraizu and frank-spano October 23, 2025 08:20

Review from buraizu is dismissed. Related teams and files:

  • documentation
    • kubernetes/manifest.json

Review from frank-spano is dismissed. Related teams and files:

  • container-integrations
    • kubernetes/assets/dashboards/kubernetes_control_plane.json
    • kubernetes/manifest.json
lavigne958
lavigne958 commented Oct 23, 2025
View reviewed changes
Copy link
Contributor Author

@lavigne958 lavigne958 left a comment

Choose a reason for hiding this comment

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

Updated links

sblumenthal
sblumenthal previously approved these changes Oct 23, 2025
View reviewed changes
@temporal-github-worker-1 temporal-github-worker-1 bot dismissed sblumenthal’s stale review October 23, 2025 14:11

Review from sblumenthal is dismissed. Related teams and files:

  • container-integrations
    • kubernetes/manifest.json
@lavigne958
Copy link
Contributor Author

fixup dashboard path.

@lavigne958 lavigne958 force-pushed the crayon/add_kube_control_plane_dashboard branch from f4c0a53 to fb4b9ea Compare October 23, 2025 14:19
@lavigne958
Copy link
Contributor Author

/merge

@dd-devflow-routing-codex
Copy link

dd-devflow-routing-codex bot commented Oct 23, 2025 •
edited
Loading

View all feedbacks in Devflow UI.

2025-10-23 14:40:11 UTC ℹ️ Start processing command /merge

2025-10-23 14:40:18 UTC ℹ️ MergeQueue: waiting for PR to be ready

This merge request is not mergeable according to GitHub. Common reasons include pending required checks, missing approvals, or merge conflicts — but it could also be blocked by other repository rules or settings.
It will be added to the queue as soon as checks pass and/or get approvals.
Note: if you pushed new commits since the last approval, you may need additional approval.
You can remove it from the waiting list with /remove command.

2025-10-23 14:41:22 UTC ⚠️ MergeQueue: This merge request was unqueued

[email protected] unqueued this merge request

@lavigne958
Copy link
Contributor Author

/merge -c

@dd-devflow-routing-codex
Copy link

dd-devflow-routing-codex bot commented Oct 23, 2025 •
edited
Loading

View all feedbacks in Devflow UI.

2025-10-23 14:41:16 UTC ℹ️ Start processing command /merge -c

sblumenthal
sblumenthal previously approved these changes Oct 23, 2025
View reviewed changes
@lavigne958
CONTINT-4831 - add kubernetes control-plane monitoring dashboard
d53df1c 
Add a new dashboard to monitor the kubernetes control-plane.

Signed-off-by: Alexandre Lavigne <[email protected]>
@lavigne958 lavigne958 force-pushed the crayon/add_kube_control_plane_dashboard branch from fb4b9ea to d53df1c Compare October 24, 2025 07:58
@temporal-github-worker-1 temporal-github-worker-1 bot dismissed sblumenthal’s stale review October 24, 2025 07:58

Review from sblumenthal is dismissed. Related teams and files:

  • container-integrations
    • kubernetes/assets/dashboards/kubernetes_control_plane.json
    • kubernetes/manifest.json
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sblumenthal sblumenthal sblumenthal left review comments

@buraizu buraizu buraizu left review comments

@frank-spano frank-spano frank-spano left review comments

At least 1 approving review is required to merge this pull request.

Assignees

@lavigne958 lavigne958

Labels

changelog/no-changelog docs/review-requested documentation ecosystems/review-requested integration/kubernetes mergequeue-status: removed product/review-requested qa/skip-qa Automatically skip this PR for the next QA team/agent-integrations team/container-integrations team/documentation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@lavigne958 @sblumenthal @buraizu @frank-spano