)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"8d3527b3f59ce3d646e66a1083833ceab6e3c77c","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Chris Dent \u003ccdent@anticdent.org\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-08-30 20:30:19 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[placememnt] Require at least one resource class in allocation"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"If an allocation was made with an empty resources object, a 500 response"},{"line_number":10,"context_line":"code would result. This change adjust the schema to use minProperties"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7f287b81_d578a790","line":7,"range":{"start_line":7,"start_character":1,"end_line":7,"end_character":11},"updated":"2017-08-30 23:18:31.000000000","message":"fix this spelling (for the sake of your own reports :)","commit_id":"ea359c70d2068106e0470d319f55d97e81377ae1"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"c7f56aea440cc3183c3393a47a9c0aa9638386d0","unresolved":false,"context_lines":[{"line_number":7,"context_line":"[placememnt] Require at least one resource class in allocation"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"If an allocation was made with an empty resources object, a 500 response"},{"line_number":10,"context_line":"code would result. This change adjust the schema to use minProperties"},{"line_number":11,"context_line":"of 1 to say there must be at least one resource class and value pair in"},{"line_number":12,"context_line":"the allocation. If there is not a 400 response is returned."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7f287b81_d07b15f2","line":10,"range":{"start_line":10,"start_character":31,"end_line":10,"end_character":37},"updated":"2017-08-31 00:38:26.000000000","message":"adjusts","commit_id":"ea359c70d2068106e0470d319f55d97e81377ae1"}],"nova/api/openstack/placement/handlers/allocation.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"c7f56aea440cc3183c3393a47a9c0aa9638386d0","unresolved":false,"context_lines":[{"line_number":51,"context_line":"                    },"},{"line_number":52,"context_line":"                    \"resources\": {"},{"line_number":53,"context_line":"                        \"type\": \"object\","},{"line_number":54,"context_line":"                        \"minProperties\": 1,"},{"line_number":55,"context_line":"                        \"patternProperties\": {"},{"line_number":56,"context_line":"                            \"^[0-9A-Z_]+$\": {"},{"line_number":57,"context_line":"                                \"type\": \"integer\","}],"source_content_type":"text/x-python","patch_set":1,"id":"7f287b81_d064354c","line":54,"updated":"2017-08-31 00:38:26.000000000","message":"I was thinking maybe we\u0027d want to mention something about this in the API reference:\n\nhttps://developer.openstack.org/api-ref/placement/#update-allocations\n\nBut then it\u0027s a slippery slope about saying that at least one allocation is provided, and the key format on the resource class, etc.","commit_id":"ea359c70d2068106e0470d319f55d97e81377ae1"}]}
