Opened 14 years ago

Last modified 9 years ago

#292 new defect

IZ1865: qconf accepts and stores invalid time format

Reported by: templedf Owned by:
Priority: lowest Milestone:
Component: sge Version: 6.0
Severity: Keywords: clients
Cc:

Description

[Imported from gridengine issuezilla http://gridengine.sunsource.net/issues/show_bug.cgi?id=1865]

        Issue #:      1865             Platform:     All      Reporter: templedf (templedf)
       Component:     gridengine          OS:        All
     Subcomponent:    clients          Version:      6.0         CC:    None defined
        Status:       NEW              Priority:     P5
      Resolution:                     Issue type:    DEFECT
                                   Target milestone: ---
      Assigned to:    roland (roland)
      QA Contact:     roland
          URL:
       * Summary:     qconf accepts and stores invalid time format
   Status whiteboard:
      Attachments:

     Issue 1865 blocks:
   Votes for issue 1865:


   Opened: Fri Nov 4 12:39:00 -0700 2005 
------------------------


For example, qconf -mconf will allow flush_time to be set to "1", even though
the format for the flush_time parameter is HH:MM:SS.  A subsequent qconf -sconf
will show that the flush_time was was indeed stored as "1".
Either qconf needs to complain about the invalid time format, or it needs to
automatically convert the numerical format into the proper time format.

   ------- Additional comments from andy Mon Nov 7 02:25:33 -0700 2005 -------
I'm voting not to change the behavior but the documented syntax (which is not
consisitnely documented by the way, e.g. for "load_report_time" nothing is
documented at all.

So my suggestion is to documetned the time interval as follows:

   [[hh:]mm:]ss

Note the this syntax and the actual intepretation allows "unusual" stuff like:

   12:61:120

which is effectively

   13:3:0

or just

   46980

Changing category to "kernel", because it's genernal parsing code used in all
components.

Suggest to close this issue and open a "doc" issue.

   ------- Additional comments from sgrell Tue Dec 6 08:12:44 -0700 2005 -------
Changed the Subcomponent.

Stephan

Change History (0)

Note: See TracTickets for help on using tickets.