Ad Widget

Collapse

PATCH : Allow item delay of 86400 (24 HOURS)

Collapse
X
 
  • Time
  • Show
Clear All
new posts
  • SAT QPass
    Member
    • Oct 2005
    • 61

    #1

    PATCH : Allow item delay of 86400 (24 HOURS)

    I think that item delay should be allowed to be set as high as 86400 (24 hours), it is currently capped at 65535. This requires only a change to the items.php file, no other modification is required.

    Code:
    --- items.php   2006-02-07 05:04:32.000000000 -0800
    +++ items_delay_86400.php       2006-02-14 14:32:50.000000000 -0800
    @@ -52,7 +52,7 @@
                    "description"=> array(T_ZBX_STR, O_OPT,  NULL,  NOT_EMPTY,'isset({save})'),
                    "key"=>         array(T_ZBX_STR, O_OPT,  NULL,  NOT_EMPTY,'isset({save})'),
                    "host"=>        array(T_ZBX_STR, O_OPT,  NULL,  NOT_EMPTY,'isset({save})'),
    -               "delay"=>       array(T_ZBX_INT, O_OPT,  NULL,  BETWEEN(0,65535),'isset({save})&&{type}!=2'),
    +               "delay"=>       array(T_ZBX_INT, O_OPT,  NULL,  BETWEEN(0,86400),'isset({save})&&{type}!=2'),
                    "history"=>     array(T_ZBX_INT, O_OPT,  NULL,  BETWEEN(0,65535),'isset({save})'),
                    "status"=>      array(T_ZBX_INT, O_OPT,  NULL,  BETWEEN(0,65535),'isset({save})'),
                    "type"=>        array(T_ZBX_INT, O_OPT,  NULL,  IN("0,1,2,3,4,5,6,7"),'isset({save})')
    Attached Files
  • dcrandall
    Member
    • Apr 2006
    • 59

    #2
    Do we know if a full 24 hours will be allowed in the stable version 1.1? I'm using 1.1beta9 and it will not let me set the delay to 24 hours.
    For things like total partition space, which is unlikely to change unexpectedly it really make sence to be able to set the delay very high. The important value is how much of that space has been used.
    Really the main reason for gathering vfs.fs.size[,total] is so that you can display the amount used against the total in the graphs.

    Comment

    • Alexei
      Founder, CEO
      Zabbix Certified Trainer
      Zabbix Certified SpecialistZabbix Certified Professional
      • Sep 2004
      • 5654

      #3
      Yes, this makes sense. The patch has been applied.
      Alexei Vladishev
      Creator of Zabbix, Product manager
      New York | Tokyo | Riga
      My Twitter

      Comment

      Working...