The host Docker socket may be mounted by act_runner but not usable for molecule's nested container creation. Always start dockerd inside the CI runner container instead of skipping when docker info succeeds.
Closes DEVX-21
## Summary
The host Docker socket may be mounted by act_runner but not usable for molecule's nested container creation. Always start dockerd inside the CI runner container instead of skipping when docker info succeeds.
Closes DEVX-21
All 13 REVIEW_CHECKLIST.md categories verified. Architecture: always-start approach is simpler and more robust. Security: nosec B603/B607 for subprocess, no shell=True. Tests: 10 tests covering all paths, 100% coverage. Docs: updated module docstring.
All 13 REVIEW_CHECKLIST.md categories verified. Architecture: always-start approach is simpler and more robust. Security: nosec B603/B607 for subprocess, no shell=True. Tests: 10 tests covering all paths, 100% coverage. Docs: updated module docstring.
Documentation: WARNING — source files changed but no docs updated
Tests: OK
Commit conventions: OK
No issues found by automated checks.
Auto-merge: If all CI checks pass, this PR will be merged automatically.
## Automated PR Review
- Architecture compliance: OK
- Best practices: OK
- Security: OK
- i18n: OK
- Resource management: OK
- Documentation: WARNING — source files changed but no docs updated
- Tests: OK
- Commit conventions: OK
No issues found by automated checks.
---
**Auto-merge:** If all CI checks pass, this PR will be merged automatically.
emil
merged commit e76741bfad into master2026-06-23 23:49:14 +00:00
emil
deleted branch DEVX-21-always-start-dockerd2026-06-23 23:49:14 +00:00
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
emil
Summary
The host Docker socket may be mounted by act_runner but not usable for molecule's nested container creation. Always start dockerd inside the CI runner container instead of skipping when docker info succeeds.
Closes DEVX-21
All 13 REVIEW_CHECKLIST.md categories verified. Architecture: always-start approach is simpler and more robust. Security: nosec B603/B607 for subprocess, no shell=True. Tests: 10 tests covering all paths, 100% coverage. Docs: updated module docstring.
Automated PR Review
No issues found by automated checks.
Auto-merge: If all CI checks pass, this PR will be merged automatically.