)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"e86479cf00c8b82c6de73aa0ccc7d58fa8b5c7a1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e74fd456_4f64699f","updated":"2022-06-03 06:23:20.000000000","message":"results look good compared to \u0027old\u0027 way we set these:\n\nhere: \n\nhttps://e4b1d932f45dd13dc874-9e40cb692266453aefdc69ab18e06808.ssl.cf1.rackcdn.com/844453/1/check/tripleo-ci-centos-9-scenario007-multinode-oooq-container/e9dfd75/logs/undercloud/var/log/tempest/tempest_run.log\n\n\nold way (another review):\n\nhttps://4b125a6587520ffd7605-3e126156a22dab1f4ba4bdb0e9add880.ssl.cf2.rackcdn.com/844006/2/check/tripleo-ci-centos-9-scenario007-multinode-oooq-container/a83702d/logs/undercloud/var/log/tempest/tempest_run.log","commit_id":"cc0a54fd962390afdf138f893c5af8629f332ff3"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"8403568bf5e130e6089228a01fa6f9781aef0803","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"35d30b88_ed2536bd","updated":"2022-06-22 11:38:55.000000000","message":"revoted as discussed on the review call\n\nplease sanity check the tests running are correct and add some comment pointing to logs that prove it\n\nplease add link to the test in the commit message ","commit_id":"cc0a54fd962390afdf138f893c5af8629f332ff3"},{"author":{"_account_id":30742,"name":"Soniya Murlidhar Vyas","email":"svyas@redhat.com","username":"svyas"},"change_message_id":"9326432a6aa3160c9355ff8533f9c144dbbdcd04","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0c6a2929_46fcd78d","in_reply_to":"35d30b88_ed2536bd","updated":"2022-06-23 13:58:41.000000000","message":"Okay","commit_id":"cc0a54fd962390afdf138f893c5af8629f332ff3"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"b585cd409067f115a10ceb42d5f59615b9f31f16","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"ff38e055_de956309","updated":"2022-06-29 11:44:44.000000000","message":"Make sense!","commit_id":"baecddecb97108769907e3b63f13c57361e3b8a9"},{"author":{"_account_id":30742,"name":"Soniya Murlidhar Vyas","email":"svyas@redhat.com","username":"svyas"},"change_message_id":"cb6fc3f662d6b6f1189f46e5a06046ba9cca8c49","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"95291079_0cb99dea","updated":"2022-07-28 13:00:33.000000000","message":"Removing \u0027Depends-on\u0027 from this patch since it is blocking the zuul gate.\nHere is the depends-on patch:- https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/844450\n\nTill then marking as WIP.","commit_id":"edd801b8bab7edc5c307dd4f82eae4c2d974e27a"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"72eb3dec6cf49e73591768f72f761a44a0d49e42","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"f644b9b0_f210a32f","updated":"2022-07-28 12:48:44.000000000","message":"WRt the zuul error (got pinged about this just now by Soniya):\n\nI believe it is referring to that https://opendev.org/openstack/tripleo-quickstart/src/commit/da32b5bf0e0ef699c749c48b8c4b856cf902a757/zuul.d/layout.yaml#L3 (    queue: tripleo) vs https://opendev.org/openstack/openstack-tempest-skiplist/src/commit/366e10984c2e5a39b1552eb78bca8ab6ab0b4477/.zuul.yaml#L1 (no queue name specified)\n\nso maybe you need to remove the depends-on, but then block this manually, until the other one merges first?","commit_id":"edd801b8bab7edc5c307dd4f82eae4c2d974e27a"},{"author":{"_account_id":30742,"name":"Soniya Murlidhar Vyas","email":"svyas@redhat.com","username":"svyas"},"change_message_id":"cd5c744c786d76485e2d073cf59d377a3e0e0033","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"e054df80_f811c704","updated":"2022-07-18 13:32:57.000000000","message":"recheck","commit_id":"edd801b8bab7edc5c307dd4f82eae4c2d974e27a"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"a3c90dcd820161444176c64310b3b0d64f67e4ab","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"e851e6a8_9a3a16c3","in_reply_to":"f644b9b0_f210a32f","updated":"2022-07-28 12:50:04.000000000","message":"15:49 \u003c marios\u003e soniya29: not sure you want to change the zuul queue in \n                openstack-tempest-skiplist to be tripleo since in theory could be used \n                by others... however in practice it will be only tripleo really so that \n                way may also work","commit_id":"edd801b8bab7edc5c307dd4f82eae4c2d974e27a"},{"author":{"_account_id":30742,"name":"Soniya Murlidhar Vyas","email":"svyas@redhat.com","username":"svyas"},"change_message_id":"0067a5ed2cf746c2b961098d25452e84b9a74b9f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"63e091e3_58471c45","updated":"2022-07-29 07:32:55.000000000","message":"recheck","commit_id":"a73200d7299436ce22af2c644c77304dec02e8d6"}],"config/general_config/featureset030.yml":[{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"d44689508949219d1a4d781765b4c4aa16245a56","unresolved":true,"context_lines":[{"line_number":85,"context_line":"test_white_regex: \u0027\u0027"},{"line_number":86,"context_line":"tempest_whitelist:"},{"line_number":87,"context_line":"  - \u0027tempest.scenario.test_network_basic_ops.TestNetworkBasicOps.test_network_basic_ops\u0027"},{"line_number":88,"context_line":"  - \u0027tempest.scenario.test_minimum_basic.TestMinimumBasicScenario.test_minimum_basic_scenario\u0027"},{"line_number":89,"context_line":"overcloud_container_cli: \u003e-"},{"line_number":90,"context_line":"  {%- if ansible_distribution in [\"RedHat\", \"CentOS\"] and ansible_distribution_major_version|int \u003e\u003d 8 %}podman"},{"line_number":91,"context_line":"  {%- else %}docker{% endif -%}"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"abfd34b7_b4bcca1c","side":"PARENT","line":88,"updated":"2022-06-22 11:39:41.000000000","message":"https://opendev.org/openstack/openstack-tempest-skiplist/src/branch/master/roles/validate-tempest/vars/tempest_skip.yml#L1061\n\nno running - in the skiplist","commit_id":"dba3e0630f1e2860751d4492a2d09407e7952822"},{"author":{"_account_id":30742,"name":"Soniya Murlidhar Vyas","email":"svyas@redhat.com","username":"svyas"},"change_message_id":"9326432a6aa3160c9355ff8533f9c144dbbdcd04","unresolved":false,"context_lines":[{"line_number":85,"context_line":"test_white_regex: \u0027\u0027"},{"line_number":86,"context_line":"tempest_whitelist:"},{"line_number":87,"context_line":"  - \u0027tempest.scenario.test_network_basic_ops.TestNetworkBasicOps.test_network_basic_ops\u0027"},{"line_number":88,"context_line":"  - \u0027tempest.scenario.test_minimum_basic.TestMinimumBasicScenario.test_minimum_basic_scenario\u0027"},{"line_number":89,"context_line":"overcloud_container_cli: \u003e-"},{"line_number":90,"context_line":"  {%- if ansible_distribution in [\"RedHat\", \"CentOS\"] and ansible_distribution_major_version|int \u003e\u003d 8 %}podman"},{"line_number":91,"context_line":"  {%- else %}docker{% endif -%}"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"109b6bb9_ad85639e","side":"PARENT","line":88,"in_reply_to":"abfd34b7_b4bcca1c","updated":"2022-06-23 13:58:41.000000000","message":"Done","commit_id":"dba3e0630f1e2860751d4492a2d09407e7952822"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"041469bdb56b729fc62b875e457d693fb9e70d13","unresolved":true,"context_lines":[{"line_number":85,"context_line":"test_white_regex: \u0027\u0027"},{"line_number":86,"context_line":"tempest_whitelist:"},{"line_number":87,"context_line":"  - \u0027tempest.scenario.test_network_basic_ops.TestNetworkBasicOps.test_network_basic_ops\u0027"},{"line_number":88,"context_line":"  - \u0027tempest.scenario.test_minimum_basic.TestMinimumBasicScenario.test_minimum_basic_scenario\u0027"},{"line_number":89,"context_line":"overcloud_container_cli: \u003e-"},{"line_number":90,"context_line":"  {%- if ansible_distribution in [\"RedHat\", \"CentOS\"] and ansible_distribution_major_version|int \u003e\u003d 8 %}podman"},{"line_number":91,"context_line":"  {%- else %}docker{% endif -%}"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"c46d51c8_fae27d39","side":"PARENT","line":88,"range":{"start_line":88,"start_character":45,"end_line":88,"end_character":46},"updated":"2022-06-24 06:24:20.000000000","message":"k... tests from this patch at [1] and from a current periodic run at [2]\n\nThe only difference is for this patch we have the tempest.scenario.test_minimum_basic.TestMinimumBasicScenario but it is skipped as hilighted by rlandy it is in the skiplist.\n\n[1] https://logserver.rdoproject.org/02/40102/7/check/periodic-tripleo-ci-centos-9-scenario007-multinode-oooq-container-master/0bf27e8/logs/undercloud/var/log/tempest/stestr_results.html.gz\n[2] https://logserver.rdoproject.org/openstack-periodic-integration-main/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-centos-9-scenario007-multinode-oooq-container-master/c40267c/logs/undercloud/var/log/tempest/stestr_results.html.gz","commit_id":"53489a06f63c76457276926bbfb9f8788badcfd1"},{"author":{"_account_id":30742,"name":"Soniya Murlidhar Vyas","email":"svyas@redhat.com","username":"svyas"},"change_message_id":"6e3e7de75c05f3293c38f38fa8995d8377d029d3","unresolved":true,"context_lines":[{"line_number":85,"context_line":"test_white_regex: \u0027\u0027"},{"line_number":86,"context_line":"tempest_whitelist:"},{"line_number":87,"context_line":"  - \u0027tempest.scenario.test_network_basic_ops.TestNetworkBasicOps.test_network_basic_ops\u0027"},{"line_number":88,"context_line":"  - \u0027tempest.scenario.test_minimum_basic.TestMinimumBasicScenario.test_minimum_basic_scenario\u0027"},{"line_number":89,"context_line":"overcloud_container_cli: \u003e-"},{"line_number":90,"context_line":"  {%- if ansible_distribution in [\"RedHat\", \"CentOS\"] and ansible_distribution_major_version|int \u003e\u003d 8 %}podman"},{"line_number":91,"context_line":"  {%- else %}docker{% endif -%}"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"136fe64c_c2251e84","side":"PARENT","line":88,"range":{"start_line":88,"start_character":45,"end_line":88,"end_character":46},"in_reply_to":"c46d51c8_fae27d39","updated":"2022-06-29 08:06:47.000000000","message":"As per the discussion with arx, TestMinimumBasicScenario is getting skipped because we have cinder disabled on sc007[1] from last 5 years. Enabling it in order to get the test pass which is disabled from so long is not so feasible. Hence, we can ignore it.\n\n[1] https://opendev.org/openstack/tripleo-heat-templates/src/branch/master/ci/environments/scenario007-standalone.yaml#L12","commit_id":"53489a06f63c76457276926bbfb9f8788badcfd1"}]}