)]}'
{"watcher/decision_engine/model/notification/nova.py":[{"author":{"_account_id":21361,"name":"Chris Spencer","email":"christopher.m.spencer@intel.com","username":"chrisspencer"},"change_message_id":"9d8bc9eb671a892d0419880af46d6cf19629aaa8","unresolved":false,"context_lines":[{"line_number":40,"context_line":"            self._nova \u003d nova_helper.NovaHelper()"},{"line_number":41,"context_line":"        return self._nova"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    def get_or_create_instance(self, instance_uuid, node_uuid\u003dNone):"},{"line_number":44,"context_line":"        try:"},{"line_number":45,"context_line":"            if node_uuid:"},{"line_number":46,"context_line":"                self.get_or_create_node(node_uuid)"}],"source_content_type":"text/x-python","patch_set":9,"id":"5a3905b3_59911652","line":43,"range":{"start_line":43,"start_character":15,"end_line":43,"end_character":30},"updated":"2017-01-24 20:34:15.000000000","message":"should the function be renamed to include the get_or_create_node addition? possibly get_or_create_node_and_instance?","commit_id":"00bfe7eb735813d73a2cede8fdd5aed71adda777"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"6528c47bbe65b114f0c3e98de7f119d46fd4ad23","unresolved":false,"context_lines":[{"line_number":40,"context_line":"            self._nova \u003d nova_helper.NovaHelper()"},{"line_number":41,"context_line":"        return self._nova"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    def get_or_create_instance(self, instance_uuid, node_uuid\u003dNone):"},{"line_number":44,"context_line":"        try:"},{"line_number":45,"context_line":"            if node_uuid:"},{"line_number":46,"context_line":"                self.get_or_create_node(node_uuid)"}],"source_content_type":"text/x-python","patch_set":9,"id":"5a3905b3_ddf072cc","line":43,"range":{"start_line":43,"start_character":15,"end_line":43,"end_character":30},"in_reply_to":"5a3905b3_59911652","updated":"2017-01-25 11:35:30.000000000","message":"well, we could but to me this kind of implied because we know that an instance is meant to be associated to a node. This extra bit of logic is more of a guard against a lack of information that should almost never occur.\nWhat I could do however is add this precision into a dosctring to make sure everyone gets it: what do you think?","commit_id":"00bfe7eb735813d73a2cede8fdd5aed71adda777"}]}