X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/73872ccc5bb6b80a6049b44b0113085a9c2b6934..059d04053d1a7ac62c796ad5757191b9c5dd5aae:/services/nodemanager/arvnodeman/computenode/driver/azure.py diff --git a/services/nodemanager/arvnodeman/computenode/driver/azure.py b/services/nodemanager/arvnodeman/computenode/driver/azure.py index 5dfa7f54a9..ae554327ca 100644 --- a/services/nodemanager/arvnodeman/computenode/driver/azure.py +++ b/services/nodemanager/arvnodeman/computenode/driver/azure.py @@ -46,6 +46,8 @@ class ComputeNodeDriver(BaseComputeNodeDriver): def arvados_create_kwargs(self, size, arvados_node): tags = { + # Set up tag indicating the Arvados assigned Cloud Size id. + 'arvados_node_size': size.id, 'booted_at': time.strftime(ARVADOS_TIMEFMT, time.gmtime()), 'arv-ping-url': self._make_ping_url(arvados_node) } @@ -78,18 +80,12 @@ echo %s > /var/tmp/arv-node-data/meta-data/instance-type def _init_image(self, urn): return "image", self.get_image(urn) - def create_node(self, size, arvados_node): - # Set up tag indicating the Arvados assigned Cloud Size id. - self.create_kwargs.setdefault('ex_tags', {}) - self.create_kwargs['ex_tags'].update({'arvados_node_size': size.id}) - return super(ComputeNodeDriver, self).create_node(size, arvados_node) - def list_nodes(self): # Azure only supports filtering node lists by resource group. # 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: