)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"c29d95930dd18b226433855210fbddb966dcd589","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Default policy for quota details was an admin only privilege"},{"line_number":10,"context_line":"but it was able to get used and reserved resource data via"},{"line_number":11,"context_line":"nova limits call even for non admin owners."},{"line_number":12,"context_line":"Horizon project used such ability instead of getting quota"},{"line_number":13,"context_line":"details it made both calls."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"This patch sets quotas detail default policy to be"},{"line_number":16,"context_line":"quota details available for project owner."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"1f1a1f67_2a41b9a9","line":13,"range":{"start_line":12,"start_character":0,"end_line":13,"end_character":27},"updated":"2017-07-19 18:14:28.000000000","message":"Does this mean that Horizon worked around the issue by using the limits API instead of the os-quota-sets API? And this removes the need to workaround it and just use the single os-quota-sets API? If that\u0027s correct, it would be nice to reword this for clarity.","commit_id":"8b0a5f8fbc77619a73e20adb62f97e2eefcf0b8f"},{"author":{"_account_id":21813,"name":"Andrey Volkov","email":"m@amadev.ru","username":"avolkov"},"change_message_id":"56082f3c92671f236ea83d3b2431916c7debff73","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Default policy for quota details was an admin only privilege"},{"line_number":10,"context_line":"but it was able to get used and reserved resource data via"},{"line_number":11,"context_line":"nova limits call even for non admin owners."},{"line_number":12,"context_line":"Horizon project used such ability instead of getting quota"},{"line_number":13,"context_line":"details it made both calls."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"This patch sets quotas detail default policy to be"},{"line_number":16,"context_line":"quota details available for project owner."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"1f1a1f67_10f5f583","line":13,"range":{"start_line":12,"start_character":0,"end_line":13,"end_character":27},"in_reply_to":"1f1a1f67_2a41b9a9","updated":"2017-07-20 11:20:48.000000000","message":"Yes, updated.","commit_id":"8b0a5f8fbc77619a73e20adb62f97e2eefcf0b8f"}],"nova/policies/quota_sets.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"500659741109a6a0e5b1bc350ba7b8736f35df8a","unresolved":false,"context_lines":[{"line_number":28,"context_line":"    policy.RuleDefault("},{"line_number":29,"context_line":"        name\u003dPOLICY_ROOT % \u0027defaults\u0027,"},{"line_number":30,"context_line":"        check_str\u003dbase.RULE_ANY),"},{"line_number":31,"context_line":"    policy.RuleDefault("},{"line_number":32,"context_line":"        name\u003dPOLICY_ROOT % \u0027show\u0027,"},{"line_number":33,"context_line":"        check_str\u003dbase.RULE_ADMIN_OR_OWNER),"},{"line_number":34,"context_line":"    policy.RuleDefault("},{"line_number":35,"context_line":"        name\u003dPOLICY_ROOT % \u0027delete\u0027,"},{"line_number":36,"context_line":"        check_str\u003dbase.RULE_ADMIN_API),"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa6399be_092a0790","line":33,"range":{"start_line":31,"start_character":4,"end_line":33,"end_character":44},"updated":"2016-10-14 21:16:10.000000000","message":"I guess since we already allow show it makes sense to have it for detail as well. Is there even any difference between these two policies and how they are used in the code?","commit_id":"9a3e620766712093253c6bd8f427c29ef9437f70"}],"releasenotes/notes/quota-show-detail-access-d6f37282d288fa33.yaml":[{"author":{"_account_id":6849,"name":"Roman Podoliaka","email":"roman.podoliaka@gmail.com","username":"rpodolyaka"},"change_message_id":"9561b01634d276f2ca00754963262c29556480c2","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"other:"},{"line_number":3,"context_line":"  - Details for quotas is available now not only for admins"},{"line_number":4,"context_line":"    but for project owners also."}],"source_content_type":"text/x-yaml","patch_set":4,"id":"ba5da102_47f59881","line":3,"updated":"2016-10-26 09:46:30.000000000","message":"Maybe:\n\n  The default policy has been changed to permit listing of quotas with  details to project users, not only to admins","commit_id":"f4a6cc1317589d866503c7ac3f3c1025c9750695"},{"author":{"_account_id":21813,"name":"Andrey Volkov","email":"m@amadev.ru","username":"avolkov"},"change_message_id":"6bb7ef9bf9298a448f252315ee663f35c3c6bf34","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"other:"},{"line_number":3,"context_line":"  - Details for quotas is available now not only for admins"},{"line_number":4,"context_line":"    but for project owners also."}],"source_content_type":"text/x-yaml","patch_set":4,"id":"ba5da102_6d1a7d19","line":3,"in_reply_to":"ba5da102_47f59881","updated":"2016-10-26 10:36:53.000000000","message":"Yes, thanks. Fixed.","commit_id":"f4a6cc1317589d866503c7ac3f3c1025c9750695"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"37f3fc0b2ace18780aeb2d09121c39a5a563b033","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"other:"},{"line_number":3,"context_line":"  - The default policy has been changed to permit listing of quotas"},{"line_number":4,"context_line":"    with details to project users, not only to admins."}],"source_content_type":"text/x-yaml","patch_set":9,"id":"1f1a1f67_6a58d1f8","line":4,"range":{"start_line":3,"start_character":4,"end_line":4,"end_character":54},"updated":"2017-07-19 18:05:29.000000000","message":"Can we put the specific policy rules in here for context so people don\u0027t have to hunt through the code to know what this is talking about?","commit_id":"8b0a5f8fbc77619a73e20adb62f97e2eefcf0b8f"},{"author":{"_account_id":21813,"name":"Andrey Volkov","email":"m@amadev.ru","username":"avolkov"},"change_message_id":"56082f3c92671f236ea83d3b2431916c7debff73","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"other:"},{"line_number":3,"context_line":"  - The default policy has been changed to permit listing of quotas"},{"line_number":4,"context_line":"    with details to project users, not only to admins."}],"source_content_type":"text/x-yaml","patch_set":9,"id":"1f1a1f67_90090597","line":4,"range":{"start_line":3,"start_character":4,"end_line":4,"end_character":54},"in_reply_to":"1f1a1f67_6a58d1f8","updated":"2017-07-20 11:20:48.000000000","message":"Done","commit_id":"8b0a5f8fbc77619a73e20adb62f97e2eefcf0b8f"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"f083a627846e4b7e8a64ba1119fa46078eaca5c2","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"other:"},{"line_number":3,"context_line":"  - The default policy for os_compute_api:os-quota-sets:detail has"},{"line_number":4,"context_line":"    been changed to permit listing of quotas with details to project"},{"line_number":5,"context_line":"    users, not only to admins."}],"source_content_type":"text/x-yaml","patch_set":10,"id":"1f1a1f67_3bc53f8f","line":2,"range":{"start_line":2,"start_character":0,"end_line":2,"end_character":5},"updated":"2017-07-21 15:06:46.000000000","message":"I hadn\u0027t noticed this was \"other\" earlier. It should be \"upgrade\" since we\u0027re changing default policy for an existing rule, so we want to bring attention to the change as a concern during an upgrade. I\u0027ll change it and re +2.","commit_id":"066f5438b6aad33d6b95866f59b4dba08834e0b1"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"59cc00be63ff1bdf5303663cc463ee0f8596e1d3","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"other:"},{"line_number":3,"context_line":"  - The default policy for os_compute_api:os-quota-sets:detail has"},{"line_number":4,"context_line":"    been changed to permit listing of quotas with details to project"},{"line_number":5,"context_line":"    users, not only to admins."}],"source_content_type":"text/x-yaml","patch_set":10,"id":"1f1a1f67_3b73ff82","line":2,"range":{"start_line":2,"start_character":0,"end_line":2,"end_character":5},"updated":"2017-07-21 15:07:40.000000000","message":"This should probably be \u0027upgrade\u0027. Changing the default value for an existing rule is an upgrade concern. If it were a new rule, then \u0027other\u0027 would be OK.","commit_id":"066f5438b6aad33d6b95866f59b4dba08834e0b1"}]}