)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":12175,"name":"Eli Qiao","email":"qiaoliyong@gmail.com","username":"Eli"},"change_message_id":"5cb228b4836f4ce8dd063b4527895fb18e93196e","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"* checkout one commit before the latest and check coverage"},{"line_number":10,"context_line":"* check coverage with the latest commit"},{"line_number":11,"context_line":"* allow 4 lines of missing tests"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This functionallity essentially compares the current change"},{"line_number":14,"context_line":"with the master branch, since the coverage job runs always"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9ad45d7e_a03115a4","line":11,"range":{"start_line":11,"start_character":0,"end_line":11,"end_character":32},"updated":"2016-08-12 07:56:24.000000000","message":"This is in-consistent.","commit_id":"8ecb1e1074ff59a6d0097c326b2d5700d8c2a92a"},{"author":{"_account_id":20498,"name":"Spyros Trigazis","email":"spyridon.trigazis@cern.ch","username":"strigazi"},"change_message_id":"10fa467f23d9203d0cef8a1f5d4c455f7c042616","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"* checkout one commit before the latest and check coverage"},{"line_number":10,"context_line":"* check coverage with the latest commit"},{"line_number":11,"context_line":"* allow 4 lines of missing tests"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This functionallity essentially compares the current change"},{"line_number":14,"context_line":"with the master branch, since the coverage job runs always"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9ad45d7e_7bc7cc63","line":11,"range":{"start_line":11,"start_character":0,"end_line":11,"end_character":32},"in_reply_to":"9ad45d7e_a03115a4","updated":"2016-08-12 08:16:49.000000000","message":"Done","commit_id":"8ecb1e1074ff59a6d0097c326b2d5700d8c2a92a"}],"cover.sh":[{"author":{"_account_id":9591,"name":"Ton Ngo","email":"ton@us.ibm.com","username":"tango"},"change_message_id":"e9439227a41c4f0793ec5c3d4cb4941704b98809","unresolved":false,"context_lines":[{"line_number":51,"context_line":"echo \"Missing lines in master            : ${baseline_missing}\""},{"line_number":52,"context_line":"echo \"Missing lines in proposed change   : ${current_missing}\""},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"if [ $allowed_missing -gt $current_missing ];"},{"line_number":55,"context_line":"then"},{"line_number":56,"context_line":"    if [ $baseline_missing -lt $current_missing ];"},{"line_number":57,"context_line":"    then"}],"source_content_type":"text/x-sh","patch_set":3,"id":"9ad45d7e_a7190b7c","line":54,"range":{"start_line":54,"start_character":23,"end_line":54,"end_character":25},"updated":"2016-08-12 16:42:30.000000000","message":"I think you want to test for \"ge\"\nOtherwise if there is no change in missing lines, the test still fails.","commit_id":"6f42fd57755b282f05319bfc93e44b0a2b877cca"},{"author":{"_account_id":20498,"name":"Spyros Trigazis","email":"spyridon.trigazis@cern.ch","username":"strigazi"},"change_message_id":"3d38d7ade86bb6a92b26eb65efd9494f458bb523","unresolved":false,"context_lines":[{"line_number":51,"context_line":"echo \"Missing lines in master            : ${baseline_missing}\""},{"line_number":52,"context_line":"echo \"Missing lines in proposed change   : ${current_missing}\""},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"if [ $allowed_missing -gt $current_missing ];"},{"line_number":55,"context_line":"then"},{"line_number":56,"context_line":"    if [ $baseline_missing -lt $current_missing ];"},{"line_number":57,"context_line":"    then"}],"source_content_type":"text/x-sh","patch_set":3,"id":"fa7ab95a_195e1c12","line":54,"range":{"start_line":54,"start_character":23,"end_line":54,"end_character":25},"in_reply_to":"9ad45d7e_255d09ed","updated":"2016-08-29 12:30:25.000000000","message":"DONE. Needed after setting the ALLOWED_EXTRA_MISSING equal to zero.","commit_id":"6f42fd57755b282f05319bfc93e44b0a2b877cca"},{"author":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"change_message_id":"cc8198a3401d328b11851e82aede74cddea5ae92","unresolved":false,"context_lines":[{"line_number":51,"context_line":"echo \"Missing lines in master            : ${baseline_missing}\""},{"line_number":52,"context_line":"echo \"Missing lines in proposed change   : ${current_missing}\""},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"if [ $allowed_missing -gt $current_missing ];"},{"line_number":55,"context_line":"then"},{"line_number":56,"context_line":"    if [ $baseline_missing -lt $current_missing ];"},{"line_number":57,"context_line":"    then"}],"source_content_type":"text/x-sh","patch_set":3,"id":"9ad45d7e_4fa93cdf","line":54,"range":{"start_line":54,"start_character":23,"end_line":54,"end_character":25},"in_reply_to":"9ad45d7e_a7190b7c","updated":"2016-08-12 19:47:35.000000000","message":"+1","commit_id":"6f42fd57755b282f05319bfc93e44b0a2b877cca"},{"author":{"_account_id":668,"name":"Adrian Otto","email":"aotto@aotto.com","username":"aotto"},"change_message_id":"2b29b1f20aadf5ea00f113d4074c9431ed93a594","unresolved":false,"context_lines":[{"line_number":51,"context_line":"echo \"Missing lines in master            : ${baseline_missing}\""},{"line_number":52,"context_line":"echo \"Missing lines in proposed change   : ${current_missing}\""},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"if [ $allowed_missing -gt $current_missing ];"},{"line_number":55,"context_line":"then"},{"line_number":56,"context_line":"    if [ $baseline_missing -lt $current_missing ];"},{"line_number":57,"context_line":"    then"}],"source_content_type":"text/x-sh","patch_set":3,"id":"9ad45d7e_255d09ed","line":54,"range":{"start_line":54,"start_character":23,"end_line":54,"end_character":25},"in_reply_to":"9ad45d7e_a7190b7c","updated":"2016-08-12 20:38:37.000000000","message":"Nice catch!","commit_id":"6f42fd57755b282f05319bfc93e44b0a2b877cca"},{"author":{"_account_id":668,"name":"Adrian Otto","email":"aotto@aotto.com","username":"aotto"},"change_message_id":"2b29b1f20aadf5ea00f113d4074c9431ed93a594","unresolved":false,"context_lines":[{"line_number":56,"context_line":"    if [ $baseline_missing -lt $current_missing ];"},{"line_number":57,"context_line":"    then"},{"line_number":58,"context_line":"        show_diff $baseline_report $current_report"},{"line_number":59,"context_line":"        echo \"I believe you can cover all your code with 100% coverage!\""},{"line_number":60,"context_line":"    else"},{"line_number":61,"context_line":"        echo \"Thank you! You are awesome! Keep writing unit tests! :)\""},{"line_number":62,"context_line":"    fi"}],"source_content_type":"text/x-sh","patch_set":3,"id":"9ad45d7e_650ee1e6","line":59,"range":{"start_line":59,"start_character":14,"end_line":59,"end_character":41},"updated":"2016-08-12 20:38:37.000000000","message":"We believe you can test","commit_id":"6f42fd57755b282f05319bfc93e44b0a2b877cca"},{"author":{"_account_id":20498,"name":"Spyros Trigazis","email":"spyridon.trigazis@cern.ch","username":"strigazi"},"change_message_id":"3d38d7ade86bb6a92b26eb65efd9494f458bb523","unresolved":false,"context_lines":[{"line_number":56,"context_line":"    if [ $baseline_missing -lt $current_missing ];"},{"line_number":57,"context_line":"    then"},{"line_number":58,"context_line":"        show_diff $baseline_report $current_report"},{"line_number":59,"context_line":"        echo \"I believe you can cover all your code with 100% coverage!\""},{"line_number":60,"context_line":"    else"},{"line_number":61,"context_line":"        echo \"Thank you! You are awesome! Keep writing unit tests! :)\""},{"line_number":62,"context_line":"    fi"}],"source_content_type":"text/x-sh","patch_set":3,"id":"fa7ab95a_39ebe014","line":59,"range":{"start_line":59,"start_character":14,"end_line":59,"end_character":41},"in_reply_to":"9ad45d7e_650ee1e6","updated":"2016-08-29 12:30:25.000000000","message":"Done","commit_id":"6f42fd57755b282f05319bfc93e44b0a2b877cca"},{"author":{"_account_id":668,"name":"Adrian Otto","email":"aotto@aotto.com","username":"aotto"},"change_message_id":"2b29b1f20aadf5ea00f113d4074c9431ed93a594","unresolved":false,"context_lines":[{"line_number":63,"context_line":"    exit_code\u003d0"},{"line_number":64,"context_line":"else"},{"line_number":65,"context_line":"    show_diff $baseline_report $current_report"},{"line_number":66,"context_line":"    echo \"Please write more unit tests, we should keep our test coverage :( \""},{"line_number":67,"context_line":"    exit_code\u003d1"},{"line_number":68,"context_line":"fi"},{"line_number":69,"context_line":""}],"source_content_type":"text/x-sh","patch_set":3,"id":"9ad45d7e_e5193124","line":66,"range":{"start_line":66,"start_character":43,"end_line":66,"end_character":54},"updated":"2016-08-12 20:38:37.000000000","message":"must maintain","commit_id":"6f42fd57755b282f05319bfc93e44b0a2b877cca"},{"author":{"_account_id":20498,"name":"Spyros Trigazis","email":"spyridon.trigazis@cern.ch","username":"strigazi"},"change_message_id":"3d38d7ade86bb6a92b26eb65efd9494f458bb523","unresolved":false,"context_lines":[{"line_number":63,"context_line":"    exit_code\u003d0"},{"line_number":64,"context_line":"else"},{"line_number":65,"context_line":"    show_diff $baseline_report $current_report"},{"line_number":66,"context_line":"    echo \"Please write more unit tests, we should keep our test coverage :( \""},{"line_number":67,"context_line":"    exit_code\u003d1"},{"line_number":68,"context_line":"fi"},{"line_number":69,"context_line":""}],"source_content_type":"text/x-sh","patch_set":3,"id":"fa7ab95a_99f6acf7","line":66,"range":{"start_line":66,"start_character":43,"end_line":66,"end_character":54},"in_reply_to":"9ad45d7e_e5193124","updated":"2016-08-29 12:30:25.000000000","message":"Done","commit_id":"6f42fd57755b282f05319bfc93e44b0a2b877cca"}],"tox.ini":[{"author":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"change_message_id":"cc8198a3401d328b11851e82aede74cddea5ae92","unresolved":false,"context_lines":[{"line_number":94,"context_line":"commands \u003d bandit -r magnum -x tests -n5 -ll"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"[testenv:cover]"},{"line_number":97,"context_line":"# NOTE(NiallBunting) Infra does not support constraints for the cover"},{"line_number":98,"context_line":"# job. While the file is set no file is there. Can be removed once infra"},{"line_number":99,"context_line":"# changes this."},{"line_number":100,"context_line":"install_command \u003d pip install -U {opts} {packages}"}],"source_content_type":"text/x-properties","patch_set":3,"id":"9ad45d7e_0c1c7276","side":"PARENT","line":97,"range":{"start_line":97,"start_character":2,"end_line":97,"end_character":6},"updated":"2016-08-12 19:47:35.000000000","message":"Just to make sure, do you intend to remove this note? Does that mean infra already support constraints for the cover job? If yest, link?","commit_id":"fc9c1a8fc0d51185a8aec2f9f57d482c52266628"},{"author":{"_account_id":20498,"name":"Spyros Trigazis","email":"spyridon.trigazis@cern.ch","username":"strigazi"},"change_message_id":"3d38d7ade86bb6a92b26eb65efd9494f458bb523","unresolved":false,"context_lines":[{"line_number":94,"context_line":"commands \u003d bandit -r magnum -x tests -n5 -ll"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"[testenv:cover]"},{"line_number":97,"context_line":"# NOTE(NiallBunting) Infra does not support constraints for the cover"},{"line_number":98,"context_line":"# job. While the file is set no file is there. Can be removed once infra"},{"line_number":99,"context_line":"# changes this."},{"line_number":100,"context_line":"install_command \u003d pip install -U {opts} {packages}"}],"source_content_type":"text/x-properties","patch_set":3,"id":"fa7ab95a_39a3e0b3","side":"PARENT","line":97,"range":{"start_line":97,"start_character":2,"end_line":97,"end_character":6},"in_reply_to":"9ad45d7e_05be05d4","updated":"2016-08-29 12:30:25.000000000","message":"Addressed by: https://review.openstack.org/#/c/356510/","commit_id":"fc9c1a8fc0d51185a8aec2f9f57d482c52266628"},{"author":{"_account_id":668,"name":"Adrian Otto","email":"aotto@aotto.com","username":"aotto"},"change_message_id":"2b29b1f20aadf5ea00f113d4074c9431ed93a594","unresolved":false,"context_lines":[{"line_number":94,"context_line":"commands \u003d bandit -r magnum -x tests -n5 -ll"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"[testenv:cover]"},{"line_number":97,"context_line":"# NOTE(NiallBunting) Infra does not support constraints for the cover"},{"line_number":98,"context_line":"# job. While the file is set no file is there. Can be removed once infra"},{"line_number":99,"context_line":"# changes this."},{"line_number":100,"context_line":"install_command \u003d pip install -U {opts} {packages}"}],"source_content_type":"text/x-properties","patch_set":3,"id":"9ad45d7e_05be05d4","side":"PARENT","line":97,"range":{"start_line":97,"start_character":2,"end_line":97,"end_character":6},"in_reply_to":"9ad45d7e_0c1c7276","updated":"2016-08-12 20:38:37.000000000","message":"Another good catch. I\u0027m curious about this too.","commit_id":"fc9c1a8fc0d51185a8aec2f9f57d482c52266628"}]}