[checkmk-commits] 3440 FIX Fixed possible BI exception "Unknown BI aggregation group ..."

Module: check_mk Branch: master Commit: a17e41f76e14d8d0dc25cda9b36b86aaf8b52e7b URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=a17e41f76e14d8… Author: Lars Michelsen <lm(a)mathias-kettner.de> Date: Wed Apr 27 11:23:48 2016 +0200 3440 FIX Fixed possible BI exception "Unknown BI aggregation group ..." An exception has been fixed which could occur when using "multi host" aggregations together with the "precompile on demand" feature. It could happen that a BI ajax fold/unfold request was the first BI request in an apache process which had not yet initialized the BI cache yet. --- .werks/3440 | 13 +++++++++++++ ChangeLog | 1 + web/htdocs/bi.py | 7 +------ 3 files changed, 15 insertions(+), 6 deletions(-) diff --git a/.werks/3440 b/.werks/3440 new file mode 100644 index 0000000..f8ce205 --- /dev/null +++ b/.werks/3440 @@ -0,0 +1,13 @@ +Title: Fixed possible BI exception "Unknown BI aggregation group ..." +Level: 1 +Component: bi +Class: fix +Compatible: compat +State: unknown +Version: 1.2.9i1 +Date: 1461748936 + +An exception has been fixed which could occur when using "multi host" aggregations +together with the "precompile on demand" feature. It could happen that a BI ajax +fold/unfold request was the first BI request in an apache process which had not +yet initialized the BI cache yet. diff --git a/ChangeLog b/ChangeLog index f1c3f66..8852cec 100644 --- a/ChangeLog +++ b/ChangeLog @@ -333,6 +333,7 @@ * 3291 FIX: Improved error handling when searching for BI aggregations with invalid regex * 3358 FIX: Fixed BI Boxes views when "Enable BI compilation diagnostics" enabled * 3439 FIX: Fixed processing of BI aggregations containing umlauts + * 3440 FIX: Fixed possible BI exception "Unknown BI aggregation group ..."... Reporting & Availability: * 3267 New availability option for reclassifying host state "unreachable"... diff --git a/web/htdocs/bi.py b/web/htdocs/bi.py index 546e12b..b5d8c34 100644 --- a/web/htdocs/bi.py +++ b/web/htdocs/bi.py @@ -303,12 +303,7 @@ def compile_forest(user, only_hosts = None, only_groups = None): num_new_host_aggrs = 0 num_new_multi_aggrs = 0 - # When only_hosts is given only use the single host aggregations for further processing. - # The only_hosts variable is only populated for single host tables. - if only_hosts: - aggr_list = [(AGGR_HOST, config.host_aggregations)] - else: - aggr_list = [(AGGR_MULTI, config.aggregations), (AGGR_HOST, config.host_aggregations)] + aggr_list = [(AGGR_MULTI, config.aggregations), (AGGR_HOST, config.host_aggregations)] single_affected_hosts = [] for aggr_type, aggregations in aggr_list: