)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"9a2c207f6c922cb86a31aeccf3730173db261e08","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     yatinkarel \u003cykarel@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2022-08-17 19:11:32 +0530"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Temporary make undercloud upgrade job non voting"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In order to switch default ML2 plugin in Undercloud to"},{"line_number":10,"context_line":"OVN[1] temporary make undercloud upgrade job non-voting."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"abd11d2b_d0cc5210","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":9},"updated":"2022-08-19 07:13:41.000000000","message":"need some more context please? \n\nalso can we please have the \u0027make voting again\u0027 patch prepared in advance and put into the review chain? with rebase and depends-on as needed and it should be green there so we know this will work. is that possible?","commit_id":"9fb1857e7e34ebc2eebbf2015bfee9fe700dfe46"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"a1a887785c9b6b2f7e7c73884752082d5d8da294","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     yatinkarel \u003cykarel@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2022-08-17 19:11:32 +0530"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Temporary make undercloud upgrade job non voting"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In order to switch default ML2 plugin in Undercloud to"},{"line_number":10,"context_line":"OVN[1] temporary make undercloud upgrade job non-voting."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"d4486e80_a0aa5454","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":9},"in_reply_to":"2f22ffd3_d1dac688","updated":"2022-08-22 10:47:24.000000000","message":"Pushed revert patch https://review.opendev.org/c/openstack/tripleo-ci/+/853947 and updated commit message","commit_id":"9fb1857e7e34ebc2eebbf2015bfee9fe700dfe46"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"9170bb738b9a2ede873f0e5e4e0eee98af099f90","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     yatinkarel \u003cykarel@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2022-08-17 19:11:32 +0530"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Temporary make undercloud upgrade job non voting"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In order to switch default ML2 plugin in Undercloud to"},{"line_number":10,"context_line":"OVN[1] temporary make undercloud upgrade job non-voting."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"b4086732_b5d24628","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":9},"in_reply_to":"4345aae0_14ed0f43","updated":"2022-09-07 08:11:09.000000000","message":"thanks for pointer to https://review.opendev.org/c/openstack/tripleo-quickstart/+/853503/ but what did you mean by \u0027parent of the revert patch\u0027 - the revert is @ tripleo-ci \n\nmainly how/where is tripleo-quickstart/+/853503 related in the merge series/order ?\n\n[EDIT]: ok i see now that is one of the main patches which you are referencing in the commit message here. ok its good that we can already see the job is green there thanks\n\nmarking this resolved but added a thought in https://review.opendev.org/c/openstack/tripleo-quickstart/+/853503/2//COMMIT_MSG wdyt we could make the job voting again there","commit_id":"9fb1857e7e34ebc2eebbf2015bfee9fe700dfe46"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"59958dea800f60be8daaa9bbaf67ea3c4f2d6ba3","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     yatinkarel \u003cykarel@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2022-08-17 19:11:32 +0530"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Temporary make undercloud upgrade job non voting"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In order to switch default ML2 plugin in Undercloud to"},{"line_number":10,"context_line":"OVN[1] temporary make undercloud upgrade job non-voting."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"2f22ffd3_d1dac688","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":9},"in_reply_to":"abd11d2b_d0cc5210","updated":"2022-08-19 07:21:55.000000000","message":"Context mostly is switch default network plugin on undercloud https://review.opendev.org/850158 which i already added in commit message, may be that\u0027s not much clear.\n\nwrt patch for voting yes i planned to do that once have results/feedback in https://review.opendev.org/q/topic:ovn-baremetal","commit_id":"9fb1857e7e34ebc2eebbf2015bfee9fe700dfe46"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"149941a6abcb13d1a18b004377f7cd4e188536db","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     yatinkarel \u003cykarel@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2022-08-17 19:11:32 +0530"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Temporary make undercloud upgrade job non voting"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In order to switch default ML2 plugin in Undercloud to"},{"line_number":10,"context_line":"OVN[1] temporary make undercloud upgrade job non-voting."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"4345aae0_14ed0f43","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":9},"in_reply_to":"b0886818_accaa81a","updated":"2022-09-05 14:23:20.000000000","message":"@Marios Upgrade job is green on parent of the revert patch https://review.opendev.org/c/openstack/tripleo-quickstart/+/853503/","commit_id":"9fb1857e7e34ebc2eebbf2015bfee9fe700dfe46"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"235560cc64bccd07ff226ba586fb527f4354385a","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     yatinkarel \u003cykarel@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2022-08-17 19:11:32 +0530"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Temporary make undercloud upgrade job non voting"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In order to switch default ML2 plugin in Undercloud to"},{"line_number":10,"context_line":"OVN[1] temporary make undercloud upgrade job non-voting."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"b0886818_accaa81a","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":9},"in_reply_to":"d4486e80_a0aa5454","updated":"2022-09-05 07:30:41.000000000","message":"(sorry for the delay i\u0027ve been out)\n\nthanks for posting https://review.opendev.org/c/openstack/tripleo-ci/+/853947\nHowever the upgrade job is not running in that patch - can you please post a DNM/test patch on tripleo-heat-templates that will depend-on tripleo-ci/+/853947 so we can be sure the job is green there?","commit_id":"9fb1857e7e34ebc2eebbf2015bfee9fe700dfe46"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"235560cc64bccd07ff226ba586fb527f4354385a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"00d5adb1_a5381eb9","updated":"2022-09-05 07:30:41.000000000","message":"Before we hit go on this we should make sure everything else is lined up.\n\nIdeally the patches at https://review.opendev.org/c/openstack/tripleo-heat-templates/+/850158/ https://review.opendev.org/c/openstack/tripleo-quickstart/+/853503/ will already have some reviews/almost ready to merge (yes i realise this one need to merge first but lets avoid merging this then waiting 2 weeks to merge the other patches before we can make the job vote again).\n\nI\u0027d like to minimize the number of days this will be non voting","commit_id":"0d1a26daf8bb606b2ba36bf8d5887f54d79b5187"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"74e074958747917aaa3fae2b9efc6b8b1097f344","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"eeb06e8f_d71ebd57","updated":"2022-09-13 15:34:32.000000000","message":"Not needed for now as not switching the default now and going with different approach https://review.opendev.org/c/openstack/tripleo-heat-templates/+/857436","commit_id":"0d1a26daf8bb606b2ba36bf8d5887f54d79b5187"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"9170bb738b9a2ede873f0e5e4e0eee98af099f90","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b315d2a3_56612121","updated":"2022-09-07 08:11:09.000000000","message":"holding on the +2 until we are closer to merging the other patches thanks","commit_id":"0d1a26daf8bb606b2ba36bf8d5887f54d79b5187"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"149941a6abcb13d1a18b004377f7cd4e188536db","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"9a967f39_75705683","in_reply_to":"00d5adb1_a5381eb9","updated":"2022-09-05 14:23:20.000000000","message":"Thanks will take care of the window, will ask merging this only once other patches are approved.","commit_id":"0d1a26daf8bb606b2ba36bf8d5887f54d79b5187"}],"zuul.d/upgrades-jobs-templates.yaml":[{"author":{"_account_id":34271,"name":"Miro Tomaska","display_name":"Miro Tomaska","email":"mtomaska@redhat.com","username":"mtomaska"},"change_message_id":"c248d0cead87c604e25f9f247a4476ff0ae59176","unresolved":true,"context_lines":[{"line_number":53,"context_line":"            dependencies:"},{"line_number":54,"context_line":"              - tripleo-ci-centos-9-content-provider"},{"line_number":55,"context_line":"        - tripleo-ci-centos-9-undercloud-upgrade: \u0026cs9_undercloud_upgrade"},{"line_number":56,"context_line":"            voting: false"},{"line_number":57,"context_line":"            vars: *consumer_vars"},{"line_number":58,"context_line":"            irrelevant-files: \u0026undercloud_ignored"},{"line_number":59,"context_line":"              - .*molecule.*"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"c7ef011e_55377d92","line":56,"range":{"start_line":56,"start_character":11,"end_line":56,"end_character":25},"updated":"2022-08-18 01:24:14.000000000","message":"I think a FIXME comment or something like that would be good here, to let others know what is going on.","commit_id":"9fb1857e7e34ebc2eebbf2015bfee9fe700dfe46"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"a1a887785c9b6b2f7e7c73884752082d5d8da294","unresolved":false,"context_lines":[{"line_number":53,"context_line":"            dependencies:"},{"line_number":54,"context_line":"              - tripleo-ci-centos-9-content-provider"},{"line_number":55,"context_line":"        - tripleo-ci-centos-9-undercloud-upgrade: \u0026cs9_undercloud_upgrade"},{"line_number":56,"context_line":"            voting: false"},{"line_number":57,"context_line":"            vars: *consumer_vars"},{"line_number":58,"context_line":"            irrelevant-files: \u0026undercloud_ignored"},{"line_number":59,"context_line":"              - .*molecule.*"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"10fcc000_758e20d3","line":56,"range":{"start_line":56,"start_character":11,"end_line":56,"end_character":25},"in_reply_to":"5d2f748f_4baaf2aa","updated":"2022-08-22 10:47:24.000000000","message":"Done","commit_id":"9fb1857e7e34ebc2eebbf2015bfee9fe700dfe46"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"59958dea800f60be8daaa9bbaf67ea3c4f2d6ba3","unresolved":true,"context_lines":[{"line_number":53,"context_line":"            dependencies:"},{"line_number":54,"context_line":"              - tripleo-ci-centos-9-content-provider"},{"line_number":55,"context_line":"        - tripleo-ci-centos-9-undercloud-upgrade: \u0026cs9_undercloud_upgrade"},{"line_number":56,"context_line":"            voting: false"},{"line_number":57,"context_line":"            vars: *consumer_vars"},{"line_number":58,"context_line":"            irrelevant-files: \u0026undercloud_ignored"},{"line_number":59,"context_line":"              - .*molecule.*"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5d2f748f_4baaf2aa","line":56,"range":{"start_line":56,"start_character":11,"end_line":56,"end_character":25},"in_reply_to":"75d25347_fc12a781","updated":"2022-08-19 07:21:55.000000000","message":"Ok can do i didn\u0027t did initially as was planning to put revert together, but adding comment also shouldn\u0027t harm.","commit_id":"9fb1857e7e34ebc2eebbf2015bfee9fe700dfe46"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"9a2c207f6c922cb86a31aeccf3730173db261e08","unresolved":true,"context_lines":[{"line_number":53,"context_line":"            dependencies:"},{"line_number":54,"context_line":"              - tripleo-ci-centos-9-content-provider"},{"line_number":55,"context_line":"        - tripleo-ci-centos-9-undercloud-upgrade: \u0026cs9_undercloud_upgrade"},{"line_number":56,"context_line":"            voting: false"},{"line_number":57,"context_line":"            vars: *consumer_vars"},{"line_number":58,"context_line":"            irrelevant-files: \u0026undercloud_ignored"},{"line_number":59,"context_line":"              - .*molecule.*"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"75d25347_fc12a781","line":56,"range":{"start_line":56,"start_character":11,"end_line":56,"end_character":25},"in_reply_to":"c7ef011e_55377d92","updated":"2022-08-19 07:13:41.000000000","message":"+1","commit_id":"9fb1857e7e34ebc2eebbf2015bfee9fe700dfe46"}]}