[ZBX-3978] time period specification should not include the upper bound Created: 2011 Jul 25  Updated: 2017 May 30  Resolved: 2011 Aug 05

Status: Closed
Project: ZABBIX BUGS AND ISSUES
Component/s: Server (S)
Affects Version/s: 1.8.5
Fix Version/s: 1.8.7

Type: Incident report Priority: Major
Reporter: Aleksandrs Saveljevs Assignee: dimir
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified


 Description   

Currently, if we specify a time period "1,12:00-12:01", it includes all seconds between 12:00:00 and 12:01:00, both ends inclusive.

That means, for instance, that if we wish to have an item checked exactly once a week, we have to specify the flexible interval above with a delay of 61 seconds (not 60!), which is somewhat non-intuitive.

It is proposed that the time period "1,12:00-12:01" includes seconds between 12:00:00 and 12:00:59.



 Comments   
Comment by dimir [ 2011 Jul 25 ]

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

Comment by richlv [ 2011 Jul 25 ]

(1) currently time periods like this can be specified in working time, user media, item flexible intervals and possibly elsewhere. will this now work like that everywhere ?

<sasha> this change will affect (on server side) following units: user media, item flexible intervals and action conditions. I think its all.

<dimir> CLOSED

(2) will this work like that in all locations in the frontend ? (working time highlighting, for example)

<sasha> working time also will be fixed in this issue

<KB> RESOLVED
<sasha> REOPENED - apache halts when we get graph with period more than 1h
<sasha> CLOSED - all changes in the frontend/php directory should be reverted. Working time in graphs work fine without any changes.

(3) this should be documented

<dimir> we decided there will be one place where time period specification format will be described

and the following sections should have a link to it:

<dimir> RESOLVED

<richlv> maybe use "d" instead of "dd" for day placeholder (only one digit used there)

<dimir> I agree. RESOLVED

<richlv> fixed typo & some minor formatting issues. CLOSED

Comment by Aleksandrs Saveljevs [ 2011 Aug 05 ]

Fixed in pre-1.8.7 in r20977.

Generated at Fri Apr 26 01:43:32 EEST 2024 using Jira 9.12.4#9120004-sha1:625303b708afdb767e17cb2838290c41888e9ff0.