Allow chris_superuser to be defined by environment variables CHRIS_US…

…ERNAME and CHRIS_PASSWORD

chrisomatic/spec/deserialize.py

@@ -1,5 +1,8 @@
+import os
+
 import serde
 import strictyaml
+import typer
 from rich.console import Console
 from rich.status import Status
 
@@ -30,4 +33,22 @@ def _load_from_yaml(input_config: str, filename: str, console: Console) -> Given
         console.print(
             f"[yellow]WARNING[/yellow]: on.chris_store_url in {filename} is deprecated."
         )
-    return serde.from_dict(GivenConfig, parsed_yaml.data)
+    # messy new feature: if on.chris_superuser is not found in the YAML,
+    # get the superuser credentials from environment variables instead.
+    data = parsed_yaml.data
+    if "chris_superuser" not in parsed_yaml.data["on"]:
+        username = os.getenv("CHRIS_USERNAME", None)
+        password = os.getenv("CHRIS_PASSWORD", None)
+        if username is None or password is None:
+            console.print(f"[red]on.chris_superuser is not defined[/red]")
+            typer.Abort()
+        data |= {
+            "on": data["on"]
+            | {
+                "chris_superuser": {
+                    "username": username,
+                    "password": password,
+                }
+            }
+        }
+    return serde.from_dict(GivenConfig, data)

chrisomatic/spec/schema.py

@@ -28,7 +28,7 @@
         "on": Map(
             {
                 "cube_url": api_url,
-                "chris_superuser": user,
+                Optional("chris_superuser"): user,
                 Optional(
                     "public_store", default=["https://cube.chrisproject.org/api/v1/"]
                 ): EmptyList()