Merge branch 'master' into 13804-no-shutdown-wanted-nodes

[arvados.git] / services / nodemanager / arvnodeman / daemon.py

diff --git a/services/nodemanager/arvnodeman/daemon.py b/services/nodemanager/arvnodeman/daemon.py
index 8ca75a270171a650f34aa2854ba59fd61b63d486..1edf4dc4792e5b7a9f638a17599c66c76410ab84 100644 (file)
--- a/services/nodemanager/arvnodeman/daemon.py
+++ b/services/nodemanager/arvnodeman/daemon.py
@@ -321,7 +321,7 @@ class NodeManagerDaemonActor(actor_class):
         busy_count = counts["busy"]
         wishlist_count = self._size_wishlist(size)
 
-        self._logger.info("%s: wishlist %i, up %i (booting %i, unpaired %i, idle %i, busy %i), down %i, shutdown %i", size.name,
+        self._logger.info("%s: wishlist %i, up %i (booting %i, unpaired %i, idle %i, busy %i), down %i, shutdown %i", size.id,
                           wishlist_count,
                           up_count,
                           counts["booting"],
@@ -341,7 +341,7 @@ class NodeManagerDaemonActor(actor_class):
             can_boot = int((self.max_total_price - total_price) / size.price)
             if can_boot == 0:
                 self._logger.info("Not booting %s (price %s) because with it would exceed max_total_price of %s (current total_price is %s)",
-                                  size.name, size.price, self.max_total_price, total_price)
+                                  size.id, size.price, self.max_total_price, total_price)
             return can_boot
         else:
             return wanted
@@ -397,7 +397,7 @@ class NodeManagerDaemonActor(actor_class):
         if not self.cancel_node_shutdown(cloud_size):
             arvados_node = self.arvados_nodes.find_stale_node(self.node_stale_after)
             self._logger.info("Want %i more %s nodes.  Booting a node.",
-                              nodes_wanted, cloud_size.name)
+                              nodes_wanted, cloud_size.id)
             new_setup = self._node_setup.start(
                 timer_actor=self._timer,
                 arvados_client=self._new_arvados(),