)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"17a9ce866ace8bb1b702ab122fe8ff1998debca7","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Amol Kahat \u003camolkahat@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2020-04-08 13:34:28 +0530"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Port tripleo-ci-centos-7-scenario012-standalone job to os_tempest"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Variables removed:"},{"line_number":10,"context_line":"run_tempest"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"df33271e_50af35e2","line":7,"range":{"start_line":7,"start_character":5,"end_line":7,"end_character":46},"updated":"2020-04-09 06:43:14.000000000","message":"as it stands on v4, this is wrong ^ you are doing it for tripleo-ci-centos-8-scenario012-standalone but leaving the centos7 one unchanged","commit_id":"3ae16dffc3821ba1e4f3f3ff049b56525d97de9b"},{"author":{"_account_id":30750,"name":"amolkahat","display_name":"Amol Kahat","email":"amolkahat@gmail.com","username":"amolkahat"},"change_message_id":"c8398311babeb82959566583a01a126d3812f422","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Amol Kahat \u003camolkahat@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2020-04-08 13:34:28 +0530"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Port tripleo-ci-centos-7-scenario012-standalone job to os_tempest"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Variables removed:"},{"line_number":10,"context_line":"run_tempest"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"df33271e_4e22f2ba","line":7,"range":{"start_line":7,"start_character":5,"end_line":7,"end_character":46},"in_reply_to":"df33271e_50af35e2","updated":"2020-04-09 09:21:32.000000000","message":"First i removed from CentOS 7. But later i changed in CentOS 8. I forgot to change the commit message.","commit_id":"3ae16dffc3821ba1e4f3f3ff049b56525d97de9b"}],"zuul.d/standalone-jobs.yaml":[{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"aa8fc092e6c48fca352a2e6a7c2ebb374c3f2eb9","unresolved":false,"context_lines":[{"line_number":648,"context_line":"      featureset: \u0027060\u0027"},{"line_number":649,"context_line":"      standalone_ceph: true"},{"line_number":650,"context_line":"      featureset_override:"},{"line_number":651,"context_line":"        run_tempest: false"},{"line_number":652,"context_line":"        use_os_tempest: true"},{"line_number":653,"context_line":"        validate_services: false"},{"line_number":654,"context_line":"        standalone_environment_files:"},{"line_number":655,"context_line":"          - \u0027environments/low-memory-usage.yaml\u0027"},{"line_number":656,"context_line":"          - \u0027ci/environments/scenario012-standalone.yaml\u0027"},{"line_number":657,"context_line":"        tempest_services:"},{"line_number":658,"context_line":"          - ironic"},{"line_number":659,"context_line":"        tempest_run_concurrency: 1"},{"line_number":660,"context_line":"        tempest_tempest_conf_overrides:"},{"line_number":661,"context_line":"          auth.create_isolated_networks: \u0027False\u0027"},{"line_number":662,"context_line":"          validation.connect_method: \u0027fixed\u0027"},{"line_number":663,"context_line":"          validation.network_for_ssh: \u0027provisioning\u0027"},{"line_number":664,"context_line":"          baremetal.use_provision_network: \u0027True\u0027"},{"line_number":665,"context_line":"        tempest_test_whitelist:"},{"line_number":666,"context_line":"          - \u0027ironic_tempest_plugin.*\u0027"},{"line_number":667,"context_line":""},{"line_number":668,"context_line":"- job:"},{"line_number":669,"context_line":"    name: tripleo-ci-centos-8-scenario012-standalone"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"df33271e_33bac009","line":666,"range":{"start_line":651,"start_character":8,"end_line":666,"end_character":37},"updated":"2020-04-07 11:06:47.000000000","message":"Move tempest related vars in featureset file https://opendev.org/openstack/tripleo-quickstart/src/branch/master/config/general_config/featureset060.yml and remove validate tempest vars from there.\n\nActions:\n1. os_tempest support in fs060\n2. Remove tempest var from job in toci and add depends on here of tq patch","commit_id":"be601c9b828af090c908f346ffc45b6f38b46779"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"6f075274c0ccea3355e9ec7d782a383d673188d4","unresolved":false,"context_lines":[{"line_number":668,"context_line":"        standalone_environment_files:"},{"line_number":669,"context_line":"          - \u0027environments/low-memory-usage.yaml\u0027"},{"line_number":670,"context_line":"          - \u0027ci/environments/scenario012-standalone.yaml\u0027"},{"line_number":671,"context_line":"        tempest_plugins:"},{"line_number":672,"context_line":"          - python-ironic-tests-tempest"},{"line_number":673,"context_line":"        test_white_regex: \u0027\u0027"},{"line_number":674,"context_line":"        tempest_workers: 1"},{"line_number":675,"context_line":"        tempest_extra_config:"},{"line_number":676,"context_line":"          auth.create_isolated_networks: \u0027False\u0027"},{"line_number":677,"context_line":"          validation.connect_method: \u0027fixed\u0027"},{"line_number":678,"context_line":"          validation.network_for_ssh: \u0027provisioning\u0027"},{"line_number":679,"context_line":"          baremetal.use_provision_network: \u0027True\u0027"},{"line_number":680,"context_line":"        tempest_whitelist:"},{"line_number":681,"context_line":"          - \u0027ironic_tempest_plugin.tests.scenario.test_baremetal_multitenancy.BaremetalMultitenancy.test_baremetal_multitenancy\u0027"},{"line_number":682,"context_line":""},{"line_number":683,"context_line":"- job:"},{"line_number":684,"context_line":"    name: tripleo-ci-centos-7-standalone-os-tempest"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"df33271e_e991699b","line":681,"range":{"start_line":671,"start_character":9,"end_line":681,"end_character":18},"updated":"2020-04-08 07:21:28.000000000","message":"this is also using fs60","commit_id":"509e9fd21b67b165b27b58b06dd7e5d087ed6f7e"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"576a24e376203af999513cc14e4369e4cfa7ed49","unresolved":false,"context_lines":[{"line_number":652,"context_line":"        standalone_environment_files:"},{"line_number":653,"context_line":"          - \u0027environments/low-memory-usage.yaml\u0027"},{"line_number":654,"context_line":"          - \u0027ci/environments/scenario012-standalone.yaml\u0027"},{"line_number":655,"context_line":"        tempest_plugins:"},{"line_number":656,"context_line":"          - python-ironic-tests-tempest"},{"line_number":657,"context_line":"        test_white_regex: \u0027\u0027"},{"line_number":658,"context_line":"        tempest_workers: 1"},{"line_number":659,"context_line":"        tempest_extra_config:"},{"line_number":660,"context_line":"          auth.create_isolated_networks: \u0027False\u0027"},{"line_number":661,"context_line":"          validation.connect_method: \u0027fixed\u0027"},{"line_number":662,"context_line":"          validation.network_for_ssh: \u0027provisioning\u0027"},{"line_number":663,"context_line":"          baremetal.use_provision_network: \u0027True\u0027"},{"line_number":664,"context_line":"        tempest_whitelist:"},{"line_number":665,"context_line":"          - \u0027ironic_tempest_plugin.tests.scenario.test_baremetal_multitenancy.BaremetalMultitenancy.test_baremetal_multitenancy\u0027"},{"line_number":666,"context_line":""},{"line_number":667,"context_line":"- job:"},{"line_number":668,"context_line":"    name: tripleo-ci-centos-8-scenario012-standalone"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"df33271e_843b64c6","line":665,"range":{"start_line":655,"start_character":8,"end_line":665,"end_character":128},"updated":"2020-04-08 08:10:13.000000000","message":"it also needs to be removed","commit_id":"3ae16dffc3821ba1e4f3f3ff049b56525d97de9b"},{"author":{"_account_id":30750,"name":"amolkahat","display_name":"Amol Kahat","email":"amolkahat@gmail.com","username":"amolkahat"},"change_message_id":"e0557fead2bec5b19d8667a3470f46d9e29686f5","unresolved":false,"context_lines":[{"line_number":652,"context_line":"        standalone_environment_files:"},{"line_number":653,"context_line":"          - \u0027environments/low-memory-usage.yaml\u0027"},{"line_number":654,"context_line":"          - \u0027ci/environments/scenario012-standalone.yaml\u0027"},{"line_number":655,"context_line":"        tempest_plugins:"},{"line_number":656,"context_line":"          - python-ironic-tests-tempest"},{"line_number":657,"context_line":"        test_white_regex: \u0027\u0027"},{"line_number":658,"context_line":"        tempest_workers: 1"},{"line_number":659,"context_line":"        tempest_extra_config:"},{"line_number":660,"context_line":"          auth.create_isolated_networks: \u0027False\u0027"},{"line_number":661,"context_line":"          validation.connect_method: \u0027fixed\u0027"},{"line_number":662,"context_line":"          validation.network_for_ssh: \u0027provisioning\u0027"},{"line_number":663,"context_line":"          baremetal.use_provision_network: \u0027True\u0027"},{"line_number":664,"context_line":"        tempest_whitelist:"},{"line_number":665,"context_line":"          - \u0027ironic_tempest_plugin.tests.scenario.test_baremetal_multitenancy.BaremetalMultitenancy.test_baremetal_multitenancy\u0027"},{"line_number":666,"context_line":""},{"line_number":667,"context_line":"- job:"},{"line_number":668,"context_line":"    name: tripleo-ci-centos-8-scenario012-standalone"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"df33271e_b76df799","line":665,"range":{"start_line":655,"start_character":8,"end_line":665,"end_character":128},"in_reply_to":"df33271e_843b64c6","updated":"2020-04-08 12:24:41.000000000","message":"It\u0027s removed here: https://review.opendev.org/#/c/718346","commit_id":"3ae16dffc3821ba1e4f3f3ff049b56525d97de9b"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"17a9ce866ace8bb1b702ab122fe8ff1998debca7","unresolved":false,"context_lines":[{"line_number":652,"context_line":"        standalone_environment_files:"},{"line_number":653,"context_line":"          - \u0027environments/low-memory-usage.yaml\u0027"},{"line_number":654,"context_line":"          - \u0027ci/environments/scenario012-standalone.yaml\u0027"},{"line_number":655,"context_line":"        tempest_plugins:"},{"line_number":656,"context_line":"          - python-ironic-tests-tempest"},{"line_number":657,"context_line":"        test_white_regex: \u0027\u0027"},{"line_number":658,"context_line":"        tempest_workers: 1"},{"line_number":659,"context_line":"        tempest_extra_config:"},{"line_number":660,"context_line":"          auth.create_isolated_networks: \u0027False\u0027"},{"line_number":661,"context_line":"          validation.connect_method: \u0027fixed\u0027"},{"line_number":662,"context_line":"          validation.network_for_ssh: \u0027provisioning\u0027"},{"line_number":663,"context_line":"          baremetal.use_provision_network: \u0027True\u0027"},{"line_number":664,"context_line":"        tempest_whitelist:"},{"line_number":665,"context_line":"          - \u0027ironic_tempest_plugin.tests.scenario.test_baremetal_multitenancy.BaremetalMultitenancy.test_baremetal_multitenancy\u0027"},{"line_number":666,"context_line":""},{"line_number":667,"context_line":"- job:"},{"line_number":668,"context_line":"    name: tripleo-ci-centos-8-scenario012-standalone"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"df33271e_f0a6490f","line":665,"range":{"start_line":655,"start_character":8,"end_line":665,"end_character":128},"in_reply_to":"df33271e_b76df799","updated":"2020-04-09 06:43:14.000000000","message":"this patch is about \"Port tripleo-ci-centos-7-scenario012-standalone job to os_tempest\"\n\nso I think that bit ^^ belongs here?","commit_id":"3ae16dffc3821ba1e4f3f3ff049b56525d97de9b"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"9ca9ec476c21ef98e544af6d62a433942c1ab24f","unresolved":false,"context_lines":[{"line_number":652,"context_line":"        standalone_environment_files:"},{"line_number":653,"context_line":"          - \u0027environments/low-memory-usage.yaml\u0027"},{"line_number":654,"context_line":"          - \u0027ci/environments/scenario012-standalone.yaml\u0027"},{"line_number":655,"context_line":"        tempest_plugins:"},{"line_number":656,"context_line":"          - python-ironic-tests-tempest"},{"line_number":657,"context_line":"        test_white_regex: \u0027\u0027"},{"line_number":658,"context_line":"        tempest_workers: 1"},{"line_number":659,"context_line":"        tempest_extra_config:"},{"line_number":660,"context_line":"          auth.create_isolated_networks: \u0027False\u0027"},{"line_number":661,"context_line":"          validation.connect_method: \u0027fixed\u0027"},{"line_number":662,"context_line":"          validation.network_for_ssh: \u0027provisioning\u0027"},{"line_number":663,"context_line":"          baremetal.use_provision_network: \u0027True\u0027"},{"line_number":664,"context_line":"        tempest_whitelist:"},{"line_number":665,"context_line":"          - \u0027ironic_tempest_plugin.tests.scenario.test_baremetal_multitenancy.BaremetalMultitenancy.test_baremetal_multitenancy\u0027"},{"line_number":666,"context_line":""},{"line_number":667,"context_line":"- job:"},{"line_number":668,"context_line":"    name: tripleo-ci-centos-8-scenario012-standalone"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"df33271e_b0896176","line":665,"range":{"start_line":655,"start_character":8,"end_line":665,"end_character":128},"in_reply_to":"df33271e_f0a6490f","updated":"2020-04-09 06:44:17.000000000","message":"i.e. i agree with chandan you should remove this too or change what the patch is about in the commit message","commit_id":"3ae16dffc3821ba1e4f3f3ff049b56525d97de9b"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"22a3e8c93385ae55690879025751ceafe79c5a54","unresolved":false,"context_lines":[{"line_number":644,"context_line":"    nodeset: single-centos-7-node"},{"line_number":645,"context_line":"    branches: ^(?!stable/(newton|ocata|pike|queens|rocky|stein|train)).*$"},{"line_number":646,"context_line":"    vars:"},{"line_number":647,"context_line":"      featureset: \u0027060\u0027"},{"line_number":648,"context_line":"      standalone_ceph: true"},{"line_number":649,"context_line":"      featureset_override:"},{"line_number":650,"context_line":"        validate_services: false"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"df33271e_f3147ceb","line":647,"range":{"start_line":647,"start_character":19,"end_line":647,"end_character":22},"updated":"2020-04-10 06:47:39.000000000","message":"my \u0027problem\u0027 here is that even though the patch is \u0027port centos8 to os-tempest\u0027 it also ports this centos-7-scenario012 job to os-tempest.\n\nI guess I don\u0027t understand why you wanted to split this across two reviews... this and https://review.opendev.org/#/c/718346\n\nDon\u0027t get me wrong, it\u0027s fine to have multiple reviews but they have to be lined up correctly. So ideally https://review.opendev.org/#/c/718346 would be first to remove/deprecate ^^^ then this patch rebased ontop of it for the centos-8 os-tempest move.\n\nIn this particular case it looks like we even stopped running this tripleo-ci-centos-7-scenario012-standalone a few days ago as far as i can see at http://zuul.openstack.org/builds?job_name\u003dtripleo-ci-centos-7-scenario012-standalone\n\nmeaning it doesn\u0027t matter since even though this patch ports centos-7-scenario012 too, it won\u0027t run.","commit_id":"5f902e7c369f5d97bfcd926de2d8e5e03ef4374b"}]}