)]}'
{"glance/store/http.py":[{"author":{"_account_id":10725,"name":"Sergey Gotliv","email":"sgotliv@redhat.com","username":"sgotliv"},"change_message_id":"84612417c31b6f49452b1211b8e021a07da5d6cd","unresolved":false,"context_lines":[{"line_number":153,"context_line":"            raise exception.BadStoreUri(reason)"},{"line_number":154,"context_line":"        except Exception:"},{"line_number":155,"context_line":"            # NOTE(flaper87): Catch more granular exceptions,"},{"line_number":156,"context_line":"            # keeping this branch for backwards compatibility."},{"line_number":157,"context_line":"            return 0"},{"line_number":158,"context_line":"        return size"},{"line_number":159,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"1ae5cdf2_d96d5b34","line":156,"updated":"2014-06-01 12:02:57.000000000","message":"Please, consider to add a log printing here.","commit_id":"19de1156c7af684a22b0ecc558c5ea6ed24b7a78"},{"author":{"_account_id":10725,"name":"Sergey Gotliv","email":"sgotliv@redhat.com","username":"sgotliv"},"change_message_id":"84612417c31b6f49452b1211b8e021a07da5d6cd","unresolved":false,"context_lines":[{"line_number":173,"context_line":"        if resp.status \u003e\u003d 400:"},{"line_number":174,"context_line":"            if resp.status \u003d\u003d httplib.NOT_FOUND:"},{"line_number":175,"context_line":"                reason \u003d _(\"HTTP datastore could not find image at URI.\")"},{"line_number":176,"context_line":"                LOG.debug(reason)"},{"line_number":177,"context_line":"                raise exception.NotFound(reason)"},{"line_number":178,"context_line":"            reason \u003d _(\"HTTP URL returned a %s status code.\") % resp.status"},{"line_number":179,"context_line":"            LOG.debug(reason)"}],"source_content_type":"text/x-python","patch_set":5,"id":"1ae5cdf2_59614b38","line":176,"updated":"2014-06-01 12:02:57.000000000","message":"It should be an error not a debug.","commit_id":"19de1156c7af684a22b0ecc558c5ea6ed24b7a78"}],"test-requirements.txt":[{"author":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"change_message_id":"360c50b29df4323e02060e5102e1dd1170b1ab12","unresolved":false,"context_lines":[{"line_number":19,"context_line":"# Optional packages that should be installed when testing"},{"line_number":20,"context_line":"MySQL-python"},{"line_number":21,"context_line":"psycopg2"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"pysendfile\u003d\u003d2.0.0"},{"line_number":24,"context_line":"qpid-python"},{"line_number":25,"context_line":"xattr\u003e\u003d0.4"}],"source_content_type":"text/plain","patch_set":2,"id":"3ae8d1ca_4e760a40","line":22,"updated":"2014-05-27 16:49:07.000000000","message":"This is change id 91547, perhaps you should back port that first rather than combining them together even being really minor?","commit_id":"f6f7ef705c694e42768d3b05ef87ddcb7833fa7f"},{"author":{"_account_id":10663,"name":"s iwata","email":"s-iwata@intellilink.co.jp","username":"s.iwata"},"change_message_id":"c74fc8fb261b0d1615811e70253890862c5367d6","unresolved":false,"context_lines":[{"line_number":19,"context_line":"# Optional packages that should be installed when testing"},{"line_number":20,"context_line":"MySQL-python"},{"line_number":21,"context_line":"psycopg2"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"pysendfile\u003d\u003d2.0.0"},{"line_number":24,"context_line":"qpid-python"},{"line_number":25,"context_line":"xattr\u003e\u003d0.4"}],"source_content_type":"text/plain","patch_set":2,"id":"3ae8d1ca_20545b53","line":22,"in_reply_to":"3ae8d1ca_4e760a40","updated":"2014-05-28 02:09:23.000000000","message":"Thank you for your review !\nI may not have understood that exactly.\nSorry... what is ”change id 91547” ?\nThings I should do...separate this patch and back port ”test-requirements.txt” ?","commit_id":"f6f7ef705c694e42768d3b05ef87ddcb7833fa7f"}]}