)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"65e5c044e72adcba984117b8a0d9dbda95245579","unresolved":false,"context_lines":[{"line_number":21,"context_line":"* recent nova master"},{"line_number":22,"context_line":"* nova/api/openstack/placement removed (except for lib.py)"},{"line_number":23,"context_line":"* nova/tests/{unit,functional}/api/openstack/placement removed"},{"line_number":24,"context_line":"* https://github.com/EdLeafe/placement/pull/2 \u0027pip install -e\u0027"},{"line_number":25,"context_line":"  into the local nova virtenv"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"And then the changes need to get both unit and functional tests"},{"line_number":28,"context_line":"passing. As the diff (a consolidated one without file removes is at"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3f79a3b5_0b7e5818","line":25,"range":{"start_line":24,"start_character":0,"end_line":25,"end_character":29},"updated":"2018-08-24 12:54:24.000000000","message":"Hm, can we add this install command to the requirements.txt to make the tests pass on the gate as well?","commit_id":"70782ac2a8c7b3434f866b8b3564fae3acfa6397"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"13d9f3561250155602051f6a0a8e6600b748388e","unresolved":false,"context_lines":[{"line_number":21,"context_line":"* recent nova master"},{"line_number":22,"context_line":"* nova/api/openstack/placement removed (except for lib.py)"},{"line_number":23,"context_line":"* nova/tests/{unit,functional}/api/openstack/placement removed"},{"line_number":24,"context_line":"* https://github.com/EdLeafe/placement/pull/2 \u0027pip install -e\u0027"},{"line_number":25,"context_line":"  into the local nova virtenv"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"And then the changes need to get both unit and functional tests"},{"line_number":28,"context_line":"passing. As the diff (a consolidated one without file removes is at"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3f79a3b5_2bc9d457","line":25,"range":{"start_line":24,"start_character":0,"end_line":25,"end_character":29},"in_reply_to":"3f79a3b5_0b7e5818","updated":"2018-08-24 12:57:35.000000000","message":"I can try, but in the past I\u0027ve never been able to get it to work right so I didn\u0027t bother to try this time around. I\u0027ll see what I can do","commit_id":"70782ac2a8c7b3434f866b8b3564fae3acfa6397"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"65e5c044e72adcba984117b8a0d9dbda95245579","unresolved":false,"context_lines":[{"line_number":58,"context_line":"  and placement for managing RequestGroup objects. However, as I"},{"line_number":59,"context_line":"  understand things, there is, or will be some divergence on how"},{"line_number":60,"context_line":"  these things are going to be used, so should be fine to have"},{"line_number":61,"context_line":"  separate versions and not an actual \"lib\""},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"* The cmd and status checks that want the database need to have a"},{"line_number":64,"context_line":"  future defined for them."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3f79a3b5_6bacac9e","line":61,"updated":"2018-08-24 12:54:24.000000000","message":"The divergence is proposed here https://review.openstack.org/#/c/568840","commit_id":"70782ac2a8c7b3434f866b8b3564fae3acfa6397"}]}