)]}'
{".zuul.yaml":[{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"fe349870bed7462f893924693e4207674024f0da","unresolved":false,"context_lines":[{"line_number":381,"context_line":"              - ^releasenotes/.*$"},{"line_number":382,"context_line":"              - ^setup.cfg$"},{"line_number":383,"context_line":"              - ^tempest/hacking/.*$"},{"line_number":384,"context_line":"              - ^tempest/tests/.*$"},{"line_number":385,"context_line":"    experimental:"},{"line_number":386,"context_line":"      jobs:"},{"line_number":387,"context_line":"        - tempest-cinder-v2-api:"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"5f7c97a3_d0714679","line":384,"updated":"2018-06-19 22:51:37.000000000","message":"Should we add tempest/api/.*$ here too since this doesn\u0027t run API tests?","commit_id":"bf91e538d87b91edc096cba7a0e64539f3f65246"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"4efd7a8efb5276b2be11037260b01b2e0b6b8132","unresolved":false,"context_lines":[{"line_number":381,"context_line":"              - ^releasenotes/.*$"},{"line_number":382,"context_line":"              - ^setup.cfg$"},{"line_number":383,"context_line":"              - ^tempest/hacking/.*$"},{"line_number":384,"context_line":"              - ^tempest/tests/.*$"},{"line_number":385,"context_line":"    experimental:"},{"line_number":386,"context_line":"      jobs:"},{"line_number":387,"context_line":"        - tempest-cinder-v2-api:"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"5f7c97a3_bad3c45e","line":384,"in_reply_to":"5f7c97a3_d0714679","updated":"2018-07-10 02:06:20.000000000","message":"nice idea, there is no dependency (and should not be) of tempest/api/.* in scenario tests so we are good to skip this job for tempest/api/.* files.","commit_id":"bf91e538d87b91edc096cba7a0e64539f3f65246"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"e596f838d089bd1656acac3a5b39c9930419641a","unresolved":false,"context_lines":[{"line_number":101,"context_line":"      - master"},{"line_number":102,"context_line":"    description: |"},{"line_number":103,"context_line":"      Base multinode integration test with Neutron networking and py27."},{"line_number":104,"context_line":"      Former names for this job where:"},{"line_number":105,"context_line":"        * neutron-tempest-multinode-full"},{"line_number":106,"context_line":"        * legacy-tempest-dsvm-neutron-multinode-full"},{"line_number":107,"context_line":"        * gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial-nv"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"5f7c97a3_f7062121","line":104,"range":{"start_line":104,"start_character":32,"end_line":104,"end_character":37},"updated":"2018-07-10 19:02:00.000000000","message":"were","commit_id":"8db070765eb61652ce0fa266ef3ab4e05d1352a8"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"6f2931c66a388728482481bf514c3fc426751c9f","unresolved":false,"context_lines":[{"line_number":101,"context_line":"      - master"},{"line_number":102,"context_line":"    description: |"},{"line_number":103,"context_line":"      Base multinode integration test with Neutron networking and py27."},{"line_number":104,"context_line":"      Former names for this job where:"},{"line_number":105,"context_line":"        * neutron-tempest-multinode-full"},{"line_number":106,"context_line":"        * legacy-tempest-dsvm-neutron-multinode-full"},{"line_number":107,"context_line":"        * gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial-nv"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"5f7c97a3_af2fe587","line":104,"range":{"start_line":104,"start_character":32,"end_line":104,"end_character":37},"in_reply_to":"5f7c97a3_240e9c3d","updated":"2018-07-11 02:10:05.000000000","message":"you are fixing it in your patch so it is ok here - https://review.openstack.org/#/c/580877","commit_id":"8db070765eb61652ce0fa266ef3ab4e05d1352a8"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"d484653ecd48c32f50ab57d8232ede47a229158a","unresolved":false,"context_lines":[{"line_number":101,"context_line":"      - master"},{"line_number":102,"context_line":"    description: |"},{"line_number":103,"context_line":"      Base multinode integration test with Neutron networking and py27."},{"line_number":104,"context_line":"      Former names for this job where:"},{"line_number":105,"context_line":"        * neutron-tempest-multinode-full"},{"line_number":106,"context_line":"        * legacy-tempest-dsvm-neutron-multinode-full"},{"line_number":107,"context_line":"        * gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial-nv"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"5f7c97a3_240e9c3d","line":104,"range":{"start_line":104,"start_character":32,"end_line":104,"end_character":37},"in_reply_to":"5f7c97a3_f7062121","updated":"2018-07-11 02:07:48.000000000","message":"i think i fixed this. done","commit_id":"8db070765eb61652ce0fa266ef3ab4e05d1352a8"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"9e690bff105ee108f3d12781dbcb3a8944f795c2","unresolved":false,"context_lines":[{"line_number":144,"context_line":""},{"line_number":145,"context_line":"# TODO(gmann): needs to migrate this to zuulv3"},{"line_number":146,"context_line":"- job:"},{"line_number":147,"context_line":"    name: tempest-scenario-all"},{"line_number":148,"context_line":"    parent: legacy-dsvm-base-multinode"},{"line_number":149,"context_line":"    description: |"},{"line_number":150,"context_line":"      This job will run all scenario tests including slow tests"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"5f7c97a3_2fb0a240","line":147,"updated":"2018-07-11 17:22:53.000000000","message":"nit: I think \"tempest-scenario-multinode-all\" is better since this is a multinode job, to be consistent with L96. Also Zuul says this is OK: https://docs.openstack.org/infra/manual/drivers.html#naming-with-zuul-v3","commit_id":"8db070765eb61652ce0fa266ef3ab4e05d1352a8"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"e596f838d089bd1656acac3a5b39c9930419641a","unresolved":false,"context_lines":[{"line_number":148,"context_line":"    parent: legacy-dsvm-base-multinode"},{"line_number":149,"context_line":"    description: |"},{"line_number":150,"context_line":"      This job will run all scenario tests including slow tests"},{"line_number":151,"context_line":"      with lvm multibackend setup. This job will not run any API tests."},{"line_number":152,"context_line":"    run: playbooks/tempest-scenario-multinode-lvm-multibackend/run.yaml"},{"line_number":153,"context_line":"    post-run: playbooks/tempest-scenario-multinode-lvm-multibackend/post.yaml"},{"line_number":154,"context_line":"    timeout: 10800"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"5f7c97a3_771bf17e","line":151,"updated":"2018-07-10 19:02:00.000000000","message":"IMO we should also do what Andrea did above for tempest-multinode-full, we should add previous job name here, e.g.:\n\n      Former names for this job were:\n        * tempest-scenario-multinode-lvm-multibackend\n\nSee L104 above. But this is a nit, so LGTM.","commit_id":"8db070765eb61652ce0fa266ef3ab4e05d1352a8"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"d484653ecd48c32f50ab57d8232ede47a229158a","unresolved":false,"context_lines":[{"line_number":148,"context_line":"    parent: legacy-dsvm-base-multinode"},{"line_number":149,"context_line":"    description: |"},{"line_number":150,"context_line":"      This job will run all scenario tests including slow tests"},{"line_number":151,"context_line":"      with lvm multibackend setup. This job will not run any API tests."},{"line_number":152,"context_line":"    run: playbooks/tempest-scenario-multinode-lvm-multibackend/run.yaml"},{"line_number":153,"context_line":"    post-run: playbooks/tempest-scenario-multinode-lvm-multibackend/post.yaml"},{"line_number":154,"context_line":"    timeout: 10800"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"5f7c97a3_a4014c0a","line":151,"in_reply_to":"5f7c97a3_771bf17e","updated":"2018-07-11 02:07:48.000000000","message":"+1, will do","commit_id":"8db070765eb61652ce0fa266ef3ab4e05d1352a8"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"6f2931c66a388728482481bf514c3fc426751c9f","unresolved":false,"context_lines":[{"line_number":148,"context_line":"    parent: legacy-dsvm-base-multinode"},{"line_number":149,"context_line":"    description: |"},{"line_number":150,"context_line":"      This job will run all scenario tests including slow tests"},{"line_number":151,"context_line":"      with lvm multibackend setup. This job will not run any API tests."},{"line_number":152,"context_line":"    run: playbooks/tempest-scenario-multinode-lvm-multibackend/run.yaml"},{"line_number":153,"context_line":"    post-run: playbooks/tempest-scenario-multinode-lvm-multibackend/post.yaml"},{"line_number":154,"context_line":"    timeout: 10800"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"5f7c97a3_8f4c6976","line":151,"in_reply_to":"5f7c97a3_a4014c0a","updated":"2018-07-11 02:10:05.000000000","message":"ditto, we can add while migrating it to zuulv3 which is your patch https://review.openstack.org/#/c/580877/","commit_id":"8db070765eb61652ce0fa266ef3ab4e05d1352a8"}]}