)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"61cfd5ebad583014daf0d9c2a88e59e92f91cc06","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6a369166_13aa8765","updated":"2022-08-19 08:41:32.000000000","message":"But the waiter is used: have you checked the reason why that retry was added?\nhttps://review.opendev.org/c/openstack/tempest/+/807909","commit_id":"a3bd8a9f0555144b9313051ba38583085796ad9c"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"c966bac5a584006c45bc26f1c8d434487f6df101","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1ebe5524_5e3a79b7","updated":"2022-08-26 18:38:26.000000000","message":"This was added recently for Ceph RADOSGW use case - https://review.opendev.org/c/openstack/tempest/+/807909\n\nIs it causing any issue for non-radosgw env? I think we can keep it to solve the Ceph RADOSGW case.","commit_id":"a3bd8a9f0555144b9313051ba38583085796ad9c"},{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"2544c7d4e40dd4fc56a6d3fe402e342d1f17ce2b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2df6f02b_3d5eb31f","in_reply_to":"1ebe5524_5e3a79b7","updated":"2022-08-26 18:54:17.000000000","message":"Ack, \nPlease note that the old code was without any waiter for object create , IMO waiter is the right approach . if the waiter is not enough we can increase the time.","commit_id":"a3bd8a9f0555144b9313051ba38583085796ad9c"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"3ae27950b031648f308cb6141e044cf310b82220","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f0a722a6_c5b70150","in_reply_to":"2df6f02b_3d5eb31f","updated":"2022-08-26 18:56:57.000000000","message":"ok but we need to test it for Ceph RADOSGW if waiter only work for them or not.","commit_id":"a3bd8a9f0555144b9313051ba38583085796ad9c"},{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"e206563f57db213c8866456a16161f06b174763d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"729a0536_acafec8b","in_reply_to":"6a369166_13aa8765","updated":"2022-08-19 08:54:18.000000000","message":"Hi Luigi, \nsee:\nhttps://review.opendev.org/c/openstack/tempest/+/850311/comments/6553fcad_b7833fff\n\nI suspect that code without waiter was jumping to the next line/ action befoe trasaction completed. a waiter may delay the transaction and prevent running action before resource allocated properly .\n\nNot sure running re-try is a good approach , we may hide bugs inside.\n\nBenny","commit_id":"a3bd8a9f0555144b9313051ba38583085796ad9c"}]}