Skip to content

Bootstrap runner registration host directory#1233

Merged
cbusillo merged 1 commit into
mainfrom
fix/runner-registration-host-bootstrap
Jun 8, 2026
Merged

Bootstrap runner registration host directory#1233
cbusillo merged 1 commit into
mainfrom
fix/runner-registration-host-bootstrap

Conversation

@cbusillo

@cbusillo cbusillo commented Jun 8, 2026

Copy link
Copy Markdown
Owner

Summary

  • Defaults the manual runner registration workflow to a service-user-owned auto root ($HOME/actions-runners) instead of root-owned /opt/actions-runners.
  • Bootstraps the official GitHub Actions runner package when config.sh is missing in a new lane directory.
  • Starts run.sh after registration so post-registration inventory can observe the lane online.

Live Evidence

The first live apply run proved the control-plane path but failed at host prep:

  • Run: https://github.com/cbusillo/launchplane/actions/runs/27157039099
  • Plan status: ready
  • GitHub registration token fetched and redacted in audit output
  • Terminal audit status: failed
  • Failure: service user could not create /opt/actions-runners/cm-website-chris-testing
  • Post-inventory: still zero runners in cbusillo/odoo-tenant-cm-website

This PR fixes that root-owned directory assumption and the follow-on missing runner package assumption.

Verification

  • uv run python -m unittest tests.test_runner_lane_registration
  • uv run --extra dev ruff format --check control_plane/workflows/runner_lane_registration_executor.py tests/test_runner_lane_registration.py
  • uv run --extra dev ruff check control_plane/workflows/runner_lane_registration_executor.py tests/test_runner_lane_registration.py
  • uv run --extra dev mypy control_plane/workflows/runner_lane_registration_executor.py tests/test_runner_lane_registration.py
  • git diff --check

@cbusillo cbusillo merged commit a2553a7 into main Jun 8, 2026
12 checks passed
@cbusillo cbusillo deleted the fix/runner-registration-host-bootstrap branch June 8, 2026 18:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant