)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"74bbf41749ee80b4ae2da204b25f2982e028a641","unresolved":true,"context_lines":[{"line_number":12,"context_line":"The discovery API does not (appear to) have existing tests, so this"},{"line_number":13,"context_line":"adds a module for that, although only usage tests are added currently."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I50c98bac50f815bdb9baae024e77afd388f74554"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":14,"id":"bae7ce92_bb338357","line":15,"updated":"2022-01-31 16:19:31.000000000","message":"Implements: blueprint quota-api","commit_id":"994da10e9f7dabbc68bf4cd645134abfd2dffcea"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"dfc75597b5b7c39838519a95d48b14d70f26c232","unresolved":false,"context_lines":[{"line_number":12,"context_line":"The discovery API does not (appear to) have existing tests, so this"},{"line_number":13,"context_line":"adds a module for that, although only usage tests are added currently."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I50c98bac50f815bdb9baae024e77afd388f74554"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":14,"id":"3089d880_0364f49d","line":15,"in_reply_to":"bae7ce92_bb338357","updated":"2022-02-02 15:29:27.000000000","message":"Done","commit_id":"994da10e9f7dabbc68bf4cd645134abfd2dffcea"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"4d1030ad05bb84684d66388f9af0f593232c0c9c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"bd85207f_2d57420d","updated":"2022-01-04 04:39:59.000000000","message":"looks good for starter, some unused variables but ignoring them as this is still WIP","commit_id":"babe54c05cf566e54050e496702548924cc93175"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"74bbf41749ee80b4ae2da204b25f2982e028a641","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"740246d1_fe1726aa","updated":"2022-01-31 16:19:31.000000000","message":"Some questions inline and I think good to add release note as  well.\nFunctional test case looks solid.\n\nThank you for working on it.","commit_id":"994da10e9f7dabbc68bf4cd645134abfd2dffcea"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"70863808747387d393a2c86ab3b500c015262117","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"f22e8d73_fe8a65af","updated":"2022-02-03 09:49:35.000000000","message":"I think with each addition of new API we increase the minor version of image API, same thing is needed to done here;\nfor example;\nhttps://review.opendev.org/c/openstack/glance/+/775860","commit_id":"ddea4c172c80815409aa72e17450247d2cbce1c8"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"4c0533004ba6340494a2de8d77e1fbb950c1184b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"b00896c3_753082c7","updated":"2022-02-01 08:49:11.000000000","message":"Looks good, all my comments are addressed.\nThank you Dan!!!","commit_id":"ddea4c172c80815409aa72e17450247d2cbce1c8"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"dfc75597b5b7c39838519a95d48b14d70f26c232","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"e0ef6680_53281455","updated":"2022-02-02 15:29:27.000000000","message":"Whoops, forgot to commit these comments from before. They\u0027re all resolved, but figure I\u0027ll put them in there for completeness :)","commit_id":"ddea4c172c80815409aa72e17450247d2cbce1c8"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"d5d947225762cd2a33957057ca83e577ec4bb80a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"ce98e1f8_b4e4fa13","in_reply_to":"f22e8d73_fe8a65af","updated":"2022-02-03 14:38:11.000000000","message":"It looks like that is done once per release though, is that right? Meaning, this patch shouldn\u0027t bump the version itself...?","commit_id":"ddea4c172c80815409aa72e17450247d2cbce1c8"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"e77d629e158aa1de2b3dd89cdc10fd34bb7bf035","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"e1778ea7_a296445c","updated":"2022-02-03 17:08:35.000000000","message":"Looks good, thank you dan!!","commit_id":"f865b8cac7fb008c870e433d22fb4216ed95e285"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"c68c511c167d898d6bb46154c22b70ddfe2c03ad","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"ed21b29d_df906560","updated":"2022-02-04 15:28:03.000000000","message":"recheck OVN","commit_id":"f865b8cac7fb008c870e433d22fb4216ed95e285"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"e2838e5936e192fc2637cb06b4ef12ccabb3370b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"65c7973f_b75445d1","updated":"2022-02-03 20:03:18.000000000","message":"recheck cinder STB?","commit_id":"f865b8cac7fb008c870e433d22fb4216ed95e285"}],"api-ref/source/v2/discovery.inc":[{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"74bbf41749ee80b4ae2da204b25f2982e028a641","unresolved":true,"context_lines":[{"line_number":124,"context_line":"Response Example"},{"line_number":125,"context_line":"----------------"},{"line_number":126,"context_line":""},{"line_number":127,"context_line":".. literalinclude:: samples/usage-response.json"},{"line_number":128,"context_line":"   :language: json"}],"source_content_type":"text/x-c++src","patch_set":14,"id":"4c627b5d_02721269","line":127,"range":{"start_line":127,"start_character":20,"end_line":127,"end_character":47},"updated":"2022-01-31 16:19:31.000000000","message":"are we missing this file or it is existing one?","commit_id":"994da10e9f7dabbc68bf4cd645134abfd2dffcea"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"dfc75597b5b7c39838519a95d48b14d70f26c232","unresolved":false,"context_lines":[{"line_number":124,"context_line":"Response Example"},{"line_number":125,"context_line":"----------------"},{"line_number":126,"context_line":""},{"line_number":127,"context_line":".. literalinclude:: samples/usage-response.json"},{"line_number":128,"context_line":"   :language: json"}],"source_content_type":"text/x-c++src","patch_set":14,"id":"335c4d29_8fd3c9bc","line":127,"range":{"start_line":127,"start_character":20,"end_line":127,"end_character":47},"in_reply_to":"4c627b5d_02721269","updated":"2022-02-02 15:29:27.000000000","message":"I\u0027m not, but you are .. sorry :/","commit_id":"994da10e9f7dabbc68bf4cd645134abfd2dffcea"}],"glance/api/v2/discovery.py":[{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"74bbf41749ee80b4ae2da204b25f2982e028a641","unresolved":true,"context_lines":[{"line_number":30,"context_line":"class InfoController(object):"},{"line_number":31,"context_line":"    def __init__(self, db_api\u003dNone, policy_enforcer\u003dNone, notifier\u003dNone,"},{"line_number":32,"context_line":"                 store_api\u003dNone):"},{"line_number":33,"context_line":"        self.db_api \u003d db_api or glance.db.get_api()"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    def get_image_import(self, req):"},{"line_number":36,"context_line":"        # TODO(jokke): All the rest of the boundaries should be implemented."}],"source_content_type":"text/x-python","patch_set":14,"id":"203b73a7_581d9772","line":33,"range":{"start_line":33,"start_character":7,"end_line":33,"end_character":51},"updated":"2022-01-31 16:19:31.000000000","message":"Is this required?\nI don\u0027t see it is getting used anywhere else.","commit_id":"994da10e9f7dabbc68bf4cd645134abfd2dffcea"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"dfc75597b5b7c39838519a95d48b14d70f26c232","unresolved":false,"context_lines":[{"line_number":30,"context_line":"class InfoController(object):"},{"line_number":31,"context_line":"    def __init__(self, db_api\u003dNone, policy_enforcer\u003dNone, notifier\u003dNone,"},{"line_number":32,"context_line":"                 store_api\u003dNone):"},{"line_number":33,"context_line":"        self.db_api \u003d db_api or glance.db.get_api()"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    def get_image_import(self, req):"},{"line_number":36,"context_line":"        # TODO(jokke): All the rest of the boundaries should be implemented."}],"source_content_type":"text/x-python","patch_set":14,"id":"149f074b_1c2630d6","line":33,"range":{"start_line":33,"start_character":7,"end_line":33,"end_character":51},"in_reply_to":"203b73a7_581d9772","updated":"2022-02-02 15:29:27.000000000","message":"Ah, probably not. Likely leftover from a previous iteration of the ultrawip being copied in here.","commit_id":"994da10e9f7dabbc68bf4cd645134abfd2dffcea"}],"glance/tests/functional/v2/test_discovery.py":[{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"74bbf41749ee80b4ae2da204b25f2982e028a641","unresolved":true,"context_lines":[{"line_number":28,"context_line":"        super(TestDiscovery, self).setUp()"},{"line_number":29,"context_line":"        self.config(use_keystone_limits\u003dTrue)"},{"line_number":30,"context_line":"        self.config(filesystem_store_datadir\u003d\u0027/tmp/foo\u0027,"},{"line_number":31,"context_line":"                    group\u003d\u0027os_glance_tasks_store\u0027)"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"        self.enforcer_mock \u003d self.useFixture("},{"line_number":34,"context_line":"            fixtures.MockPatchObject(ks_quota, \u0027limit\u0027)).mock"}],"source_content_type":"text/x-python","patch_set":14,"id":"a5c17e54_f6f75ba9","line":31,"range":{"start_line":31,"start_character":27,"end_line":31,"end_character":48},"updated":"2022-01-31 16:19:31.000000000","message":"question: why do we need this and not os_glance_staging_store?","commit_id":"994da10e9f7dabbc68bf4cd645134abfd2dffcea"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"dfc75597b5b7c39838519a95d48b14d70f26c232","unresolved":false,"context_lines":[{"line_number":28,"context_line":"        super(TestDiscovery, self).setUp()"},{"line_number":29,"context_line":"        self.config(use_keystone_limits\u003dTrue)"},{"line_number":30,"context_line":"        self.config(filesystem_store_datadir\u003d\u0027/tmp/foo\u0027,"},{"line_number":31,"context_line":"                    group\u003d\u0027os_glance_tasks_store\u0027)"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"        self.enforcer_mock \u003d self.useFixture("},{"line_number":34,"context_line":"            fixtures.MockPatchObject(ks_quota, \u0027limit\u0027)).mock"}],"source_content_type":"text/x-python","patch_set":14,"id":"8051985c_eebe9950","line":31,"range":{"start_line":31,"start_character":27,"end_line":31,"end_character":48},"in_reply_to":"a5c17e54_f6f75ba9","updated":"2022-02-02 15:29:27.000000000","message":"Just copy pasta I think :)","commit_id":"994da10e9f7dabbc68bf4cd645134abfd2dffcea"}]}