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

expose report metrics section as prometheus metrics #148

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

UiP9AV6Y
Copy link

@UiP9AV6Y UiP9AV6Y commented Sep 8, 2024

this change parses additional sections from the report metrics.

while all section have a total metric, some have additional information which is also exposed by the collector. for good measure additional testdata was added representing various major puppet versions. the reports where generated using puppet apply --test against a minimal catalog. the puppet5 report was generated by a puppet installation from Ubuntu 18.04 upstream, while the others where generated from a Puppet installation from the Puppetlabs repo (also against an Ubuntu 18.04 container)

@UiP9AV6Y UiP9AV6Y force-pushed the feature_report_metrics branch from 486f065 to c6c0590 Compare September 8, 2024 18:48
this change parses additional sections from the report metrics.

while all section have a *total* metric, some have additional
information which is also exposed by the collector. for good measure
additional testdata was added representing various major puppet
versions. the reports where generated using `puppet apply --test`
against a minimal catalog. the puppet5 report was generated by
a puppet installation from Ubuntu 18.04 upstream, while the others
where generated from a Puppet installation from the Puppetlabs
repo (also against an Ubuntu 18.04 container)

Signed-off-by: Gordon Bleux <[email protected]>
@UiP9AV6Y UiP9AV6Y force-pushed the feature_report_metrics branch from c6c0590 to c741935 Compare September 9, 2024 08:40
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