Skip to content

Conversation

@Katsuyu
Copy link
Contributor

@Katsuyu Katsuyu commented Jan 6, 2025

Your checklist for this pull request

Description

Adding a documentation on how to send custom metrics from a k8s cluster to Cockpit using a custom datasource
with either Kubernetes or Terraform

@Katsuyu Katsuyu requested a review from nerda-codes as a code owner January 6, 2025 16:39
@Katsuyu
Copy link
Contributor Author

Katsuyu commented Jan 6, 2025

This PR has the branch in #4149 as base branch, we should first merge the log documentation before this one or just simply close the #4149 and only validate this PR

@nerda-codes nerda-codes added the do not merge PR that shouldn't be merged before a specific date (eg release) label Jan 7, 2025
@Katsuyu
Copy link
Contributor Author

Katsuyu commented Jan 8, 2025

This documentation will be updated with the release of the 2.0 version of k8s-monitoring, i'll refactor the code, dont merge until it's done, i'll leave another message with it's done !

@nerda-codes
Copy link
Contributor

Finished testing and reviewing. There is just one thing that I am not sure about, which is the step where we create the Helm release resource. We tell the user to name their log/metric-ingester "my-log/metric-ingester". We can discuss this face to face. I've also added images of what the output looks like on Grafana, I'll add them once we merge this PR 😄

@nerda-codes nerda-codes added status: doc review Waiting for doc team review and removed neda-reviewed labels Jan 20, 2025
Co-authored-by: Jessica <[email protected]>
Co-authored-by: Rowena Jones <[email protected]>
@nerda-codes nerda-codes merged commit 83bba68 into scaleway:main Jan 21, 2025
Laure-di pushed a commit to Laure-di/docs-content that referenced this pull request Mar 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not merge PR that shouldn't be merged before a specific date (eg release) status: doc review Waiting for doc team review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants