[ZBX-2517] Changes on ZBX_MIN_PERIOD don't alter min graph zoom behavior Created: 2010 Jun 07  Updated: 2017 May 30  Resolved: 2016 Jan 25

Status: Closed
Project: ZABBIX BUGS AND ISSUES
Component/s: Frontend (F)
Affects Version/s: 1.8.2, 2.0.6, 2.1.0
Fix Version/s: 3.0.0alpha4

Type: Incident report Priority: Major
Reporter: Murilo Moreira de Oliveira Assignee: Unassigned
Resolution: Fixed Votes: 26
Labels: graphs, patch, timeperiodselection
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified
Environment:

CentOS 5.4 - Zabbix 1.8.2 - PostgreSQL 8.4.2


Attachments: PNG File screen-capture.png     PNG File x-axis.png     Text File zabbix-2.0.7-zoom.patch     Text File zabbix-2.2.1-zoom.patch     File zabbix-2.2.8-zoom.patch     File zabbix-2.4.2-zoom.patch     File zabbix-2.4.4-zoom.patch    
Issue Links:
Duplicate
is duplicated by ZBXNEXT-753 Make 5 minutes the lowest possible gr... Closed
is duplicated by ZBXNEXT-2764 The scale of the graphs 30 and 15 min... Closed

 Description   

As discussed on http://www.zabbix.com/forum/showthread.php?t=14745, it is desired to reduce the min zoom period of graphs in some cases (default is 1 hour). There is a variable named ZBX_MIN_PERIOD in include/defines.inc.php file that is supposed to achieve this goal, but changes in it doesn't appear to take practical effect on zabbix graphs.



 Comments   
Comment by alessandro ciancaglini [ 2011 Jan 27 ]

I've done a patch in order to make a 15min zoom level. It's working for me.
I'll try to submit.

Comment by Raymond Kuiper [ 2011 Mar 04 ]

I hacked up the graph javascript to reduce min. time to 300 seconds.
Seems to work for me like this:

http://www.zabbix.com/forum/showpost.php?p=80997&postcount=14

Maybe it should be made to use the vars from the define file?

Comment by Benoit Dubé [ 2011 Jun 30 ]

qix has proposed a modification to gtlc.js and it's working fine. See http://www.zabbix.com/forum/showthread.php?t=14745&page=2

Comment by Raymond Kuiper [ 2012 Aug 30 ]

Any update on this?

Comment by zbx user [ 2013 Sep 04 ]

+100500 on this feature

why there is no progress on it (whilst new versions are constantly coming..)

Comment by Michael H [ 2013 Dec 05 ]

I have attached zabbix-2.0.7-zoom.patch which will allow zooming down to the minute. It also adds 5m, 15m, and 30m timeframes, as well as the same values to the timeframe shifter.

It applies cleanly against 2.0.9.

Comment by zbx user [ 2013 Dec 18 ]

+100500 on this!

Comment by Michael H [ 2014 Jan 06 ]

Attached patch for 2.2.1

Comment by Cal Sawyer [ 2014 Aug 14 ]

Still works great in 2.2.5 - thank you!

+1, would be +100 if Zabbix would finally adopt this patch

Comment by Marc [ 2014 Aug 14 ]

For the time this issue isn't fixed the documentation should be corrected accordingly:
https://www.zabbix.com/documentation/2.2/manual/web_interface/definitions

Comment by richlv [ 2014 Aug 16 ]

while i'm all for documenting everything possible, documenting bugs will just make some lazy ass dev go "works as intended" later and no bugs will be fixed, ever

Comment by Guido Jäkel [ 2014 Nov 22 ]

I have attached a bumped version of the patch for Zabbix 2.4.2, based on zabbix-2.2.1-zoom.patch uploaded by Michael H.

Comment by OFN Team [ 2015 Feb 11 ]

Is a chance to add this patch to official zabbix tree?

Comment by Tomasz Kłoczko [ 2015 Feb 11 ]

Uploaded fixed version for zabbix 2.2.8

Comment by Tomasz Kłoczko [ 2015 Feb 24 ]

Uploaded updated patch for zabbix 2.4.4.

Comment by makemeasandwich [ 2015 Sep 04 ]

Hello guys,

I'm using the zoom patch and it works fine. Is there a way to display the hour with the seconds? It is only displaying in Minutes steps. Like 10:21 10:22 10:23. Would be epic if you can show it in like 10:21:01, 10:21:02 ; 10:21:03 like in the latest value field. If i zoom in into 1 minute I cannot see the time in seconds but the graph is moving forward.

I would be really cool if there is a way to see the time in seconds as i am polling every second.

Thanks for the help =)

Comment by Ivan Fernandez [ 2015 Sep 07 ]

The last pach 2.4.4 work in 2.4.5 and 2.4.6 versions? I need this version for debian jessie.

Thx

Comment by makemeasandwich [ 2015 Sep 08 ]

@Ivan Fernandez. It works for 2.4.5. as I am using it

Comment by Vladimir [ 2015 Oct 01 ]

In zabbix 2.4.6 this patch is not work.
Scale is possible, but graph is not draw.

Comment by Alexei Vladishev [ 2015 Oct 05 ]

Fixed in dev branch svn://svn.zabbix.com/branches/dev/ZBX-2517

Comment by Ivo Kurzemnieks [ 2015 Oct 06 ]

(1) Translation string changes?

Strings removed:

  • Minimum time period to display is %1$s hour.
  • Minimum time period to display is %1$s hours.

Strings added:

  • Minimum time period to display is %1$s minute.
  • Minimum time period to display is %1$s minutes.

iivs CLOSED

Comment by Ivo Kurzemnieks [ 2015 Oct 06 ]

(2) Confusing message when time period is invalid. For example in events, type "&period=59" in URL. The message is green and is says: "Minimum time period to display is 0.016666666666667 hours." Same can be achieved for screens. Different function, but same message there. I think it is better to display error message and the message could be "Minimum time period to display is 1 minute.".

alexei RESOLVED in r56076.

iivs Thanks! Looks better. Minor coding style fix in r56178, please review.

sasha CLOSED

Comment by Ivo Kurzemnieks [ 2015 Oct 06 ]

Problems with graph widths and grid can be addressed to ZBXNEXT-1762

Comment by Ivo Kurzemnieks [ 2015 Oct 06 ]

(3) Fixed several coding style issues in r55949. Please, review.

alexei CLOSED

Comment by Guido Jäkel [ 2015 Oct 06 ]

Please also add a sufficient number of convenient entries to the appendZoomLinks and appendNavLinks functions. I'm currently use the set 5min,15min, 30min, 1h, 2h, 3h, 6h, 1d, 2d, 4d, 7d, 14d, 1m, 3m, 6m, 1y, All and maybe one should even add 1min ...

zabbix-2.4.4/frontends/php/js/gtlc.js

        appendZoomLinks: function() {
[...]
               var zooms = [300, 900, 1800, 3600, 7200, 10800, 21600, 43200, 86400, 172800, 345600, 604800, 1209600, 2592000, 7776000, 15552000, 31536000];
[...]

        appendNavLinks: function() {
[...]
               var moves = [300, 900, 1800, 3600, 7200, 10800, 21600, 43200, 86400, 172800, 345600, 604800, 2592000, 15552000, 31536000];
[...]
Comment by Ivo Kurzemnieks [ 2015 Oct 15 ]

TESTED,

but please close (2) before merging.

Comment by Alexander Vladishev [ 2015 Oct 21 ]

Available in pre-3.0.0alpha4 r56295.

Comment by Alexander Vladishev [ 2015 Oct 21 ]

(4) Documentation

martins-v Updated sections:

RESOLVED

asaveljevs Wonderful! CLOSED.

Comment by Aleksandrs Saveljevs [ 2015 Oct 28 ]

(5) That does not look very good:

oleg.egorov Will be fixed under ZBXNEXT-1762
CLOSED

Comment by Aleksandrs Saveljevs [ 2015 Oct 28 ]

(6) Interval selection seems to be broken in screens, at least in Firefox. The mouse was clicked at the red dot, but it started drawing the interval where the yellow is:

asaveljevs This issue seems to have existed before this task was merged. Moved to ZBX-10019. CLOSED.

Comment by richlv [ 2015 Oct 30 ]

(7) changelog entry does not follow the guidelines (does not start with a verb in the past tense)

asaveljevs RESOLVED in r57961 and r57963.

sasha CLOSED

Generated at Sat Apr 27 05:41:25 EEST 2024 using Jira 9.12.4#9120004-sha1:625303b708afdb767e17cb2838290c41888e9ff0.