Skip to content

Commit 8677762

Browse files
committed
test(robot): add retry mechanism for flaky longhorn rest api snapshotList
Signed-off-by: Yang Chiu <yang.chiu@suse.com>
1 parent f49c1d4 commit 8677762

1 file changed

Lines changed: 8 additions & 2 deletions

File tree

e2e/libs/snapshot/rest.py

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ def create(self, volume_name, snapshot_id, waiting):
2424

2525
snapshot_created = False
2626
for i in range(self.retry_count):
27-
snapshots = volume.snapshotList().data
27+
snapshots = self.list(volume_name)
2828
for vs in snapshots:
2929
if vs.name == snap_name:
3030
snapshot_created = True
@@ -88,7 +88,13 @@ def wait_for_snapshot_to_be_deleted(self, volume_name, snapshot_name):
8888
assert False, f"Failed to wait for volume {volume_name} snapshot {snapshot_name} to be deleted"
8989

9090
def list(self, volume_name):
91-
return self.volume.get(volume_name).snapshotList().data
91+
for i in range(self.retry_count):
92+
try:
93+
return self.volume.get(volume_name).snapshotList().data
94+
except Exception as e:
95+
logging(f"Failed to list volume {volume_name} snapshots: {e}")
96+
time.sleep(self.retry_interval)
97+
assert False, f"Failed to list volume {volume_name} snapshots"
9298

9399
def delete(self, volume_name, snapshot_id):
94100
logging(f"Deleting volume {volume_name} snapshot {snapshot_id}")

0 commit comments

Comments
 (0)