gistfile1.txt

Provider
--------

def destroy
  managers.each { |m| m.destroy }
  super
end

Manager
-------

def destroy
  disable! if enabled?
  
  kill_ems_workers
  
  super
end

Worker
------

def kill
  Process.kill(9, pid).wait!
  
  destroy
end
  

Provider
--------

def destroy(task_id = nil)
  managers.destroy_all
  super().tap do
    if task_id
      # update_status_and_log
    end
  end
end

Manager
-------

# Remove  the dependent destroys from the leaf classes

def destroy(task_id = nil)
  disable! if enabled?

  # destroy child managers if there are any

  # kill ems workers

  super().tap do
    if task_id
      # update_status_and_log
    end
  end
end

Worker
------

def kill
  Process.kill(9, pid)

  loop do
    break unless alive?
    sleep 0.01
  end

  destroy
end

Should the provider's destroy method have the following line at the top?

disable! if enabled?

Providers don't have a disable concept today. And, we might be moving away from the provider concept as a whole. So, we probably aren't going to add it. Jason, me, Adam, and others need to have a good long discussion on the whole provider/manager concept. We just keep putting it on the back burner.