mirror of
https://github.com/kevinveenbirkenbach/computer-playbook.git
synced 2026-11-06 02:54:26 +00:00
kevinveenbirkenbachandClaude Opus 5
f3b478b79c
compose_networks, container_networks and compose_external_networks each rebuilt the same Ansible-side context before calling their renderer: resolve the variables, settle the deployment mode, load applications, build the service registry, and close over the config and database lookups. Around forty lines, three times, each with its own copy of _resolve_var. A change to how the registry or the closures are obtained could reach one lookup and miss another. build_context() returns that context once. honour_mode_force carries the only real difference between the callers: compose_external_networks never consulted compose_mode_force, and still does not. The lookups keep only what is theirs - term validation, their own arguments, their renderer call - and drop from 94/90/87 lines to 59/54/53. No assertion changed in the tests; only the mock targets moved with the seam, from the plugin modules to utils.networks.lookup_context. Twelve legacy comments in those test files lost their grandfathering when the files entered the lint's diff scope: two became docstrings naming the call contract, the rest were narration and are gone. Behaviour-preserving: the same 13 lookup tests pass before and after. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
60 lines
2.0 KiB
Python
60 lines
2.0 KiB
Python
"""Lookup ``compose_networks``: emit the top-level ``networks:`` block for
|
|
a compose file. Reads the service_registry, the rendering role's
|
|
``application_id`` and ``DEPLOYMENT_MODE`` from variables.
|
|
|
|
Usage in any ``compose.yml.j2``:
|
|
|
|
{{ lookup('compose_networks') }}
|
|
|
|
The output starts at column 0 (no caller-side indent).
|
|
"""
|
|
|
|
from __future__ import annotations
|
|
|
|
from typing import Any
|
|
|
|
from ansible.errors import AnsibleError
|
|
from ansible.plugins.lookup import LookupBase
|
|
|
|
from utils.networks.lookup_context import build_context, resolve_var
|
|
from utils.networks.render import render_compose_networks
|
|
from utils.roles.entity.name import get_entity_name
|
|
|
|
|
|
class LookupModule(LookupBase):
|
|
def run(
|
|
self,
|
|
terms: list[Any] | None,
|
|
variables: dict[str, Any] | None = None,
|
|
**kwargs: Any,
|
|
) -> list[str]:
|
|
if terms:
|
|
raise AnsibleError("compose_networks lookup takes no positional terms")
|
|
|
|
ctx = build_context(self, variables)
|
|
|
|
application_id = resolve_var(ctx.templar, ctx.vars.get("application_id"))
|
|
if not application_id:
|
|
raise AnsibleError(
|
|
"compose_networks lookup: application_id is required in variables"
|
|
)
|
|
|
|
swarm_cfg = resolve_var(ctx.templar, ctx.vars.get("swarm", {})) or {}
|
|
net_cfg = swarm_cfg.get("network", {}) if isinstance(swarm_cfg, dict) else {}
|
|
swarm_encrypted = True
|
|
if isinstance(net_cfg, dict) and "encryption" in net_cfg:
|
|
swarm_encrypted = bool(net_cfg.get("encryption"))
|
|
|
|
return [
|
|
render_compose_networks(
|
|
application_id=str(application_id),
|
|
deployment_mode=ctx.deployment_mode,
|
|
registry=ctx.registry,
|
|
get_entity_name=get_entity_name,
|
|
lookup_config=ctx.lookup_config,
|
|
lookup_database=ctx.lookup_database,
|
|
swarm_encrypted=swarm_encrypted,
|
|
node_local=bool(kwargs.get("node_local", False)),
|
|
)
|
|
]
|