Merge pull request #20996 from edx/usama/learner-6648-certificate-generation-after-id-verification

Added logs in Certificate Generation after Id Verification
This commit is contained in:
Usama Sadiq
2019-07-11 12:49:23 +05:00
committed by GitHub
4 changed files with 22 additions and 8 deletions

View File

@@ -188,6 +188,10 @@ def generate_user_certificates(student, course_key, course=None, insecure=False,
generate_pdf=generate_pdf,
forced_grade=forced_grade
)
message = u'Queued Certificate Generation task for {user} : {course}'
log.info(message.format(user=student.id, course=course_key))
# If cert_status is not present in certificate valid_statuses (for example unverified) then
# add_cert returns None and raises AttributeError while accesing cert attributes.
if cert is None:

View File

@@ -146,6 +146,9 @@ def fire_ungenerated_certificate_task(user, course_key, expected_verification_st
traffic to workers.
"""
message = u'Entered into Ungenerated Certificate task for {user} : {course}'
log.info(message.format(user=user.id, course=course_key))
allowed_enrollment_modes_list = [
CourseMode.VERIFIED,
CourseMode.CREDIT_MODE,
@@ -169,3 +172,6 @@ def fire_ungenerated_certificate_task(user, course_key, expected_verification_st
kwargs['expected_verification_status'] = six.text_type(expected_verification_status)
generate_certificate.apply_async(countdown=CERTIFICATE_DELAY_SECONDS, kwargs=kwargs)
return True
message = u'Certificate Generation task failed for {user} : {course}'
log.info(message.format(user=user.id, course=course_key))

View File

@@ -38,13 +38,14 @@ def generate_certificate(self, **kwargs):
actual_verification_status = IDVerificationService.user_status(student)
actual_verification_status = actual_verification_status['status']
if expected_verification_status != actual_verification_status:
logger.warn(u'Expected verification status {expected} '
u'differs from actual verification status {actual} '
u'for user {user} in course {course}'.format(
expected=expected_verification_status,
actual=actual_verification_status,
user=student.id,
course=course_key
))
logger.warning(
u'Expected verification status {expected} '
u'differs from actual verification status {actual} '
u'for user {user} in course {course}'.format(
expected=expected_verification_status,
actual=actual_verification_status,
user=student.id,
course=course_key
))
raise self.retry(kwargs=original_kwargs)
generate_user_certificates(student=student, course_key=course_key, **kwargs)

View File

@@ -420,6 +420,9 @@ class PhotoVerification(IDVerificationAttempt):
user=self.user
)
message = u'LEARNER_NOW_VERIFIED signal fired for {user}'
log.info(message.format(user=self.user.username))
@status_before_must_be("must_retry", "submitted", "approved", "denied")
def deny(self,
error_msg,