fix: avoid retriggering the CONFIG_LOADED action multiple times (#186)

The get_init_tasks function is called a few times inside the Jinja2
templates. The function used internal Tutor modules to load the user
configuration, this caused the `CONFIG_LOADED` action to trigger hooks
multiple times.

Instead of manually load the Tutor configuration using Tutor's internal
functions we now rely on the very same `CONFIG_LOADED` action to
register a hook that will store the value of the configuration in a
global variable.

Author: MoisesGSalas

CHANGELOG.md

@@ -13,6 +13,16 @@ See the fragment files in the [changelog.d/ directory](./changelog.d).
 
 <!-- scriv-insert-here -->
 
+<a id='changelog-21.2.1'></a>
+## 21.2.1 — 2026-05-28
+
+### Fixed
+
+- Properly mirror the LMS caddy block in the default catch-all block used by `DRYDOCK_ENABLE_MULTITENANCY`.
+
+- Avoid retriggering the CONFIG_LOADED action multiple times when redering the
+  jobs templates.
+
 <a id='changelog-21.2.0'></a>
 ## 21.2.0 — 2026-04-24
 

changelog.d/20260525_165246_deimer.morales_refactor_extra_lms_rules.md

@@ -5,7 +5,6 @@
 from glob import glob
 
 import click
-from tutor import config as tutor_config
 from tutor import env as tutor_env
 from tutor import hooks as tutor_hooks
 from tutor import serialize, types
@@ -17,6 +16,8 @@
 
 INIT_JOBS_SYNC_WAVE = 1
 
+TUTOR_CONFIG: types.Config = {}
+
 tutor_hooks.Filters.CONFIG_DEFAULTS.add_items(
     [
         ("DRYDOCK_VERSION", __version__),
@@ -59,6 +60,12 @@
 )
 
 
+@tutor_hooks.Actions.CONFIG_LOADED.add()
+def _capture_config(config: types.Config) -> None:
+    global TUTOR_CONFIG
+    TUTOR_CONFIG = config
+
+
 # This function is taken from
 # https://github.com/overhangio/tutor/blob/v16.1.8/tutor/commands/k8s.py#L182
 def _load_jobs(tutor_conf: types.Config) -> t.Iterable[t.Any]:
@@ -73,21 +80,21 @@ def _load_jobs(tutor_conf: types.Config) -> t.Iterable[t.Any]:
 # and https://github.com/overhangio/tutor/blob/v16.1.8/tutor/commands/k8s.py#L82
 def get_init_tasks():
     """Return the list of init tasks to run."""
+    if not TUTOR_CONFIG:
+        return []
     init_tasks = list(tutor_hooks.Filters.CLI_DO_INIT_TASKS.iterate())
-    context = click.get_current_context().obj
-    tutor_conf = tutor_config.load(context.root)
-    jobs = tutor_conf.get("DRYDOCK_INIT_JOBS_EXCLUDED", [])
+    jobs = TUTOR_CONFIG.get("DRYDOCK_INIT_JOBS_EXCLUDED", [])
     if not isinstance(jobs, list):
         click.secho("'DRYDOCK_INIT_JOBS_EXCLUDED' must be a list. Ignoring.", fg="yellow")
         jobs = []
     excluded_init_jobs = set(jobs)
 
     for i, (service, command) in enumerate(init_tasks):
-        for template in _load_jobs(tutor_conf):
+        for template in _load_jobs(TUTOR_CONFIG):
             if template["metadata"]["name"] != service + "-job" or template["metadata"]["name"] in excluded_init_jobs:
                 continue
 
-            render_command = tutor_env.render_str(tutor_conf, command)
+            render_command = tutor_env.render_str(TUTOR_CONFIG, command)
 
             template["metadata"]["name"] = "drydock-" + template["metadata"]["name"] + "-" + str(i)
             template["metadata"]["labels"].update(

drydock/plugin.py

@@ -1,6 +1,6 @@
 [project]
 name = "tutor-contrib-drydock"
-version = "21.2.0"
+version = "21.2.1"
 description = "A Tutor plugin to manage our opinionated Open edX operations"
 readme = "README.md"
 requires-python = ">=3.10"