)]}'
{"placement/objects/resource_provider.py":[{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"59de94af3a9b29bb13be1b77a749d38ae6ac2471","unresolved":false,"context_lines":[{"line_number":3417,"context_line":"            LOG.debug(\"considering %d sharing providers with %d %s, \""},{"line_number":3418,"context_line":"                      \"now we\u0027ve got %d provider trees\","},{"line_number":3419,"context_line":"                      len(sharing_providers), amount, rc_name,"},{"line_number":3420,"context_line":"                      len(provs_with_inv_rc.trees()))"},{"line_number":3421,"context_line":""},{"line_number":3422,"context_line":"        # Adding the resource providers we\u0027ve got for this resource class,"},{"line_number":3423,"context_line":"        # filter provs_with_inv to have only trees with enough inventories"}],"source_content_type":"text/x-python","patch_set":2,"id":"9fdfeff1_11f5f182","line":3420,"range":{"start_line":3420,"start_character":49,"end_line":3420,"end_character":51},"updated":"2019-03-01 15:09:42.000000000","message":"whoops","commit_id":"b105f76f9c7233bf4f13c2d5b035963187b858bf"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"59de94af3a9b29bb13be1b77a749d38ae6ac2471","unresolved":false,"context_lines":[{"line_number":3466,"context_line":"    LOG.debug(\"found %d providers under %d trees after applying \""},{"line_number":3467,"context_line":"              \"traits filter - required: %s, forbidden: %s\","},{"line_number":3468,"context_line":"              len(provs_with_inv.rps), len(provs_with_inv_rc.trees),"},{"line_number":3469,"context_line":"              list(required_traits.keys()), list(forbidden_traits.keys()))"},{"line_number":3470,"context_line":""},{"line_number":3471,"context_line":"    return provs_with_inv"},{"line_number":3472,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9fdfeff1_f13e2d27","line":3469,"range":{"start_line":3469,"start_character":65,"end_line":3469,"end_character":72},"updated":"2019-03-01 15:09:42.000000000","message":"ditto","commit_id":"b105f76f9c7233bf4f13c2d5b035963187b858bf"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"59de94af3a9b29bb13be1b77a749d38ae6ac2471","unresolved":false,"context_lines":[{"line_number":3466,"context_line":"    LOG.debug(\"found %d providers under %d trees after applying \""},{"line_number":3467,"context_line":"              \"traits filter - required: %s, forbidden: %s\","},{"line_number":3468,"context_line":"              len(provs_with_inv.rps), len(provs_with_inv_rc.trees),"},{"line_number":3469,"context_line":"              list(required_traits.keys()), list(forbidden_traits.keys()))"},{"line_number":3470,"context_line":""},{"line_number":3471,"context_line":"    return provs_with_inv"},{"line_number":3472,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9fdfeff1_5134d948","line":3469,"range":{"start_line":3469,"start_character":34,"end_line":3469,"end_character":41},"updated":"2019-03-01 15:09:42.000000000","message":"nit:redundant\n\nin py3 it\u0027s a no-op, but in py2 you\u0027re creating an extra list.","commit_id":"b105f76f9c7233bf4f13c2d5b035963187b858bf"}]}