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,
"""
pass
+
+ async def is_leader_from_status(self):
+ """
+ Check to see if this unit is the leader. Returns True if so, and
+ False if it is not, or if leadership does not make sense
+ (e.g., there is no leader in this application.)
+
+ This method is a kluge that calls FullStatus in the
+ ClientFacade to get its information. Once
+ https://bugs.launchpad.net/juju/+bug/1643691 is resolved, we
+ should add a simple .is_leader property, and deprecate this
+ method.
+
+ """
+ app = self.name.split("/")[0]
+
+ c = client.ClientFacade()
+ c.connect(self.model.connection)
+
+ status = await c.FullStatus(None)
+
+ try:
+ return status.applications[app]['units'][self.name].get(
+ 'leader', False)
+ except KeyError:
+ # FullStatus may be more up-to-date than the model
+ # referenced by this class. If this unit has been
+ # destroyed between the time the class was created and the
+ # time that we call this method, we'll get a KeyError. In
+ # that case, we simply return False, as a destroyed unit
+ # is not a leader.
+ return False
+
+ async def get_metrics(self):
+ metrics = await self.model.get_metrics(self.tag)
+ return metrics[self.name]