Merge pull request #13182 from edx/efischer/add-logging

Add Logging for CohortMembership

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.

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: