[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 (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. |