)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":30742,"name":"Soniya Murlidhar Vyas","email":"svyas@redhat.com","username":"svyas"},"change_message_id":"c3847137ee46a151e1d33fce10b78286fb912df9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"08900d10_8a53a1bd","updated":"2022-08-03 11:22:09.000000000","message":"recheck","commit_id":"f2f3d7e62b2ec44134f88a01aa4789c6d3cee0a5"},{"author":{"_account_id":8367,"name":"Arx Cruz","email":"arxcruz@redhat.com","username":"arxcruz"},"change_message_id":"905f7ead346a63f08a3868c64d6b0a51c50026c9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"55033ab4_4ed0f686","updated":"2022-09-13 08:54:47.000000000","message":"Works fine, the test shows that for train is running the 7 tests properly","commit_id":"23e24e13e6bd635f5ad127a0717ed73af6a5a43a"},{"author":{"_account_id":30742,"name":"Soniya Murlidhar Vyas","email":"svyas@redhat.com","username":"svyas"},"change_message_id":"227f3cc64be09cf5bc436c1ed1a03b7f39566cf7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"e35a23ef_58f05f3c","updated":"2022-09-15 15:27:44.000000000","message":"recheck","commit_id":"23e24e13e6bd635f5ad127a0717ed73af6a5a43a"}],"config/general_config/featureset062.yml":[{"author":{"_account_id":30742,"name":"Soniya Murlidhar Vyas","email":"svyas@redhat.com","username":"svyas"},"change_message_id":"7fb6488e2f9f676bf544463ea0774c45c5a29514","unresolved":true,"context_lines":[{"line_number":36,"context_line":"tempest_allowed_group_check: \"featureset062\""},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"# In Integration and periodic line, we run only smoke tests, hence the special"},{"line_number":39,"context_line":"# group is defined here fs62_periodic"},{"line_number":40,"context_line":"tempest_allowed_group_periodic: \"featureset062_periodic\""},{"line_number":41,"context_line":"tempest_test_periodic: \u003e-"},{"line_number":42,"context_line":"  {% if \u0027train\u0027 in release -%}"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"b4be0040_7e1a917e","line":39,"updated":"2022-08-09 13:46:07.000000000","message":"I think sc10 standalone because of this variable. in periodic we are running tempest tests on all releases but in fs62, the tests are indeed classified..so not sure what to do here?\n\nhttps://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/851395/3/roles/validate-tempest/vars/tempest_allow.yml\n\nArx and marios... may be you both can have a look here?","commit_id":"4704081505cfc99181e4c8a54606beadc1c92b7d"},{"author":{"_account_id":8367,"name":"Arx Cruz","email":"arxcruz@redhat.com","username":"arxcruz"},"change_message_id":"ff2e9d02332f94a8261c6b8be1a59440c5472044","unresolved":true,"context_lines":[{"line_number":38,"context_line":"# In Integration and periodic line, we run only smoke tests, hence the special"},{"line_number":39,"context_line":"# group is defined here fs62_periodic"},{"line_number":40,"context_line":"tempest_allowed_group_periodic: \"featureset062_periodic\""},{"line_number":41,"context_line":"tempest_test_periodic: \u003e-"},{"line_number":42,"context_line":"  {% if \u0027train\u0027 in release -%}"},{"line_number":43,"context_line":"  {{ tempest_allowed_group_check }}"},{"line_number":44,"context_line":"  {%- else -%}"},{"line_number":45,"context_line":"  {{ tempest_allowed_group_periodic }}"},{"line_number":46,"context_line":"  {%- endif -%}"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"tempest_allowed_group: \u003e-"},{"line_number":49,"context_line":"  {% if (\u0027periodic\u0027 in zuul.pipeline and not job.force_non_periodic|default(false)|bool) or (job.force_periodic|default(false)|bool) or (job.component is defined|default(false)|bool) -%}"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"11dc8aa1_4f726a3a","line":46,"range":{"start_line":41,"start_character":0,"end_line":46,"end_character":15},"updated":"2022-08-17 12:59:40.000000000","message":"We don\u0027t need it here, because we can have groups with same name and different releases in the tempest_allow, just need to add a featureset062_periodic just for train, this will reduce the complexity here because checking the logs, the list of tests to be executed is returning empty, so sc010 is failing.","commit_id":"4704081505cfc99181e4c8a54606beadc1c92b7d"},{"author":{"_account_id":8367,"name":"Arx Cruz","email":"arxcruz@redhat.com","username":"arxcruz"},"change_message_id":"905f7ead346a63f08a3868c64d6b0a51c50026c9","unresolved":false,"context_lines":[{"line_number":38,"context_line":"# In Integration and periodic line, we run only smoke tests, hence the special"},{"line_number":39,"context_line":"# group is defined here fs62_periodic"},{"line_number":40,"context_line":"tempest_allowed_group_periodic: \"featureset062_periodic\""},{"line_number":41,"context_line":"tempest_test_periodic: \u003e-"},{"line_number":42,"context_line":"  {% if \u0027train\u0027 in release -%}"},{"line_number":43,"context_line":"  {{ tempest_allowed_group_check }}"},{"line_number":44,"context_line":"  {%- else -%}"},{"line_number":45,"context_line":"  {{ tempest_allowed_group_periodic }}"},{"line_number":46,"context_line":"  {%- endif -%}"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"tempest_allowed_group: \u003e-"},{"line_number":49,"context_line":"  {% if (\u0027periodic\u0027 in zuul.pipeline and not job.force_non_periodic|default(false)|bool) or (job.force_periodic|default(false)|bool) or (job.component is defined|default(false)|bool) -%}"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"906e5606_f5bad8fe","line":46,"range":{"start_line":41,"start_character":0,"end_line":46,"end_character":15},"in_reply_to":"11dc8aa1_4f726a3a","updated":"2022-09-13 08:54:47.000000000","message":"Done","commit_id":"4704081505cfc99181e4c8a54606beadc1c92b7d"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"24b7ba5ff4b555bc70800a2ad4ba67dc1d278977","unresolved":true,"context_lines":[{"line_number":40,"context_line":"tempest_allowed_group_periodic: \"featureset062_periodic\""},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"tempest_allowed_group: \u003e-"},{"line_number":43,"context_line":"  {% if (\u0027periodic\u0027 in zuul.pipeline and not job.force_non_periodic|default(false)|bool) or (job.force_periodic|default(false)|bool) or (job.component is defined|default(false)|bool) -%}"},{"line_number":44,"context_line":"  {{ tempest_allowed_group_periodic }}"},{"line_number":45,"context_line":"  {%- else -%}"},{"line_number":46,"context_line":"  {{ tempest_allowed_group_check }}"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"498b17f3_ff34bacd","line":43,"range":{"start_line":43,"start_character":137,"end_line":43,"end_character":161},"updated":"2022-09-01 07:02:54.000000000","message":"I think here we want to run check specific tests na?\nhttps://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/851395/3/roles/validate-tempest/vars/tempest_allow.yml#132\n\nas per this in train, we only run `featureset062` in check job not in component. It is going to break the job.\n\nplease also check https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/851395/3#message-6979237ab7d3a73fa48eae2430582487832decc8","commit_id":"e1569c1acb2d08511cbdaff4cb3ca0cb74eaeb7b"},{"author":{"_account_id":30742,"name":"Soniya Murlidhar Vyas","email":"svyas@redhat.com","username":"svyas"},"change_message_id":"87437ed16e3332fc8e6e93336b4100fef21770da","unresolved":false,"context_lines":[{"line_number":40,"context_line":"tempest_allowed_group_periodic: \"featureset062_periodic\""},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"tempest_allowed_group: \u003e-"},{"line_number":43,"context_line":"  {% if (\u0027periodic\u0027 in zuul.pipeline and not job.force_non_periodic|default(false)|bool) or (job.force_periodic|default(false)|bool) or (job.component is defined|default(false)|bool) -%}"},{"line_number":44,"context_line":"  {{ tempest_allowed_group_periodic }}"},{"line_number":45,"context_line":"  {%- else -%}"},{"line_number":46,"context_line":"  {{ tempest_allowed_group_check }}"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"42f1654c_1fa74867","line":43,"range":{"start_line":43,"start_character":137,"end_line":43,"end_character":161},"in_reply_to":"498b17f3_ff34bacd","updated":"2022-09-01 09:59:43.000000000","message":"Ack","commit_id":"e1569c1acb2d08511cbdaff4cb3ca0cb74eaeb7b"}]}