Added logs while importing course

cms/djangoapps/contentstore/tasks.py

@@ -526,9 +526,11 @@ def get_dir_for_filename(directory, filename):
         return
 
     if not user_has_access(user):
+        logging.info(f'Course import {course_key_string}: User is not allowed to import')
         return
 
     if not file_is_supported():
+        logging.info(f'Course import {course_key_string}: File not supported')
         return
 
     is_library = isinstance(courselike_key, LibraryLocator)
@@ -737,14 +739,18 @@ def validate_course_olx(courselike_key, course_dir, status):
     if not course_import_olx_validation_is_enabled():
         return olx_is_valid
     try:
+        if str(courselike_key) == "course-v1:ArbiX+CS101+2014_T3":
+            logging.info(log_prefix + "Validating. Calling olxcleaner.validate")
         __, errorstore, __ = olxcleaner.validate(
             filename=course_dir,
             steps=settings.COURSE_OLX_VALIDATION_STAGE,
             ignore=settings.COURSE_OLX_VALIDATION_IGNORE_LIST,
             allowed_xblocks=ALL_ALLOWED_XBLOCKS
         )
-    except Exception:  # pylint: disable=broad-except
-        LOGGER.exception(f'{log_prefix}: CourseOlx could not be validated')
+        if str(courselike_key) == "course-v1:ArbiX+CS101+2014_T3":
+            logging.info(log_prefix + "Course validated. No errors")
+    except Exception as e:  # pylint: disable=broad-except
+        LOGGER.exception(f'{log_prefix}: CourseOlx could not be validated' + str(e))
         return olx_is_valid
 
     has_errors = errorstore.return_error(ErrorLevel.ERROR.value)

cms/djangoapps/contentstore/views/import_export.py

@@ -247,6 +247,9 @@ def import_status_handler(request, course_key_string, filename=None):
         4 : Import successful
 
     """
+    if str(courselike_key) == "course-v1:ArbiX+CS101+2014_T3":
+        log.info(f"Import Status Handler {course_key_string}: Begin} ")
+
     course_key = CourseKey.from_string(course_key_string)
     if not has_course_author_access(request.user, course_key):
         raise PermissionDenied()
@@ -259,6 +262,8 @@ def import_status_handler(request, course_key_string, filename=None):
     for status_filter in STATUS_FILTERS:
         task_status = status_filter().filter_queryset(request, task_status, import_status_handler)
     task_status = task_status.order_by('-created').first()
+    if str(courselike_key) == "course-v1:ArbiX+CS101+2014_T3":
+        log.info("Selected task status for course " + str(task_status.__dict__))
     if task_status is None:
         # The task hasn't been initialized yet; did we store info in the session already?
         try:
@@ -276,6 +281,9 @@ def import_status_handler(request, course_key_string, filename=None):
     else:
         status = min(task_status.completed_steps + 1, 3)
 
+    if str(courselike_key) == "course-v1:ArbiX+CS101+2014_T3":
+        log.info("Status for course " + str(status))
+
     return JsonResponse({"ImportStatus": status, "Message": message})
 
 

common/djangoapps/util/monitoring.py

@@ -13,9 +13,11 @@ def monitor_import_failure(course_key, import_step, message=None, exception=None
     """
     set_custom_attribute('course_import_failure', import_step)
     set_custom_attributes_for_course_key(course_key)
+    logging.info(f"Course import failed at step {import_step} for course with key {course_key}")
 
     if message:
         set_custom_attribute('course_import_failure_message', message)
+        logging.info(f"Course import failed message {message}")
 
     if exception is not None:
         exception_module = getattr(exception, '__module__', '')