)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"40429962e264b2cdcc99b4353910fc9d62c1c8ae","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e4e7d95f_b78f5ff3","updated":"2025-05-22 13:14:53.000000000","message":"I\u0027m a bit torn here - the slow label should be added to tests which takes more time, and not all the tests marked here are \"slow\".\nWe should check the run time, ragardless of whether the tests are failing right now. The failure may be related to the concurrent execution of a really slow test.\n\nhttps://docs.openstack.org/tempest/latest/HACKING.html#slow-attribute\n\nThe type\u003d\u0027slow\u0027 attribute is used to signify that a test takes a long time to run, relatively speaking. This attribute is usually applied to scenario tests, which involve a complicated series of API operations, the total runtime of which can be relatively long. This long runtime has performance implications on Zuul jobs, which is why the slow attribute is leveraged to run slow tests on a selective basis, to keep total Zuul job runtime down to a reasonable time frame.","commit_id":"a48cca8fa7fab4be95037f59dd34c7a2b1986ef9"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"1b2e6fe5a812b1ef6ca6aba859ee2f07e0cd63f6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2427db09_d71a043d","updated":"2025-05-22 17:57:07.000000000","message":"I\u0027ve noticed there is also a \"serial\" decorator, but even if we decided to use it, we shouldn\u0027t just stamp it on all tests:\nhttps://review.opendev.org/c/openstack/tempest/+/821732","commit_id":"a48cca8fa7fab4be95037f59dd34c7a2b1986ef9"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"ff36774ff14f956f2410bed57706e1242c985c0b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d653e3a6_4dbc1279","updated":"2025-05-21 16:49:28.000000000","message":"Number in the first column is the number of failures out of 60 runs\n\n```\nalready marked \u0027slow\u0027 in tempest:\n   21 tempest.scenario.test_volume_boot_pattern.TestVolumeBootPattern.test_volume_boot_pattern \n   10 tempest.scenario.test_stamp_pattern.TestStampPattern.test_stamp_pattern\n\nadded the \u0027slow\u0027 marker in this patch:\n   20 cinder_tempest_plugin.api.volume.test_volume_dependency.VolumeDependencyTests.test_delete_source_snapshot\n   19 cinder_tempest_plugin.api.volume.test_volume_dependency.VolumeDependencyTests.test_delete_source_volume\n   19 cinder_tempest_plugin.api.volume.test_volume_dependency.VolumeDependencyTests.test_delete_dep_chain_2\n   19 cinder_tempest_plugin.api.volume.test_volume_dependency.VolumeDependencyTests.test_delete_dep_chain\n   19 cinder_tempest_plugin.api.volume.test_multiple_volume_from_resource.CreateVolumesFromImageTest.test_create_from_image_multiple\n   18 cinder_tempest_plugin.api.volume.test_multiple_volume_from_resource.CreateVolumesFromSourceVolumeTest.test_create_multiple_volume_from_source_volume\n   17 cinder_tempest_plugin.api.volume.test_volume_revert.VolumeRevertTests.test_volume_revert_to_snapshot_after_extended_negative\n   17 cinder_tempest_plugin.api.volume.test_volume_revert.VolumeRevertTests.test_volume_revert_to_snapshot\n   17 cinder_tempest_plugin.api.volume.test_create_from_image.VolumeAndVolumeTypeFromImageTest.test_create_from_image_with_volume_type_image_property\n   16 cinder_tempest_plugin.scenario.test_snapshots.SnapshotDataIntegrityTests.test_snapshot_data_integrity\n   16 cinder_tempest_plugin.api.volume.test_volume_dependency.VolumeImageDependencyTests.test_image_volume_dependencies_3\n   16 cinder_tempest_plugin.api.volume.test_volume_dependency.VolumeImageDependencyTests.test_image_volume_dependencies_2\n   16 cinder_tempest_plugin.api.volume.test_volume_dependency.VolumeImageDependencyTests.test_image_volume_dependencies_1\n   15 cinder_tempest_plugin.api.volume.test_multiple_volume_from_resource.CreateVolumesFromSnapshotTest.test_create_multiple_volume_from_snapshot\n   14 cinder_tempest_plugin.scenario.test_snapshots.SnapshotDependencyTests.test_snapshot_removal\n\nthese tests don\u0027t look particularly heavy-duty; issue may resolve itself\nonce the other tests are run separately\n   14 setUpClass(cinder_tempest_plugin.api.volume.test_volume_unicode.CinderUnicodeTest)\n\nnot currently marked slow in tempest; if failures persist after we make the\nother changes, can put up a tempest patch\n   15 tempest.scenario.test_volume_boot_pattern.TestVolumeBootPattern.test_image_defined_boot_from_volume\n\nthese are already marked for serial execution, we should run them serially:\n   15 tempest.serial_tests.scenario.test_aggregates_basic_ops.TestAggregatesBasicOps.test_aggregate_basic_ops\n   15 tempest.serial_tests.api.compute.admin.test_aggregates.AggregatesAdminTestJSON.test_aggregate_add_host_create_server_with_az\n```","commit_id":"a48cca8fa7fab4be95037f59dd34c7a2b1986ef9"}]}