Merge pull request #2469 from edx/feanil/xml_modulestore_settings

Remove the xml modulestore from the list of modulestore that cat called.

CHANGELOG.md

+- Role: edxapp
+  - Removed XmlModuleStore from the default list of modulestores for the LMS.
+  - EDXAPP_XML_MAPPINGS variable no longer exists by default and is not used by the edxapp role.
+
 - Role: ecommerce
   - Removed ECOMMERCE_ORDER_NUMBER_PREFIX variable
 

playbooks/roles/edxapp/defaults/main.yml

@@ -219,7 +219,6 @@ EDXAPP_LOGGING_ENV:  'sandbox'
 
 EDXAPP_SYSLOG_SERVER: ""
 EDXAPP_RABBIT_HOSTNAME: 'localhost'
-EDXAPP_XML_MAPPINGS: {}
 
 EDXAPP_LMS_NGINX_PORT: 18000
 EDXAPP_LMS_SSL_NGINX_PORT: 48000
@@ -873,7 +872,7 @@ lms_auth_config:
     default:
         ENGINE: 'xmodule.modulestore.mixed.MixedModuleStore'
         OPTIONS:
-          mappings: "{{ EDXAPP_XML_MAPPINGS }}"
+          mappings: {}
           stores:
             - NAME: 'split'
               ENGINE: 'xmodule.modulestore.split_mongo.split_draft.DraftVersioningModuleStore'
@@ -889,11 +888,6 @@ lms_auth_config:
                 default_class: 'xmodule.hidden_module.HiddenDescriptor'
                 fs_root:  "{{ edxapp_course_data_dir }}"
                 render_template: 'edxmako.shortcuts.render_to_string'
-            - NAME: 'xml'
-              ENGINE: 'xmodule.modulestore.xml.XMLModuleStore'
-              OPTIONS:
-                data_dir: "{{ edxapp_course_data_dir }}"
-                default_class: 'xmodule.hidden_module.HiddenDescriptor'
   PROCTORING_BACKEND_PROVIDER: "{{ EDXAPP_PROCTORING_BACKEND_PROVIDER }}"
 
 
@@ -936,8 +930,6 @@ cms_auth_config:
     default:
         ENGINE: 'xmodule.modulestore.mixed.MixedModuleStore'
         OPTIONS:
-# See commented section below. LMS-11258
-#          mappings: "{{ EDXAPP_XML_MAPPINGS }}"
           mappings: {}
           stores:
             - NAME: 'split'
@@ -954,9 +946,6 @@ cms_auth_config:
                 default_class: 'xmodule.hidden_module.HiddenDescriptor'
                 fs_root:  "{{ edxapp_course_data_dir }}"
                 render_template: 'edxmako.shortcuts.render_to_string'
-
-# Commented for now so that it can be tested first: LMS-11258
-#            - *edxapp_generic_xml_modulestore
   PARSE_KEYS: "{{ EDXAPP_PARSE_KEYS }}"
 cms_env_config:
   <<: *edxapp_generic_env