)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"7b41f8cd52f151dc3b0822a7cfde570dbbb2d79c","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"CI: Fix check-logs.sh"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This script has a few issues:"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"* It catches false positives, due to log levels in config options."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3fa7e38b_a777b23a","line":9,"range":{"start_line":9,"start_character":12,"end_line":9,"end_character":15},"updated":"2019-09-20 10:40:27.000000000","message":"had :-)","commit_id":"7b4e3d4e38aa0f111b19b65594acacc917cf8241"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"7b41f8cd52f151dc3b0822a7cfde570dbbb2d79c","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"* It catches false positives, due to log levels in config options."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"* It doesn\u0027t fail on CRITICAL logs, due to variable scope issue."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I50c859eb2991e498eeb64bca45daf1e6f237761f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3fa7e38b_8792b64a","line":13,"range":{"start_line":13,"start_character":52,"end_line":13,"end_character":57},"updated":"2019-09-20 10:40:27.000000000","message":"value reset","commit_id":"7b4e3d4e38aa0f111b19b65594acacc917cf8241"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"6054a0d71606a99a9cb8cbc56c7a4a458c598c29","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"* It catches false positives, due to log levels in config options."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"* It doesn\u0027t fail on CRITICAL logs, due to variable scope issue."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I50c859eb2991e498eeb64bca45daf1e6f237761f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3fa7e38b_72ec1629","line":13,"range":{"start_line":13,"start_character":52,"end_line":13,"end_character":57},"in_reply_to":"3fa7e38b_8792b64a","updated":"2019-09-24 09:21:45.000000000","message":"damn, how did I miss that? Blaming lack of sleep","commit_id":"7b4e3d4e38aa0f111b19b65594acacc917cf8241"}],"tests/check-logs.sh":[{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"7b41f8cd52f151dc3b0822a7cfde570dbbb2d79c","unresolved":false,"context_lines":[{"line_number":13,"context_line":"    # $1: file"},{"line_number":14,"context_line":"    # $2: log level"},{"line_number":15,"context_line":"    # Filter out false positives from logged config options."},{"line_number":16,"context_line":"    sudo egrep \"^.* .* .* $level\" $f | egrep -v \"(logging_exception_prefix|rate_limit_except_level)\""},{"line_number":17,"context_line":"}"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"any_critical\u003d0"}],"source_content_type":"text/x-sh","patch_set":1,"id":"3fa7e38b_0779e62d","line":16,"range":{"start_line":16,"start_character":9,"end_line":16,"end_character":33},"updated":"2019-09-20 10:40:27.000000000","message":"not better to grep with perl syntax and do just:\n\"^\\S+ \\S+ \\S+ $level\"\n?","commit_id":"7b4e3d4e38aa0f111b19b65594acacc917cf8241"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"6054a0d71606a99a9cb8cbc56c7a4a458c598c29","unresolved":false,"context_lines":[{"line_number":13,"context_line":"    # $1: file"},{"line_number":14,"context_line":"    # $2: log level"},{"line_number":15,"context_line":"    # Filter out false positives from logged config options."},{"line_number":16,"context_line":"    sudo egrep \"^.* .* .* $level\" $f | egrep -v \"(logging_exception_prefix|rate_limit_except_level)\""},{"line_number":17,"context_line":"}"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"any_critical\u003d0"}],"source_content_type":"text/x-sh","patch_set":1,"id":"3fa7e38b_3281be4b","line":16,"range":{"start_line":16,"start_character":9,"end_line":16,"end_character":33},"in_reply_to":"3fa7e38b_0779e62d","updated":"2019-09-24 09:21:45.000000000","message":"Should probably fix this - there are actually 4 groups before the level in the files I checked.","commit_id":"7b4e3d4e38aa0f111b19b65594acacc917cf8241"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"aba5b0042307b08bfb36ebdab2a551489d01cd46","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"function check_file_for_level {"},{"line_number":13,"context_line":"    # $1: file"},{"line_number":14,"context_line":"    # $2: log level"},{"line_number":15,"context_line":"    # Filter out false positives from logged config options."},{"line_number":16,"context_line":"    sudo egrep \" $level \" $f | egrep -v \"(logging_exception_prefix|rate_limit_except_level)\""},{"line_number":17,"context_line":"}"}],"source_content_type":"text/x-sh","patch_set":2,"id":"3fa7e38b_db1642f2","line":14,"range":{"start_line":13,"start_character":0,"end_line":14,"end_character":19},"updated":"2019-09-20 15:22:01.000000000","message":"you ain\u0027t no using that, homie ;-)","commit_id":"eab8505cd87686570ae21063a3a85ce7d74a927e"}]}