)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"005b5aa8f551a022f67a64474ac96af8c7e94f4a","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add support of keystone unified limits from glance to use quotas"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Adding \u0027use_keystone_limits, which will allow Glance to retrieve"},{"line_number":10,"context_line":"limits set in keystone for resource consumption and enforce them"},{"line_number":11,"context_line":"against API users."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"d9b6c53b_ea3ea2f7","line":9,"range":{"start_line":9,"start_character":27,"end_line":9,"end_character":28},"updated":"2021-10-05 12:50:54.000000000","message":"\u0027 is missing.","commit_id":"dac0d59c9d0ff2d1a429c90b8809c7831dcd0ba1"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"f18e29dc491ac27662f037fbd81317a9c4623b10","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add support of keystone unified limits from glance to use quotas"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Adding \u0027use_keystone_limits, which will allow Glance to retrieve"},{"line_number":10,"context_line":"limits set in keystone for resource consumption and enforce them"},{"line_number":11,"context_line":"against API users."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"0e94a0bb_53a97c23","line":9,"range":{"start_line":9,"start_character":8,"end_line":9,"end_character":27},"updated":"2021-10-05 12:49:12.000000000","message":"nit: the closing quote mark is missing","commit_id":"dac0d59c9d0ff2d1a429c90b8809c7831dcd0ba1"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"8e69b026d80fdbb5339ab9806dfeca4f2e066b77","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add support of keystone unified limits from glance to use quotas"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Adding \u0027use_keystone_limits, which will allow Glance to retrieve"},{"line_number":10,"context_line":"limits set in keystone for resource consumption and enforce them"},{"line_number":11,"context_line":"against API users."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"c2373223_898ae0c3","line":9,"range":{"start_line":9,"start_character":8,"end_line":9,"end_character":27},"in_reply_to":"0e94a0bb_53a97c23","updated":"2021-10-08 11:20:23.000000000","message":"Done","commit_id":"dac0d59c9d0ff2d1a429c90b8809c7831dcd0ba1"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"8e69b026d80fdbb5339ab9806dfeca4f2e066b77","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add support of keystone unified limits from glance to use quotas"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Adding \u0027use_keystone_limits, which will allow Glance to retrieve"},{"line_number":10,"context_line":"limits set in keystone for resource consumption and enforce them"},{"line_number":11,"context_line":"against API users."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"b80583c6_14d24549","line":9,"range":{"start_line":9,"start_character":27,"end_line":9,"end_character":28},"in_reply_to":"d9b6c53b_ea3ea2f7","updated":"2021-10-08 11:20:23.000000000","message":"Done","commit_id":"dac0d59c9d0ff2d1a429c90b8809c7831dcd0ba1"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"499e64ab73c8853808a1a8b4bd548396467ab0dc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"8c9bcde2_347ffc05","updated":"2021-10-11 02:30:49.000000000","message":"Do you mind adding the glance::limit class depending on the new oslo::limit ? IIUC the use_keystone_limit parameter requires the [oslo_limit] parameters and it makes more sense to implement both at the same time.\n\nAlso, I think it\u0027s better to implement the parameter in that glance::limit class instead of glance::api, to gather all parameters for unified limits.","commit_id":"b8c69da610530bb83459ee42595563c6e4a4708d"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"872441567512d703583ee50b8d9ce0c1f52c88c8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"1f40cfef_cd5f4007","in_reply_to":"8c9bcde2_347ffc05","updated":"2021-10-11 05:52:54.000000000","message":"Yes sure, I will add the same, Thanks !","commit_id":"b8c69da610530bb83459ee42595563c6e4a4708d"}]}