)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"234d312fbb50beb033ee07b54568362249870a78","unresolved":true,"context_lines":[{"line_number":7,"context_line":"Remove irrelevant-files from content provider jobs"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Patches to code in tripleo-quickstart-extras that"},{"line_number":10,"context_line":"trigger tripleo-ci-centos-8-undercloud-upgrade"},{"line_number":11,"context_line":"jobs are failing as"},{"line_number":12,"context_line":"tripleo-ci-centos-8-content-provider-wallaby"},{"line_number":13,"context_line":"was not run."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"18ac2691_2718159b","line":10,"range":{"start_line":10,"start_character":11,"end_line":10,"end_character":42},"updated":"2021-07-20 06:03:06.000000000","message":"I\u0027d rather go with https://review.opendev.org/c/openstack/tripleo-ci/+/801266\n\nlet\u0027s limit where the undercloud-upgrade job runs to prevent the problem","commit_id":"00221b9c5017abe84d792e95a83010751dd4f9fc"}],"zuul.d/multinode-jobs.yaml":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"234d312fbb50beb033ee07b54568362249870a78","unresolved":true,"context_lines":[{"line_number":196,"context_line":"        - tripleo-ci-centos-8-content-provider-victoria:"},{"line_number":197,"context_line":"            irrelevant-files: *multinode_ignored"},{"line_number":198,"context_line":"        - tripleo-ci-centos-8-content-provider-ussuri:"},{"line_number":199,"context_line":"            irrelevant-files: *multinode_ignored"},{"line_number":200,"context_line":"        - tripleo-ci-centos-8-content-provider-train:"},{"line_number":201,"context_line":"            irrelevant-files: *multinode_ignored"},{"line_number":202,"context_line":"        - tripleo-ci-centos-8-containers-multinode-wallaby: *multi_consumer_job_wallaby"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"aef6f0c2_361b5cf3","side":"PARENT","line":199,"updated":"2021-07-20 06:03:06.000000000","message":"doing this means we will have these content providers run unnecessarily in some cases \n\nan example is there https://review.opendev.org/c/openstack/tripleo-ci/+/779447/1#message-f88911fba103f47d5a641bf7ddfaa0c53f0b2958 (this is the reason we added the irrelevant files here in the first place) ... where a change in standalone template caused these content providers to run with nothing using them.\n\nInstead we can limit where the undercloud-upgrade job runs to prevent that, i.e. lets go with https://review.opendev.org/c/openstack/tripleo-ci/+/801266/3/zuul.d/upgrades-jobs-templates.yaml ? If the issue happens again I\u0027d be happier to deal with it, instaed of just running redundant providers.","commit_id":"ba710b8056922ad8f00e4f8440039e63e60d180f"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"53dc1949a7a3f0c0fb546cac1e0b73b8d606f5be","unresolved":true,"context_lines":[{"line_number":196,"context_line":"        - tripleo-ci-centos-8-content-provider-victoria:"},{"line_number":197,"context_line":"            irrelevant-files: *multinode_ignored"},{"line_number":198,"context_line":"        - tripleo-ci-centos-8-content-provider-ussuri:"},{"line_number":199,"context_line":"            irrelevant-files: *multinode_ignored"},{"line_number":200,"context_line":"        - tripleo-ci-centos-8-content-provider-train:"},{"line_number":201,"context_line":"            irrelevant-files: *multinode_ignored"},{"line_number":202,"context_line":"        - tripleo-ci-centos-8-containers-multinode-wallaby: *multi_consumer_job_wallaby"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"a444b12e_faa85c7b","side":"PARENT","line":199,"in_reply_to":"aef6f0c2_361b5cf3","updated":"2021-07-20 07:22:39.000000000","message":"just had an irc chat with sagi and I added a comment there https://review.opendev.org/c/openstack/tripleo-ci/+/801266/3#message-713232745c3e5003fe0ede0cc3fbe6c75bdb1f48\n\nI reluctantly remove my -1 from here (not cos I disagree with Sagi\u0027s point, but because I am reluctant to start running redundant/unused providers again... but Sagi has some ideas about how we might improve it to be discussed in next scrum)","commit_id":"ba710b8056922ad8f00e4f8440039e63e60d180f"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"87f614e797668071a9766abf41717eee4eca362c","unresolved":true,"context_lines":[{"line_number":197,"context_line":"        - tripleo-ci-centos-8-containers-multinode-train: *multi_consumer_job_train"},{"line_number":198,"context_line":"    experimental:"},{"line_number":199,"context_line":"      jobs:"},{"line_number":200,"context_line":"        - tripleo-ci-centos-7-containers-multinode-stein"},{"line_number":201,"context_line":""},{"line_number":202,"context_line":"## CentOS 8 jobs"},{"line_number":203,"context_line":"- job:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"f941dfd8_b5882c16","line":200,"range":{"start_line":200,"start_character":9,"end_line":200,"end_character":56},"updated":"2021-07-19 21:32:57.000000000","message":"this can actually stay, but anyway it\u0027s experimental","commit_id":"00221b9c5017abe84d792e95a83010751dd4f9fc"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"fc050a48d080dc30ab90fc23c4a3968f84eaf5a1","unresolved":true,"context_lines":[{"line_number":206,"context_line":"    experimental:"},{"line_number":207,"context_line":"      jobs:"},{"line_number":208,"context_line":"        - tripleo-ci-centos-7-containers-multinode-stein:"},{"line_number":209,"context_line":"            irrelevant-files: *multinode_ignored"},{"line_number":210,"context_line":""},{"line_number":211,"context_line":"## CentOS 8 jobs"},{"line_number":212,"context_line":"- job:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"2257db44_537dc131","side":"PARENT","line":209,"range":{"start_line":209,"start_character":12,"end_line":209,"end_character":48},"updated":"2021-07-20 07:37:37.000000000","message":"I think this got removed by mistake?","commit_id":"01aa4d081601ff5548a51135cc8e55e75bb0d44c"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"16664169b56000d66c533d3b01e3fb11dbf09892","unresolved":true,"context_lines":[{"line_number":206,"context_line":"    experimental:"},{"line_number":207,"context_line":"      jobs:"},{"line_number":208,"context_line":"        - tripleo-ci-centos-7-containers-multinode-stein:"},{"line_number":209,"context_line":"            irrelevant-files: *multinode_ignored"},{"line_number":210,"context_line":""},{"line_number":211,"context_line":"## CentOS 8 jobs"},{"line_number":212,"context_line":"- job:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"eedf73b5_6767f377","side":"PARENT","line":209,"range":{"start_line":209,"start_character":12,"end_line":209,"end_character":48},"in_reply_to":"2257db44_537dc131","updated":"2021-07-20 12:00:54.000000000","message":"yeah, but we need delete it anyways","commit_id":"01aa4d081601ff5548a51135cc8e55e75bb0d44c"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"89b66921c9b112ca7d9f63f9170c9b942cbba573","unresolved":true,"context_lines":[{"line_number":206,"context_line":"    experimental:"},{"line_number":207,"context_line":"      jobs:"},{"line_number":208,"context_line":"        - tripleo-ci-centos-7-containers-multinode-stein:"},{"line_number":209,"context_line":"            irrelevant-files: *multinode_ignored"},{"line_number":210,"context_line":""},{"line_number":211,"context_line":"## CentOS 8 jobs"},{"line_number":212,"context_line":"- job:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"c16cc68a_0c0dba05","side":"PARENT","line":209,"range":{"start_line":209,"start_character":12,"end_line":209,"end_character":48},"in_reply_to":"eedf73b5_6767f377","updated":"2021-07-20 12:03:20.000000000","message":"Ok, let get it merged then.","commit_id":"01aa4d081601ff5548a51135cc8e55e75bb0d44c"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"3ecfe91a65670bc04e32d39a3a629d12f823e9ed","unresolved":true,"context_lines":[{"line_number":171,"context_line":"              - ^playbooks/deprecated/.*$"},{"line_number":172,"context_line":"              - ^playbooks/tripleo-buildcontainers/.*$"},{"line_number":173,"context_line":"              - ^playbooks/tripleo-buildimages/.*$"},{"line_number":174,"context_line":"              - ^vars/sova-patterns.yml$"},{"line_number":175,"context_line":"            dependencies:"},{"line_number":176,"context_line":"              - openstack-tox-linters"},{"line_number":177,"context_line":"        - tripleo-ci-centos-8-content-provider-victoria:"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"fd68ea76_3a7f6c4c","line":174,"updated":"2021-07-20 12:17:34.000000000","message":"i think this will fix the job freeze graph problem, but it doesn\u0027t fix the issue of \u0027redundant jobs\u0027 \n\nFor example changes to featuresets will now trigger these multinode jobs even if they aren\u0027t necessary like changes to the standalone featuresets for example\n\ni guess we can add some more of those here, as long as we also add them to the undercloud_ignored ? i.e. if we are also keeping https://review.opendev.org/c/openstack/tripleo-ci/+/801266 then (has to be a followup, after 801266 merges) we can add some of those featuresets here too","commit_id":"24fbeb95cc214cf99c7d4ebd8f48e13cf1011700"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"0d49bda98978e5f6bfc141960c20074b3b13dd2c","unresolved":true,"context_lines":[{"line_number":171,"context_line":"              - ^playbooks/deprecated/.*$"},{"line_number":172,"context_line":"              - ^playbooks/tripleo-buildcontainers/.*$"},{"line_number":173,"context_line":"              - ^playbooks/tripleo-buildimages/.*$"},{"line_number":174,"context_line":"              - ^vars/sova-patterns.yml$"},{"line_number":175,"context_line":"            dependencies:"},{"line_number":176,"context_line":"              - openstack-tox-linters"},{"line_number":177,"context_line":"        - tripleo-ci-centos-8-content-provider-victoria:"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"a5b1b4ac_b8f7f2eb","line":174,"in_reply_to":"fd68ea76_3a7f6c4c","updated":"2021-07-20 12:41:17.000000000","message":"Agree, we can add more to here from https://review.opendev.org/c/openstack/tripleo-ci/+/801266","commit_id":"24fbeb95cc214cf99c7d4ebd8f48e13cf1011700"}]}