13480: Ignore nodes with no tags instead of crashing.
authorLucas Di Pentima <ldipentima@veritasgenetics.com>
Wed, 6 Jun 2018 18:37:04 +0000 (15:37 -0300)
committerLucas Di Pentima <ldipentima@veritasgenetics.com>
Wed, 6 Jun 2018 18:37:04 +0000 (15:37 -0300)
Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima <ldipentima@veritasgenetics.com>

services/nodemanager/arvnodeman/computenode/driver/azure.py

index e0f260ab86542252102e28459381505833998d10..152817253589cfbe7e4f8f1870437f4e8c97c382 100644 (file)
@@ -83,7 +83,7 @@ echo %s > /var/tmp/arv-node-data/meta-data/instance-type
         # Do our own filtering based on tag.
         nodes = [node for node in
                 super(ComputeNodeDriver, self).list_nodes(ex_fetch_nic=False, ex_fetch_power_state=False)
-                if node.extra["tags"].get("arvados-class") == self.tags["arvados-class"]]
+                if node.extra.get("tags", {}).get("arvados-class") == self.tags["arvados-class"]]
         for n in nodes:
             # Need to populate Node.size
             if not n.size: