[ARVADOS] updated: da39f7ee464b4be805591ff5901bdacaa005b0a8

Git user git at public.curoverse.com
Fri Apr 15 14:06:33 EDT 2016


Summary of changes:
 services/nodemanager/arvnodeman/computenode/dispatch/__init__.py | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

       via  da39f7ee464b4be805591ff5901bdacaa005b0a8 (commit)
      from  da6e931b990c7edde55213196fc8a47f4dc4998d (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit da39f7ee464b4be805591ff5901bdacaa005b0a8
Author: Peter Amstutz <peter.amstutz at curoverse.com>
Date:   Fri Apr 15 14:06:29 2016 -0400

    8953: Assign to tuple (eligible, reason)

diff --git a/services/nodemanager/arvnodeman/computenode/dispatch/__init__.py b/services/nodemanager/arvnodeman/computenode/dispatch/__init__.py
index e7ae10f..412a5d7 100644
--- a/services/nodemanager/arvnodeman/computenode/dispatch/__init__.py
+++ b/services/nodemanager/arvnodeman/computenode/dispatch/__init__.py
@@ -400,13 +400,13 @@ class ComputeNodeMonitorActor(config.actor_class):
 
     def consider_shutdown(self):
         try:
-            eligible = self.shutdown_eligible()
+            eligible, reason = self.shutdown_eligible()
             next_opening = self._shutdowns.next_opening()
-            if eligible[0]:
-                self._debug("Suggesting shutdown because %s", eligible[1])
+            if eligible:
+                self._debug("Suggesting shutdown because %s", reason)
                 _notify_subscribers(self.actor_ref.proxy(), self.subscribers)
             else:
-                self._debug("Not eligible for shut down because %s", eligible[1])
+                self._debug("Not eligible for shut down because %s", reason)
 
                 if self.last_shutdown_opening != next_opening:
                     self._debug("Shutdown window closed.  Next at %s.",

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list