Skip to content
This repository has been archived by the owner on Jun 1, 2023. It is now read-only.

Fix archive dashboard query for compile time #64

Merged

Conversation

Sharpie
Copy link
Member

@Sharpie Sharpie commented Jun 19, 2019

This commit updates the "Archive Puppetserver Performance" dashboard
template with two fixes for the catalog compile time metric:

  • The mean() function is replaced with distinct() so that null
    values are properly handled.

  • The catalog-metrics measurement is an array of different
    timings of operations that make up compilation. The query
    is updated to select only compile and static_compile
    operations.

This commit updates the "Archive Puppetserver Performance" dashboard
template with two fixes for the catalog compile time metric:

  - The `mean()` function is replaced with `distinct()` so that null
    values are properly handled.

  - The `catalog-metrics` measurement is an array of different
    timings of operations that make up compilation. The query
    is updated to select only `compile` and `static_compile`
    operations.
@jarretlavallee
Copy link
Contributor

Kicking the travis job as it looks transient.

Copy link
Contributor

@jarretlavallee jarretlavallee left a comment

Choose a reason for hiding this comment

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

Looks good.

@suckatrash suckatrash merged commit cf763f5 into puppetlabs:master Jun 19, 2019
@Sharpie Sharpie deleted the fix-puppetserver-compile-query branch June 19, 2019 23:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants