)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"007aa21d07b77018e3aa6aaa6064ad15c03f198b","unresolved":false,"context_lines":[{"line_number":10,"context_line":"coverage in the Octavia code. Until then, omit it from"},{"line_number":11,"context_line":"the report so the coverage test will pass, so we can"},{"line_number":12,"context_line":"make it voting."},{"line_number":13,"context_line":"For now we have test coverage at level of 66%."},{"line_number":14,"context_line":"As a good start set 65% to start voting and then"},{"line_number":15,"context_line":"successfully increase it."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"[0] https://review.opendev.org/#/c/601948/"},{"line_number":18,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"9fb8cfa7_c451f30f","line":15,"range":{"start_line":13,"start_character":0,"end_line":15,"end_character":25},"updated":"2019-07-01 09:39:38.000000000","message":"sigh","commit_id":"a4b02a3b6d04e8504a4fccdaef255d8561ff2d03"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"007aa21d07b77018e3aa6aaa6064ad15c03f198b","unresolved":false,"context_lines":[{"line_number":15,"context_line":"successfully increase it."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"[0] https://review.opendev.org/#/c/601948/"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: I0d63fc127da9ab37bb217678a1c47961c17d3eda"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"9fb8cfa7_6454c7fe","line":18,"updated":"2019-07-01 09:39:38.000000000","message":"Co-Authored-By: Maciej Józefczyk \u003cmjozefcz@redhat.com\u003e","commit_id":"a4b02a3b6d04e8504a4fccdaef255d8561ff2d03"}],"tox.ini":[{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"9eb34313ba44c3fba728fb3881461abe98015718","unresolved":false,"context_lines":[{"line_number":82,"context_line":"commands \u003d"},{"line_number":83,"context_line":"  stestr run --no-subunit-trace {posargs}"},{"line_number":84,"context_line":"  coverage combine"},{"line_number":85,"context_line":"  coverage report --fail-under\u003d65 --skip-covered"},{"line_number":86,"context_line":"  coverage html -d cover"},{"line_number":87,"context_line":"  coverage xml -o cover/coverage.xml"},{"line_number":88,"context_line":""}],"source_content_type":"text/x-properties","patch_set":3,"id":"9fb8cfa7_a22926f3","line":85,"updated":"2019-06-17 21:13:13.000000000","message":"We are currently failing the cover job due to the Octavia driver lacking unittests. There\u0027s a patch implementing it as pointed out by Kuba in a previous review [0] which will fix it.\n\nBut, IMHO 70% is _already_ pretty low. I wonder if we could just ignore the contents of networking_ovn/octavia/ instead of lowering it even more. Then, in [0] we can just remove the regex to ignore it.\n\nI believe it can be done by adding a \"--omit networking_ovn/octavia\" parameter to the \"coverage report\" command.\n\nWhat do you think ?\n\n[0] https://review.opendev.org/#/c/601948/","commit_id":"ffeb5a76b79a22fc69bfc1237ffcbfb78cfdcdc5"}]}