From 3ba09221c1aa59c0500b0d996c2740ea850f5cb4 Mon Sep 17 00:00:00 2001 From: Eric Fischer Date: Fri, 5 Aug 2016 16:59:57 -0400 Subject: [PATCH] Add Logging for CohortMembership --- lms/djangoapps/verified_track_content/models.py | 2 ++ openedx/core/djangoapps/course_groups/models.py | 2 ++ 2 files changed, 4 insertions(+) diff --git a/lms/djangoapps/verified_track_content/models.py b/lms/djangoapps/verified_track_content/models.py index 5bf3b2271d..4b5b4f6134 100644 --- a/lms/djangoapps/verified_track_content/models.py +++ b/lms/djangoapps/verified_track_content/models.py @@ -49,6 +49,8 @@ def move_to_verified_cohort(sender, instance, **kwargs): # pylint: disable=unus 'verified_cohort_name': verified_cohort_name, 'default_cohort_name': random_cohort.name } + log.info("Queuing automatic cohorting for user '%s' in course '%s'", instance.user.username, course_key) + # Do the update with a 3-second delay in hopes that the CourseEnrollment transaction has been # completed before the celery task runs. We want a reasonably short delay in case the learner # immediately goes to the courseware. diff --git a/openedx/core/djangoapps/course_groups/models.py b/openedx/core/djangoapps/course_groups/models.py index bb0103d026..14ce7a7fe8 100644 --- a/openedx/core/djangoapps/course_groups/models.py +++ b/openedx/core/djangoapps/course_groups/models.py @@ -90,6 +90,8 @@ class CohortMembership(models.Model): def save(self, *args, **kwargs): self.full_clean(validate_unique=False) + log.info("Saving CohortMembership for '%s' (id=%s) in '%s'", self.user.username, self.user.id, self.course_id) + # Avoid infinite recursion if creating from get_or_create() call below. # This block also allows middleware to use CohortMembership.get_or_create without worrying about outer_atomic if 'force_insert' in kwargs and kwargs['force_insert'] is True: