Skip to content

Commit

Permalink
Merge pull request #880
Browse files Browse the repository at this point in the history
* stv/fix/oom-origin:
  Add missing import
  Handle out-of-memory exception on Sysadmin Courses
  • Loading branch information
stvstnfrd committed Mar 5, 2019
2 parents aacd832 + c20f3a7 commit 5d5080c
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions lms/djangoapps/dashboard/sysadmin.py
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@
from django.views.generic.base import TemplateView
from opaque_keys.edx.locations import SlashSeparatedCourseKey
from path import Path as path
from six import text_type
import sys

import dashboard.git_import as git_import
Expand Down Expand Up @@ -403,6 +404,8 @@ def git_info_for_course(self, cdir):
info = [output_json['commit'],
output_json['date'],
output_json['author'], ]
except OSError as error:
log.warning(text_type(u"Error fetching git data: %s - %s"), text_type(cdir), text_type(error))
except (ValueError, subprocess.CalledProcessError):
pass

Expand Down

0 comments on commit 5d5080c

Please sign in to comment.