)]}'
{"nova/cells/rpcapi.py":[{"author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"change_message_id":"c83cf2dbb89c42db84cce866c0e2b2fd6cb52399","unresolved":false,"context_lines":[{"line_number":224,"context_line":"                         version\u003d\u00271.4\u0027)"},{"line_number":225,"context_line":""},{"line_number":226,"context_line":"    def actions_get(self, ctxt, instance):"},{"line_number":227,"context_line":"        if not instance.get(\u0027cell_name\u0027):"},{"line_number":228,"context_line":"            raise exception.InstanceUnknownCell(instance_uuid\u003dinstance[\u0027uuid\u0027])"},{"line_number":229,"context_line":"        return self.call(ctxt, self.make_msg(\u0027actions_get\u0027,"},{"line_number":230,"context_line":"                                             cell_name\u003dinstance[\u0027cell_name\u0027],"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2BnAM%3D","line":227,"updated":"2013-02-25 22:44:45.000000000","message":"Would prefer these to be instance[\u0027cell_name\u0027].... but kind of a nit.","commit_id":"9b5a53d69aa15942610b1cb6a433dd2dd7096c62"},{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"e39d72fcddf8eb8e30b755b8da519952c7a3742a","unresolved":false,"context_lines":[{"line_number":224,"context_line":"                         version\u003d\u00271.4\u0027)"},{"line_number":225,"context_line":""},{"line_number":226,"context_line":"    def actions_get(self, ctxt, instance):"},{"line_number":227,"context_line":"        if not instance.get(\u0027cell_name\u0027):"},{"line_number":228,"context_line":"            raise exception.InstanceUnknownCell(instance_uuid\u003dinstance[\u0027uuid\u0027])"},{"line_number":229,"context_line":"        return self.call(ctxt, self.make_msg(\u0027actions_get\u0027,"},{"line_number":230,"context_line":"                                             cell_name\u003dinstance[\u0027cell_name\u0027],"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2BmzM%3D","line":227,"in_reply_to":"AAAAM3%2F%2BnAM%3D","updated":"2013-02-26 00:33:16.000000000","message":"Done.  .get() is overkill since it should always exist in the model, but it makes testing with fakes slightly easier.","commit_id":"9b5a53d69aa15942610b1cb6a433dd2dd7096c62"},{"author":{"_account_id":2835,"name":"Brian Elliott","email":"bdelliott@gmail.com","username":"belliott"},"change_message_id":"ffe20e4279b53273f3394c04bc55b9ead875cf29","unresolved":false,"context_lines":[{"line_number":224,"context_line":"                         version\u003d\u00271.4\u0027)"},{"line_number":225,"context_line":""},{"line_number":226,"context_line":"    def actions_get(self, ctxt, instance):"},{"line_number":227,"context_line":"        if not instance[\u0027cell_name\u0027]:"},{"line_number":228,"context_line":"            raise exception.InstanceUnknownCell(instance_uuid\u003dinstance[\u0027uuid\u0027])"},{"line_number":229,"context_line":"        return self.call(ctxt, self.make_msg(\u0027actions_get\u0027,"},{"line_number":230,"context_line":"                                             cell_name\u003dinstance[\u0027cell_name\u0027],"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAM3%2F%2Blvo%3D","line":227,"updated":"2013-02-26 17:08:22.000000000","message":"It\u0027d be more consistent to make these checks with \u0027validate_cell\u0027 in nova/compute/cells_api","commit_id":"ab2920726c0e2633c033a31a324f30a97fdce6bd"},{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"038b96c838ec7b73d6c7ef1a08de5e91e533930b","unresolved":false,"context_lines":[{"line_number":224,"context_line":"                         version\u003d\u00271.4\u0027)"},{"line_number":225,"context_line":""},{"line_number":226,"context_line":"    def actions_get(self, ctxt, instance):"},{"line_number":227,"context_line":"        if not instance[\u0027cell_name\u0027]:"},{"line_number":228,"context_line":"            raise exception.InstanceUnknownCell(instance_uuid\u003dinstance[\u0027uuid\u0027])"},{"line_number":229,"context_line":"        return self.call(ctxt, self.make_msg(\u0027actions_get\u0027,"},{"line_number":230,"context_line":"                                             cell_name\u003dinstance[\u0027cell_name\u0027],"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAM3%2F%2Blgc%3D","line":227,"in_reply_to":"AAAAM3%2F%2Blvo%3D","updated":"2013-02-26 18:14:02.000000000","message":"I considered making this check in nova/compute/cells_api.py but went with it here because this is where cell_name is used.  I can move it over there for consistency if I need to make another change and resubmit.  I didn\u0027t notice that wrapper earlier.","commit_id":"ab2920726c0e2633c033a31a324f30a97fdce6bd"}]}