fixup! Make it easier to enable (rather than copying the default)

edx · Feb 13, 2025 · edeeaea · edeeaea
1 parent 9d81e2a
commit edeeaea
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/docs/codejail.rst b/docs/codejail.rst
@@ -19,7 +19,7 @@ In order to run the codejail devstack component:
 
 1. Install AppArmor: ``sudo apt install apparmor``
 2. Add the codejail AppArmor profile to your OS, or update it: ``sudo apparmor_parser --replace -W codejail.profile``
-3. Configure LMS and CMS to use the codejail-service by changing ``ENABLE_CODEJAIL_REST_SERVICE`` to ``True`` in ``py_configuration_files/{lms,cms}.py``
+3. Configure LMS and CMS to use the codejail-service by uncommenting ``# ENABLE_CODEJAIL_REST_SERVICE = True`` in ``py_configuration_files/{lms,cms}.py``
 4. Run ``make codejail-up``
 
 The service does not need any provisioning, and does not have dependencies.

diff --git a/py_configuration_files/cms.py b/py_configuration_files/cms.py
@@ -316,7 +316,8 @@ def should_show_debug_toolbar(request):  # lint-amnesty, pylint: disable=missing
 
 # Disabled by default since codejail service needs to be configured
 # and started separately. See docs/codejail.rst for details.
-ENABLE_CODEJAIL_REST_SERVICE = False
+
+# ENABLE_CODEJAIL_REST_SERVICE = True
 CODE_JAIL_REST_SERVICE_HOST = "http://edx.devstack.codejail:8080"
 
 

diff --git a/py_configuration_files/lms.py b/py_configuration_files/lms.py
@@ -558,7 +558,8 @@ def should_show_debug_toolbar(request):  # lint-amnesty, pylint: disable=missing
 
 # Disabled by default since codejail service needs to be configured
 # and started separately. See docs/codejail.rst for details.
-ENABLE_CODEJAIL_REST_SERVICE = False
+
+# ENABLE_CODEJAIL_REST_SERVICE = True
 CODE_JAIL_REST_SERVICE_HOST = "http://edx.devstack.codejail:8080"
 
 ################# New settings must go ABOVE this line #################