Opened 11 years ago

Last modified 4 years ago

#596 new defect

IZ2788: qconf -Aq $FILE prints wrong error if "hostname" list is empty

Reported by: pollinger Owned by:
Priority: lowest Milestone:
Component: sge Version: 6.2
Severity: minor Keywords: clients
Cc:

Description

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

        Issue #:      2788             Platform:     All      Reporter: pollinger (pollinger)
       Component:     gridengine          OS:        All
     Subcomponent:    clients          Version:      6.2         CC:    None defined
        Status:       NEW              Priority:     P5
      Resolution:                     Issue type:    DEFECT
                                   Target milestone: ---
      Assigned to:    roland (roland)
      QA Contact:     roland
          URL:
       * Summary:     qconf -Aq $FILE prints wrong error if "hostname" list is empty
   Status whiteboard:
      Attachments:

     Issue 2788 blocks:
   Votes for issue 2788:


   Opened: Thu Nov 13 03:32:00 -0700 2008 
------------------------


"qconf -Aq broken_queue.txt" (see definition file below) prints the error message:
'error: " " is the only character allowed between the attribute name and the
value in line 18'

But the error is not in line 18, it's the empty value for key "hostlist" a few
lines above. So the error should be "hostlist must not be empty, set it to NONE
if you don't want to add hosts to the queue.", or the parser could interpret an
empty hostlist as "NONE".

This issue might apply for all other keys that allow "NONE" as a value, too.

broken_queue.txt
----------------
s_rt INFINITY
s_fsize INFINITY
qtype BATCH INTERACTIVE
slots 100
load_thresholds np_load_avg=8.00
notify 00:00:60
priority 0
initial_state default
hostlist
complex_values NONE
terminate_method NONE
h_data INFINITY
s_cpu INFINITY
s_data INFINITY
seq_no 0
tmpdir /tmp/testsuite_8104
epilog NONE
h_core INFINITY
suspend_method NONE
processors UNDEFINED
s_core INFINITY
h_stack INFINITY
pe_list make
shell_start_mode posix_compliant
xprojects NONE
subordinate_list NONE
h_vmem INFINITY
xuser_lists NONE
min_cpu_interval 00:05:00
h_cpu INFINITY
s_vmem INFINITY
shell /bin/csh
h_fsize INFINITY
resume_method NONE
projects NONE
calendar NONE
s_rss INFINITY
rerun TRUE
user_lists NONE
h_rt INFINITY
prolog NONE
starter_method NONE
owner_list NONE
nsuspend 1
s_stack INFINITY
suspend_thresholds NONE
qname re_true.q
ckpt_list NONE
suspend_interval 00:05:00
h_rss INFINITY

Change History (1)

comment:1 Changed 4 years ago by wish

  • Severity set to minor

Just seen this misleading error. Apparently caused by an empty pe_list on line 13 reported
as line 26.

Note: See TracTickets for help on using tickets.