Changeset 146


Ignore:
Timestamp:
06/20/05 09:08:38 (17 years ago)
Author:
bastiaans
Message:

daemon/togad.py:

  • Fixed: rrdtool pipes were never closed, cuasing memory hogs and zombie processes
File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/daemon/togad.py

    r136 r146  
    317317                debug_msg( 8, self.binary + ' ' + action + ' ' + filename + ' ' + arg_string  )
    318318
    319                 for line in os.popen( self.binary + ' ' + action + ' ' + filename + ' ' + arg_string ).readlines():
     319                cmd = os.popen( self.binary + ' ' + action + ' ' + filename + ' ' + arg_string )
     320                lines = cmd.readlines()
     321                cmd.close()
     322
     323                for line in lines:
    320324
    321325                        if line.find( 'ERROR' ) != -1:
     
    11251129                                for period, pmetric in metric_serial_table.items():
    11261130
    1127                                         self.createCheck( hostname, metricname, period )       
     1131                                        create_ret = self.createCheck( hostname, metricname, period )   
    11281132
    11291133                                        update_ret = self.update( hostname, metricname, period, pmetric )
     
    11351139                                                debug_msg( 9, 'metric update failed' )
    11361140
     1141                                        update_rets.append( create_ret )
    11371142                                        update_rets.append( update_ret )
    11381143
Note: See TracChangeset for help on using the changeset viewer.