)]}'
{"zuul.d/layout.yaml":[{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"97fd8c12be6ac99e100b8edbb36eff86b009041c","unresolved":false,"context_lines":[{"line_number":11,"context_line":"      - tripleo-ci-centos-7-scenario004-standalone:"},{"line_number":12,"context_line":"          files:"},{"line_number":13,"context_line":"            - ^tripleo_ansible/.*"},{"line_number":14,"context_line":"            - ^zuul.d/.*"},{"line_number":15,"context_line":"            - requirements.txt"},{"line_number":16,"context_line":"      - tripleo-ci-centos-7-scenario010-standalone:"},{"line_number":17,"context_line":"          files:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"9fb8cfa7_1db22895","line":14,"range":{"start_line":14,"start_character":14,"end_line":14,"end_character":24},"updated":"2019-06-27 12:59:10.000000000","message":"I\u0027m not sure if we need to run a scenario test when things in zuul.d change?","commit_id":"02c308d9b73319afefb09b4b01ecc6fbbba36fa3"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"c9c7129d233d8f7d4c12edd526655f1f49acce5e","unresolved":false,"context_lines":[{"line_number":11,"context_line":"      - tripleo-ci-centos-7-scenario004-standalone:"},{"line_number":12,"context_line":"          files:"},{"line_number":13,"context_line":"            - ^tripleo_ansible/.*"},{"line_number":14,"context_line":"            - ^zuul.d/.*"},{"line_number":15,"context_line":"            - requirements.txt"},{"line_number":16,"context_line":"      - tripleo-ci-centos-7-scenario010-standalone:"},{"line_number":17,"context_line":"          files:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"9fb8cfa7_93060986","line":14,"range":{"start_line":14,"start_character":14,"end_line":14,"end_character":24},"in_reply_to":"9fb8cfa7_1db22895","updated":"2019-06-27 14:17:28.000000000","message":"When you change jobs set or definition it\u0027s always good to run it to be sure you didn\u0027t break anything.","commit_id":"02c308d9b73319afefb09b4b01ecc6fbbba36fa3"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"97fd8c12be6ac99e100b8edbb36eff86b009041c","unresolved":false,"context_lines":[{"line_number":15,"context_line":"            - requirements.txt"},{"line_number":16,"context_line":"      - tripleo-ci-centos-7-scenario010-standalone:"},{"line_number":17,"context_line":"          files:"},{"line_number":18,"context_line":"            - ^tripleo_ansible/octavia.*"},{"line_number":19,"context_line":"            - ^zuul.d/.*"},{"line_number":20,"context_line":"      - tripleo-ci-centos-7-scenario003-standalone:"},{"line_number":21,"context_line":"          files:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"9fb8cfa7_fd8b94bc","line":18,"range":{"start_line":18,"start_character":14,"end_line":18,"end_character":39},"updated":"2019-06-27 12:59:10.000000000","message":"Maybe this should be \n  \n  ^tripleo_ansible/.*octavia.*\n\nThe octavia playbooks and roles are spread across the playbooks and roles directories.","commit_id":"02c308d9b73319afefb09b4b01ecc6fbbba36fa3"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"c9c7129d233d8f7d4c12edd526655f1f49acce5e","unresolved":false,"context_lines":[{"line_number":15,"context_line":"            - requirements.txt"},{"line_number":16,"context_line":"      - tripleo-ci-centos-7-scenario010-standalone:"},{"line_number":17,"context_line":"          files:"},{"line_number":18,"context_line":"            - ^tripleo_ansible/octavia.*"},{"line_number":19,"context_line":"            - ^zuul.d/.*"},{"line_number":20,"context_line":"      - tripleo-ci-centos-7-scenario003-standalone:"},{"line_number":21,"context_line":"          files:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"9fb8cfa7_d3efa1ae","line":18,"range":{"start_line":18,"start_character":14,"end_line":18,"end_character":39},"in_reply_to":"9fb8cfa7_fd8b94bc","updated":"2019-06-27 14:17:28.000000000","message":"Sure, will change it","commit_id":"02c308d9b73319afefb09b4b01ecc6fbbba36fa3"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"bcb560b9daafb18b5f7c6ade64ecd89d8478d20c","unresolved":false,"context_lines":[{"line_number":21,"context_line":"          files:"},{"line_number":22,"context_line":"            - ^tripleo_ansible/roles/tripleo-create-admin/.*"},{"line_number":23,"context_line":"            - ^zuul.d/.*"},{"line_number":24,"context_line":"      - tripleo-ci-centos-7-scenario012-standalone:"},{"line_number":25,"context_line":"          files:"},{"line_number":26,"context_line":"            - ^tripleo_ansible/roles/tripleo-module-load/.*"},{"line_number":27,"context_line":"            - ^zuul.d/.*"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"9fb8cfa7_e5af7a7a","line":24,"range":{"start_line":24,"start_character":8,"end_line":24,"end_character":50},"updated":"2019-06-27 16:06:42.000000000","message":"This scenario fails pretty consistently, Is this something not tested elsewhere?","commit_id":"f3aa0b0c336081cddbc04484365d9b788e81f621"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"73803b1ffeab70cc83a687b3b11000fe5b44f2d4","unresolved":false,"context_lines":[{"line_number":21,"context_line":"          files:"},{"line_number":22,"context_line":"            - ^tripleo_ansible/roles/tripleo-create-admin/.*"},{"line_number":23,"context_line":"            - ^zuul.d/.*"},{"line_number":24,"context_line":"      - tripleo-ci-centos-7-scenario012-standalone:"},{"line_number":25,"context_line":"          files:"},{"line_number":26,"context_line":"            - ^tripleo_ansible/roles/tripleo-module-load/.*"},{"line_number":27,"context_line":"            - ^zuul.d/.*"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"9fb8cfa7_f4b3a608","line":24,"range":{"start_line":24,"start_character":8,"end_line":24,"end_character":50},"in_reply_to":"9fb8cfa7_e5af7a7a","updated":"2019-06-27 20:02:03.000000000","message":"According to what I saw this is only job that can test tripleo-module-load role. If this role could be tested somewhere else, I\u0027d be only for it. \n\nI hope this job will be fixed one day.","commit_id":"f3aa0b0c336081cddbc04484365d9b788e81f621"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"d463f227d2dcd400369f4cd16a5b3f469273171d","unresolved":false,"context_lines":[{"line_number":21,"context_line":"          files:"},{"line_number":22,"context_line":"            - ^tripleo_ansible/roles/tripleo-create-admin/.*"},{"line_number":23,"context_line":"            - ^zuul.d/.*"},{"line_number":24,"context_line":"      - tripleo-ci-centos-7-scenario012-standalone:"},{"line_number":25,"context_line":"          files:"},{"line_number":26,"context_line":"            - ^tripleo_ansible/roles/tripleo-module-load/.*"},{"line_number":27,"context_line":"            - ^zuul.d/.*"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"9fb8cfa7_4801b604","line":24,"range":{"start_line":24,"start_character":8,"end_line":24,"end_character":50},"in_reply_to":"9fb8cfa7_f4b3a608","updated":"2019-06-27 20:43:32.000000000","message":"++","commit_id":"f3aa0b0c336081cddbc04484365d9b788e81f621"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"ac192406f0c1b564cf1af7d771f34c697cf48320","unresolved":false,"context_lines":[{"line_number":26,"context_line":"            - ^tripleo_ansible/roles/tripleo-module-load/.*"},{"line_number":27,"context_line":"            - ^zuul.d/.*"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"    gate:"},{"line_number":30,"context_line":"      jobs:"},{"line_number":31,"context_line":"      - openstack-tox-linters"},{"line_number":32,"context_line":"      - tripleo-ansible-docs"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"9fb8cfa7_5f7413fe","line":29,"updated":"2019-06-27 14:57:06.000000000","message":"We likely will want to add this to queue: tripleo to prevent the possibility of things to be merged out of order but that\u0027ll slow down jobs for this repo.","commit_id":"f3aa0b0c336081cddbc04484365d9b788e81f621"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"fcafea6e9c54fa6c3f99b9e89a85eea177e4fd77","unresolved":false,"context_lines":[{"line_number":8,"context_line":"      jobs:"},{"line_number":9,"context_line":"      - openstack-tox-linters"},{"line_number":10,"context_line":"      - tripleo-ansible-docs"},{"line_number":11,"context_line":"      - tripleo-ci-centos-7-scenario004-standalone:"},{"line_number":12,"context_line":"          files:"},{"line_number":13,"context_line":"            - ^tripleo_ansible/.*"},{"line_number":14,"context_line":"            - ^zuul.d/.*"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9fb8cfa7_d366bced","line":11,"updated":"2019-06-27 17:24:48.000000000","message":"In these scenarios maybe we could add irrelevant_files to the job definition so we can further limit the impact of these long running jobs?\n\nhttps://zuul-ci.org/docs/zuul/user/config.html#attr-job.irrelevant-files\n\nSomething like so\n\n  irrelevant-files:\n    - ^tripleo_ansible/.*molecule.*\n    - ^tripleo_ansible/.*meta.*\n    - .*md$\n    - .*rst$\n\nThis would further limit the job impact within the project by ensuring we\u0027re not running scenario tests when making changes to molecule scenarios, ansible metadata, or documentation.","commit_id":"7c11e3cb8b7ce9cab97b4c6ed121c5b0ec185d10"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"46a9862ba29a2a655cb22742cb23494d71ceae1f","unresolved":false,"context_lines":[{"line_number":8,"context_line":"      jobs:"},{"line_number":9,"context_line":"      - openstack-tox-linters"},{"line_number":10,"context_line":"      - tripleo-ansible-docs"},{"line_number":11,"context_line":"      - tripleo-ci-centos-7-scenario004-standalone:"},{"line_number":12,"context_line":"          files:"},{"line_number":13,"context_line":"            - ^tripleo_ansible/.*"},{"line_number":14,"context_line":"            - ^zuul.d/.*"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9fb8cfa7_e8218ab5","line":11,"in_reply_to":"9fb8cfa7_482a167f","updated":"2019-06-27 21:38:13.000000000","message":"yeah, it doesn\u0027t make sense to have 2, usually we rebase followup review on previous, but I don\u0027t mind just to leave one of them.","commit_id":"7c11e3cb8b7ce9cab97b4c6ed121c5b0ec185d10"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"d2e49222d7bd4667a084cc9651f42fea87dd119f","unresolved":false,"context_lines":[{"line_number":8,"context_line":"      jobs:"},{"line_number":9,"context_line":"      - openstack-tox-linters"},{"line_number":10,"context_line":"      - tripleo-ansible-docs"},{"line_number":11,"context_line":"      - tripleo-ci-centos-7-scenario004-standalone:"},{"line_number":12,"context_line":"          files:"},{"line_number":13,"context_line":"            - ^tripleo_ansible/.*"},{"line_number":14,"context_line":"            - ^zuul.d/.*"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9fb8cfa7_d41ea20a","line":11,"in_reply_to":"9fb8cfa7_d366bced","updated":"2019-06-27 19:58:32.000000000","message":"Agree","commit_id":"7c11e3cb8b7ce9cab97b4c6ed121c5b0ec185d10"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"578c3bbec3c7f72f87fe4961fe4b89adcc16e0cd","unresolved":false,"context_lines":[{"line_number":8,"context_line":"      jobs:"},{"line_number":9,"context_line":"      - openstack-tox-linters"},{"line_number":10,"context_line":"      - tripleo-ansible-docs"},{"line_number":11,"context_line":"      - tripleo-ci-centos-7-scenario004-standalone:"},{"line_number":12,"context_line":"          files:"},{"line_number":13,"context_line":"            - ^tripleo_ansible/.*"},{"line_number":14,"context_line":"            - ^zuul.d/.*"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9fb8cfa7_482a167f","line":11,"in_reply_to":"9fb8cfa7_d41ea20a","updated":"2019-06-27 20:45:09.000000000","message":"should we squash the two reviews?","commit_id":"7c11e3cb8b7ce9cab97b4c6ed121c5b0ec185d10"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"700d1b6330acd4246381ee29f0c777a53cb8b179","unresolved":false,"context_lines":[{"line_number":8,"context_line":"      jobs:"},{"line_number":9,"context_line":"      - openstack-tox-linters"},{"line_number":10,"context_line":"      - tripleo-ansible-docs"},{"line_number":11,"context_line":"      - tripleo-ci-centos-7-scenario004-standalone:"},{"line_number":12,"context_line":"          files:"},{"line_number":13,"context_line":"            - ^tripleo_ansible/.*"},{"line_number":14,"context_line":"            - ^zuul.d/.*"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9fb8cfa7_80c8da54","line":11,"in_reply_to":"9fb8cfa7_e8218ab5","updated":"2019-06-28 15:33:43.000000000","message":"sounds good, i\u0027ll rebase the follow-on review when this one is approved.","commit_id":"7c11e3cb8b7ce9cab97b4c6ed121c5b0ec185d10"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"3a63b5478c43eba698a96e2c4359d217221959ce","unresolved":false,"context_lines":[{"line_number":11,"context_line":"      - tripleo-ci-centos-7-scenario004-standalone:"},{"line_number":12,"context_line":"          files:"},{"line_number":13,"context_line":"            - ^tripleo_ansible/.*"},{"line_number":14,"context_line":"            - ^zuul.d/.*"},{"line_number":15,"context_line":"            - requirements.txt"},{"line_number":16,"context_line":"      - tripleo-ci-centos-7-scenario010-standalone:"},{"line_number":17,"context_line":"          files:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9fb8cfa7_53a08cf4","line":14,"range":{"start_line":14,"start_character":12,"end_line":14,"end_character":24},"updated":"2019-06-27 17:14:50.000000000","message":"Let\u0027s not run these on zuul.d changes. it makes it hard quickly address ci issues. We shouldn\u0027t be changing anything in zuul.d in this repo that is actually tested by these jobs","commit_id":"7c11e3cb8b7ce9cab97b4c6ed121c5b0ec185d10"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"8876e71167510a88643eacc934f497ab65129e9d","unresolved":false,"context_lines":[{"line_number":11,"context_line":"      - tripleo-ci-centos-7-scenario004-standalone:"},{"line_number":12,"context_line":"          files:"},{"line_number":13,"context_line":"            - ^tripleo_ansible/.*"},{"line_number":14,"context_line":"            - ^zuul.d/.*"},{"line_number":15,"context_line":"            - requirements.txt"},{"line_number":16,"context_line":"      - tripleo-ci-centos-7-scenario010-standalone:"},{"line_number":17,"context_line":"          files:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9fb8cfa7_b4a44ea7","line":14,"range":{"start_line":14,"start_character":12,"end_line":14,"end_character":24},"in_reply_to":"9fb8cfa7_14299a35","updated":"2019-06-27 20:12:46.000000000","message":"Yes but delays turning things off by hours. I\u0027d rather not continue this pattern for things outside of oooq/tripleo-ci unless the job definition includes some variable that can actually affect things","commit_id":"7c11e3cb8b7ce9cab97b4c6ed121c5b0ec185d10"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"f6640881957045cf579731916e28ef1565ebc353","unresolved":false,"context_lines":[{"line_number":11,"context_line":"      - tripleo-ci-centos-7-scenario004-standalone:"},{"line_number":12,"context_line":"          files:"},{"line_number":13,"context_line":"            - ^tripleo_ansible/.*"},{"line_number":14,"context_line":"            - ^zuul.d/.*"},{"line_number":15,"context_line":"            - requirements.txt"},{"line_number":16,"context_line":"      - tripleo-ci-centos-7-scenario010-standalone:"},{"line_number":17,"context_line":"          files:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9fb8cfa7_7d9f7601","line":14,"range":{"start_line":14,"start_character":12,"end_line":14,"end_character":24},"in_reply_to":"9fb8cfa7_33306156","updated":"2019-07-01 13:20:14.000000000","message":"Once again we\u0027re not eliminating CI, we only running jobs that actually test the code.","commit_id":"7c11e3cb8b7ce9cab97b4c6ed121c5b0ec185d10"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"d2e49222d7bd4667a084cc9651f42fea87dd119f","unresolved":false,"context_lines":[{"line_number":11,"context_line":"      - tripleo-ci-centos-7-scenario004-standalone:"},{"line_number":12,"context_line":"          files:"},{"line_number":13,"context_line":"            - ^tripleo_ansible/.*"},{"line_number":14,"context_line":"            - ^zuul.d/.*"},{"line_number":15,"context_line":"            - requirements.txt"},{"line_number":16,"context_line":"      - tripleo-ci-centos-7-scenario010-standalone:"},{"line_number":17,"context_line":"          files:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9fb8cfa7_14299a35","line":14,"range":{"start_line":14,"start_character":12,"end_line":14,"end_character":24},"in_reply_to":"9fb8cfa7_53a08cf4","updated":"2019-06-27 19:58:32.000000000","message":"I think it does make it easier to address ci issues. If you change something in zuul.d/ you have to see it in the same patch. Without sending any dummy DNM patches or kind of","commit_id":"7c11e3cb8b7ce9cab97b4c6ed121c5b0ec185d10"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"560729aafcfd70dacecaee1c40f0502bf3636ef6","unresolved":false,"context_lines":[{"line_number":11,"context_line":"      - tripleo-ci-centos-7-scenario004-standalone:"},{"line_number":12,"context_line":"          files:"},{"line_number":13,"context_line":"            - ^tripleo_ansible/.*"},{"line_number":14,"context_line":"            - ^zuul.d/.*"},{"line_number":15,"context_line":"            - requirements.txt"},{"line_number":16,"context_line":"      - tripleo-ci-centos-7-scenario010-standalone:"},{"line_number":17,"context_line":"          files:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9fb8cfa7_f3bcf706","line":14,"range":{"start_line":14,"start_character":12,"end_line":14,"end_character":24},"in_reply_to":"9fb8cfa7_780c64eb","updated":"2019-07-01 14:14:21.000000000","message":"But is currently not exercised by those jobs. This isn\u0027t zuul/zuul","commit_id":"7c11e3cb8b7ce9cab97b4c6ed121c5b0ec185d10"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"6bf73dbad0fa22eba5f35b0a24543e084c8c4ea1","unresolved":false,"context_lines":[{"line_number":11,"context_line":"      - tripleo-ci-centos-7-scenario004-standalone:"},{"line_number":12,"context_line":"          files:"},{"line_number":13,"context_line":"            - ^tripleo_ansible/.*"},{"line_number":14,"context_line":"            - ^zuul.d/.*"},{"line_number":15,"context_line":"            - requirements.txt"},{"line_number":16,"context_line":"      - tripleo-ci-centos-7-scenario010-standalone:"},{"line_number":17,"context_line":"          files:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9fb8cfa7_780c64eb","line":14,"range":{"start_line":14,"start_character":12,"end_line":14,"end_character":24},"in_reply_to":"9fb8cfa7_7d9f7601","updated":"2019-07-01 13:41:11.000000000","message":"Yes, zuul.d/layout.yaml is also a code imho.","commit_id":"7c11e3cb8b7ce9cab97b4c6ed121c5b0ec185d10"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"5b35594fef9be49b666af678c5f7ef4f07460e3c","unresolved":false,"context_lines":[{"line_number":11,"context_line":"      - tripleo-ci-centos-7-scenario004-standalone:"},{"line_number":12,"context_line":"          files:"},{"line_number":13,"context_line":"            - ^tripleo_ansible/.*"},{"line_number":14,"context_line":"            - ^zuul.d/.*"},{"line_number":15,"context_line":"            - requirements.txt"},{"line_number":16,"context_line":"      - tripleo-ci-centos-7-scenario010-standalone:"},{"line_number":17,"context_line":"          files:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9fb8cfa7_a543b06b","line":14,"range":{"start_line":14,"start_character":12,"end_line":14,"end_character":24},"in_reply_to":"9fb8cfa7_8538ecd2","updated":"2019-06-28 15:07:33.000000000","message":"\u003e Beside of syntax could be done a lot of errors of configuration,\n \u003e including files: irrelevant_files: job vars, nodesets and god knows\n \u003e what else. It\u0027s all written by blood.\n \u003e \n \u003e I don\u0027t understand the point to argue here, CIs role is to test\n \u003e changes, even if it takes time. If we don\u0027t want to wait - let\u0027s\n \u003e just remove all jobs here, they all take time and delay our work.\n\nSince we don\u0027t change any of those configurations in here, this tests nothing. What you are trying to prevent is not being configured here so you\u0027re currently not testing that. CI\u0027s job is to actually test the change.  irrerelvant_files don\u0027t get tested by this configuration.  We\u0027re not configuring job vars, nodesets in this file. If we were then your argument makes sense. It currently does not.\n\nWe need to be smarter about CI job configurations. This is not smarter because it is including jobs that don\u0027t actually test anything here. If/When we actually add job vars to this file, then it would make sense to include it and would only make sense to include it if zuul.d/layout.yaml is modified. Since we are not doing that, this does not make sense.","commit_id":"7c11e3cb8b7ce9cab97b4c6ed121c5b0ec185d10"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"31a11ff9ee8adbfebb02c7b36366b40e0bd1b109","unresolved":false,"context_lines":[{"line_number":11,"context_line":"      - tripleo-ci-centos-7-scenario004-standalone:"},{"line_number":12,"context_line":"          files:"},{"line_number":13,"context_line":"            - ^tripleo_ansible/.*"},{"line_number":14,"context_line":"            - ^zuul.d/.*"},{"line_number":15,"context_line":"            - requirements.txt"},{"line_number":16,"context_line":"      - tripleo-ci-centos-7-scenario010-standalone:"},{"line_number":17,"context_line":"          files:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9fb8cfa7_e5d3e85e","line":14,"range":{"start_line":14,"start_character":12,"end_line":14,"end_character":24},"in_reply_to":"9fb8cfa7_8538ecd2","updated":"2019-06-28 14:33:51.000000000","message":"If you don\u0027t want to run all on molecule job changes - just set zuul.d/molecule.yaml to irrelevant files for standalone jobs.","commit_id":"7c11e3cb8b7ce9cab97b4c6ed121c5b0ec185d10"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"0a803cae805d705dbe7f43efc664f9484e43de91","unresolved":false,"context_lines":[{"line_number":11,"context_line":"      - tripleo-ci-centos-7-scenario004-standalone:"},{"line_number":12,"context_line":"          files:"},{"line_number":13,"context_line":"            - ^tripleo_ansible/.*"},{"line_number":14,"context_line":"            - ^zuul.d/.*"},{"line_number":15,"context_line":"            - requirements.txt"},{"line_number":16,"context_line":"      - tripleo-ci-centos-7-scenario010-standalone:"},{"line_number":17,"context_line":"          files:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9fb8cfa7_33306156","line":14,"range":{"start_line":14,"start_character":12,"end_line":14,"end_character":24},"in_reply_to":"9fb8cfa7_a543b06b","updated":"2019-07-01 11:40:30.000000000","message":"It\u0027s bad practice to run a change and not to see results of this change applied in CI run. It\u0027s too big price for taking two more VMs once a year.\nBut I\u0027d prefer to merge at least something so I don\u0027t need to fix then promotion jobs later.","commit_id":"7c11e3cb8b7ce9cab97b4c6ed121c5b0ec185d10"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"27e508d4f4f830ca4c6ab52b118adbe09d5997c8","unresolved":false,"context_lines":[{"line_number":11,"context_line":"      - tripleo-ci-centos-7-scenario004-standalone:"},{"line_number":12,"context_line":"          files:"},{"line_number":13,"context_line":"            - ^tripleo_ansible/.*"},{"line_number":14,"context_line":"            - ^zuul.d/.*"},{"line_number":15,"context_line":"            - requirements.txt"},{"line_number":16,"context_line":"      - tripleo-ci-centos-7-scenario010-standalone:"},{"line_number":17,"context_line":"          files:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9fb8cfa7_c80fe634","line":14,"range":{"start_line":14,"start_character":12,"end_line":14,"end_character":24},"in_reply_to":"9fb8cfa7_a852b233","updated":"2019-06-27 21:40:35.000000000","message":"These jobs don\u0027t actually test anything in this file other than existing.  zuul handles any broken syntax problems so they solve nothing by triggering on the zuul.d files. In fact they would be triggered on molecule job changes which means they do more harm than good.","commit_id":"7c11e3cb8b7ce9cab97b4c6ed121c5b0ec185d10"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"46a9862ba29a2a655cb22742cb23494d71ceae1f","unresolved":false,"context_lines":[{"line_number":11,"context_line":"      - tripleo-ci-centos-7-scenario004-standalone:"},{"line_number":12,"context_line":"          files:"},{"line_number":13,"context_line":"            - ^tripleo_ansible/.*"},{"line_number":14,"context_line":"            - ^zuul.d/.*"},{"line_number":15,"context_line":"            - requirements.txt"},{"line_number":16,"context_line":"      - tripleo-ci-centos-7-scenario010-standalone:"},{"line_number":17,"context_line":"          files:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9fb8cfa7_a852b233","line":14,"range":{"start_line":14,"start_character":12,"end_line":14,"end_character":24},"in_reply_to":"9fb8cfa7_b4a44ea7","updated":"2019-06-27 21:38:13.000000000","message":"I think hours are small price for having non-broken CI. Also it\u0027s not place of frequent changes here, usually it\u0027s pretty stable file.","commit_id":"7c11e3cb8b7ce9cab97b4c6ed121c5b0ec185d10"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"76a33b5f0d94fcc60fae5ed672a3217afa5d3a2e","unresolved":false,"context_lines":[{"line_number":11,"context_line":"      - tripleo-ci-centos-7-scenario004-standalone:"},{"line_number":12,"context_line":"          files:"},{"line_number":13,"context_line":"            - ^tripleo_ansible/.*"},{"line_number":14,"context_line":"            - ^zuul.d/.*"},{"line_number":15,"context_line":"            - requirements.txt"},{"line_number":16,"context_line":"      - tripleo-ci-centos-7-scenario010-standalone:"},{"line_number":17,"context_line":"          files:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9fb8cfa7_8538ecd2","line":14,"range":{"start_line":14,"start_character":12,"end_line":14,"end_character":24},"in_reply_to":"9fb8cfa7_c80fe634","updated":"2019-06-28 14:25:04.000000000","message":"Beside of syntax could be done a lot of errors of configuration, including files: irrelevant_files: job vars, nodesets and god knows what else. It\u0027s all written by blood.\n\nI don\u0027t understand the point to argue here, CIs role is to test changes, even if it takes time. If we don\u0027t want to wait - let\u0027s just remove all jobs here, they all take time and delay our work.","commit_id":"7c11e3cb8b7ce9cab97b4c6ed121c5b0ec185d10"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"7705f4273c0531596586fc258775a9613f65b995","unresolved":false,"context_lines":[{"line_number":11,"context_line":"      - tripleo-ci-centos-7-scenario004-standalone:"},{"line_number":12,"context_line":"          files:"},{"line_number":13,"context_line":"            - ^tripleo_ansible/.*"},{"line_number":14,"context_line":"            - ^zuul.d/.*"},{"line_number":15,"context_line":"            - requirements.txt"},{"line_number":16,"context_line":"      - tripleo-ci-centos-7-scenario010-standalone:"},{"line_number":17,"context_line":"          files:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9fb8cfa7_79069cd0","line":14,"range":{"start_line":14,"start_character":12,"end_line":14,"end_character":24},"in_reply_to":"9fb8cfa7_f3bcf706","updated":"2019-07-02 07:09:08.000000000","message":"i agree with Sagi - this change is adding jobs to the gate of this project without actually seeing those jobs run on this project.","commit_id":"7c11e3cb8b7ce9cab97b4c6ed121c5b0ec185d10"}]}