41c3377b96
--- ci--- project: acdl phase: 67b milestone: v1.12 status: execute --- /ci--- The modules-lifecycle pipeline now defaults to plan-only (fast, no AWS mutation, no credentials, no cost) so it runs on every PR. A CI variable ACDL_LIFECYCLE_MODE (workflow_dispatch input 'lifecycle_mode', default 'plan') overrides to 'full' for the real apply->modify->destroy against live AWS. Scripts: run_lifecycle_test.sh / run_lifecycle_destroy.sh / run_l2_lifecycle_test.sh / run_l2_lifecycle_destroy.sh read the flag and dispatch to --plan-only (plan mode) or --apply/--destroy (full mode). Destroy is a no-op exit 0 in plan mode (nothing was applied). VPC-output injection is gated on full mode. Workflows: both .github + .gitea (byte-identical) expose lifecycle_mode as a workflow_dispatch input (choice: plan/full), pass it via env: ACDL_LIFECYCLE_MODE to every lifecycle step, skip ci-vpc-apply + ci-vpc-destroy + Read-CI-VPC-outputs in plan mode, and run the lifecycle + l2-lifecycle jobs with if: always() so they execute (plan-only) even when ci-vpc-apply is skipped. Contract + schema: pipelines/modules-lifecycle.yml gains default_mode: plan; the schema accepts default_mode (enum plan|full) and a richer workflow_dispatch inputs shape. Tests: 14 new tests in test_lifecycle_mode_flag.py (script dispatch) + 10 new tests in TestModulesLifecyclePipeline (workflow flag wiring, byte-identity, plan-mode skips). Updated test_platform_vpc_destroy to reflect the plan-mode skip. 516 tests pass; smoke-tested plan mode on the s3 module (--plan-only green, no AWS apply).
104 lines
4.9 KiB
Python
104 lines
4.9 KiB
Python
"""Tests for the lifecycle test scripts' plan-only/full mode flag (REQ-134).
|
|
|
|
The lifecycle scripts (run_lifecycle_test.sh, run_lifecycle_destroy.sh,
|
|
run_l2_lifecycle_test.sh, run_l2_lifecycle_destroy.sh) wrap run_platform.sh.
|
|
REQ-134 (v1.12) adds the ACDL_LIFECYCLE_MODE env var: default "plan" runs
|
|
`run_platform.sh --plan-only` (fast, no AWS mutation); "full" runs the real
|
|
`--apply`/`--destroy` against live AWS.
|
|
|
|
These tests verify the dispatch logic offline by inspecting script content
|
|
(running the scripts end-to-end requires AWS credentials in full mode).
|
|
"""
|
|
import re
|
|
|
|
import pytest
|
|
|
|
ROOT = __import__("pathlib").Path(__file__).resolve().parent.parent
|
|
|
|
SCRIPTS = [
|
|
"scripts/run_lifecycle_test.sh",
|
|
"scripts/run_lifecycle_destroy.sh",
|
|
"scripts/run_l2_lifecycle_test.sh",
|
|
"scripts/run_l2_lifecycle_destroy.sh",
|
|
]
|
|
|
|
|
|
def _read(path):
|
|
return (ROOT / path).read_text()
|
|
|
|
|
|
class TestLifecycleModeFlag:
|
|
"""REQ-134: the lifecycle scripts dispatch to plan-only by default."""
|
|
|
|
@pytest.mark.parametrize("script", SCRIPTS)
|
|
def test_script_reads_acdl_lifecycle_mode(self, script):
|
|
"""Every lifecycle script reads ACDL_LIFECYCLE_MODE with a 'plan' default."""
|
|
src = _read(script)
|
|
# The default must be 'plan' (the speed-up default). The scripts use
|
|
# an intermediate LIFECYCLE_MODE var sourced from ACDL_LIFECYCLE_MODE.
|
|
assert "${ACDL_LIFECYCLE_MODE:-plan}" in src, \
|
|
f"{script} must read ACDL_LIFECYCLE_MODE defaulting to 'plan'"
|
|
assert "LIFECYCLE_MODE=" in src, \
|
|
f"{script} must assign LIFECYCLE_MODE from the env var"
|
|
|
|
@pytest.mark.parametrize("script", [
|
|
"scripts/run_lifecycle_test.sh",
|
|
"scripts/run_l2_lifecycle_test.sh",
|
|
])
|
|
def test_apply_scripts_dispatch_to_plan_only_by_default(self, script):
|
|
"""Apply/modify scripts run --plan-only when mode != full."""
|
|
src = _read(script)
|
|
assert "--plan-only" in src, f"{script} must support --plan-only (plan mode)"
|
|
assert "--apply" in src, f"{script} must support --apply (full mode)"
|
|
# The dispatch must branch on LIFECYCLE_MODE
|
|
assert "LIFECYCLE_MODE" in src and "full" in src, \
|
|
f"{script} must branch on LIFECYCLE_MODE == 'full'"
|
|
|
|
@pytest.mark.parametrize("script", [
|
|
"scripts/run_lifecycle_destroy.sh",
|
|
"scripts/run_l2_lifecycle_destroy.sh",
|
|
])
|
|
def test_destroy_scripts_noop_in_plan_mode(self, script):
|
|
"""Destroy scripts are a no-op (exit 0) in plan mode — nothing was applied."""
|
|
src = _read(script)
|
|
# The destroy script must check the mode and exit 0 in plan mode.
|
|
assert "nothing to destroy" in src, \
|
|
f"{script} must no-op (with a message) in plan mode"
|
|
assert "--destroy" in src, f"{script} must still support --destroy in full mode"
|
|
assert "LIFECYCLE_MODE" in src, f"{script} must read LIFECYCLE_MODE"
|
|
|
|
def test_apply_script_plan_mode_uses_plan_only(self):
|
|
"""In plan mode, run_lifecycle_test.sh dispatches to --plan-only."""
|
|
src = _read("scripts/run_lifecycle_test.sh")
|
|
# The if/else: full -> --apply, else -> --plan-only
|
|
assert re.search(r'if.*LIFECYCLE_MODE.*=.*"full".*;.*then', src)
|
|
assert re.search(r'else\s+bash scripts/run_platform\.sh --plan-only', src, re.DOTALL)
|
|
|
|
def test_apply_script_full_mode_uses_apply(self):
|
|
"""In full mode, run_lifecycle_test.sh dispatches to --apply."""
|
|
src = _read("scripts/run_lifecycle_test.sh")
|
|
assert re.search(r'LIFECYCLE_MODE.*=.*"full".*bash scripts/run_platform\.sh --apply', src, re.DOTALL)
|
|
|
|
def test_destroy_script_plan_mode_exits_zero(self):
|
|
"""In plan mode, the L1 destroy script exits 0 without calling run_platform."""
|
|
src = _read("scripts/run_lifecycle_destroy.sh")
|
|
assert re.search(r'LIFECYCLE_MODE.*!=.*"full".*exiting 0', src, re.DOTALL)
|
|
|
|
def test_l2_apply_script_dispatches_correctly(self):
|
|
"""The L2 apply script dispatches to plan-only by default, apply in full."""
|
|
src = _read("scripts/run_l2_lifecycle_test.sh")
|
|
assert "--plan-only" in src and "--apply" in src
|
|
assert re.search(r'if.*LIFECYCLE_MODE.*=.*"full"', src)
|
|
|
|
def test_l2_destroy_script_noop_in_plan_mode(self):
|
|
"""The L2 destroy script is a no-op in plan mode."""
|
|
src = _read("scripts/run_l2_lifecycle_destroy.sh")
|
|
assert "nothing to destroy" in src
|
|
assert re.search(r'LIFECYCLE_MODE.*!=.*"full".*exiting 0', src, re.DOTALL)
|
|
|
|
def test_vpc_injection_gated_on_full_mode(self):
|
|
"""VPC output injection is gated on full mode (plan mode skips it)."""
|
|
src = _read("scripts/run_lifecycle_test.sh")
|
|
# The VPC injection block must be gated on LIFECYCLE_MODE == full.
|
|
assert re.search(r'LIFECYCLE_MODE.*=.*"full".*&&.*echo.*VPC_DEPENDENT', src, re.DOTALL) or \
|
|
re.search(r'\[ "\$LIFECYCLE_MODE" = "full" \] && echo.*VPC_DEPENDENT', src, re.DOTALL) |