From b813f3f807019fa4e87de644674fc41051b63118 Mon Sep 17 00:00:00 2001 From: John Griffith Date: Wed, 29 Apr 2015 17:25:12 -0600 Subject: [PATCH] Add retry to lvm delete Seems we have another issue related to lvm and snapshots, but this time it's on the delete side. We used a simple retry mechanism for snapshot create here: https://review.openstack.org/#/c/149360/8 I'm hesitant to just add another retry without looking at dm issues, but this should address the problem temporarily and give us a chance to collect some data on the issue. Change-Id: I37607a09e2f9af6f2f32ddfaa8567400541e533a Closes-Bug: #1450110 --- cinder/brick/local_dev/lvm.py | 1 + 1 file changed, 1 insertion(+) diff --git a/cinder/brick/local_dev/lvm.py b/cinder/brick/local_dev/lvm.py index 9a0c26464..f4c7753c8 100644 --- a/cinder/brick/local_dev/lvm.py +++ b/cinder/brick/local_dev/lvm.py @@ -625,6 +625,7 @@ class LVM(executor.Executor): LOG.error(_LE('StdErr :%s'), err.stderr) raise + @utils.retry(putils.ProcessExecutionError) def delete(self, name): """Delete logical volume or snapshot. -- 2.45.2