##// END OF EJS Templates
we must rollback if repo2db mapper cleanup fails ! Session blows up, and that code still throws an error without it
marcink -
r2635:d6fa7805 beta
parent child Browse files
Show More
@@ -479,7 +479,7 b' def repo2db_mapper(initial_repo_list, re'
479 # remove from database those repositories that are not in the filesystem
479 # remove from database those repositories that are not in the filesystem
480 for repo in sa.query(Repository).all():
480 for repo in sa.query(Repository).all():
481 if repo.repo_name not in initial_repo_list.keys():
481 if repo.repo_name not in initial_repo_list.keys():
482 log.debug("Removing non existing repository found in db %s" %
482 log.debug("Removing non existing repository found in db `%s`" %
483 repo.repo_name)
483 repo.repo_name)
484 try:
484 try:
485 sa.delete(repo)
485 sa.delete(repo)
@@ -488,6 +488,7 b' def repo2db_mapper(initial_repo_list, re'
488 except:
488 except:
489 #don't hold further removals on error
489 #don't hold further removals on error
490 log.error(traceback.format_exc())
490 log.error(traceback.format_exc())
491 sa.rollback()
491
492
492 # clear cache keys
493 # clear cache keys
493 log.debug("Clearing cache keys now...")
494 log.debug("Clearing cache keys now...")
General Comments 0
You need to be logged in to leave comments. Login now