)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"439a9008c8360fab4b596d6899a53b11e1b44cc6","unresolved":false,"context_lines":[{"line_number":14,"context_line":"plumbing underpinning this job so that we can start using some of"},{"line_number":15,"context_line":"the latest cool stuff that Tempest has to offer."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"The switch must be made conditional to branches since the it only"},{"line_number":18,"context_line":"makes sense from Newton onward."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Depends-on: Ia233aa162746845f6ae08a8157dcd242dcd58eab"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"5a18252c_9472ab0f","line":17,"range":{"start_line":17,"start_character":54,"end_line":17,"end_character":57},"updated":"2016-04-14 21:09:42.000000000","message":"s/the//","commit_id":"75ed3f55ee33b185788482b7154ee2cf8fc09c9a"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"58948da477592d482fda54bb98b61c645e133006","unresolved":false,"context_lines":[{"line_number":14,"context_line":"plumbing underpinning this job so that we can start using some of"},{"line_number":15,"context_line":"the latest cool stuff that Tempest has to offer."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"The switch must be made conditional to branches since the it only"},{"line_number":18,"context_line":"makes sense from Newton onward."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Depends-on: Ia233aa162746845f6ae08a8157dcd242dcd58eab"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"5a18252c_217c2eb4","line":17,"range":{"start_line":17,"start_character":54,"end_line":17,"end_character":57},"in_reply_to":"5a18252c_9472ab0f","updated":"2016-04-15 00:59:10.000000000","message":"meh :)","commit_id":"75ed3f55ee33b185788482b7154ee2cf8fc09c9a"}],"jenkins/jobs/neutron.yaml":[{"author":{"_account_id":8873,"name":"Assaf Muller","email":"amuller@redhat.com","username":"amuller"},"change_message_id":"b3531e4b789e2bb7f8cdb50d222593e236bdf01a","unresolved":false,"context_lines":[{"line_number":31,"context_line":"              export DEVSTACK_GATE_TEMPEST_NOTESTS\u003d1"},{"line_number":32,"context_line":"              export DEVSTACK_GATE_INSTALL_TESTONLY\u003d1"},{"line_number":33,"context_line":"          else"},{"line_number":34,"context_line":"              export DEVSTACK_GATE_TEMPEST_ALL_PLUGINS\u003d1"},{"line_number":35,"context_line":"          fi"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"          function gate_hook {{"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"5a18252c_1874c9cd","line":34,"updated":"2016-04-11 21:58:52.000000000","message":"Doesn\u0027t this mean we\u0027ll run all tempest API tests + Neutron\u0027s plugin? To remain at parity with the current API job we need to run *only* the Neutron plugin tests.","commit_id":"aea048b423312909e17c885441ace38978f6ea55"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"894b0cf25df7f494c37e3efe9c92eae7da8fbb17","unresolved":false,"context_lines":[{"line_number":31,"context_line":"              export DEVSTACK_GATE_TEMPEST_NOTESTS\u003d1"},{"line_number":32,"context_line":"              export DEVSTACK_GATE_INSTALL_TESTONLY\u003d1"},{"line_number":33,"context_line":"          else"},{"line_number":34,"context_line":"              export DEVSTACK_GATE_TEMPEST_ALL_PLUGINS\u003d1"},{"line_number":35,"context_line":"          fi"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"          function gate_hook {{"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"5a18252c_f83aade6","line":34,"in_reply_to":"5a18252c_1874c9cd","updated":"2016-04-11 22:01:06.000000000","message":"good point, I should probably add a regex","commit_id":"aea048b423312909e17c885441ace38978f6ea55"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"439a9008c8360fab4b596d6899a53b11e1b44cc6","unresolved":false,"context_lines":[{"line_number":27,"context_line":"          # branches below are dropped. Prior to Newton, the API Tempest tests are"},{"line_number":28,"context_line":"          # invoked explicitly and thus we need the following variables set. From"},{"line_number":29,"context_line":"          # Newton onward, everything is taken care of by the Tempest plugin framework"},{"line_number":30,"context_line":"          if [ \"$ZUUL_BRANCH\" \u003d\u003d \"stable/kilo\" -o \"$ZUUL_BRANCH\" \u003d\u003d \"stable/liberty\" -o \"$ZUUL_BRANCH\" \u003d\u003d \"stable/mitaka\" ] ; then"},{"line_number":31,"context_line":"              export DEVSTACK_GATE_TEMPEST_NOTESTS\u003d1"},{"line_number":32,"context_line":"              export DEVSTACK_GATE_INSTALL_TESTONLY\u003d1"},{"line_number":33,"context_line":"          else"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"5a18252c_ef834654","line":30,"range":{"start_line":30,"start_character":15,"end_line":30,"end_character":121},"updated":"2016-04-14 21:09:42.000000000","message":"Could also do [ \"stable/kilo stable/liberty stable/mitaka\" \u003d~ \"$ZUUL_BRANCH\" ]","commit_id":"75ed3f55ee33b185788482b7154ee2cf8fc09c9a"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"6aea545254dafe4e3b6719783a1104700cd2e336","unresolved":false,"context_lines":[{"line_number":40,"context_line":"          }}"},{"line_number":41,"context_line":"          export -f gate_hook"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"          function post_test_hook {{"},{"line_number":44,"context_line":"              bash -xe $BASE/new/neutron/neutron/tests/contrib/post_test_hook.sh api{which}"},{"line_number":45,"context_line":"          }}"},{"line_number":46,"context_line":"          export -f post_test_hook"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"5a18252c_4f535a6b","line":43,"updated":"2016-04-14 21:21:13.000000000","message":"It seems that the post hook will trigger api tests one more time for us. Should we make it conditional to the branch too, so that the hook is not defined for master?","commit_id":"75ed3f55ee33b185788482b7154ee2cf8fc09c9a"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"b40e0ab59b50dc00b29a905589633fe1f4b6f6bb","unresolved":false,"context_lines":[{"line_number":40,"context_line":"          }}"},{"line_number":41,"context_line":"          export -f gate_hook"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"          function post_test_hook {{"},{"line_number":44,"context_line":"              bash -xe $BASE/new/neutron/neutron/tests/contrib/post_test_hook.sh api{which}"},{"line_number":45,"context_line":"          }}"},{"line_number":46,"context_line":"          export -f post_test_hook"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"5a18252c_017bf2c1","line":43,"in_reply_to":"5a18252c_4f535a6b","updated":"2016-04-15 00:58:45.000000000","message":"Once [1] lands it becomes a no-op for the master branch, and I see no point to skip it in case we do need it later on. If we can tolerate the bump, then I think it\u0027s fine, that\u0027s why the job is non-voting ;)\n\n[1] https://review.openstack.org/#/c/304324/5/neutron/tests/contrib/post_test_hook.sh","commit_id":"75ed3f55ee33b185788482b7154ee2cf8fc09c9a"}]}