Skip to content

Conversation

vicwicker
Copy link
Member

@vicwicker vicwicker commented Mar 6, 2025

How to categorize this PR?

/area documentation
/kind enhancement

What this PR does / why we need it:

This motivates the federation from the shoot Prometheus as the proper way for shoot owners to collect control-plane metrics and includes a configuration example.

Special notes for your reviewer:

/cc @rickardsjp @chrkl @vpnachev

Release note:

Add documentation on how to federate metrics from the shoot Prometheus into an external Prometheus instance

@gardener-prow gardener-prow bot requested review from chrkl, rickardsjp and vpnachev March 6, 2025 10:57
@gardener-prow gardener-prow bot added area/documentation Documentation related kind/enhancement Enhancement, improvement, extension cla: yes Indicates the PR's author has signed the cla-assistant.io CLA. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Mar 6, 2025
@chrkl
Copy link
Member

chrkl commented Mar 6, 2025

/lgtm

@gardener-prow gardener-prow bot added the lgtm Indicates that a PR is ready to be merged. label Mar 6, 2025
Copy link
Contributor

gardener-prow bot commented Mar 6, 2025

LGTM label has been added.

Git tree hash: 384655fda48e5a967149e3d60aae94a9d8f3ede0

Copy link
Member

@rickardsjp rickardsjp left a comment

Choose a reason for hiding this comment

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

This guide is sort of specific to the API server metrics, and I don't really see a reason not to generalize this into a guide on federating from control plane Prometheus instances in general.
Additionally, I think we should mention that we've got an OTel based approach in the works.

WDYT?

@gardener-prow gardener-prow bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed lgtm Indicates that a PR is ready to be merged. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Mar 6, 2025
@vicwicker vicwicker requested a review from rickardsjp March 6, 2025 15:54
Copy link
Member

@rickardsjp rickardsjp left a comment

Choose a reason for hiding this comment

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

Last nit I have to pick. Looks good otherwise!

Co-authored-by: Jeremy Rickards <jeremy.rickards@sap.com>
@vicwicker vicwicker force-pushed the add-prometheus-federation-documentation branch from f43448b to 0a8ea26 Compare March 6, 2025 16:03
Copy link
Member

@rickardsjp rickardsjp left a comment

Choose a reason for hiding this comment

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

Thank you!
/lgtm

@gardener-prow gardener-prow bot added the lgtm Indicates that a PR is ready to be merged. label Mar 6, 2025
Copy link
Contributor

gardener-prow bot commented Mar 6, 2025

LGTM label has been added.

Git tree hash: e6d0b1903595b7213acc390df86bbb7dcb0fe467

Copy link
Member

@rfranzke rfranzke left a comment

Choose a reason for hiding this comment

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

/approve

Copy link
Contributor

gardener-prow bot commented Mar 6, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: rfranzke, rickardsjp

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@gardener-prow gardener-prow bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Mar 6, 2025
@gardener-prow gardener-prow bot merged commit 05a5e27 into gardener:master Mar 6, 2025
18 checks passed
@vicwicker vicwicker deleted the add-prometheus-federation-documentation branch March 6, 2025 19:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. area/documentation Documentation related cla: yes Indicates the PR's author has signed the cla-assistant.io CLA. kind/enhancement Enhancement, improvement, extension lgtm Indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants