vbatts/quay
Archived
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Switch to generic RedisError

Browse source 
Fixes #1558
This commit is contained in:
Joseph Schorr 2016-06-20 11:20:17 -04:00
parent e6d214c955
commit 986d20bcad
2 changed files with 8 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

10
data/buildlogs.py
View file

@ -61,8 +61,8 @@ class RedisBuildLogs(object):
llen = self._redis.llen(self._logs_key(build_id))
log_entries = self._redis.lrange(self._logs_key(build_id), start_index, -1)
return (llen, (json.loads(entry) for entry in log_entries))
except redis.ConnectionError as ce:
raise BuildStatusRetrievalError('Cannot retrieve build logs: %s' % ce)
except redis.RedisError as re:
raise BuildStatusRetrievalError('Cannot retrieve build logs: %s' % re)
def expire_log_entries(self, build_id):
"""
@ -87,8 +87,8 @@ class RedisBuildLogs(object):
"""
try:
fetched = self._redis.get(self._status_key(build_id))
except redis.ConnectionError as ce:
raise BuildStatusRetrievalError('Cannot retrieve build status: %s' % ce)
except redis.RedisError as re:
raise BuildStatusRetrievalError('Cannot retrieve build status: %s' % re)
return json.loads(fetched) if fetched else None
@ -110,7 +110,7 @@ class RedisBuildLogs(object):
connection.get(self._health_key())
return True
except redis.ConnectionError:
except redis.RedisError:
return False