Commit 065300ba by Renzo Lucioni

Extend program certification task logging to facilitate monitoring during backpopulation

Log a message at the info level when the task is called for a user with no completed programs, and another when the task is called for a user who is not eligible for any new program certificates. Part of ECOM-3924.
parent 0a61ab75
...@@ -171,6 +171,7 @@ def award_program_certificates(self, username): ...@@ -171,6 +171,7 @@ def award_program_certificates(self, username):
if not program_ids: if not program_ids:
# Again, no reason to continue beyond this point unless/until this # Again, no reason to continue beyond this point unless/until this
# task gets updated to support revocation of program certs. # task gets updated to support revocation of program certs.
LOGGER.info('Task award_program_certificates was called for user %s with no completed programs', username)
return return
# Determine which program certificates the user has already been # Determine which program certificates the user has already been
...@@ -213,5 +214,7 @@ def award_program_certificates(self, username): ...@@ -213,5 +214,7 @@ def award_program_certificates(self, username):
# N.B. This logic assumes that this task is idempotent # N.B. This logic assumes that this task is idempotent
LOGGER.info('Retrying task to award failed certificates to user %s', username) LOGGER.info('Retrying task to award failed certificates to user %s', username)
raise self.retry(countdown=countdown, max_retries=config.max_retries) raise self.retry(countdown=countdown, max_retries=config.max_retries)
else:
LOGGER.info('User %s is not eligible for any new program certificates', username)
LOGGER.info('Successfully completed the task award_program_certificates for username %s', username) LOGGER.info('Successfully completed the task award_program_certificates for username %s', username)
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment