)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"1816933552ed1a05db5ade1cea8a0260fdc10be6","unresolved":false,"context_lines":[{"line_number":15,"context_line":"modified to return the cell mapping uuid rather than 0. In that case"},{"line_number":16,"context_line":"there may still be unmapped instances so getting access to that uuid is"},{"line_number":17,"context_line":"beneficial."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: Ia2d82243fd8e92f270801bcad2222808f2ee1a98"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1a122d0e_9080d3f8","line":18,"updated":"2016-04-28 21:28:00.000000000","message":"one could nitpick that you\u0027re not explaining that you\u0027re doing this because of your magic do-it-once cellsv2 nova-manage command but that\u0027s fine.","commit_id":"64f5f99326d7313c9a9b6341445d98bc7fc76332"}],"nova/cmd/manage.py":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"f19ac63822c0b7d817a136d0bf8e176387f8e19b","unresolved":false,"context_lines":[{"line_number":1405,"context_line":"        compute_nodes \u003d objects.ComputeNodeList.get_all(ctxt)"},{"line_number":1406,"context_line":"        if not compute_nodes:"},{"line_number":1407,"context_line":"            print(_(\u0027No hosts found to map to cell, exiting.\u0027))"},{"line_number":1408,"context_line":"            return(0)"},{"line_number":1409,"context_line":"        missing_nodes \u003d []"},{"line_number":1410,"context_line":"        for compute_node in compute_nodes:"},{"line_number":1411,"context_line":"            try:"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a1b2928_0a20eb9a","line":1408,"range":{"start_line":1408,"start_character":12,"end_line":1408,"end_character":21},"updated":"2016-04-10 23:15:22.000000000","message":"It would be clear meaning if returning None here instead of 0.","commit_id":"f10c073c62e0da7bfea88aef7ecfc5ec4f1d99e6"},{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"b7578a15fc61b56af0c6c55c192ee500ed6337b6","unresolved":false,"context_lines":[{"line_number":1405,"context_line":"        compute_nodes \u003d objects.ComputeNodeList.get_all(ctxt)"},{"line_number":1406,"context_line":"        if not compute_nodes:"},{"line_number":1407,"context_line":"            print(_(\u0027No hosts found to map to cell, exiting.\u0027))"},{"line_number":1408,"context_line":"            return(0)"},{"line_number":1409,"context_line":"        missing_nodes \u003d []"},{"line_number":1410,"context_line":"        for compute_node in compute_nodes:"},{"line_number":1411,"context_line":"            try:"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a122d0e_5ca9117b","line":1408,"range":{"start_line":1408,"start_character":12,"end_line":1408,"end_character":21},"in_reply_to":"7a1b2928_0a20eb9a","updated":"2016-04-21 12:59:30.000000000","message":"Okay, I\u0027ll change this.","commit_id":"f10c073c62e0da7bfea88aef7ecfc5ec4f1d99e6"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"42c655276fa0fd15dbb491982e5eb32e85555b93","unresolved":false,"context_lines":[{"line_number":1250,"context_line":"            host_mapping.create()"},{"line_number":1251,"context_line":"        if verbose:"},{"line_number":1252,"context_line":"            print(cell_mapping_uuid)"},{"line_number":1253,"context_line":"        return cell_mapping_uuid"},{"line_number":1254,"context_line":""},{"line_number":1255,"context_line":"    # TODO(melwitt): Remove this when the oslo.messaging function"},{"line_number":1256,"context_line":"    # for assembling a transport url from ConfigOpts is available"}],"source_content_type":"text/x-python","patch_set":2,"id":"dab17558_17286585","line":1253,"updated":"2016-05-10 22:30:09.000000000","message":"Woo, bringing back my return value","commit_id":"64f5f99326d7313c9a9b6341445d98bc7fc76332"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"1816933552ed1a05db5ade1cea8a0260fdc10be6","unresolved":false,"context_lines":[{"line_number":1275,"context_line":"        self._map_cell_and_hosts(transport_url, name, verbose)"},{"line_number":1276,"context_line":"        # online_data_migrations established a pattern of 0 meaning everything"},{"line_number":1277,"context_line":"        # is done, 1 means run again to do more work. This command doesn\u0027t do"},{"line_number":1278,"context_line":"        # partial work so 0 is appropriate."},{"line_number":1279,"context_line":"        return 0"},{"line_number":1280,"context_line":""},{"line_number":1281,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"1a122d0e_30a61fc1","line":1278,"updated":"2016-04-28 21:28:00.000000000","message":"tbc, this change has nothing to do with online_data_migrations but the fact that it returns the same status code. Fine by me but a bit confusing.","commit_id":"64f5f99326d7313c9a9b6341445d98bc7fc76332"}]}