)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"db6ed977be6ed1902b584e2686c2af35470e43cd","unresolved":false,"context_lines":[{"line_number":10,"context_line":"\u0027force\u0027 parameter was missing from the \u0027PUT /os-quota-sets/{tenant_id}\u0027"},{"line_number":11,"context_line":"API request schema so users could not force quota updates with"},{"line_number":12,"context_line":"microversion 2.36 or later. The bug is now fixed so that the \u0027force\u0027"},{"line_number":13,"context_line":"parameter can once again be specified during quota updates. There is"},{"line_number":14,"context_line":"no new microversion for this change since it is an admin-only API."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I947e927802f755ccb25a91efd82cac895779d19e"},{"line_number":17,"context_line":"Closes-Bug: #1733886"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ff82abbf_fbb088df","line":14,"range":{"start_line":13,"start_character":60,"end_line":14,"end_character":66},"updated":"2017-11-28 18:34:52.000000000","message":"I am not sure this is a valid reason to avoid a new microversion bump. We had similar discussions with Ironic team for considering when we need to bump a microversion.\n\nThey wanted to avoid bumping microversions if changes are backwards compatible but that is different from the principle of microversions of Nova. We said we could handle it as Ironic specific because most Ironic APIs are for admin users. However, the conclusion was Ironic follows the same way of Nova without any exceptions. So admin-only API could not be the reason to avoid micorversion bump.\n\nIn addition, the rule[1] doesn\u0027t contain exception of admin-only API.\n\n[1]: https://github.com/openstack/nova/blob/master/doc/source/contributor/microversions.rst#when-do-i-need-a-new-microversion","commit_id":"e31dd087fb090d0db70ad5dbf4e92eed416fc4ed"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"c99ad0ea4c554bd5bff02d5913c0eca2c07acfee","unresolved":false,"context_lines":[{"line_number":10,"context_line":"\u0027force\u0027 parameter was missing from the \u0027PUT /os-quota-sets/{tenant_id}\u0027"},{"line_number":11,"context_line":"API request schema so users could not force quota updates with"},{"line_number":12,"context_line":"microversion 2.36 or later. The bug is now fixed so that the \u0027force\u0027"},{"line_number":13,"context_line":"parameter can once again be specified during quota updates. There is"},{"line_number":14,"context_line":"no new microversion for this change since it is an admin-only API."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I947e927802f755ccb25a91efd82cac895779d19e"},{"line_number":17,"context_line":"Closes-Bug: #1733886"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ff82abbf_bede6b3b","line":14,"range":{"start_line":13,"start_character":60,"end_line":14,"end_character":66},"in_reply_to":"ff82abbf_1b01a9af","updated":"2017-11-28 20:59:02.000000000","message":"From this case, we could add an exception of microversion bump to [1] like\n\n - the target is admin-only API\n - Stable branch which the bug happened is still maintained at upstream\n\nI can see the point, but I still am not sure this is valid way.\nI just want to see another feedback.","commit_id":"e31dd087fb090d0db70ad5dbf4e92eed416fc4ed"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"9189b98e20b40b26466b9d20a0a1c700930917ea","unresolved":false,"context_lines":[{"line_number":10,"context_line":"\u0027force\u0027 parameter was missing from the \u0027PUT /os-quota-sets/{tenant_id}\u0027"},{"line_number":11,"context_line":"API request schema so users could not force quota updates with"},{"line_number":12,"context_line":"microversion 2.36 or later. The bug is now fixed so that the \u0027force\u0027"},{"line_number":13,"context_line":"parameter can once again be specified during quota updates. There is"},{"line_number":14,"context_line":"no new microversion for this change since it is an admin-only API."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I947e927802f755ccb25a91efd82cac895779d19e"},{"line_number":17,"context_line":"Closes-Bug: #1733886"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ff82abbf_6b6538e1","line":14,"range":{"start_line":13,"start_character":60,"end_line":14,"end_character":66},"in_reply_to":"ff82abbf_8af1b697","updated":"2017-12-01 03:52:10.000000000","message":"yea, its hard to follow microversion in this case. I am also ok to fix this without microversion as we are going to backport this till newton. That is suitable as per engineering decision. \n\nIMO, \"admin API criteria\" only is not convincing to avoid microverion as microversion is not just for interop but for other reason also like change (bug fix etc) publishing. \nBut if it is bug fix and backporting fix till all stable branch makes it suitable to avoid microversion bump.\n\nas kenichi mentioned, we should put that criteria in microversion doc also. Which should be very clear about bug fix only not any changes on admin api is ok without microversion.","commit_id":"e31dd087fb090d0db70ad5dbf4e92eed416fc4ed"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"b4470237546f80cd3913ac2be44ecc5c9e4c9461","unresolved":false,"context_lines":[{"line_number":10,"context_line":"\u0027force\u0027 parameter was missing from the \u0027PUT /os-quota-sets/{tenant_id}\u0027"},{"line_number":11,"context_line":"API request schema so users could not force quota updates with"},{"line_number":12,"context_line":"microversion 2.36 or later. The bug is now fixed so that the \u0027force\u0027"},{"line_number":13,"context_line":"parameter can once again be specified during quota updates. There is"},{"line_number":14,"context_line":"no new microversion for this change since it is an admin-only API."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I947e927802f755ccb25a91efd82cac895779d19e"},{"line_number":17,"context_line":"Closes-Bug: #1733886"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ff82abbf_8af1b697","line":14,"range":{"start_line":13,"start_character":60,"end_line":14,"end_character":66},"in_reply_to":"ff82abbf_bede6b3b","updated":"2017-11-30 06:38:25.000000000","message":"I remember we fix the server name\u0027s regex in the past without microversion. I remember the reason is that really a basic function, we must fix it. Just like we break the server create API, without fix it the nova isn\u0027t usable. At that case, we also can fix it without microversion.","commit_id":"e31dd087fb090d0db70ad5dbf4e92eed416fc4ed"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"22214313b7acf62b7f21b5b8a9f1d09f76da44a3","unresolved":false,"context_lines":[{"line_number":10,"context_line":"\u0027force\u0027 parameter was missing from the \u0027PUT /os-quota-sets/{tenant_id}\u0027"},{"line_number":11,"context_line":"API request schema so users could not force quota updates with"},{"line_number":12,"context_line":"microversion 2.36 or later. The bug is now fixed so that the \u0027force\u0027"},{"line_number":13,"context_line":"parameter can once again be specified during quota updates. There is"},{"line_number":14,"context_line":"no new microversion for this change since it is an admin-only API."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I947e927802f755ccb25a91efd82cac895779d19e"},{"line_number":17,"context_line":"Closes-Bug: #1733886"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ff82abbf_1b01a9af","line":14,"range":{"start_line":13,"start_character":60,"end_line":14,"end_character":66},"in_reply_to":"ff82abbf_fbb088df","updated":"2017-11-28 20:20:13.000000000","message":"We can update [1] because when it was written it wasn\u0027t really considering a case like this, for a 1-line bug change on an admin-only API. Admin-only APIs are not held to the same interoperability concerns, and this isn\u0027t a change in the request or response format for new behavior, it\u0027s just fixing the documented behavior which was broken way back in Newton. A new microversion seems excessive here and would be something we can\u0027t backport to stable branches to fix this.","commit_id":"e31dd087fb090d0db70ad5dbf4e92eed416fc4ed"}],"nova/api/openstack/compute/rest_api_version_history.rst":[{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"c1a82d12200f681edfb5abec349159744ca4677d","unresolved":false,"context_lines":[{"line_number":407,"context_line":"  \u0027/os-baremetal-nodes\u0027"},{"line_number":408,"context_line":"  \u0027/os-fping\u0027"},{"line_number":409,"context_line":""},{"line_number":410,"context_line":".. note:: A `regression`_ was introduced in this microversion which broke the"},{"line_number":411,"context_line":"    ``force`` parameter in the ``PUT /os-quota-sets`` API. The fix will have"},{"line_number":412,"context_line":"    to be applied to restore this functionality."},{"line_number":413,"context_line":""},{"line_number":414,"context_line":".. _regression: https://bugs.launchpad.net/nova/+bug/1733886"},{"line_number":415,"context_line":""},{"line_number":416,"context_line":"2.37"},{"line_number":417,"context_line":"----"}],"source_content_type":"text/x-rst","patch_set":2,"id":"df87a7cf_abfd156f","line":414,"range":{"start_line":410,"start_character":0,"end_line":414,"end_character":60},"updated":"2017-12-07 07:45:09.000000000","message":"Thanks, this is what i\u0027m looking for.","commit_id":"023e295e1cd95064db8b55b4a3ce28d067ffbf40"}],"nova/api/openstack/compute/schemas/quota_sets.py":[{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"6a56a755cafe5022665abaf76f536c243b6ce1ca","unresolved":false,"context_lines":[{"line_number":46,"context_line":"update_quota_set \u003d copy.deepcopy(quota_resources)"},{"line_number":47,"context_line":"update_quota_set.update({\u0027force\u0027: parameter_types.boolean})"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"update_quota_set_v236 \u003d copy.deepcopy(update_quota_set)"},{"line_number":50,"context_line":"del update_quota_set_v236[\u0027fixed_ips\u0027]"},{"line_number":51,"context_line":"del update_quota_set_v236[\u0027floating_ips\u0027]"},{"line_number":52,"context_line":"del update_quota_set_v236[\u0027security_groups\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff82abbf_fa7422d1","line":49,"updated":"2017-11-27 19:12:58.000000000","message":"the bug wouldn\u0027t have happened if update_quota_set had been fully expanded. I kind of think the deepcopy makes these sorts of breaks more likely as you have to think through the entire transform (and people miss it).","commit_id":"e31dd087fb090d0db70ad5dbf4e92eed416fc4ed"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"b4470237546f80cd3913ac2be44ecc5c9e4c9461","unresolved":false,"context_lines":[{"line_number":46,"context_line":"update_quota_set \u003d copy.deepcopy(quota_resources)"},{"line_number":47,"context_line":"update_quota_set.update({\u0027force\u0027: parameter_types.boolean})"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"update_quota_set_v236 \u003d copy.deepcopy(update_quota_set)"},{"line_number":50,"context_line":"del update_quota_set_v236[\u0027fixed_ips\u0027]"},{"line_number":51,"context_line":"del update_quota_set_v236[\u0027floating_ips\u0027]"},{"line_number":52,"context_line":"del update_quota_set_v236[\u0027security_groups\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff82abbf_aae91ae3","line":49,"in_reply_to":"ff82abbf_1a11c65d","updated":"2017-11-30 06:38:25.000000000","message":"In the 2.37, we missed the device tag...that sounds like the api sample test can\u0027t prevent all of this kind of mistake.","commit_id":"e31dd087fb090d0db70ad5dbf4e92eed416fc4ed"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"9189b98e20b40b26466b9d20a0a1c700930917ea","unresolved":false,"context_lines":[{"line_number":46,"context_line":"update_quota_set \u003d copy.deepcopy(quota_resources)"},{"line_number":47,"context_line":"update_quota_set.update({\u0027force\u0027: parameter_types.boolean})"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"update_quota_set_v236 \u003d copy.deepcopy(update_quota_set)"},{"line_number":50,"context_line":"del update_quota_set_v236[\u0027fixed_ips\u0027]"},{"line_number":51,"context_line":"del update_quota_set_v236[\u0027floating_ips\u0027]"},{"line_number":52,"context_line":"del update_quota_set_v236[\u0027security_groups\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff82abbf_90b6ab8a","line":49,"in_reply_to":"ff82abbf_aa610114","updated":"2017-12-01 03:52:10.000000000","message":"yea, these deepcopy things always problematic. Now we try to make all new schema as raw as they can be. \ni prefer if we can define this schema as saw without deepcopy and del part.","commit_id":"e31dd087fb090d0db70ad5dbf4e92eed416fc4ed"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"bf7bfa94c0f16fd75321551c95c7cb4c06cb7550","unresolved":false,"context_lines":[{"line_number":46,"context_line":"update_quota_set \u003d copy.deepcopy(quota_resources)"},{"line_number":47,"context_line":"update_quota_set.update({\u0027force\u0027: parameter_types.boolean})"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"update_quota_set_v236 \u003d copy.deepcopy(update_quota_set)"},{"line_number":50,"context_line":"del update_quota_set_v236[\u0027fixed_ips\u0027]"},{"line_number":51,"context_line":"del update_quota_set_v236[\u0027floating_ips\u0027]"},{"line_number":52,"context_line":"del update_quota_set_v236[\u0027security_groups\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff82abbf_aa610114","line":49,"in_reply_to":"ff82abbf_aae91ae3","updated":"2017-11-30 23:20:15.000000000","message":"I thought about 2.37 and the main difference there is 2.37 is a user API, not an admin API, so interoperability and discoverability of the fix is more important.","commit_id":"e31dd087fb090d0db70ad5dbf4e92eed416fc4ed"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"1d17c04175362b07fa54e7cefc76341219a261f0","unresolved":false,"context_lines":[{"line_number":46,"context_line":"update_quota_set \u003d copy.deepcopy(quota_resources)"},{"line_number":47,"context_line":"update_quota_set.update({\u0027force\u0027: parameter_types.boolean})"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"update_quota_set_v236 \u003d copy.deepcopy(update_quota_set)"},{"line_number":50,"context_line":"del update_quota_set_v236[\u0027fixed_ips\u0027]"},{"line_number":51,"context_line":"del update_quota_set_v236[\u0027floating_ips\u0027]"},{"line_number":52,"context_line":"del update_quota_set_v236[\u0027security_groups\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff82abbf_1a11c65d","line":49,"in_reply_to":"ff82abbf_fa7422d1","updated":"2017-11-27 19:19:36.000000000","message":"Sure, however, I think the main reason this was missed was simply because there was no API sample testing of the 2.36 microversion, and we usually always have API sample tests for new microversions. That\u0027s actually how I found this bug, by adding API sample tests for a new microversion on the os-quota-sets API schema.","commit_id":"e31dd087fb090d0db70ad5dbf4e92eed416fc4ed"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"9189b98e20b40b26466b9d20a0a1c700930917ea","unresolved":false,"context_lines":[{"line_number":46,"context_line":"update_quota_set \u003d copy.deepcopy(quota_resources)"},{"line_number":47,"context_line":"update_quota_set.update({\u0027force\u0027: parameter_types.boolean})"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"update_quota_set_v236 \u003d copy.deepcopy(update_quota_set)"},{"line_number":50,"context_line":"del update_quota_set_v236[\u0027fixed_ips\u0027]"},{"line_number":51,"context_line":"del update_quota_set_v236[\u0027floating_ips\u0027]"},{"line_number":52,"context_line":"del update_quota_set_v236[\u0027security_groups\u0027]"},{"line_number":53,"context_line":"del update_quota_set_v236[\u0027security_group_rules\u0027]"},{"line_number":54,"context_line":"del update_quota_set_v236[\u0027networks\u0027]"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"update \u003d {"},{"line_number":57,"context_line":"    \u0027type\u0027: \u0027object\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"ff82abbf_906b0bf0","line":54,"range":{"start_line":49,"start_character":0,"end_line":54,"end_character":37},"updated":"2017-12-01 03:52:10.000000000","message":"i prefer to make it a raw schema to avoid to introduce any other hidden bugs.\n\nupdate_quota_set_v236 \u003d {\n    \u0027instances\u0027: common_quota,\n    \u0027cores\u0027: common_quota,\n    \u0027ram\u0027: common_quota,\n    \u0027metadata_items\u0027: common_quota,\n    \u0027key_pairs\u0027: common_quota,\n    \u0027injected_files\u0027: common_quota,\n    \u0027injected_file_content_bytes\u0027: common_quota,\n    \u0027injected_file_path_bytes\u0027: common_quota,\n    \u0027server_groups\u0027: common_quota,\n    \u0027server_group_members\u0027: common_quota,\n    \u0027force\u0027: parameter_types.boolean}\n}","commit_id":"023e295e1cd95064db8b55b4a3ce28d067ffbf40"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"8bbb13e4c2a11c9137ea6896f51958c5d444c77e","unresolved":false,"context_lines":[{"line_number":46,"context_line":"update_quota_set \u003d copy.deepcopy(quota_resources)"},{"line_number":47,"context_line":"update_quota_set.update({\u0027force\u0027: parameter_types.boolean})"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"update_quota_set_v236 \u003d copy.deepcopy(update_quota_set)"},{"line_number":50,"context_line":"del update_quota_set_v236[\u0027fixed_ips\u0027]"},{"line_number":51,"context_line":"del update_quota_set_v236[\u0027floating_ips\u0027]"},{"line_number":52,"context_line":"del update_quota_set_v236[\u0027security_groups\u0027]"},{"line_number":53,"context_line":"del update_quota_set_v236[\u0027security_group_rules\u0027]"},{"line_number":54,"context_line":"del update_quota_set_v236[\u0027networks\u0027]"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"update \u003d {"},{"line_number":57,"context_line":"    \u0027type\u0027: \u0027object\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"df87a7cf_51b1098f","line":54,"range":{"start_line":49,"start_character":0,"end_line":54,"end_character":37},"in_reply_to":"df87a7cf_41607233","updated":"2017-12-03 11:47:02.000000000","message":"sure, we can do.","commit_id":"023e295e1cd95064db8b55b4a3ce28d067ffbf40"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"4e9386718692e3a0a0e44dcad0cf19ebaf855d44","unresolved":false,"context_lines":[{"line_number":46,"context_line":"update_quota_set \u003d copy.deepcopy(quota_resources)"},{"line_number":47,"context_line":"update_quota_set.update({\u0027force\u0027: parameter_types.boolean})"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"update_quota_set_v236 \u003d copy.deepcopy(update_quota_set)"},{"line_number":50,"context_line":"del update_quota_set_v236[\u0027fixed_ips\u0027]"},{"line_number":51,"context_line":"del update_quota_set_v236[\u0027floating_ips\u0027]"},{"line_number":52,"context_line":"del update_quota_set_v236[\u0027security_groups\u0027]"},{"line_number":53,"context_line":"del update_quota_set_v236[\u0027security_group_rules\u0027]"},{"line_number":54,"context_line":"del update_quota_set_v236[\u0027networks\u0027]"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"update \u003d {"},{"line_number":57,"context_line":"    \u0027type\u0027: \u0027object\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"df87a7cf_41607233","line":54,"range":{"start_line":49,"start_character":0,"end_line":54,"end_character":37},"in_reply_to":"ff82abbf_906b0bf0","updated":"2017-12-01 14:16:22.000000000","message":"Can we do that in a follow up change? Right now this is a one-line fix so I\u0027d like to keep that simple, the tests are verifying the correct behavior which was missing from the original 2.36 change.","commit_id":"023e295e1cd95064db8b55b4a3ce28d067ffbf40"}],"nova/tests/functional/api_sample_tests/api_samples/os-quota-sets/v2.36/quotas-update-force-resp.json.tpl":[{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"4877f97e1736ab5770a1be5e130f119f13420560","unresolved":false,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"    \"quota_set\": {"},{"line_number":3,"context_line":"        \"cores\": 20,"},{"line_number":4,"context_line":"        \"injected_file_content_bytes\": 10240,"},{"line_number":5,"context_line":"        \"injected_file_path_bytes\": 255,"}],"source_content_type":"text/x-smarty","patch_set":1,"id":"ff82abbf_2a842ad2","line":2,"updated":"2017-11-30 06:43:20.000000000","message":"why doesn\u0027t this file show up in the sample file directory?","commit_id":"e31dd087fb090d0db70ad5dbf4e92eed416fc4ed"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"bf7bfa94c0f16fd75321551c95c7cb4c06cb7550","unresolved":false,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"    \"quota_set\": {"},{"line_number":3,"context_line":"        \"cores\": 20,"},{"line_number":4,"context_line":"        \"injected_file_content_bytes\": 10240,"},{"line_number":5,"context_line":"        \"injected_file_path_bytes\": 255,"}],"source_content_type":"text/x-smarty","patch_set":1,"id":"ff82abbf_4a72b5e5","line":2,"in_reply_to":"ff82abbf_2a5d0a78","updated":"2017-11-30 23:20:15.000000000","message":"Oh yeah, I copied this and noticed it wasn\u0027t used in the API samples docs and deleted that one, but forgot about this one. Nice catch.","commit_id":"e31dd087fb090d0db70ad5dbf4e92eed416fc4ed"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"97cc5063e69da1093917f610bfbbe611e7540be3","unresolved":false,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"    \"quota_set\": {"},{"line_number":3,"context_line":"        \"cores\": 20,"},{"line_number":4,"context_line":"        \"injected_file_content_bytes\": 10240,"},{"line_number":5,"context_line":"        \"injected_file_path_bytes\": 255,"}],"source_content_type":"text/x-smarty","patch_set":1,"id":"ff82abbf_2a5d0a78","line":2,"in_reply_to":"ff82abbf_2a842ad2","updated":"2017-11-30 06:44:55.000000000","message":"looks like this file should be deleted","commit_id":"e31dd087fb090d0db70ad5dbf4e92eed416fc4ed"}],"releasenotes/notes/bug-1733886-os-quota-sets-force-2.36-5866924621ecc857.yaml":[{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"b4470237546f80cd3913ac2be44ecc5c9e4c9461","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    This release includes a fix for `bug 1733886`_ which was a regression"},{"line_number":5,"context_line":"    introduced in the 2.36 API microversion where the ``force`` parameter was"},{"line_number":6,"context_line":"    missing from the ``PUT /os-quota-sets/{tenant_id}`` API request schema so"},{"line_number":7,"context_line":"    users could not force quota updates with microversion 2.36 or later. The"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"ff82abbf_4adf2e0c","line":4,"updated":"2017-11-30 06:38:25.000000000","message":"should we mark this in the rest_api_version_history.rst?\nI try to find out which kind of bug fix for the API without microversion, but looks like it is hard.","commit_id":"e31dd087fb090d0db70ad5dbf4e92eed416fc4ed"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"bf7bfa94c0f16fd75321551c95c7cb4c06cb7550","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    This release includes a fix for `bug 1733886`_ which was a regression"},{"line_number":5,"context_line":"    introduced in the 2.36 API microversion where the ``force`` parameter was"},{"line_number":6,"context_line":"    missing from the ``PUT /os-quota-sets/{tenant_id}`` API request schema so"},{"line_number":7,"context_line":"    users could not force quota updates with microversion 2.36 or later. The"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"ff82abbf_6a6bf9f0","line":4,"in_reply_to":"ff82abbf_4adf2e0c","updated":"2017-11-30 23:20:15.000000000","message":"Like, link to the bug report in the 2.36 REST API history doc as a known issue?","commit_id":"e31dd087fb090d0db70ad5dbf4e92eed416fc4ed"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"9189b98e20b40b26466b9d20a0a1c700930917ea","unresolved":false,"context_lines":[{"line_number":7,"context_line":"    users could not force quota updates with microversion 2.36 or later. The"},{"line_number":8,"context_line":"    bug is now fixed so that the ``force`` parameter can once again be"},{"line_number":9,"context_line":"    specified during quota updates. There is no new microversion for this"},{"line_number":10,"context_line":"    change since it is an admin-only API."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"    .. _bug 1733886: https://bugs.launchpad.net/nova/+bug/1733886"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"ff82abbf_b0edef8c","line":10,"range":{"start_line":10,"start_character":4,"end_line":10,"end_character":41},"updated":"2017-12-01 03:52:10.000000000","message":"we can add backport info also saying fix is being backported till newton.","commit_id":"023e295e1cd95064db8b55b4a3ce28d067ffbf40"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"4e9386718692e3a0a0e44dcad0cf19ebaf855d44","unresolved":false,"context_lines":[{"line_number":7,"context_line":"    users could not force quota updates with microversion 2.36 or later. The"},{"line_number":8,"context_line":"    bug is now fixed so that the ``force`` parameter can once again be"},{"line_number":9,"context_line":"    specified during quota updates. There is no new microversion for this"},{"line_number":10,"context_line":"    change since it is an admin-only API."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"    .. _bug 1733886: https://bugs.launchpad.net/nova/+bug/1733886"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"df87a7cf_6161f639","line":10,"range":{"start_line":10,"start_character":4,"end_line":10,"end_character":41},"in_reply_to":"ff82abbf_b0edef8c","updated":"2017-12-01 14:16:22.000000000","message":"I\u0027d rather not do that if it doesn\u0027t end up happening.","commit_id":"023e295e1cd95064db8b55b4a3ce28d067ffbf40"}]}