[Supervisor-checkins] r788 - in supervisor/trunk/src/supervisor: . tests

[Chris McDonough]

Author: Chris McDonough <chrism at agendaless.com>
Date: Fri Jul 25 23:07:43 2008
New Revision: 788

Log:
Remove reloadProcess RPC interface: not yet ready for primetime.


Modified:
   supervisor/trunk/src/supervisor/rpcinterface.py
   supervisor/trunk/src/supervisor/tests/test_rpcinterfaces.py

Modified: supervisor/trunk/src/supervisor/rpcinterface.py
==============================================================================
--- supervisor/trunk/src/supervisor/rpcinterface.py	(original)
+++ supervisor/trunk/src/supervisor/rpcinterface.py	Fri Jul 25 23:07:43 2008
@@ -162,20 +162,6 @@
         self.supervisord.options.mood = SupervisorStates.RESTARTING
         return True
 
-    def reloadConfig(self):
-        """
-        Reload configuration
-
-        @return boolean result  always return True unless error
-        """
-        self._update('reloadConfig')
-        added, changed, removed = self.supervisord.options.process_config_file()
-
-        added = [group.name for group in added]
-        changed = [group.name for group in changed]
-        removed = [group.name for group in removed]
-        return [[added, changed, removed]] # cannot return len > 1, apparently
-
     def addProcess(self, name):
         """ Update the config for a running process from config file.
 

Modified: supervisor/trunk/src/supervisor/tests/test_rpcinterfaces.py
==============================================================================
--- supervisor/trunk/src/supervisor/tests/test_rpcinterfaces.py	(original)
+++ supervisor/trunk/src/supervisor/tests/test_rpcinterfaces.py	Fri Jul 25 23:07:43 2008
@@ -218,21 +218,6 @@
         self.assertEqual(value, True)
         self.assertEqual(supervisord.options.mood, 0)
 
-    def test_reloadConfig(self):
-        options = DummyOptions()
-        supervisord = DummySupervisor(options)
-        interface = self._makeOne(supervisord)
-
-        changes = [ [DummyPGroupConfig(options, 'added')],
-                    [DummyPGroupConfig(options, 'changed')],
-                    [DummyPGroupConfig(options, 'dropped')] ]
-
-        supervisord.options.process_config_file = \
-            lambda : changes
-
-        value = interface.reloadConfig()
-        self.assertEqual(value, [[['added'], ['changed'], ['dropped']]])
-
     def test_addProcess(self):
         from supervisor.supervisord import Supervisor
         from supervisor import xmlrpc