Andal.LND/edx-platform
4
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #19240 from edx/aj/program_cache_logging

Browse Source 
Updated logging to include stack trace
This commit is contained in:
Albert (AJ) St. Aubin
2018-11-07 09:07:26 -05:00
committed by GitHub
parent 95aa21870d caf89550f2
commit f5ef546652
1 changed files with 6 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
openedx/core/djangoapps/catalog/management/commands/cache_programs.py
View File

@@ -39,7 +39,7 @@ class Command(BaseCommand):
try: try:
user = User.objects.get(username=username) user = User.objects.get(username=username)
except User.DoesNotExist: except User.DoesNotExist:
logger.error( logger.exception(
'Failed to create API client. Service user {username} does not exist.'.format(username=username) 'Failed to create API client. Service user {username} does not exist.'.format(username=username)
) )
raise raise
@@ -108,7 +108,7 @@ class Command(BaseCommand):
logger.info('Requesting program UUIDs for {domain}.'.format(domain=site.domain)) logger.info('Requesting program UUIDs for {domain}.'.format(domain=site.domain))
uuids = client.programs.get(**querystring) uuids = client.programs.get(**querystring)
except: # pylint: disable=bare-except except: # pylint: disable=bare-except
logger.error('Failed to retrieve program UUIDs for site: {domain}.'.format(domain=site.domain)) logger.exception('Failed to retrieve program UUIDs for site: {domain}.'.format(domain=site.domain))
failure = True failure = True
logger.info('Received {total} UUIDs for site {domain}'.format( logger.info('Received {total} UUIDs for site {domain}'.format(
@@ -149,7 +149,9 @@ class Command(BaseCommand):
next_page = next_page + 1 if new_pathways['next'] else None next_page = next_page + 1 if new_pathways['next'] else None
except: # pylint: disable=bare-except except: # pylint: disable=bare-except
logger.error('Failed to retrieve pathways for site: {domain}.'.format(domain=site.domain)) logger.exception(
msg='Failed to retrieve pathways for site: {domain}.'.format(domain=site.domain),
)
failure = True failure = True
logger.info('Received {total} pathways for site {domain}'.format( logger.info('Received {total} pathways for site {domain}'.format(
@@ -183,6 +185,6 @@ class Command(BaseCommand):
del pathway['programs'] del pathway['programs']
pathway['program_uuids'] = uuids pathway['program_uuids'] = uuids
except: # pylint: disable=bare-except except: # pylint: disable=bare-except
logger.error('Failed to process pathways for {domain}'.format(domain=site.domain)) logger.exception('Failed to process pathways for {domain}'.format(domain=site.domain))
failure = True failure = True
return processed_pathways, programs, failure return processed_pathways, programs, failure