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

Add blackbox mixin #49

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from
Draft

Add blackbox mixin #49

wants to merge 5 commits into from

Conversation

janhorstmann
Copy link
Contributor

No description provided.

Monitoring-mixins ([1]) allow upstream collaboration on alerts and
grafana dashboards and convergence on monitoring best practices through
bundling mixins with the corresponding software.
Downstream customization of alerts and dashboards may be done using
jsonnet ([2]).
Usage of jsonnet-bundler ([3]) gives a clear reference of where
dashboards and alerts originated and which version is used through
`.src/jsonnetfile.json` and `.src/jsonnetfile.lock.json`.

[1]
https://monitoring.mixins.dev/

[2]
https://github.com/google/jsonnet

[3]
https://github.com/jsonnet-bundler/jsonnet-bundler

Signed-off-by: Jan Horstmann <[email protected]>
With the proposed workflow mixin output is generated offline and
checked into version control for direct usage.
A test is added to check whether the generated rules and dashboards
match the checked-in version.
Additionally a check for the script generating the mixins is added.

Signed-off-by: Jan Horstmann <[email protected]>
With the alerts of the added node-exporter mixin some alerts became
redundant and are removed.

Signed-off-by: Jan Horstmann <[email protected]>
With the alerts of the added prometheus-exporter mixin some alerts
became redundant and are removed.

Signed-off-by: Jan Horstmann <[email protected]>
Signed-off-by: Jan Horstmann <[email protected]>
@janhorstmann
Copy link
Contributor Author

Depends on #48

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.

1 participant