Opened 17 years ago

Closed 17 years ago

Last modified 16 years ago

#25 closed defect (fixed)

global Names not defined

Reported by: anonymous Owned by: bastiaans
Priority: normal Milestone: 0.3
Component: jobmond Version: 0.2
Keywords: Cc: pk@…
Estimated Number of Hours:

Description

Hello,

in jobmond.py v0.2 two global variables are used but not defined. The attached patch fixes it. The first error:

Traceback (most recent call last):
  File "/usr/sbin/jobmond.py", line 816, in ?
    main()
  File "/usr/sbin/jobmond.py", line 811, in main
    gather.run()
  File "/usr/sbin/jobmond.py", line 341, in run
    self.jobs = self.getJobData( self.jobs )
  File "/usr/sbin/jobmond.py", line 530, in getJobData
    except PBSError:
NameError: global name 'PBSError' is not defined

And the second error:

Traceback (most recent call last):
  File "/usr/sbin/jobmond.py", line 819, in ?
    main()
  File "/usr/sbin/jobmond.py", line 814, in main
    gather.run()
  File "/usr/sbin/jobmond.py", line 341, in run
    self.jobs = self.getJobData( self.jobs )
  File "/usr/sbin/jobmond.py", line 532, in getJobData
    time.sleep( TORQUE_POLL_INTERVAL )
NameError: global name 'TORQUE_POLL_INTERVAL' is not defined

Regards,

Peter

Attachments (1)

jobmond.py.patch (488 bytes) - added by pk@… 17 years ago.

Download all attachments as: .zip

Change History (4)

Changed 17 years ago by pk@…

comment:1 Changed 17 years ago by bastiaans

  • Component changed from general to jobmond
  • Milestone set to 0.2.1
  • Owner changed from somebody to bastiaans
  • Status changed from new to assigned

Thanks for reporting it, must have missed this while switching to new config style.

comment:2 Changed 17 years ago by bastiaans

  • Cc pk@… added
  • Resolution set to fixed
  • Status changed from assigned to closed

The sleep on the polling interval was already removed since the trunk version.

It was a redundant statement which now just returns nothing, while jobmond skips to the next polling interval now handled by the superclass.

I have still fixed the PBSError declaration in changeset r352, thanks Peter.

Note: See TracTickets for help on using tickets.