)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"ec520958a5bb1a248ddd181f4351f79215ab6ff6","unresolved":false,"context_lines":[{"line_number":16,"context_line":"caveats:"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"* Dealing with nova/db/sqlalchemy/resource_class_cache.py has not"},{"line_number":19,"context_line":"  yet been accomplished, so it remains where it is an imports"},{"line_number":20,"context_line":"  exceptions from the placement hierarchy. A TODO indicating some of the"},{"line_number":21,"context_line":"  options is left for future work."},{"line_number":22,"context_line":"* Exceptions with the name ResourceProviderInUse and InventoryInUse are"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":16,"id":"df7087c5_3bc46340","line":19,"range":{"start_line":19,"start_character":51,"end_line":19,"end_character":53},"updated":"2018-03-19 15:08:32.000000000","message":"and?","commit_id":"84317b290b74d530d38181257f3713ebe5f1b4e4"}],"nova/api/openstack/placement/exception.py":[{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"ec520958a5bb1a248ddd181f4351f79215ab6ff6","unresolved":false,"context_lines":[{"line_number":61,"context_line":""},{"line_number":62,"context_line":"    def format_message(self):"},{"line_number":63,"context_line":"        # NOTE(mrodden): use the first argument to the python Exception object"},{"line_number":64,"context_line":"        # which should be our full NovaException message, (see __init__)"},{"line_number":65,"context_line":"        return self.args[0]"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":""}],"source_content_type":"text/x-python","patch_set":16,"id":"df7087c5_1b187fd1","line":64,"range":{"start_line":64,"start_character":35,"end_line":64,"end_character":48},"updated":"2018-03-19 15:08:32.000000000","message":"change this? also, it seems to be referring to Exception.__init__, not NovaException.__init__.","commit_id":"84317b290b74d530d38181257f3713ebe5f1b4e4"}],"nova/db/sqlalchemy/resource_class_cache.py":[{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"18057feb12117792bde78b9fe61050084b253917","unresolved":false,"context_lines":[{"line_number":15,"context_line":"import sqlalchemy as sa"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"# TODO(cdent): This file and its location is problematic for placement"},{"line_number":18,"context_line":"# extraction but we probalby want to switch to os-resource-classes (like"},{"line_number":19,"context_line":"# os-traits) instead of moving it?"},{"line_number":20,"context_line":"from nova.api.openstack.placement import exception"},{"line_number":21,"context_line":"from nova.db.sqlalchemy import api as db_api"}],"source_content_type":"text/x-python","patch_set":15,"id":"df7087c5_98e25501","line":18,"range":{"start_line":18,"start_character":20,"end_line":18,"end_character":28},"updated":"2018-03-19 14:26:07.000000000","message":"probably","commit_id":"1a4291824ea7da01145d711fe59a8c03fe7dd625"}],"nova/tests/functional/api/openstack/placement/test_report_client.py":[{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"18057feb12117792bde78b9fe61050084b253917","unresolved":false,"context_lines":[{"line_number":23,"context_line":"# TODO(cdent): This points to the nova, not placement, exception for"},{"line_number":24,"context_line":"# InvalidResourceClass. This test should probably move out of the"},{"line_number":25,"context_line":"# placement hierarchy since it expects a \"standard\" placement server"},{"line_number":26,"context_line":"# and not testing it directly."},{"line_number":27,"context_line":"from nova import exception"},{"line_number":28,"context_line":"from nova import objects"},{"line_number":29,"context_line":"from nova import rc_fields as fields"}],"source_content_type":"text/x-python","patch_set":15,"id":"df7087c5_b8a95991","line":26,"range":{"start_line":26,"start_character":2,"end_line":26,"end_character":21},"updated":"2018-03-19 14:26:07.000000000","message":"English no parse for me","commit_id":"1a4291824ea7da01145d711fe59a8c03fe7dd625"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"ec520958a5bb1a248ddd181f4351f79215ab6ff6","unresolved":false,"context_lines":[{"line_number":23,"context_line":"# TODO(cdent): This points to the nova, not placement, exception for"},{"line_number":24,"context_line":"# InvalidResourceClass. This test should probably move out of the"},{"line_number":25,"context_line":"# placement hierarchy since it expects a \"standard\" placement server"},{"line_number":26,"context_line":"# and is not testing the placement service itself."},{"line_number":27,"context_line":"from nova import exception"},{"line_number":28,"context_line":"from nova import objects"},{"line_number":29,"context_line":"from nova import rc_fields as fields"}],"source_content_type":"text/x-python","patch_set":16,"id":"df7087c5_5b4b57af","line":26,"updated":"2018-03-19 15:08:32.000000000","message":"++","commit_id":"84317b290b74d530d38181257f3713ebe5f1b4e4"}]}
