b7cd58ac2c
* puppetboard/app.py: Refactoring the report event counts. Writing the report event counts code in the node(), reports() and reports_node() functions to iterate through report.events() instead of querying the event counts endpoint for each report. This solution is heavier than the original because we have to query for all full event objects for each report and iterate through them to interpret their statuses. I originally wanted to replace the report.events() function with an events variable for the Report object but that turned out to be not technically possible presently because the report extended events' timestamps for in a different format which python can't interpret. Specifically the timezone values contain ':', there is no Python 2.x documentation that states timezones containing colons is supported. This does, however, lighten our dependency on the event-counts endpoint which is marked as experimental. Which means that it may change or be removed in a future release. This also resolves a silent bug which may or may not include environment filters on the event-counts queries. report.events() searches the Events endpoint based on the report hash which eliminates the possibility of mistaken relationships. * puppetboard/app.py: Replacing url_for_* functions with a single url_for_field() The url_for_pagination and url_for_environments functions only worked with a single, fixed request argument, 'page' and 'env' respectively. The new url_for_field() excepts 2 arguments, field: the name of the argument to update, and value: its intended value. Should consider adding a url_for_fields() function that accepts a dict argument and updates all the request arguments using a dict.update(). There is currently no requirement for it so it will remain in the backlog. * puppetboard/templates/reports.html: Adding a dropdown menu to limit the report count This fixes https://github.com/voxpupuli/puppetboard/issues/202 This new dropdown allows users to select their desired number of reports on the reports() and reports_node() pages. The available options are app.config['REPORTS_COUNT'], 25, 50, 100 or All. The default value is determined by the REPORTS_COUNT configuration value. Had to modify url_for_field() to merge the request args to the view args in order to generate the links that include the limit query string.
18 lines
1.1 KiB
HTML
18 lines
1.1 KiB
HTML
{% extends 'layout.html' %}
|
|
{% import '_macros.html' as macros %}
|
|
{% block content %}
|
|
{{ macros.reports_table(reports, reports_count, report_event_counts, condensed=False, hash_truncate=False, show_conf_col=True, show_agent_col=True, show_host_col=True, show_search_bar=True, searchable=True, current_env=current_env)}}
|
|
{{ macros.render_pagination(pagination)}}
|
|
<div class="ui dropdown" style="float:right;">
|
|
<div class="text">Limit</div>
|
|
<i class="dropdown icon"></i>
|
|
<div class="menu">
|
|
<a class="{% if limit == config.REPORTS_COUNT %}active {% endif %}item" href="{{url_for_field('limit', config.REPORTS_COUNT)}}">{{config.REPORTS_COUNT}}</a>
|
|
<a class="{% if limit == 25 %}active {% endif %}item" href="{{url_for_field('limit', 25)}}">25</a>
|
|
<a class="{% if limit == 50 %}active {% endif %}item" href="{{url_for_field('limit', 50)}}">50</a>
|
|
<a class="{% if limit == 100 %}active {% endif %}item" href="{{url_for_field('limit', 100)}}">100</a>
|
|
<a class="{% if limit == '*' %}active {% endif %}item" href="{{url_for_field('limit', '*')}}">All</a>
|
|
</div>
|
|
</div>
|
|
{% endblock content %}
|