Skip to content
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.

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

docs: Metrics configuration guide & FAQ reorg #2414

Merged
merged 3 commits into from
May 10, 2024

Conversation

lambdanis
Copy link
Contributor

@lambdanis lambdanis commented May 3, 2024

The main thing here is a general rewrite of the Metrics page.

The page is moved from the "Concepts" section to "Installation and
Configuration" - it barely touches on the concept of metrics, but it does guide
the user through the metrics configuration, so it makes more sense there.

The page covers:

  • The purpose of metrics and a link to the metrics reference
  • How to enable/disable metrics in a Kubernetes and non-Kubernetes deployments
  • How to verify that metrics are exposed
  • How to configure labels on events metrics
  • How to enable ServiceMonitor and scrape metrics

I also did a small reorg of the installation docs. The "Installation" section is renamed to "Installation and Configuration", and the FAQ page is moved there. Two CI-related FAQ questions are moved under the Contribution Guide.

To avoid breaking links I also introduced aliases: docs/concepts/metrics -> docs/installation/metrics and docs/faq -> docs/installation/faq.

@lambdanis lambdanis added area/documentation Improvements or additions to documentation release-note/docs This PR updates the documentation. labels May 3, 2024
@lambdanis lambdanis requested review from mtardy and a team as code owners May 3, 2024 16:35
@lambdanis lambdanis marked this pull request as draft May 3, 2024 16:36
@lambdanis lambdanis force-pushed the pr/lambdanis/docs-metrics branch 2 times, most recently from ae4c47d to f505272 Compare May 3, 2024 16:48
lambdanis added 2 commits May 3, 2024 18:07
The FAQ page included some installation and some CI questions. They have
different audiences, so let's split them:
* Installation questions are under "Installation and Configuration" section
* CI questions are a part of "Submitting a pull request" page

Signed-off-by: Anna Kapuscinska <[email protected]>
This is a general rewrite of the Metrics page.

The page is moved from the "Concepts" section to "Installation and
Configuration" - it barely touches on the concept of metrics, but it does guide
the use through the metrics configuration, so it makes more sense there.

The page covers:
* The purpose of metrics and a link to the metrics reference
* How to enable/disable metrics in a Kubernetes and non-Kubernetes deployments
* How to verify that metrics are exposed
* How to configure labels on events metrics
* How to enable ServiceMonitor and scrape metrics

Signed-off-by: Anna Kapuscinska <[email protected]>
@lambdanis lambdanis force-pushed the pr/lambdanis/docs-metrics branch from f505272 to 06f8058 Compare May 3, 2024 17:07
Copy link

netlify bot commented May 3, 2024

Deploy Preview for tetragon ready!

Name Link
🔨 Latest commit 06f8058
🔍 Latest deploy log https://app.netlify.com/sites/tetragon/deploys/663519e19a365e00089f7453
😎 Deploy Preview https://deploy-preview-2414--tetragon.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@lambdanis lambdanis marked this pull request as ready for review May 3, 2024 17:10
@lambdanis
Copy link
Contributor Author

The CI is failing because https://tetragon.cilium.io/docs/installation/faq/ doesn't exist yet, but it will after this PR is merged.

Copy link
Contributor

@kkourt kkourt left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@lambdanis lambdanis merged commit e435da7 into cilium:main May 10, 2024
47 of 48 checks passed
description: "List of frequently asked questions"
aliases: ["/docs/faq"]
Copy link
Member

Choose a reason for hiding this comment

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

thanks for adding that!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation Improvements or additions to documentation release-note/docs This PR updates the documentation.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants