Skip to content

Commit

Permalink
bug 790022 failure is always an option
Browse files Browse the repository at this point in the history
  • Loading branch information
rhelmer committed Sep 10, 2012
1 parent 4d9e18d commit 214919b
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion socorro/cron/dailyMatviews.py
Expand Up @@ -38,6 +38,8 @@ def update(config, targetDate):
)

failed = set()
# returning false is ok in some cases, see bug 790022
failureOk = ('update_adu')
databaseDSN = ""
if 'databaseHost' in config:
databaseDSN += 'host=%(databaseHost)s '
Expand All @@ -63,7 +65,7 @@ def update(config, targetDate):
cursor.callproc(funcname, parameters)
# fetchone() returns a tuple of length 1
result = cursor.fetchone()
if result and result[0]:
if result and result[0] or funcname in failureOk:
success = True
else:
# "expected" error
Expand Down

0 comments on commit 214919b

Please sign in to comment.