From af0909e8cf3dda84ff4efe7da51284ded8a904d5 Mon Sep 17 00:00:00 2001 From: Sanford Student Date: Fri, 19 Jan 2018 09:58:20 -0500 Subject: [PATCH] reducing logging for EDUCATOR-1290 --- .../xmodule/xmodule/library_content_module.py | 35 ++++++++++--------- 1 file changed, 19 insertions(+), 16 deletions(-) diff --git a/common/lib/xmodule/xmodule/library_content_module.py b/common/lib/xmodule/xmodule/library_content_module.py index 6f458374a3..babd349a5f 100644 --- a/common/lib/xmodule/xmodule/library_content_module.py +++ b/common/lib/xmodule/xmodule/library_content_module.py @@ -154,13 +154,15 @@ class LibraryContentModule(LibraryContentFields, XModule, StudioEditableModule): 'added' (set) of newly added (block_type, block_id) tuples """ selected = set(tuple(k) for k in selected) # set of (block_type, block_id) tuples assigned to this student - log_id = uuid4() - logger.info( - "EDUCATOR-1290: LibraryContentModule.make_selection executing. selected: {0} | log ID: {1}".format( - selected, - log_id + course_is_mit_supply_chain = len(selected) != 0 and "MITx+CTL" in next(iter(selected))[1] # used for temporary logging for EDUCATOR-1290 + if course_is_mit_supply_chain: + log_id = uuid4() + logger.info( + "EDUCATOR-1290: LibraryContentModule.make_selection executing. selected: {0} | log ID: {1}".format( + selected, + log_id + ) ) - ) # Determine which of our children we will show: valid_block_keys = set([(c.block_type, c.block_id) for c in children]) @@ -188,17 +190,18 @@ class LibraryContentModule(LibraryContentFields, XModule, StudioEditableModule): else: raise NotImplementedError("Unsupported mode.") selected |= added_block_keys - logger.info( - "EDUCATOR-1290: LibraryContentModule.make_selection executed. " - "valid_block_keys: {0} | selected: {1} | invalid: {2} | overlimit: {3} | added: {4} | log ID: {5}".format( - valid_block_keys, - selected, - invalid_block_keys, - overlimit_block_keys, - added_block_keys, - log_id + if course_is_mit_supply_chain: + logger.info( + "EDUCATOR-1290: LibraryContentModule.make_selection executed. " + "valid_block_keys: {0} | selected: {1} | invalid: {2} | overlimit: {3} | added: {4} | log ID: {5}".format( + valid_block_keys, + selected, + invalid_block_keys, + overlimit_block_keys, + added_block_keys, + log_id + ) ) - ) return { 'selected': selected, 'invalid': invalid_block_keys,