Project

General

Profile

Bug #3901

Need better error reporting when reconfiging the factory with wrong schedd_name in the entry

Added by Parag Mhashilkar over 7 years ago. Updated over 7 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Parag Mhashilkar
Category:
-
Target version:
Start date:
05/16/2013
Due date:
% Done:

0%

Estimated time:
First Occurred:
Occurs In:
Stakeholders:
Duration:

Description

sl5 glideinwms-factory-2.7.1-1.0
  • I copied and pasted an entry point element into my fermicloud320 rpm install
    from another factory I had. It referenced the wrong schedd_name.
    ... entry name="ress_ITB_INSTALL_TEST_2"... schedd_name=" cms-xen21.fnal.gov "
    a reconfig resulted in this stacktrace
    Traceback (most recent call last):
      File "/usr/sbin/reconfig_glidein", line 218, in ?
        main(params, old_params, update_scripts, update_def_cfg)
      File "/usr/sbin/reconfig_glidein", line 46, in main
        glidein_dicts_obj.populate()
      File "/usr/lib/python2.4/site-packages/glideinwms/creation/lib/cgWParamDict.py", line 430, in populate
        self.local_populate(params)
      File "/usr/lib/python2.4/site-packages/glideinwms/creation/lib/cgWParamDict.py", line 459, in local_populate
        global_schedd_count[params.entries[sub_name].schedd_name]+=1
    KeyError: u'cms-xen21.fnal.gov'
    

John Weigand

See #3841 for details.


Related issues

Related to GlideinWMS - Bug #3841: V2.7.1 RPM testingClosed05/09/2013

History

#1 Updated by Parag Mhashilkar over 7 years ago

  • Status changed from New to Feedback
  • Assignee set to Burt Holzman
  • Target version changed from v2_7_x to v2_7_2

Fixed in branch_v2plus_3901: commit:70ca919

#2 Updated by Burt Holzman over 7 years ago

Looks ok to merge (after I reverted the whitespace changes).

But this also needs to be added to master.

#3 Updated by Parag Mhashilkar over 7 years ago

  • Assignee changed from Burt Holzman to Parag Mhashilkar

#4 Updated by Burt Holzman over 7 years ago

  • Status changed from Feedback to Closed

Merged in v2plus and master

Also available in: Atom PDF