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.

Sign up for GitHub

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

Jump to bottom

Fix aggregate's link in case of a boolean value #26

Open

jrRibeiro wants to merge 1 commit into sensu:master from jrRibeiro:master

Contributor

jrRibeiro commented Apr 6, 2019 •

edited

Loading

As per Sensu's documentation - https://docs.sensu.io/sensu-core/1.7/reference/checks/#check-attributes - named aggregates were only introduced in version 0.24.

NOTE: named aggregates are new to Sensu version 0.24, now being defined with a String data type rather than a Boolean (i.e. true or false). Legacy check definitions with "aggregate": true attributes will default to using the check name as the aggregate name.

If the aggregation attribute is set to true, an aggregation is created using the check's name.

This PR fixes an issue in Uchiwa, where an incorrect link is created, pointing to an (obviously un-existing) aggregation's page called true.

The fix is simply to point to the correct aggregation by using the check's name in the link if true is found.


          Fix aggregate's link in case of a boolean value

df4660a

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