summaryrefslogtreecommitdiffstats
path: root/extras
diff options
context:
space:
mode:
authorNiels de Vos <ndevos@redhat.com>2018-06-12 09:49:09 +0200
committerAmar Tumballi <amarts@redhat.com>2018-06-18 03:51:47 +0000
commit26b52694feb04c98e6c9436bcd4e23e1687f0237 (patch)
tree76e74631b2ae99b3312e0eb4a44b3b3f9c9d456d /extras
parent6ac82e8306dba6639bb5d4ea5193d5a262ade589 (diff)
gcron: catch OSError as well as IOError
In case os.open() fails because the file does not exist, an OSError is raised. To prevent the script to abort uncleanly, catch the OSError in addition to the IOError. Change-Id: I48e5b23e17d63639cc33db51b4229249a9887880 Fixes: bz#1590193 Signed-off-by: Niels de Vos <ndevos@redhat.com>
Diffstat (limited to 'extras')
-rwxr-xr-xextras/snap_scheduler/gcron.py4
1 files changed, 2 insertions, 2 deletions
diff --git a/extras/snap_scheduler/gcron.py b/extras/snap_scheduler/gcron.py
index 8324231a82c..b4c1146759b 100755
--- a/extras/snap_scheduler/gcron.py
+++ b/extras/snap_scheduler/gcron.py
@@ -105,10 +105,10 @@ def doJob(name, lockFile, jobFunc, volname):
else:
log.info("Job %s has been processed already", name)
fcntl.flock(f, fcntl.LOCK_UN)
- except IOError as (errno, strerror):
+ except (OSError, IOError) as (errno, strerror):
log.info("Job %s is being processed by another agent", name)
os.close(f)
- except IOError as (errno, strerror):
+ except (OSError, IOError) as (errno, strerror):
log.debug("Failed to open lock file %s : %s", lockFile, strerror)
log.error("Failed to process job %s", name)
success = False