raise Exception('Unknown action error: %s' % error.serialize())
action_id = action.tag[len('action-'):]
log.debug('Action started as %s', action_id)
- # we can't use wait_for_new here because we don't
- # consistently (ever?) get an "add" delta for the action
- return await self.model._wait('action', action_id, None)
+ # we mustn't use wait_for_action because that blocks until the
+ # action is complete, rather than just being in the model
+ return await self.model._wait_for_new('action', action_id)
def scp(
self, source_path, user=None, destination_path=None, proxy=False,
status = await c.FullStatus(None)
- return status.applications[app]['units'][self.name].get(
- 'leader', False)
+ # FullStatus may be more up to date than our model, and the
+ # unit may have gone away, or we may be doing something silly,
+ # like trying to fetch leadership for a subordinate, which
+ # will not be filed where we expect in the model. In those
+ # cases, we may simply return False, as a nonexistent or
+ # subordinate unit is not a leader.
+ if not status.applications.get(app):
+ return False
+
+ if not status.applications[app].get('units'):
+ return False
+
+ if not status.applications[app]['units'].get(self.name):
+ return False
+
+ return status.applications[app]['units'][self.name].get('leader', False)
+
+
+ async def get_metrics(self):
+ metrics = await self.model.get_metrics(self.tag)
+ return metrics[self.name]