[ZBX-6491] Macro {ITEM.VALUE} not consistent in System status and Last 20 issues. and in some other places too Created: 2013 Apr 16 Updated: 2017 May 30 Resolved: 2016 May 09 |
|
Status: | Closed |
Project: | ZABBIX BUGS AND ISSUES |
Component/s: | Frontend (F) |
Affects Version/s: | 2.0.5, 2.2.3 |
Fix Version/s: | 2.2.12rc1, 3.0.2rc1, 3.2.0alpha1 |
Type: | Incident report | Priority: | Blocker |
Reporter: | Nicola Canepa | Assignee: | Unassigned |
Resolution: | Fixed | Votes: | 2 |
Labels: | dashboard, lastvalue, macros | ||
Remaining Estimate: | Not Specified | ||
Time Spent: | Not Specified | ||
Original Estimate: | Not Specified |
Attachments: |
![]() |
||||||||
Issue Links: |
|
Description |
{ITEM.VALUE}
is expanded as {ITEM.LASTVALUE}(current value of item) in System status, and as expected (value at the time the trigger was fired) in Last 20 issues |
Comments |
Comment by Oleksii Zagorskyi [ 2014 May 19 ] |
I confirm it, see attached picture. BUT ! We see trigger name in both widgets, but we know that trigger names uses also for events name. I want to ask us all - what are shown in the widgets? events or triggers? Also, using latest value ({ITEM.LASTVALUE} logic) is better in regard of performance (in 2.2 we take it from historical tables only). So I ask to resolve it this way: zalex_ua At the end it was fixed. On the widget the value expanded as {ITEM.VALUE}. |
Comment by Marc [ 2014 May 19 ] |
Comment by Oleksii Zagorskyi [ 2014 May 20 ] |
(1) After the fix, documentation have to be reviewed. zalex_ua see (3) |
Comment by Ivo Kurzemnieks [ 2014 May 26 ] |
We had a meeting about this and it seems like a lot of work, because we need to consider all places this could affect, not only widget. And so we decided to leave it as is for now. That being said, this will could probably be done post 2.4 release unfortunately. Current behaviour will be documented so there is no confusion. |
Comment by Kodai Terashima [ 2015 Apr 08 ] |
Related to ZBXNEXT-2768 zalex_ua Discussion will be continued there. |
Comment by Oleksii Zagorskyi [ 2016 Feb 25 ] |
Let's collect in this issue other cases with the same effect. |
Comment by Oleksii Zagorskyi [ 2016 Feb 25 ] |
(2) Another bad point here is that being actually resolved like {ITEM.LASTVALUE} it's affected by ZBX_MAX_PERIOD (defines.inc.php), which is 1 day by default. b) Actual for 2.2, not for 3.0: gunarspujats RESOLVED in development branch svn://svn.zabbix.com/branches/dev/ZBX-6491 , r59171 gunarspujats Solution increased memory usage and count of SQL queries on tr_status.php page, but execution time remained the same. It correlates with configured displayed rows count in frontend - the more rows the more resources are used. sasha Successfully tested! CLOSED zalex_ua I can confirm the fix on pages: REOPENED zalex_ua Sasha explained that we have decided to not fix these 2 points here because they were actual for 2.2 only, and they are simply missing in 3.0 at all. so actually this sub issue CLOSED as won't fix. |
Comment by Alexander Vladishev [ 2016 Apr 04 ] |
Fixed in:
|
Comment by Oleksii Zagorskyi [ 2016 Apr 05 ] |
(3) Documentation Suggested changes, from Resolved to either: 1) the latest value of the Nth item in the trigger expression, if used for displaying triggers. In this case, works the same as {ITEM.LASTVALUE}. 2) the historical (at-the-time-of-event) value of the Nth item in the trigger expression, if used for displaying events and notifications. In both cases it will resolve to UNKNOWN if the history value has already been deleted or has never been stored. Supported since 1.4.3. to: Resolved to either: 1) the historical (at-the-time-of-event) value of the Nth item in the trigger expression, if used in context when trigger changes its status, for example for displaying events, sending notifications. 2) the latest value of the Nth item in the trigger expression, if used without context to trigger status, for example to display list of triggers in pop-up selection window. In other words, in this case, works the same as {ITEM.LASTVALUE}. In first case it will resolve to *UNKNOWN* if the history value has already been deleted or has never been stored. In second case, if consider frontend context only, it will resolve to *UNKNOWN* also if latest history value has been collected more than //ZBX_HISTORY_PERIOD// time ago (defined in [[:manual/web_interface/definitions|defines.inc.php]]) . Supported since 1.4.3. Suggestion for upgrade notes: When displaying trigger name, previously {ITEM.VALUE} macro was resolved like {ITEM.LASTVALUE}: - on a mouse over pop-up on "System status" widget on "Dashboard" - at Monitoring -> Triggers page Now on both these places the macro will be resolved as {ITEM.VALUE}. The details about ZBX_HISTORY_PERIOD could be added to {ITEM.LASTVALUE} macro description too. martins-v Updated documentation:
RESOLVED. zalex_ua I like it. Would be good to get final Sasha's review and then can be closed. martins-v Yes, I've added the star characters around UNKNOWN. sasha A screen element "System status" also was affected in this issue. It must be mentioned in the Upgrade notes. REOPENED sasha I updated upgrade notes. RESOLVED martins-v Reviewed, with some small changes. CLOSED sasha Thanks a lot! |
Comment by richlv [ 2016 Jul 23 ] |
somebody actually relied on the bug in |