Skip to content

Commit 80ad544

Browse files
diegocepedawjwon
authored andcommitted
remove email message retry
1 parent 6542dd5 commit 80ad544

2 files changed

Lines changed: 3 additions & 13 deletions

File tree

src/iris/api.py

Lines changed: 2 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -4041,18 +4041,8 @@ def create_email_message(self, application, dest, subject, body):
40414041
'body': body,
40424042
'destination': dest
40434043
}
4044-
retries = 0
4045-
while retries < 3:
4046-
try:
4047-
message_id = session.execute(sql, data).lastrowid
4048-
session.commit()
4049-
except Exception as e:
4050-
retries += 1
4051-
if retries == 3:
4052-
logger.exception("failed to create email message")
4053-
sleep(0.2)
4054-
else:
4055-
break
4044+
message_id = session.execute(sql, data).lastrowid
4045+
session.commit()
40564046

40574047
session.close()
40584048
return True, message_id

src/iris/bin/sync_targets.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -391,7 +391,7 @@ def sync_from_oncall(config, engine, purge_old_users=True):
391391
except SQLAlchemyError as e:
392392
logger.exception('Error inserting oncall_team %s: %s', t, e)
393393
continue
394-
session.commit()
394+
session.commit()
395395
session.close()
396396

397397
# mark users/teams inactive

0 commit comments

Comments
 (0)