Skip to content

Add documentation for using metrics in health checks#744

Open
gianlucam76 wants to merge 1 commit into
projectsveltos:mainfrom
gianlucam76:metrics
Open

Add documentation for using metrics in health checks#744
gianlucam76 wants to merge 1 commit into
projectsveltos:mainfrom
gianlucam76:metrics

Conversation

@gianlucam76
Copy link
Copy Markdown
Member

No description provided.

@gianlucam76 gianlucam76 requested a review from egrosdou01 May 29, 2026 14:11
Copy link
Copy Markdown
Contributor

@egrosdou01 egrosdou01 left a comment

Choose a reason for hiding this comment

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

Only minor changes. The context and examples look clean.

Each check can:

Lua's scripting capabilities offer flexibility in defining complex validation logic tailored to specific add-ons or applications.
- **Inspect Kubernetes resources** — fetch objects by Group/Version/Kind and evaluate them with a Lua script or CEL rules.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can we keep the same format for listing items?

-Property Name: The description

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

The problem is there is no single field for each of above. It is the entire validateHealth structure (group, version, kind) to select resources. Then the same "script" can contain Lua to evaluate either resources or metrics or both.

Comment thread docs/addons/clusterprofile.md Outdated
When the Prometheus endpoint requires a bearer token, first create the Secret:

```bash
kubectl create secret generic prometheus-token \
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Same command outline format starting with $

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants