)]}'
{"nova/image/glance.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"5248bd61c063060dcfd534c089c1fad04ffe399b","unresolved":false,"context_lines":[{"line_number":481,"context_line":"    def _add_location(self, context, image_id, location):"},{"line_number":482,"context_line":"        # \u0027show_multiple_locations\u0027 must be enabled in glance api conf file."},{"line_number":483,"context_line":"        try:"},{"line_number":484,"context_line":"            return self._client.call("},{"line_number":485,"context_line":"                context, 2, \u0027add_location\u0027, args\u003d(image_id, location, {}))"},{"line_number":486,"context_line":"        except glanceclient.exc.HTTPBadRequest:"},{"line_number":487,"context_line":"            _reraise_translated_exception()"},{"line_number":488,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_25d5fc48","line":485,"range":{"start_line":484,"start_character":12,"end_line":485,"end_character":74},"updated":"2018-11-16 15:04:17.000000000","message":"good catch.\nbased on the docs you found \n\nhttps://docs.openstack.org/python-glanceclient/latest/reference/api/glanceclient.v2.images.html#glanceclient.v2.images.Controller\n\nthe final argument is the location metadata which is required.\n\nthis also echos the code in horizon \n\nhttp://git.openstack.org/cgit/openstack/horizon/tree/openstack_dashboard/api/glance.py#n542\n\nso this all seam correct to me","commit_id":"fd540e2135c26d8c297695a3fa73d993655f0ad8"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"21b83a87bb71ffd96c522e6ded4fad47df5f7dc3","unresolved":false,"context_lines":[{"line_number":481,"context_line":"    def _add_location(self, context, image_id, location):"},{"line_number":482,"context_line":"        # \u0027show_multiple_locations\u0027 must be enabled in glance api conf file."},{"line_number":483,"context_line":"        try:"},{"line_number":484,"context_line":"            return self._client.call("},{"line_number":485,"context_line":"                context, 2, \u0027add_location\u0027, args\u003d(image_id, location, {}))"},{"line_number":486,"context_line":"        except glanceclient.exc.HTTPBadRequest:"},{"line_number":487,"context_line":"            _reraise_translated_exception()"},{"line_number":488,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_c5476879","line":485,"range":{"start_line":484,"start_character":12,"end_line":485,"end_character":74},"in_reply_to":"3f79a3b5_25d5fc48","updated":"2018-11-16 15:42:58.000000000","message":"yes, good catch. Ugh, the glance client API is amazingly inconsistent...","commit_id":"fd540e2135c26d8c297695a3fa73d993655f0ad8"}]}