)]}'
{"neutron/tests/contrib/post_test_hook.sh":[{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"f1a28a566c164231793c6a37d09e780cc2b2c345","unresolved":false,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"function generate_log_index {"},{"line_number":24,"context_line":"    virtualenv /tmp/os-log-merger"},{"line_number":25,"context_line":"    /tmp/os-log-merger/bin/pip install -U os-log-merger\u003d\u003d1.0.6"},{"line_number":26,"context_line":"    # switch directory to avoid full log path inclusion into index log (name is enough)"},{"line_number":27,"context_line":"    pushd /opt/stack/logs/$venv-logs/"},{"line_number":28,"context_line":"    /tmp/os-log-merger/bin/os-log-merger *.txt | sudo tee /opt/stack/logs/$venv-index.txt"}],"source_content_type":"text/x-sh","patch_set":4,"id":"9a30ddce_2172cb7b","line":25,"updated":"2017-03-09 22:14:29.000000000","message":"I wonder if we should negotiate that with infra, like maybe they should mirror it properly, or smth.","commit_id":"b27111ebc47b554fdb903bc4c22e2a78be6f3712"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"f1a28a566c164231793c6a37d09e780cc2b2c345","unresolved":false,"context_lines":[{"line_number":25,"context_line":"    /tmp/os-log-merger/bin/pip install -U os-log-merger\u003d\u003d1.0.6"},{"line_number":26,"context_line":"    # switch directory to avoid full log path inclusion into index log (name is enough)"},{"line_number":27,"context_line":"    pushd /opt/stack/logs/$venv-logs/"},{"line_number":28,"context_line":"    /tmp/os-log-merger/bin/os-log-merger *.txt | sudo tee /opt/stack/logs/$venv-index.txt"},{"line_number":29,"context_line":"    popd"},{"line_number":30,"context_line":"}"},{"line_number":31,"context_line":""}],"source_content_type":"text/x-sh","patch_set":4,"id":"9a30ddce_8107f7c8","line":28,"updated":"2017-03-09 22:14:29.000000000","message":"gotta include subdirs for fullstack.","commit_id":"b27111ebc47b554fdb903bc4c22e2a78be6f3712"},{"author":{"_account_id":8788,"name":"Miguel Angel Ajo","email":"mangelajo@redhat.com","username":"mangelajo"},"change_message_id":"53e3b677bab348efa839953059b81384d4c10819","unresolved":false,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"function generate_log_index {"},{"line_number":24,"context_line":"    virtualenv /tmp/os-log-merger"},{"line_number":25,"context_line":"    /tmp/os-log-merger/bin/pip install -U os-log-merger\u003d\u003d1.0.6"},{"line_number":26,"context_line":"    # switch directory to avoid full log path inclusion into index log (name is enough)"},{"line_number":27,"context_line":"    pushd /opt/stack/logs/$venv-logs/"},{"line_number":28,"context_line":"    /tmp/os-log-merger/bin/os-log-merger `find . -name \u0027*.txt\u0027` | sudo tee /opt/stack/logs/$venv-index.txt"}],"source_content_type":"text/x-sh","patch_set":6,"id":"9a30ddce_99a2f597","line":25,"updated":"2017-03-10 11:42:37.000000000","message":"awesome! ;)","commit_id":"1e82a85dabcdb1a838711fc61d43bad0c8332c7f"},{"author":{"_account_id":8788,"name":"Miguel Angel Ajo","email":"mangelajo@redhat.com","username":"mangelajo"},"change_message_id":"53e3b677bab348efa839953059b81384d4c10819","unresolved":false,"context_lines":[{"line_number":25,"context_line":"    /tmp/os-log-merger/bin/pip install -U os-log-merger\u003d\u003d1.0.6"},{"line_number":26,"context_line":"    # switch directory to avoid full log path inclusion into index log (name is enough)"},{"line_number":27,"context_line":"    pushd /opt/stack/logs/$venv-logs/"},{"line_number":28,"context_line":"    /tmp/os-log-merger/bin/os-log-merger `find . -name \u0027*.txt\u0027` | sudo tee /opt/stack/logs/$venv-index.txt"},{"line_number":29,"context_line":"    popd"},{"line_number":30,"context_line":"}"},{"line_number":31,"context_line":""}],"source_content_type":"text/x-sh","patch_set":6,"id":"9a30ddce_f9905145","line":28,"updated":"2017-03-10 11:42:37.000000000","message":"Please, let\u0027s try adding -a3 as an option to os-log-merger, It will find the common parts (at least the front) of the filenames and strip it.","commit_id":"1e82a85dabcdb1a838711fc61d43bad0c8332c7f"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"7f27f3a01444e6417798dd14eb210be21569007c","unresolved":false,"context_lines":[{"line_number":24,"context_line":"    virtualenv /tmp/os-log-merger"},{"line_number":25,"context_line":"    /tmp/os-log-merger/bin/pip install -U os-log-merger\u003d\u003d1.0.6"},{"line_number":26,"context_line":"    # -a3 to truncate common path prefix"},{"line_number":27,"context_line":"    contents\u003d$(/tmp/os-log-merger/bin/os-log-merger -a3 `find . -name \u0027*.txt\u0027 -o -name \u0027*.log\u0027`)"},{"line_number":28,"context_line":"    echo $contents | sudo tee /opt/stack/logs/$venv-index.txt"},{"line_number":29,"context_line":"}"},{"line_number":30,"context_line":""}],"source_content_type":"text/x-sh","patch_set":9,"id":"9a30ddce_38eadaac","line":27,"range":{"start_line":27,"start_character":56,"end_line":27,"end_character":57},"updated":"2017-03-10 15:23:23.000000000","message":"Would be good to stick to one format: $(command) instead of `command` as $() is preferred.","commit_id":"b422f190c18f05e6267da9c131d89b15afc15bdf"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"ee6e60790d7d497ba928a007995cb16f06daaa9d","unresolved":false,"context_lines":[{"line_number":24,"context_line":"    virtualenv /tmp/os-log-merger"},{"line_number":25,"context_line":"    /tmp/os-log-merger/bin/pip install -U os-log-merger\u003d\u003d1.0.6"},{"line_number":26,"context_line":"    # -a3 to truncate common path prefix"},{"line_number":27,"context_line":"    contents\u003d$(/tmp/os-log-merger/bin/os-log-merger -a3 `find . -name \u0027*.txt\u0027 -o -name \u0027*.log\u0027`)"},{"line_number":28,"context_line":"    echo $contents | sudo tee /opt/stack/logs/$venv-index.txt"},{"line_number":29,"context_line":"}"},{"line_number":30,"context_line":""}],"source_content_type":"text/x-sh","patch_set":9,"id":"9a30ddce_731f1508","line":27,"range":{"start_line":27,"start_character":56,"end_line":27,"end_character":57},"in_reply_to":"9a30ddce_38eadaac","updated":"2017-03-10 15:36:19.000000000","message":"Oh code style police! :) Will it work with two-layered $()? I can try.","commit_id":"b422f190c18f05e6267da9c131d89b15afc15bdf"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"5fbba051bb756825613e229a7908ca559564528a","unresolved":false,"context_lines":[{"line_number":24,"context_line":"    virtualenv /tmp/os-log-merger"},{"line_number":25,"context_line":"    /tmp/os-log-merger/bin/pip install -U os-log-merger\u003d\u003d1.0.6"},{"line_number":26,"context_line":"    # -a3 to truncate common path prefix"},{"line_number":27,"context_line":"    contents\u003d$(/tmp/os-log-merger/bin/os-log-merger -a3 `find . -name \u0027*.txt\u0027 -o -name \u0027*.log\u0027`)"},{"line_number":28,"context_line":"    echo $contents | sudo tee /opt/stack/logs/$venv-index.txt"},{"line_number":29,"context_line":"}"},{"line_number":30,"context_line":""}],"source_content_type":"text/x-sh","patch_set":9,"id":"9a30ddce_b3ec2df4","line":27,"range":{"start_line":27,"start_character":56,"end_line":27,"end_character":57},"in_reply_to":"9a30ddce_731f1508","updated":"2017-03-10 15:36:56.000000000","message":"Ofc I can always add another line to calculate the file names.","commit_id":"b422f190c18f05e6267da9c131d89b15afc15bdf"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"13c6b9fd2878a89263300f5362b90c690e11530f","unresolved":false,"context_lines":[{"line_number":24,"context_line":"    virtualenv /tmp/os-log-merger"},{"line_number":25,"context_line":"    /tmp/os-log-merger/bin/pip install -U os-log-merger\u003d\u003d1.0.6"},{"line_number":26,"context_line":"    # -a3 to truncate common path prefix"},{"line_number":27,"context_line":"    contents\u003d$(/tmp/os-log-merger/bin/os-log-merger -a3 `find . -name \u0027*.txt\u0027 -o -name \u0027*.log\u0027`)"},{"line_number":28,"context_line":"    echo $contents | sudo tee /opt/stack/logs/$venv-index.txt"},{"line_number":29,"context_line":"}"},{"line_number":30,"context_line":""}],"source_content_type":"text/x-sh","patch_set":9,"id":"9a30ddce_42a5dc44","line":27,"range":{"start_line":27,"start_character":56,"end_line":27,"end_character":57},"in_reply_to":"9a30ddce_b3ec2df4","updated":"2017-03-10 16:24:18.000000000","message":"Nested $() works. That\u0027s the difference from `` :)","commit_id":"b422f190c18f05e6267da9c131d89b15afc15bdf"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"82fbfda0d4d0ab1fc8c294c34a3be213fe66e7ea","unresolved":false,"context_lines":[{"line_number":25,"context_line":"    /tmp/os-log-merger/bin/pip install -U os-log-merger\u003d\u003d1.0.6"},{"line_number":26,"context_line":"    files\u003d$(find /opt/stack/logs/$venv-logs -name \u0027*.txt\u0027 -o -name \u0027*.log\u0027)"},{"line_number":27,"context_line":"    # -a3 to truncate common path prefix"},{"line_number":28,"context_line":"    contents\u003d$(/tmp/os-log-merger/bin/os-log-merger -a3 $files)"},{"line_number":29,"context_line":"    echo $contents | sudo tee /opt/stack/logs/$venv-index.txt"},{"line_number":30,"context_line":"}"},{"line_number":31,"context_line":""}],"source_content_type":"text/x-sh","patch_set":10,"id":"9a30ddce_f4fbe676","line":28,"updated":"2017-03-10 18:18:47.000000000","message":"we should strip messages lower than INFO since those are not indexed, and to save space.","commit_id":"137d6ebd32921dd168747493b7fc1042feb142dd"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"f03d9a47adaf6b14f3b372b22f1b8c3284525b42","unresolved":false,"context_lines":[{"line_number":25,"context_line":"    /tmp/os-log-merger/bin/pip install -U os-log-merger\u003d\u003d1.0.6"},{"line_number":26,"context_line":"    files\u003d$(find /opt/stack/logs/$venv-logs -name \u0027*.txt\u0027 -o -name \u0027*.log\u0027)"},{"line_number":27,"context_line":"    # -a3 to truncate common path prefix"},{"line_number":28,"context_line":"    contents\u003d$(/tmp/os-log-merger/bin/os-log-merger -a3 $files)"},{"line_number":29,"context_line":"    echo $contents | sudo tee /opt/stack/logs/$venv-index.txt"},{"line_number":30,"context_line":"}"},{"line_number":31,"context_line":""}],"source_content_type":"text/x-sh","patch_set":10,"id":"9a30ddce_14da3ae8","line":28,"in_reply_to":"9a30ddce_f4fbe676","updated":"2017-03-10 18:24:23.000000000","message":"this seems to collect everything into a single string now. :-x","commit_id":"137d6ebd32921dd168747493b7fc1042feb142dd"},{"author":{"_account_id":8726,"name":"Victor Morales","email":"chipahuac@hotmail.com","username":"electrocucaracha"},"change_message_id":"44d2b0e8edd0d703d89f3ea83639e3fc0bc217c7","unresolved":false,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"function generate_log_index {"},{"line_number":24,"context_line":"    virtualenv /tmp/os-log-merger"},{"line_number":25,"context_line":"    /tmp/os-log-merger/bin/pip install -U os-log-merger\u003d\u003d1.0.6"},{"line_number":26,"context_line":"    files\u003d$(find /opt/stack/logs/$venv-logs -name \u0027*.txt\u0027 -o -name \u0027*.log\u0027)"},{"line_number":27,"context_line":"    # -a3 to truncate common path prefix"},{"line_number":28,"context_line":"    contents\u003d$(/tmp/os-log-merger/bin/os-log-merger -a3 $files)"}],"source_content_type":"text/x-sh","patch_set":11,"id":"9a30ddce_287b6229","line":25,"updated":"2017-03-10 18:44:52.000000000","message":"I just wondering if this installation could affect the others dependencies and maybe could be required to add an uninstall process before ending this function.","commit_id":"ffc55a0334fed225b5e214657c6d2630d3793549"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"da586831d402e6f1d5cdd3b53507c8dfc641daae","unresolved":false,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"function generate_log_index {"},{"line_number":24,"context_line":"    virtualenv /tmp/os-log-merger"},{"line_number":25,"context_line":"    /tmp/os-log-merger/bin/pip install -U os-log-merger\u003d\u003d1.0.6"},{"line_number":26,"context_line":"    files\u003d$(find /opt/stack/logs/$venv-logs -name \u0027*.txt\u0027 -o -name \u0027*.log\u0027)"},{"line_number":27,"context_line":"    # -a3 to truncate common path prefix"},{"line_number":28,"context_line":"    contents\u003d$(/tmp/os-log-merger/bin/os-log-merger -a3 $files)"}],"source_content_type":"text/x-sh","patch_set":11,"id":"9a30ddce_482096f7","line":25,"in_reply_to":"9a30ddce_287b6229","updated":"2017-03-10 18:49:52.000000000","message":"The gate node will be scrapped after run.","commit_id":"ffc55a0334fed225b5e214657c6d2630d3793549"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"da55b8d34c24eece03f673fd2536bf6d40e72564","unresolved":false,"context_lines":[{"line_number":32,"context_line":"    contents\u003d$(/tmp/os-log-merger/bin/os-log-merger -a3 $files)"},{"line_number":33,"context_line":"    # don\u0027t store DEBUG level messages because they are not very useful,"},{"line_number":34,"context_line":"    # and are not indexed by logstash anyway"},{"line_number":35,"context_line":"    echo \"$contents\" | grep -v DEBUG | sudo tee /opt/stack/logs/$venv-index.txt"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"    $xtrace"},{"line_number":38,"context_line":"}"}],"source_content_type":"text/x-sh","patch_set":12,"id":"9a30ddce_da2d08ff","line":35,"range":{"start_line":35,"start_character":44,"end_line":35,"end_character":47},"updated":"2017-03-13 10:42:05.000000000","message":"It\u0027s actually \u0027tee\u0027 who does it. From man page:\n\n\"Copy standard input to each FILE, and also to standard output.\"\n\nso we probably need \"\u003e /dev/null\"","commit_id":"f6d474d3b5c0c43af3d10fc9250291c64758ede6"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"ba6de790cc76652c0703c0c6ad8aa46722bdbab4","unresolved":false,"context_lines":[{"line_number":32,"context_line":"    contents\u003d$(/tmp/os-log-merger/bin/os-log-merger -a3 $files)"},{"line_number":33,"context_line":"    # don\u0027t store DEBUG level messages because they are not very useful,"},{"line_number":34,"context_line":"    # and are not indexed by logstash anyway"},{"line_number":35,"context_line":"    echo \"$contents\" | grep -v DEBUG | sudo tee /opt/stack/logs/$venv-index.txt"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"    $xtrace"},{"line_number":38,"context_line":"}"}],"source_content_type":"text/x-sh","patch_set":12,"id":"9a30ddce_9ae63042","line":35,"range":{"start_line":35,"start_character":9,"end_line":35,"end_character":20},"updated":"2017-03-13 10:31:26.000000000","message":"This is still dumping all the INFO level logs to the console. Maybe just following would be sufficient?\n\n /tmp/os-log-merger/bin/os-log-merger -a3 $files | grep -v DEBUG | sude tee /opt/stack/logs/$venv-index.txt","commit_id":"f6d474d3b5c0c43af3d10fc9250291c64758ede6"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"09d89cad39e962c6ca7186fe0d855fad9ae979e1","unresolved":false,"context_lines":[{"line_number":32,"context_line":"    contents\u003d$(/tmp/os-log-merger/bin/os-log-merger -a3 $files)"},{"line_number":33,"context_line":"    # don\u0027t store DEBUG level messages because they are not very useful,"},{"line_number":34,"context_line":"    # and are not indexed by logstash anyway"},{"line_number":35,"context_line":"    echo \"$contents\" | grep -v DEBUG | sudo tee /opt/stack/logs/$venv-index.txt"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"    $xtrace"},{"line_number":38,"context_line":"}"}],"source_content_type":"text/x-sh","patch_set":12,"id":"9a30ddce_62246a0c","line":35,"range":{"start_line":35,"start_character":9,"end_line":35,"end_character":20},"in_reply_to":"9a30ddce_02400e06","updated":"2017-03-13 15:16:36.000000000","message":"We do want to index INFO messages - my complaint was that we still see the \"$contents\" in console.log: http://logs.openstack.org/04/442804/12/check/gate-neutron-dsvm-functional-ubuntu-xenial/aafd4ca/console.html#_2017-03-13_05_33_46_950689","commit_id":"f6d474d3b5c0c43af3d10fc9250291c64758ede6"},{"author":{"_account_id":8726,"name":"Victor Morales","email":"chipahuac@hotmail.com","username":"electrocucaracha"},"change_message_id":"44be9e3ce052aa6f32f93cb16699d424933096b3","unresolved":false,"context_lines":[{"line_number":32,"context_line":"    contents\u003d$(/tmp/os-log-merger/bin/os-log-merger -a3 $files)"},{"line_number":33,"context_line":"    # don\u0027t store DEBUG level messages because they are not very useful,"},{"line_number":34,"context_line":"    # and are not indexed by logstash anyway"},{"line_number":35,"context_line":"    echo \"$contents\" | grep -v DEBUG | sudo tee /opt/stack/logs/$venv-index.txt"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"    $xtrace"},{"line_number":38,"context_line":"}"}],"source_content_type":"text/x-sh","patch_set":12,"id":"9a30ddce_02400e06","line":35,"range":{"start_line":35,"start_character":9,"end_line":35,"end_character":20},"in_reply_to":"9a30ddce_9ae63042","updated":"2017-03-13 15:10:17.000000000","message":"If the idea is to avoid the INFO messages too, shouldn\u0027t need to be defined in this way?\n\n  echo \"$contents\" | grep -v \"DEBUG\\|INFO\" | sudo tee /opt/stack/logs/$venv-index.txt\n\nAnd also update the comment","commit_id":"f6d474d3b5c0c43af3d10fc9250291c64758ede6"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"72cd900bdade3a660ae84b83a7e0c2ab0d17db93","unresolved":false,"context_lines":[{"line_number":32,"context_line":"    contents\u003d$(/tmp/os-log-merger/bin/os-log-merger -a3 $files)"},{"line_number":33,"context_line":"    # don\u0027t store DEBUG level messages because they are not very useful,"},{"line_number":34,"context_line":"    # and are not indexed by logstash anyway"},{"line_number":35,"context_line":"    echo \"$contents\" | grep -v DEBUG | sudo tee /opt/stack/logs/$venv-index.txt \u003e /dev/null"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"    $xtrace"},{"line_number":38,"context_line":"}"}],"source_content_type":"text/x-sh","patch_set":13,"id":"9a30ddce_d62b1780","line":35,"range":{"start_line":35,"start_character":31,"end_line":35,"end_character":36},"updated":"2017-03-13 15:24:01.000000000","message":"Now we must decide how paranoid we want to be: what if the important ERROR level message contains the word \"DEBUG\" :)","commit_id":"cfd47928ffa469656192a4cc12b7547fb01ed69e"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"6ca0c68aa382b6e02709a428dc5fbd3346543d21","unresolved":false,"context_lines":[{"line_number":32,"context_line":"    contents\u003d$(/tmp/os-log-merger/bin/os-log-merger -a3 $files)"},{"line_number":33,"context_line":"    # don\u0027t store DEBUG level messages because they are not very useful,"},{"line_number":34,"context_line":"    # and are not indexed by logstash anyway"},{"line_number":35,"context_line":"    echo \"$contents\" | grep -v DEBUG | sudo tee /opt/stack/logs/$venv-index.txt \u003e /dev/null"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"    $xtrace"},{"line_number":38,"context_line":"}"}],"source_content_type":"text/x-sh","patch_set":13,"id":"7a25e98e_cf7b757a","line":35,"range":{"start_line":35,"start_character":31,"end_line":35,"end_character":36},"in_reply_to":"7a25e98e_8f9a5dc4","updated":"2017-03-13 19:05:44.000000000","message":"Or was it fixed lately in pypi? In the past it was possible.","commit_id":"cfd47928ffa469656192a4cc12b7547fb01ed69e"},{"author":{"_account_id":8788,"name":"Miguel Angel Ajo","email":"mangelajo@redhat.com","username":"mangelajo"},"change_message_id":"9f0bb8e3c86d71bf6054723980a8830b20d7281b","unresolved":false,"context_lines":[{"line_number":32,"context_line":"    contents\u003d$(/tmp/os-log-merger/bin/os-log-merger -a3 $files)"},{"line_number":33,"context_line":"    # don\u0027t store DEBUG level messages because they are not very useful,"},{"line_number":34,"context_line":"    # and are not indexed by logstash anyway"},{"line_number":35,"context_line":"    echo \"$contents\" | grep -v DEBUG | sudo tee /opt/stack/logs/$venv-index.txt \u003e /dev/null"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"    $xtrace"},{"line_number":38,"context_line":"}"}],"source_content_type":"text/x-sh","patch_set":13,"id":"3a1ff146_0fa1eaa9","line":35,"range":{"start_line":35,"start_character":31,"end_line":35,"end_character":36},"in_reply_to":"7a25e98e_cf7b757a","updated":"2017-03-15 10:00:30.000000000","message":":P :D\n\nAbout the log level filtering, it\u0027s been proposed before, it\u0027d be a nice enhancement.","commit_id":"cfd47928ffa469656192a4cc12b7547fb01ed69e"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"f001eaf2fb6cde948d066cdd634608bb837a3ac6","unresolved":false,"context_lines":[{"line_number":32,"context_line":"    contents\u003d$(/tmp/os-log-merger/bin/os-log-merger -a3 $files)"},{"line_number":33,"context_line":"    # don\u0027t store DEBUG level messages because they are not very useful,"},{"line_number":34,"context_line":"    # and are not indexed by logstash anyway"},{"line_number":35,"context_line":"    echo \"$contents\" | grep -v DEBUG | sudo tee /opt/stack/logs/$venv-index.txt \u003e /dev/null"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"    $xtrace"},{"line_number":38,"context_line":"}"}],"source_content_type":"text/x-sh","patch_set":13,"id":"7a25e98e_8f9a5dc4","line":35,"range":{"start_line":35,"start_character":31,"end_line":35,"end_character":36},"in_reply_to":"7a25e98e_ec5e7f75","updated":"2017-03-13 19:05:07.000000000","message":"ajo may still drop the version from pypi!","commit_id":"cfd47928ffa469656192a4cc12b7547fb01ed69e"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"a8f80ce07bcdff3463dbafd89c71a075e1ecea83","unresolved":false,"context_lines":[{"line_number":32,"context_line":"    contents\u003d$(/tmp/os-log-merger/bin/os-log-merger -a3 $files)"},{"line_number":33,"context_line":"    # don\u0027t store DEBUG level messages because they are not very useful,"},{"line_number":34,"context_line":"    # and are not indexed by logstash anyway"},{"line_number":35,"context_line":"    echo \"$contents\" | grep -v DEBUG | sudo tee /opt/stack/logs/$venv-index.txt \u003e /dev/null"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"    $xtrace"},{"line_number":38,"context_line":"}"}],"source_content_type":"text/x-sh","patch_set":13,"id":"9a30ddce_794dfcc2","line":35,"range":{"start_line":35,"start_character":31,"end_line":35,"end_character":36},"in_reply_to":"9a30ddce_3970e40a","updated":"2017-03-13 15:58:56.000000000","message":"Talking about paranoia, we should probably take ajo\u0027s children as hostages for in case he uploads a broken package to pypi.","commit_id":"cfd47928ffa469656192a4cc12b7547fb01ed69e"},{"author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"change_message_id":"700475b4e75fbf60a91cb14b84317663173aae75","unresolved":false,"context_lines":[{"line_number":32,"context_line":"    contents\u003d$(/tmp/os-log-merger/bin/os-log-merger -a3 $files)"},{"line_number":33,"context_line":"    # don\u0027t store DEBUG level messages because they are not very useful,"},{"line_number":34,"context_line":"    # and are not indexed by logstash anyway"},{"line_number":35,"context_line":"    echo \"$contents\" | grep -v DEBUG | sudo tee /opt/stack/logs/$venv-index.txt \u003e /dev/null"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"    $xtrace"},{"line_number":38,"context_line":"}"}],"source_content_type":"text/x-sh","patch_set":13,"id":"7a25e98e_ec5e7f75","line":35,"range":{"start_line":35,"start_character":31,"end_line":35,"end_character":36},"in_reply_to":"9a30ddce_794dfcc2","updated":"2017-03-13 18:54:36.000000000","message":"we\u0027re protected by pinning to a specific version. I don\u0027t believe pypi lets you replace anymore at the same version level :)","commit_id":"cfd47928ffa469656192a4cc12b7547fb01ed69e"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"1cbab59f371b7b37aea309aef8dad41747d9b221","unresolved":false,"context_lines":[{"line_number":32,"context_line":"    contents\u003d$(/tmp/os-log-merger/bin/os-log-merger -a3 $files)"},{"line_number":33,"context_line":"    # don\u0027t store DEBUG level messages because they are not very useful,"},{"line_number":34,"context_line":"    # and are not indexed by logstash anyway"},{"line_number":35,"context_line":"    echo \"$contents\" | grep -v DEBUG | sudo tee /opt/stack/logs/$venv-index.txt \u003e /dev/null"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"    $xtrace"},{"line_number":38,"context_line":"}"}],"source_content_type":"text/x-sh","patch_set":13,"id":"9a30ddce_3970e40a","line":35,"range":{"start_line":35,"start_character":31,"end_line":35,"end_character":36},"in_reply_to":"9a30ddce_d62b1780","updated":"2017-03-13 15:57:16.000000000","message":"Well, that will be a good RFE for os-log-merger. ;)","commit_id":"cfd47928ffa469656192a4cc12b7547fb01ed69e"}]}