)]}'
{"playbooks/legacy/grenade-dsvm-ironic/post.yaml":[{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"c458acb3469802722d5b51f5b6b9ea9dab54c374","unresolved":false,"context_lines":[{"line_number":1,"context_line":"- hosts: primary"},{"line_number":2,"context_line":"  tasks:"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"    - name: Copy files from {{ ansible_user_dir }}/workspace/ on node"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"3f4b6375_1521734e","line":1,"updated":"2017-10-18 14:25:53.000000000","message":"all post files are the same, do we really need to keepa separate file per each job?","commit_id":"ac71fe5f0884a8d1140eb1211263492a0a943abf"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"aeff3b8f57e32e245d4d5062a017369f4d646fd9","unresolved":false,"context_lines":[{"line_number":1,"context_line":"- hosts: primary"},{"line_number":2,"context_line":"  tasks:"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"    - name: Copy files from {{ ansible_user_dir }}/workspace/ on node"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"3f4b6375_75f4275c","line":1,"in_reply_to":"3f4b6375_1521734e","updated":"2017-10-18 14:36:18.000000000","message":"Those files are autogenerated. The migration instructions proposes to copy over 1:1 - and then rework this.\n\nSee https://docs.openstack.org/infra/manual/zuulv3.html#moving-legacy-jobs-to-projects.\n\nSo, let\u0027s merge - and know everything works. And then update.\n\nThis change should be step one in a series.","commit_id":"ac71fe5f0884a8d1140eb1211263492a0a943abf"}],"zuul.d/legacy-ironic-jobs.yaml":[{"author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"change_message_id":"10a2ed9cb1d0b7b58b916a8c6295534074294b5e","unresolved":false,"context_lines":[{"line_number":1,"context_line":"- job:"},{"line_number":2,"context_line":"    name: legacy-ironic-dsvm-base"},{"line_number":3,"context_line":"    parent: legacy-dsvm-base"},{"line_number":4,"context_line":"    irrelevant-files: \u0026irrelevant_files"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"5f4e5783_2c686e1f","line":1,"updated":"2017-10-11 21:45:40.000000000","message":"Where did this job come from? I\u0027m trying to match it up with os-zuul-jobs","commit_id":"ac71fe5f0884a8d1140eb1211263492a0a943abf"},{"author":{"_account_id":6637,"name":"Sam Betts","email":"sam@code-smash.net","username":"sambetts"},"change_message_id":"85c86857665d5d32e36f9743b28a6be1bcce3265","unresolved":false,"context_lines":[{"line_number":1,"context_line":"- job:"},{"line_number":2,"context_line":"    name: legacy-ironic-dsvm-base"},{"line_number":3,"context_line":"    parent: legacy-dsvm-base"},{"line_number":4,"context_line":"    irrelevant-files: \u0026irrelevant_files"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"3f4b6375_1594d343","line":1,"in_reply_to":"5f4e5783_2c686e1f","updated":"2017-10-18 14:43:41.000000000","message":"Openstack-infra team suggested in a previous patch to create this job as a parent to our other jobs to reduce the amount of duplication of irrelevant-files.","commit_id":"ac71fe5f0884a8d1140eb1211263492a0a943abf"},{"author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"change_message_id":"10a2ed9cb1d0b7b58b916a8c6295534074294b5e","unresolved":false,"context_lines":[{"line_number":1,"context_line":"- job:"},{"line_number":2,"context_line":"    name: legacy-ironic-dsvm-base"},{"line_number":3,"context_line":"    parent: legacy-dsvm-base"},{"line_number":4,"context_line":"    irrelevant-files: \u0026irrelevant_files"},{"line_number":5,"context_line":"      - ^(test-|)requirements.txt$"},{"line_number":6,"context_line":"      - ^.*\\.rst$"},{"line_number":7,"context_line":"      - ^api-ref/.*$"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"5f4e5783_4c772ad6","line":4,"range":{"start_line":4,"start_character":22,"end_line":4,"end_character":39},"updated":"2017-10-11 21:45:40.000000000","message":"Would be nice to have a comment on what this is doing\n\nWhat does \u0027\u0026\u0027 do?","commit_id":"ac71fe5f0884a8d1140eb1211263492a0a943abf"},{"author":{"_account_id":6637,"name":"Sam Betts","email":"sam@code-smash.net","username":"sambetts"},"change_message_id":"9f9cafd810178dd1d5def6c9c4279c062621b9b1","unresolved":false,"context_lines":[{"line_number":1,"context_line":"- job:"},{"line_number":2,"context_line":"    name: legacy-ironic-dsvm-base"},{"line_number":3,"context_line":"    parent: legacy-dsvm-base"},{"line_number":4,"context_line":"    irrelevant-files: \u0026irrelevant_files"},{"line_number":5,"context_line":"      - ^(test-|)requirements.txt$"},{"line_number":6,"context_line":"      - ^.*\\.rst$"},{"line_number":7,"context_line":"      - ^api-ref/.*$"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"5f4e5783_4503660e","line":4,"range":{"start_line":4,"start_character":22,"end_line":4,"end_character":39},"in_reply_to":"5f4e5783_4c772ad6","updated":"2017-10-12 16:29:40.000000000","message":"This is standard (although often forgotten) yaml syntax called anchors and allows you to reference back to things you defined before to reduce duplication http://blog.daemonl.com/2016/02/yaml.html","commit_id":"ac71fe5f0884a8d1140eb1211263492a0a943abf"},{"author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"change_message_id":"10a2ed9cb1d0b7b58b916a8c6295534074294b5e","unresolved":false,"context_lines":[{"line_number":26,"context_line":"- job:"},{"line_number":27,"context_line":"    name: legacy-ironic-dsvm-base-multinode"},{"line_number":28,"context_line":"    parent: legacy-dsvm-base-multinode"},{"line_number":29,"context_line":"    irrelevant-files: *irrelevant_files"},{"line_number":30,"context_line":"    required-projects:"},{"line_number":31,"context_line":"      - openstack-infra/devstack-gate"},{"line_number":32,"context_line":"      - openstack/ironic"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"5f4e5783_ec7b76bb","line":29,"range":{"start_line":29,"start_character":22,"end_line":29,"end_character":39},"updated":"2017-10-11 21:45:40.000000000","message":"Would be nice to have a comment on what this is doing\n\nWhat does \u0027*\u0027 do?","commit_id":"ac71fe5f0884a8d1140eb1211263492a0a943abf"},{"author":{"_account_id":1926,"name":"Derek Higgins","email":"derekh@redhat.com","username":"derekh"},"change_message_id":"2f0c04cefe73fef86814ef77f974ddbd1f38b178","unresolved":false,"context_lines":[{"line_number":26,"context_line":"- job:"},{"line_number":27,"context_line":"    name: legacy-ironic-dsvm-base-multinode"},{"line_number":28,"context_line":"    parent: legacy-dsvm-base-multinode"},{"line_number":29,"context_line":"    irrelevant-files: *irrelevant_files"},{"line_number":30,"context_line":"    required-projects:"},{"line_number":31,"context_line":"      - openstack-infra/devstack-gate"},{"line_number":32,"context_line":"      - openstack/ironic"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"5f4e5783_5a646522","line":29,"range":{"start_line":29,"start_character":22,"end_line":29,"end_character":39},"in_reply_to":"5f4e5783_ec7b76bb","updated":"2017-10-12 16:25:04.000000000","message":"ya, I guess its a reference to irrelevant_files in the previous job but that just a guess, is it documented somewhere?","commit_id":"ac71fe5f0884a8d1140eb1211263492a0a943abf"},{"author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"change_message_id":"10a2ed9cb1d0b7b58b916a8c6295534074294b5e","unresolved":false,"context_lines":[{"line_number":39,"context_line":"      - openstack/networking-generic-switch"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"- job:"},{"line_number":42,"context_line":"    name: ironic-tempest-dsvm-ironic-pxe_ipmitool-postgres"},{"line_number":43,"context_line":"    parent: legacy-ironic-dsvm-base"},{"line_number":44,"context_line":"    run: playbooks/legacy/tempest-dsvm-ironic-pxe_ipmitool-postgres/run"},{"line_number":45,"context_line":"    post-run: playbooks/legacy/tempest-dsvm-ironic-pxe_ipmitool-postgres/post"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"5f4e5783_8c140250","line":42,"updated":"2017-10-11 21:45:40.000000000","message":"It makes it difficult to review with the job names changing from what they are in os-zuul-jobs :(","commit_id":"ac71fe5f0884a8d1140eb1211263492a0a943abf"},{"author":{"_account_id":6637,"name":"Sam Betts","email":"sam@code-smash.net","username":"sambetts"},"change_message_id":"85c86857665d5d32e36f9743b28a6be1bcce3265","unresolved":false,"context_lines":[{"line_number":39,"context_line":"      - openstack/networking-generic-switch"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"- job:"},{"line_number":42,"context_line":"    name: ironic-tempest-dsvm-ironic-pxe_ipmitool-postgres"},{"line_number":43,"context_line":"    parent: legacy-ironic-dsvm-base"},{"line_number":44,"context_line":"    run: playbooks/legacy/tempest-dsvm-ironic-pxe_ipmitool-postgres/run"},{"line_number":45,"context_line":"    post-run: playbooks/legacy/tempest-dsvm-ironic-pxe_ipmitool-postgres/post"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"3f4b6375_d5763b3f","line":42,"in_reply_to":"5f4e5783_8c140250","updated":"2017-10-18 14:43:41.000000000","message":"We have to change the names because the jobs have to be globally unique and for a period of time both these jobs and the jobs in openstack-zuul-job will exist so if we don\u0027t change them they conflict.","commit_id":"ac71fe5f0884a8d1140eb1211263492a0a943abf"},{"author":{"_account_id":1926,"name":"Derek Higgins","email":"derekh@redhat.com","username":"derekh"},"change_message_id":"2f0c04cefe73fef86814ef77f974ddbd1f38b178","unresolved":false,"context_lines":[{"line_number":103,"context_line":"    timeout: 5400"},{"line_number":104,"context_line":""},{"line_number":105,"context_line":"- job:"},{"line_number":106,"context_line":"    name: ironic-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-multinode"},{"line_number":107,"context_line":"    parent: legacy-ironic-dsvm-base-multinode"},{"line_number":108,"context_line":"    run: playbooks/legacy/tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-multinode/run"},{"line_number":109,"context_line":"    post-run: playbooks/legacy/tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-multinode/post"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"5f4e5783_9a777d03","line":106,"updated":"2017-10-12 16:25:04.000000000","message":"This job appears twice","commit_id":"ac71fe5f0884a8d1140eb1211263492a0a943abf"},{"author":{"_account_id":6637,"name":"Sam Betts","email":"sam@code-smash.net","username":"sambetts"},"change_message_id":"85c86857665d5d32e36f9743b28a6be1bcce3265","unresolved":false,"context_lines":[{"line_number":103,"context_line":"    timeout: 5400"},{"line_number":104,"context_line":""},{"line_number":105,"context_line":"- job:"},{"line_number":106,"context_line":"    name: ironic-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-multinode"},{"line_number":107,"context_line":"    parent: legacy-ironic-dsvm-base-multinode"},{"line_number":108,"context_line":"    run: playbooks/legacy/tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-multinode/run"},{"line_number":109,"context_line":"    post-run: playbooks/legacy/tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-multinode/post"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"3f4b6375_55b12be1","line":106,"in_reply_to":"5f4e5783_9a777d03","updated":"2017-10-18 14:43:41.000000000","message":"Done","commit_id":"ac71fe5f0884a8d1140eb1211263492a0a943abf"},{"author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"change_message_id":"10a2ed9cb1d0b7b58b916a8c6295534074294b5e","unresolved":false,"context_lines":[{"line_number":125,"context_line":"    timeout: 9600"},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"- job:"},{"line_number":128,"context_line":"    name: ironic-ironic-dsvm-standalone"},{"line_number":129,"context_line":"    parent: legacy-ironic-dsvm-base"},{"line_number":130,"context_line":"    run: playbooks/legacy/ironic-dsvm-standalone/run"},{"line_number":131,"context_line":"    post-run: playbooks/legacy/ironic-dsvm-standalone/post"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"5f4e5783_4cdbaaec","line":128,"updated":"2017-10-11 21:45:40.000000000","message":"Why \"ironic-ironic\" ?","commit_id":"ac71fe5f0884a8d1140eb1211263492a0a943abf"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"31875ba78b878537d48adb6750f0e3de583b29c0","unresolved":false,"context_lines":[{"line_number":125,"context_line":"    timeout: 9600"},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"- job:"},{"line_number":128,"context_line":"    name: ironic-ironic-dsvm-standalone"},{"line_number":129,"context_line":"    parent: legacy-ironic-dsvm-base"},{"line_number":130,"context_line":"    run: playbooks/legacy/ironic-dsvm-standalone/run"},{"line_number":131,"context_line":"    post-run: playbooks/legacy/ironic-dsvm-standalone/post"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"5f4e5783_4fc6fc9c","line":128,"in_reply_to":"5f4e5783_4cdbaaec","updated":"2017-10-11 21:59:38.000000000","message":"I believe all the jobs were renamed from \u0027legacy-\u0027 to \u0027ironic-\u0027. See the discussion culminating with http://eavesdrop.openstack.org/irclogs/%23openstack-ironic/%23openstack-ironic.2017-10-11.log.html#t2017-10-11T15:29:57.","commit_id":"ac71fe5f0884a8d1140eb1211263492a0a943abf"},{"author":{"_account_id":6637,"name":"Sam Betts","email":"sam@code-smash.net","username":"sambetts"},"change_message_id":"85c86857665d5d32e36f9743b28a6be1bcce3265","unresolved":false,"context_lines":[{"line_number":125,"context_line":"    timeout: 9600"},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"- job:"},{"line_number":128,"context_line":"    name: ironic-ironic-dsvm-standalone"},{"line_number":129,"context_line":"    parent: legacy-ironic-dsvm-base"},{"line_number":130,"context_line":"    run: playbooks/legacy/ironic-dsvm-standalone/run"},{"line_number":131,"context_line":"    post-run: playbooks/legacy/ironic-dsvm-standalone/post"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"3f4b6375_951323b9","line":128,"in_reply_to":"5f4e5783_4fc6fc9c","updated":"2017-10-18 14:43:41.000000000","message":"I\u0027ve cleaned up the jobs names to remove duplication of ironic","commit_id":"ac71fe5f0884a8d1140eb1211263492a0a943abf"},{"author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"change_message_id":"10a2ed9cb1d0b7b58b916a8c6295534074294b5e","unresolved":false,"context_lines":[{"line_number":207,"context_line":"      - openstack/networking-generic-switch"},{"line_number":208,"context_line":""},{"line_number":209,"context_line":"- job:"},{"line_number":210,"context_line":"    name: ironic-ironic-dsvm-functional"},{"line_number":211,"context_line":"    parent: legacy-ironic-dsvm-base"},{"line_number":212,"context_line":"    run: playbooks/legacy/ironic-dsvm-functional/run"},{"line_number":213,"context_line":"    post-run: playbooks/legacy/ironic-dsvm-functional/post"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"5f4e5783_2c098e30","line":210,"updated":"2017-10-11 21:45:40.000000000","message":"Why \"ironic-ironic\"?","commit_id":"ac71fe5f0884a8d1140eb1211263492a0a943abf"},{"author":{"_account_id":6637,"name":"Sam Betts","email":"sam@code-smash.net","username":"sambetts"},"change_message_id":"85c86857665d5d32e36f9743b28a6be1bcce3265","unresolved":false,"context_lines":[{"line_number":207,"context_line":"      - openstack/networking-generic-switch"},{"line_number":208,"context_line":""},{"line_number":209,"context_line":"- job:"},{"line_number":210,"context_line":"    name: ironic-ironic-dsvm-functional"},{"line_number":211,"context_line":"    parent: legacy-ironic-dsvm-base"},{"line_number":212,"context_line":"    run: playbooks/legacy/ironic-dsvm-functional/run"},{"line_number":213,"context_line":"    post-run: playbooks/legacy/ironic-dsvm-functional/post"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"3f4b6375_550dab8e","line":210,"in_reply_to":"5f4e5783_2c098e30","updated":"2017-10-18 14:43:41.000000000","message":"Done","commit_id":"ac71fe5f0884a8d1140eb1211263492a0a943abf"},{"author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"change_message_id":"c58a99996fcf85844f5d640a33a5940523289681","unresolved":false,"context_lines":[{"line_number":1,"context_line":"- job:"},{"line_number":2,"context_line":"    name: legacy-ironic-dsvm-base"},{"line_number":3,"context_line":"    parent: legacy-dsvm-base"},{"line_number":4,"context_line":"    irrelevant-files: \u0026irrelevant_files"},{"line_number":5,"context_line":"      - ^(test-|)requirements.txt$"},{"line_number":6,"context_line":"      - ^.*\\.rst$"},{"line_number":7,"context_line":"      - ^api-ref/.*$"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"3f4b6375_f01dc50f","line":4,"updated":"2017-10-18 14:58:31.000000000","message":"Please document this special YAML stuff with a comment.\n\nVery few people know what this is.","commit_id":"b19bba6953908702b4755663e5cd049a9e335970"}],"zuul.d/project.yaml":[{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"bf8454be5e5e607ab49d3e75db9cdfe01f7de53e","unresolved":false,"context_lines":[{"line_number":3,"context_line":"    check:"},{"line_number":4,"context_line":"      jobs:"},{"line_number":5,"context_line":"        - ironic-tempest-dsvm-ironic-pxe_ipmitool-postgres:"},{"line_number":6,"context_line":"            voting: false"},{"line_number":7,"context_line":"            irrelevant-files:"},{"line_number":8,"context_line":"              - ^(test-|)requirements.txt$"},{"line_number":9,"context_line":"              - ^.*\\.rst$"},{"line_number":10,"context_line":"              - ^api-ref/.*$"},{"line_number":11,"context_line":"              - ^doc/.*$"},{"line_number":12,"context_line":"              - ^install-guide/.*$"},{"line_number":13,"context_line":"              - ^ironic/locale/.*$"},{"line_number":14,"context_line":"              - ^ironic/tests/.*$"},{"line_number":15,"context_line":"              - ^releasenotes/.*$"},{"line_number":16,"context_line":"              - ^setup.cfg$"},{"line_number":17,"context_line":"              - ^tools/.*$"},{"line_number":18,"context_line":"              - ^tox.ini$"},{"line_number":19,"context_line":"        - ironic-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa:"},{"line_number":20,"context_line":"            irrelevant-files:"},{"line_number":21,"context_line":"              - ^(test-|)requirements.txt$"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"5f4e5783_cdb3214d","line":18,"range":{"start_line":6,"start_character":0,"end_line":18,"end_character":25},"updated":"2017-10-11 15:37:40.000000000","message":"you can add the irrelevant-files and to the job directly so that you don\u0027t need to duplicate those for both check and gate.\n\nYou can also move the voting line - I would leave it here to make it clearer what is non-voting.","commit_id":"125e5acd637939e0da50fbce208ea63129e4d4af"},{"author":{"_account_id":6637,"name":"Sam Betts","email":"sam@code-smash.net","username":"sambetts"},"change_message_id":"9f9cafd810178dd1d5def6c9c4279c062621b9b1","unresolved":false,"context_lines":[{"line_number":3,"context_line":"    check:"},{"line_number":4,"context_line":"      jobs:"},{"line_number":5,"context_line":"        - ironic-tempest-dsvm-ironic-pxe_ipmitool-postgres:"},{"line_number":6,"context_line":"            voting: false"},{"line_number":7,"context_line":"            irrelevant-files:"},{"line_number":8,"context_line":"              - ^(test-|)requirements.txt$"},{"line_number":9,"context_line":"              - ^.*\\.rst$"},{"line_number":10,"context_line":"              - ^api-ref/.*$"},{"line_number":11,"context_line":"              - ^doc/.*$"},{"line_number":12,"context_line":"              - ^install-guide/.*$"},{"line_number":13,"context_line":"              - ^ironic/locale/.*$"},{"line_number":14,"context_line":"              - ^ironic/tests/.*$"},{"line_number":15,"context_line":"              - ^releasenotes/.*$"},{"line_number":16,"context_line":"              - ^setup.cfg$"},{"line_number":17,"context_line":"              - ^tools/.*$"},{"line_number":18,"context_line":"              - ^tox.ini$"},{"line_number":19,"context_line":"        - ironic-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa:"},{"line_number":20,"context_line":"            irrelevant-files:"},{"line_number":21,"context_line":"              - ^(test-|)requirements.txt$"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"5f4e5783_d0717a0e","line":18,"range":{"start_line":6,"start_character":0,"end_line":18,"end_character":25},"in_reply_to":"5f4e5783_cdb3214d","updated":"2017-10-12 16:29:40.000000000","message":"Done","commit_id":"125e5acd637939e0da50fbce208ea63129e4d4af"},{"author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"change_message_id":"89fbedc2b39441bd60f50c9c72d047001c49adc4","unresolved":false,"context_lines":[{"line_number":1,"context_line":"- project:"},{"line_number":2,"context_line":"    name: openstack/ironic"},{"line_number":3,"context_line":"    check:"},{"line_number":4,"context_line":"      jobs:"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"5f4e5783_8f351449","line":1,"updated":"2017-10-11 22:23:23.000000000","message":"What is the source of this file? Sorry if dumb question.","commit_id":"ac71fe5f0884a8d1140eb1211263492a0a943abf"},{"author":{"_account_id":6637,"name":"Sam Betts","email":"sam@code-smash.net","username":"sambetts"},"change_message_id":"85c86857665d5d32e36f9743b28a6be1bcce3265","unresolved":false,"context_lines":[{"line_number":1,"context_line":"- project:"},{"line_number":2,"context_line":"    name: openstack/ironic"},{"line_number":3,"context_line":"    check:"},{"line_number":4,"context_line":"      jobs:"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"3f4b6375_f5513764","line":1,"in_reply_to":"5f4e5783_04856708","updated":"2017-10-18 14:43:41.000000000","message":"Yes, all the non-standard job and project definitions move into our tree leaving only the standard set of infra jobs in the project-config repo","commit_id":"ac71fe5f0884a8d1140eb1211263492a0a943abf"},{"author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"change_message_id":"ff46afccf7893d6fef7ffb77e02c9d7bd2e8b295","unresolved":false,"context_lines":[{"line_number":1,"context_line":"- project:"},{"line_number":2,"context_line":"    name: openstack/ironic"},{"line_number":3,"context_line":"    check:"},{"line_number":4,"context_line":"      jobs:"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"5f4e5783_04856708","line":1,"in_reply_to":"5f4e5783_8f351449","updated":"2017-10-12 23:52:11.000000000","message":"Maybe answering my own question. I think this comes from:\n\n openstack-infra/project-config/zuul/layout.yaml\n\nhttps://github.com/openstack-infra/project-config/blob/15b1ad5fa4d38c012d86bb946ca15cf8e67b5025/zuul/layout.yaml#L10591-L10647\n\nAnd probably other parts of the file too","commit_id":"ac71fe5f0884a8d1140eb1211263492a0a943abf"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"36ba5da0b6ad57c6e82ca36b5ec69cec680244c6","unresolved":false,"context_lines":[{"line_number":34,"context_line":"        - ironic-tempest-dsvm-ironic-inspector:"},{"line_number":35,"context_line":"            voting: false"},{"line_number":36,"context_line":"        - ironic-grenade-dsvm:"},{"line_number":37,"context_line":"            branches: ^(?!(driverfixes|stable/(mitaka|newton))).*$"},{"line_number":38,"context_line":"        - ironic-grenade-dsvm-multinode-multitenant:"},{"line_number":39,"context_line":"            branches: ^(?!stable/(newton|ocata)).*$"},{"line_number":40,"context_line":"        - ironic-tempest-dsvm-multitenant-network:"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"3f4b6375_f05b8551","line":37,"range":{"start_line":37,"start_character":27,"end_line":37,"end_character":38},"updated":"2017-10-18 15:15:12.000000000","message":"what is this we don\u0027t have such branch.","commit_id":"b19bba6953908702b4755663e5cd049a9e335970"},{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"c8e7259bac7a04fcd9e943643b2c673eedc63dab","unresolved":false,"context_lines":[{"line_number":50,"context_line":"        - ironic-tempest-dsvm-ipa-wholedisk-bios-pxe_ipmitool-tinyipa:"},{"line_number":51,"context_line":"            branches: ^stable/(newton|ocata).*$"},{"line_number":52,"context_line":"        - ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa:"},{"line_number":53,"context_line":"            voting: false"},{"line_number":54,"context_line":"        - ironic-tempest-dsvm-ipa-partition-redfish-tinyipa:"},{"line_number":55,"context_line":"            branches: ^(?!stable/(newton|ocata)).*$"},{"line_number":56,"context_line":"        - ironic-tempest-dsvm-bfv:"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"3f4b6375_b0174dee","line":53,"range":{"start_line":53,"start_character":12,"end_line":53,"end_character":25},"updated":"2017-10-18 14:59:55.000000000","message":"non-voting jobs should not be on gate pipeline.\n\nbesides, this particular job has to be voting IMO, it was wrongly set as non-voting during zuul3 transition","commit_id":"b19bba6953908702b4755663e5cd049a9e335970"},{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"c8e7259bac7a04fcd9e943643b2c673eedc63dab","unresolved":false,"context_lines":[{"line_number":59,"context_line":"            branches: ^(?!stable/(newton|ocata)).*$"},{"line_number":60,"context_line":"        - ironic-grenade-dsvm:"},{"line_number":61,"context_line":"            branches: ^(?!(driverfixes|stable/(mitaka|newton))).*$"},{"line_number":62,"context_line":"            voting: false"},{"line_number":63,"context_line":"        - ironic-grenade-dsvm-multinode-multitenant:"},{"line_number":64,"context_line":"            branches: ^(?!stable/(newton|ocata)).*$"},{"line_number":65,"context_line":"        - ironic-tempest-dsvm-multitenant-network:"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"3f4b6375_f0e26501","line":62,"range":{"start_line":62,"start_character":12,"end_line":62,"end_character":25},"updated":"2017-10-18 14:59:55.000000000","message":"non-voting job on gate, must be voting, also mitaka may be cleaned up already","commit_id":"b19bba6953908702b4755663e5cd049a9e335970"},{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"c8e7259bac7a04fcd9e943643b2c673eedc63dab","unresolved":false,"context_lines":[{"line_number":65,"context_line":"        - ironic-tempest-dsvm-multitenant-network:"},{"line_number":66,"context_line":"            branches: ^stable/(newton|ocata).*$"},{"line_number":67,"context_line":"        - ironic-tempest-dsvm-ipa-wholedisk-agent_ipmitool-tinyipa-multinode:"},{"line_number":68,"context_line":"            branches: ^stable/(newton|ocata).*$"},{"line_number":69,"context_line":"            voting: false"},{"line_number":70,"context_line":"    experimental:"},{"line_number":71,"context_line":"      jobs:"},{"line_number":72,"context_line":"        - ironic-dsvm-functional"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"3f4b6375_b0c50d5e","line":69,"range":{"start_line":68,"start_character":12,"end_line":69,"end_character":25},"updated":"2017-10-18 14:59:55.000000000","message":"must be voting, and running NOT on this branches (same regex fix as in check queue)","commit_id":"b19bba6953908702b4755663e5cd049a9e335970"},{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"1778cc66f8b721faf510aaf0296bcee9088786a2","unresolved":false,"context_lines":[{"line_number":4,"context_line":"      jobs:"},{"line_number":5,"context_line":"        - ironic-tempest-dsvm-pxe_ipmitool-postgres:"},{"line_number":6,"context_line":"            voting: false"},{"line_number":7,"context_line":"        - ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa:"},{"line_number":8,"context_line":"            branches: ^stable/(newton|ocata).*$"},{"line_number":9,"context_line":"        - ironic-tempest-dsvm-ipa-wholedisk-bios-pxe_ipmitool-tinyipa:"},{"line_number":10,"context_line":"            branches: ^stable/(newton|ocata).*$"},{"line_number":11,"context_line":"        - ironic-tempest-dsvm-ipa-wholedisk-bios-pxe_snmp-tinyipa:"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"3f4b6375_f61a5e15","line":8,"range":{"start_line":7,"start_character":7,"end_line":8,"end_character":47},"updated":"2017-10-18 15:42:09.000000000","message":"I definitely remember this job as voting during Pike and early in Queens\n\nexample from Pike times https://review.openstack.org/#/c/482982/\n\nexample from early Queens\nhttps://review.openstack.org/#/c/503274/\n\nthough I could understand if we ditch it in favor of multinode job that tests the same combination (whole-disk + agent_ipmitool), I just don\u0027t remember us explicitly agreeing on such change.","commit_id":"455df9d704204cf419a05d1c43ecc70b17e24ad8"},{"author":{"_account_id":6637,"name":"Sam Betts","email":"sam@code-smash.net","username":"sambetts"},"change_message_id":"9501ba264937e14fb528cf877c0a99172d1b9659","unresolved":false,"context_lines":[{"line_number":4,"context_line":"      jobs:"},{"line_number":5,"context_line":"        - ironic-tempest-dsvm-pxe_ipmitool-postgres:"},{"line_number":6,"context_line":"            voting: false"},{"line_number":7,"context_line":"        - ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa:"},{"line_number":8,"context_line":"            branches: ^stable/(newton|ocata).*$"},{"line_number":9,"context_line":"        - ironic-tempest-dsvm-ipa-wholedisk-bios-pxe_ipmitool-tinyipa:"},{"line_number":10,"context_line":"            branches: ^stable/(newton|ocata).*$"},{"line_number":11,"context_line":"        - ironic-tempest-dsvm-ipa-wholedisk-bios-pxe_snmp-tinyipa:"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"3f4b6375_766c2e08","line":8,"range":{"start_line":7,"start_character":7,"end_line":8,"end_character":47},"in_reply_to":"3f4b6375_f61a5e15","updated":"2017-10-18 15:53:19.000000000","message":"I misread a comment in project-config, this should be enabled still sorry.","commit_id":"455df9d704204cf419a05d1c43ecc70b17e24ad8"},{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"1778cc66f8b721faf510aaf0296bcee9088786a2","unresolved":false,"context_lines":[{"line_number":49,"context_line":"            branches: ^stable/(newton|ocata).*$"},{"line_number":50,"context_line":"        - ironic-tempest-dsvm-ipa-wholedisk-bios-pxe_ipmitool-tinyipa:"},{"line_number":51,"context_line":"            branches: ^stable/(newton|ocata).*$"},{"line_number":52,"context_line":"        - ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa:"},{"line_number":53,"context_line":"            branches: ^stable/(newton|ocata).*$"},{"line_number":54,"context_line":"        - ironic-tempest-dsvm-ipa-partition-redfish-tinyipa:"},{"line_number":55,"context_line":"            branches: ^(?!stable/(newton|ocata)).*$"},{"line_number":56,"context_line":"        - ironic-tempest-dsvm-bfv:"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"3f4b6375_76dc2e46","line":53,"range":{"start_line":52,"start_character":8,"end_line":53,"end_character":47},"updated":"2017-10-18 15:42:09.000000000","message":"ditto","commit_id":"455df9d704204cf419a05d1c43ecc70b17e24ad8"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"88dfdfab59859e19514186bb46b97901cbd4775a","unresolved":false,"context_lines":[{"line_number":58,"context_line":"        - ironic-dsvm-standalone:"},{"line_number":59,"context_line":"            branches: ^(?!stable/(newton|ocata)).*$"},{"line_number":60,"context_line":"        - ironic-grenade-dsvm:"},{"line_number":61,"context_line":"            branches: ^(?!(driverfixes|stable/(mitaka|newton))).*$"},{"line_number":62,"context_line":"            voting: false"},{"line_number":63,"context_line":"        - ironic-grenade-dsvm-multinode-multitenant:"},{"line_number":64,"context_line":"            branches: ^(?!stable/(newton|ocata)).*$"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"3f4b6375_f0094531","line":61,"range":{"start_line":61,"start_character":27,"end_line":61,"end_character":38},"updated":"2017-10-18 15:17:58.000000000","message":"we don\u0027t have suck branch.","commit_id":"455df9d704204cf419a05d1c43ecc70b17e24ad8"},{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"1778cc66f8b721faf510aaf0296bcee9088786a2","unresolved":false,"context_lines":[{"line_number":58,"context_line":"        - ironic-dsvm-standalone:"},{"line_number":59,"context_line":"            branches: ^(?!stable/(newton|ocata)).*$"},{"line_number":60,"context_line":"        - ironic-grenade-dsvm:"},{"line_number":61,"context_line":"            branches: ^(?!(driverfixes|stable/(mitaka|newton))).*$"},{"line_number":62,"context_line":"            voting: false"},{"line_number":63,"context_line":"        - ironic-grenade-dsvm-multinode-multitenant:"},{"line_number":64,"context_line":"            branches: ^(?!stable/(newton|ocata)).*$"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"3f4b6375_76342e4b","line":61,"range":{"start_line":61,"start_character":27,"end_line":61,"end_character":38},"in_reply_to":"3f4b6375_56b6d2db","updated":"2017-10-18 15:42:09.000000000","message":"also, mitaka is over already, could be dropped too","commit_id":"455df9d704204cf419a05d1c43ecc70b17e24ad8"},{"author":{"_account_id":6637,"name":"Sam Betts","email":"sam@code-smash.net","username":"sambetts"},"change_message_id":"b1d3ec7be4249bf8eaad1a73626c8f451baf5828","unresolved":false,"context_lines":[{"line_number":58,"context_line":"        - ironic-dsvm-standalone:"},{"line_number":59,"context_line":"            branches: ^(?!stable/(newton|ocata)).*$"},{"line_number":60,"context_line":"        - ironic-grenade-dsvm:"},{"line_number":61,"context_line":"            branches: ^(?!(driverfixes|stable/(mitaka|newton))).*$"},{"line_number":62,"context_line":"            voting: false"},{"line_number":63,"context_line":"        - ironic-grenade-dsvm-multinode-multitenant:"},{"line_number":64,"context_line":"            branches: ^(?!stable/(newton|ocata)).*$"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"3f4b6375_56b6d2db","line":61,"range":{"start_line":61,"start_character":27,"end_line":61,"end_character":38},"in_reply_to":"3f4b6375_f0094531","updated":"2017-10-18 15:36:17.000000000","message":"not really sure where this came from, but we shouldn\u0027t need it going forward","commit_id":"455df9d704204cf419a05d1c43ecc70b17e24ad8"},{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"1778cc66f8b721faf510aaf0296bcee9088786a2","unresolved":false,"context_lines":[{"line_number":57,"context_line":"            branches: ^(?!stable/(newton|ocata)).*$"},{"line_number":58,"context_line":"        - ironic-dsvm-standalone:"},{"line_number":59,"context_line":"            branches: ^(?!stable/(newton|ocata)).*$"},{"line_number":60,"context_line":"        - ironic-grenade-dsvm:"},{"line_number":61,"context_line":"            branches: ^(?!(driverfixes|stable/(mitaka|newton))).*$"},{"line_number":62,"context_line":"            voting: false"},{"line_number":63,"context_line":"        - ironic-grenade-dsvm-multinode-multitenant:"},{"line_number":64,"context_line":"            branches: ^(?!stable/(newton|ocata)).*$"},{"line_number":65,"context_line":"        - ironic-tempest-dsvm-multitenant-network:"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"3f4b6375_b68e2625","line":62,"range":{"start_line":60,"start_character":0,"end_line":62,"end_character":25},"updated":"2017-10-18 15:42:09.000000000","message":"still non-voting on gate pipeline? why?","commit_id":"455df9d704204cf419a05d1c43ecc70b17e24ad8"},{"author":{"_account_id":6637,"name":"Sam Betts","email":"sam@code-smash.net","username":"sambetts"},"change_message_id":"9501ba264937e14fb528cf877c0a99172d1b9659","unresolved":false,"context_lines":[{"line_number":57,"context_line":"            branches: ^(?!stable/(newton|ocata)).*$"},{"line_number":58,"context_line":"        - ironic-dsvm-standalone:"},{"line_number":59,"context_line":"            branches: ^(?!stable/(newton|ocata)).*$"},{"line_number":60,"context_line":"        - ironic-grenade-dsvm:"},{"line_number":61,"context_line":"            branches: ^(?!(driverfixes|stable/(mitaka|newton))).*$"},{"line_number":62,"context_line":"            voting: false"},{"line_number":63,"context_line":"        - ironic-grenade-dsvm-multinode-multitenant:"},{"line_number":64,"context_line":"            branches: ^(?!stable/(newton|ocata)).*$"},{"line_number":65,"context_line":"        - ironic-tempest-dsvm-multitenant-network:"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"3f4b6375_590de535","line":62,"range":{"start_line":60,"start_character":0,"end_line":62,"end_character":25},"in_reply_to":"3f4b6375_b68e2625","updated":"2017-10-18 15:53:19.000000000","message":"Not really sure... will fix...","commit_id":"455df9d704204cf419a05d1c43ecc70b17e24ad8"},{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"63e25bb911e4df8e8797f4f6c290d28aa3d16add","unresolved":false,"context_lines":[{"line_number":49,"context_line":"        - ironic-tempest-dsvm-ipa-wholedisk-bios-pxe_ipmitool-tinyipa:"},{"line_number":50,"context_line":"            branches: ^stable/(newton|ocata).*$"},{"line_number":51,"context_line":"        - ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa:"},{"line_number":52,"context_line":"            branches: ^stable/(newton|ocata).*$"},{"line_number":53,"context_line":"        - ironic-tempest-dsvm-ipa-partition-redfish-tinyipa:"},{"line_number":54,"context_line":"            branches: ^(?!stable/(newton|ocata)).*$"},{"line_number":55,"context_line":"        - ironic-tempest-dsvm-bfv:"}],"source_content_type":"text/x-yaml","patch_set":11,"id":"3f4b6375_5945e516","line":52,"range":{"start_line":52,"start_character":12,"end_line":52,"end_character":47},"updated":"2017-10-18 15:57:35.000000000","message":"this branch specifier should be removed too, but we could do it later, as it is already made voting on check pipeline already","commit_id":"4b37c29b5e180d68e744e8114e2f414023d1376a"}]}