)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":13997,"name":"SHIGEMATSU Mitsuhiro","email":"shigematsu.mitsuhiro@lab.ntt.co.jp","username":"pshige"},"change_message_id":"0d9f1cc555aea53d0bc7f7bfae413fc025e1f053","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix functional tests job"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Fuctional tests job are broken. This change proposes add post"},{"line_number":10,"context_line":"test hook in ironicclient which will be executed inside"},{"line_number":11,"context_line":"post_test_hook function in devstack gate."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"da20952f_769cc069","line":9,"updated":"2015-08-26 23:51:21.000000000","message":"s/Fuctional/Functional/","commit_id":"2fb091ef5332bf1918525847024186d15bc80901"},{"author":{"_account_id":13636,"name":"aarefiev","display_name":"Anton","email":"arefyevanton@gmail.com","username":"aarefiev"},"change_message_id":"05ef3e1dc8388eb62cb5afb7b2c8f0426ea84f13","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix functional tests job"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Fuctional tests job are broken. This change proposes add post"},{"line_number":10,"context_line":"test hook in ironicclient which will be executed inside"},{"line_number":11,"context_line":"post_test_hook function in devstack gate."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"da20952f_5ee98019","line":9,"in_reply_to":"da20952f_769cc069","updated":"2015-08-27 09:03:20.000000000","message":"Done","commit_id":"2fb091ef5332bf1918525847024186d15bc80901"},{"author":{"_account_id":13997,"name":"SHIGEMATSU Mitsuhiro","email":"shigematsu.mitsuhiro@lab.ntt.co.jp","username":"pshige"},"change_message_id":"0d9f1cc555aea53d0bc7f7bfae413fc025e1f053","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"The benefit from this approach is it will immediately pick up"},{"line_number":14,"context_line":"proposed fixes on ironicclient side and runs job. It allows us"},{"line_number":15,"context_line":"to see the change breakes tests or not w/o merging related"},{"line_number":16,"context_line":"changes."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"post_test_hook runs functional tests and generate testr results"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"da20952f_d6810cfd","line":15,"updated":"2015-08-26 23:51:21.000000000","message":"s/breakes/breaks/","commit_id":"2fb091ef5332bf1918525847024186d15bc80901"},{"author":{"_account_id":13636,"name":"aarefiev","display_name":"Anton","email":"arefyevanton@gmail.com","username":"aarefiev"},"change_message_id":"05ef3e1dc8388eb62cb5afb7b2c8f0426ea84f13","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"The benefit from this approach is it will immediately pick up"},{"line_number":14,"context_line":"proposed fixes on ironicclient side and runs job. It allows us"},{"line_number":15,"context_line":"to see the change breakes tests or not w/o merging related"},{"line_number":16,"context_line":"changes."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"post_test_hook runs functional tests and generate testr results"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"da20952f_1ef3080c","line":15,"in_reply_to":"da20952f_d6810cfd","updated":"2015-08-27 09:03:20.000000000","message":"Done","commit_id":"2fb091ef5332bf1918525847024186d15bc80901"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"9f707d40b65d8473e59c4353fa38fba9a462fb9f","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix functional tests job"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Functional tests job are broken. This change proposes add post"},{"line_number":10,"context_line":"test hook in ironicclient, which will be executed inside"},{"line_number":11,"context_line":"post_test_hook function in devstack gate."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"da20952f_e66c4b9e","line":9,"updated":"2015-09-03 10:02:36.000000000","message":"Do we have a bug for it?","commit_id":"3fc1e547335bf33cb9324823fa6393aee8021372"},{"author":{"_account_id":13636,"name":"aarefiev","display_name":"Anton","email":"arefyevanton@gmail.com","username":"aarefiev"},"change_message_id":"eb256f420cc68500a5a5c914c610e979d4bfbb74","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix functional tests job"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Functional tests job are broken. This change proposes add post"},{"line_number":10,"context_line":"test hook in ironicclient, which will be executed inside"},{"line_number":11,"context_line":"post_test_hook function in devstack gate."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"da20952f_2955344f","line":9,"in_reply_to":"da20952f_e66c4b9e","updated":"2015-09-03 10:36:17.000000000","message":"I\u0027ll file new one","commit_id":"3fc1e547335bf33cb9324823fa6393aee8021372"}],"ironicclient/tests/functional/hooks/post_test_hook.sh":[{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"9f707d40b65d8473e59c4353fa38fba9a462fb9f","unresolved":false,"context_lines":[{"line_number":18,"context_line":"    if [ -f .testrepository/0 ]; then"},{"line_number":19,"context_line":"        sudo .tox/functional/bin/testr last --subunit \u003e $WORKSPACE/testrepository.subunit"},{"line_number":20,"context_line":"        sudo mv $WORKSPACE/testrepository.subunit $BASE/logs/testrepository.subunit"},{"line_number":21,"context_line":"        sudo .tox/functional/bin/python /usr/local/jenkins/slave_scripts/subunit2html.py $BASE/logs/testrepository.subunit $BASE/logs/testr_results.html"},{"line_number":22,"context_line":"        sudo gzip -9 $BASE/logs/testrepository.subunit"},{"line_number":23,"context_line":"        sudo gzip -9 $BASE/logs/testr_results.html"},{"line_number":24,"context_line":"        sudo chown jenkins:jenkins $BASE/logs/testrepository.subunit.gz $BASE/logs/testr_results.html.gz"}],"source_content_type":"text/x-sh","patch_set":4,"id":"da20952f_26cfa380","line":21,"updated":"2015-09-03 10:02:36.000000000","message":"For this line, should we care about: https://review.openstack.org/#/c/219835/ ?\n\nIt seems that the subunit2html.py have changed places on the images in the gate.","commit_id":"3fc1e547335bf33cb9324823fa6393aee8021372"},{"author":{"_account_id":13636,"name":"aarefiev","display_name":"Anton","email":"arefyevanton@gmail.com","username":"aarefiev"},"change_message_id":"eb256f420cc68500a5a5c914c610e979d4bfbb74","unresolved":false,"context_lines":[{"line_number":18,"context_line":"    if [ -f .testrepository/0 ]; then"},{"line_number":19,"context_line":"        sudo .tox/functional/bin/testr last --subunit \u003e $WORKSPACE/testrepository.subunit"},{"line_number":20,"context_line":"        sudo mv $WORKSPACE/testrepository.subunit $BASE/logs/testrepository.subunit"},{"line_number":21,"context_line":"        sudo .tox/functional/bin/python /usr/local/jenkins/slave_scripts/subunit2html.py $BASE/logs/testrepository.subunit $BASE/logs/testr_results.html"},{"line_number":22,"context_line":"        sudo gzip -9 $BASE/logs/testrepository.subunit"},{"line_number":23,"context_line":"        sudo gzip -9 $BASE/logs/testr_results.html"},{"line_number":24,"context_line":"        sudo chown jenkins:jenkins $BASE/logs/testrepository.subunit.gz $BASE/logs/testr_results.html.gz"}],"source_content_type":"text/x-sh","patch_set":4,"id":"da20952f_a97944fe","line":21,"in_reply_to":"da20952f_26cfa380","updated":"2015-09-03 10:36:17.000000000","message":"Thanks, will update","commit_id":"3fc1e547335bf33cb9324823fa6393aee8021372"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"9f707d40b65d8473e59c4353fa38fba9a462fb9f","unresolved":false,"context_lines":[{"line_number":35,"context_line":"# Run tests"},{"line_number":36,"context_line":"echo \"Running ironicclient functional test suite\""},{"line_number":37,"context_line":"set +e"},{"line_number":38,"context_line":"# Preserve env for OS_ credentials"},{"line_number":39,"context_line":"sudo -E -H -u jenkins ./tools/run_functional.sh"},{"line_number":40,"context_line":"EXIT_CODE\u003d$?"},{"line_number":41,"context_line":"set -e"}],"source_content_type":"text/x-sh","patch_set":4,"id":"da20952f_46f41786","line":38,"updated":"2015-09-03 10:02:36.000000000","message":"Ironic being a admin only API, do we need a way to ensure we have the admin credentials here?","commit_id":"3fc1e547335bf33cb9324823fa6393aee8021372"},{"author":{"_account_id":13636,"name":"aarefiev","display_name":"Anton","email":"arefyevanton@gmail.com","username":"aarefiev"},"change_message_id":"eb256f420cc68500a5a5c914c610e979d4bfbb74","unresolved":false,"context_lines":[{"line_number":35,"context_line":"# Run tests"},{"line_number":36,"context_line":"echo \"Running ironicclient functional test suite\""},{"line_number":37,"context_line":"set +e"},{"line_number":38,"context_line":"# Preserve env for OS_ credentials"},{"line_number":39,"context_line":"sudo -E -H -u jenkins ./tools/run_functional.sh"},{"line_number":40,"context_line":"EXIT_CODE\u003d$?"},{"line_number":41,"context_line":"set -e"}],"source_content_type":"text/x-sh","patch_set":4,"id":"da20952f_49698824","line":38,"in_reply_to":"da20952f_46f41786","updated":"2015-09-03 10:36:17.000000000","message":"I\u0027ll add explicitly set of admin credentials","commit_id":"3fc1e547335bf33cb9324823fa6393aee8021372"}]}