)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},"change_message_id":"446be99d6f904e3fbf4caf8984ca3aa8086059f5","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add models sync with migration and functional tests"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Add the neutron standard model-migration sync test and functional tests."},{"line_number":10,"context_line":"This requires that networking-bagpipe uses the python-db-jobs template"},{"line_number":11,"context_line":"in the gate."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Signed-off-by: Thomas Morin \u003cthomas.morin@orange.com\u003e"},{"line_number":14,"context_line":"Submitted on behalf of a third-party: Orange"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3a71b18c_bf6842f6","line":11,"range":{"start_line":10,"start_character":0,"end_line":11,"end_character":12},"updated":"2016-12-07 10:02:25.000000000","message":"We already do for the python-35 job, but what we need in fact is a new gate job to trigger the new tox \u0027functional\u0027 target.","commit_id":"19655bba4f03055c59d597117f1117275be681c0"},{"author":{"_account_id":19990,"name":"Yannick Thomas","email":"ythomas1.ext@orange.com","username":"ythomas1"},"change_message_id":"08388bce0b941e8a67ca43287555376f8e9fdcca","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add models sync with migration and functional tests"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Add the neutron standard model-migration sync test and functional tests."},{"line_number":10,"context_line":"This requires that networking-bagpipe uses the python-db-jobs template"},{"line_number":11,"context_line":"in the gate."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Signed-off-by: Thomas Morin \u003cthomas.morin@orange.com\u003e"},{"line_number":14,"context_line":"Submitted on behalf of a third-party: Orange"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3a71b18c_7912ca63","line":11,"range":{"start_line":10,"start_character":0,"end_line":11,"end_character":12},"in_reply_to":"3a71b18c_bf6842f6","updated":"2016-12-09 16:48:27.000000000","message":"Remove deprecated line","commit_id":"19655bba4f03055c59d597117f1117275be681c0"}],"tox.ini":[{"author":{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},"change_message_id":"5042cc945860ee89ae7d88dec56c48c70c82364b","unresolved":false,"context_lines":[{"line_number":13,"context_line":"       -r{toxinidir}/test-requirements.txt"},{"line_number":14,"context_line":"commands \u003d"},{"line_number":15,"context_line":"    find . -type f -name \"*.pyc\" -delete"},{"line_number":16,"context_line":"    python setup.py testr --slowest --testr-args\u003d\u0027{posargs}\u0027"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"[testenv:common]"},{"line_number":19,"context_line":"# Fake job to define environment variables shared between dsvm/non-dsvm jobs"}],"source_content_type":"text/x-properties","patch_set":4,"id":"3a71b18c_4aa9c07a","line":16,"range":{"start_line":16,"start_character":0,"end_line":16,"end_character":60},"updated":"2016-12-09 11:14:55.000000000","message":"I\u0027m not sure how necessary it is, but it seems sane to  follow what neutron does and replace by:\n\n  {toxinidir}/tools/ostestr_compat_shim.sh {posargs}","commit_id":"341faa758914aa65ad5f42cf347b815edb2cbc35"},{"author":{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},"change_message_id":"5042cc945860ee89ae7d88dec56c48c70c82364b","unresolved":false,"context_lines":[{"line_number":24,"context_line":"basepython \u003d python2.7"},{"line_number":25,"context_line":"setenv \u003d {[testenv]setenv}"},{"line_number":26,"context_line":"         {[testenv:common]setenv}"},{"line_number":27,"context_line":"         OS_TEST_PATH\u003d./networking_bagpipe/tests/functional"},{"line_number":28,"context_line":"         OS_LOG_PATH\u003d{env:OS_LOG_PATH:/opt/stack/logs}"},{"line_number":29,"context_line":"deps \u003d"},{"line_number":30,"context_line":"  {[testenv]deps}"}],"source_content_type":"text/x-properties","patch_set":4,"id":"3a71b18c_ea028c3d","line":27,"range":{"start_line":27,"start_character":0,"end_line":27,"end_character":59},"updated":"2016-12-09 11:14:55.000000000","message":"This is ignored because, unlike neutron/.testr.conf, networking-bagpipe/.testr.conf does not use this variable.\n\nWe need to update our .testr.conf to something similar to the one from neutron.","commit_id":"341faa758914aa65ad5f42cf347b815edb2cbc35"},{"author":{"_account_id":12021,"name":"Thomas Morin","email":"tmmorin.orange@gmail.com","username":"tmorin"},"change_message_id":"ff357af55f1075123a15a672b078ea89ab181fe2","unresolved":false,"context_lines":[{"line_number":24,"context_line":"basepython \u003d python2.7"},{"line_number":25,"context_line":"setenv \u003d {[testenv]setenv}"},{"line_number":26,"context_line":"         {[testenv:common]setenv}"},{"line_number":27,"context_line":"         OS_TEST_PATH\u003d./networking_bagpipe/tests/functional"},{"line_number":28,"context_line":"         OS_LOG_PATH\u003d{env:OS_LOG_PATH:/opt/stack/logs}"},{"line_number":29,"context_line":"deps \u003d"},{"line_number":30,"context_line":"  {[testenv]deps}"}],"source_content_type":"text/x-properties","patch_set":4,"id":"3a71b18c_99b326e9","line":27,"range":{"start_line":27,"start_character":0,"end_line":27,"end_character":59},"in_reply_to":"3a71b18c_ea028c3d","updated":"2016-12-09 16:36:25.000000000","message":"Now that .testr.conf is modified to use this variable, you can use it, and you\u0027ll have to to have the \u0027functional\u0027 target run the functional suite.","commit_id":"341faa758914aa65ad5f42cf347b815edb2cbc35"}]}