so close ...in 1058077_patch3.diff you have:
+ def delete_invalid_token(self, ctxt, instance_uuid): + topic = self.make_msg('delete_invalid_token', + instance_uuid=instance_uuid) + return self.call(ctxt, topic, version='1.1')
I don't think topic makes sense as the variable name, because it's the message being sent, not the topic. If you change the variable name to something else (msg works for me), then I'm +2 on it.
so close ...in 1058077_patch3.diff you have:
+ def delete_ invalid_ token(self, ctxt, instance_uuid): msg('delete_ invalid_ token', uuid=instance_ uuid)
+ topic = self.make_
+ instance_
+ return self.call(ctxt, topic, version='1.1')
I don't think topic makes sense as the variable name, because it's the message being sent, not the topic. If you change the variable name to something else (msg works for me), then I'm +2 on it.