docs: Update quickstart for multi-repo support [doc-updater]#551
Conversation
Update troubleshooting guidance to reflect that the orchestrator now supports parent directories containing multiple repos, not just single git repositories. This was changed in commit 7409c68 which implemented per-repo volume mounts and added _resolve_pipeline() and _collect_all_pipelines() functions. Authored-by: egg
There was a problem hiding this comment.
Review: APPROVED
This documentation change accurately reflects the multi-repo support added in commit 7409c68.
Verification
I confirmed the documentation claim against the implementation:
-
_resolve_pipeline()(orchestrator/routes/pipelines.py:86-120): Whenbase_pathlacks a.gitdirectory, the function iterates over subdirectories to find git repos and load pipelines from each. -
_collect_all_pipelines()(orchestrator/routes/pipelines.py:123-150): Similarly scans repo subdirectories when the base path is not itself a git repo. -
get_repo_path()(orchestrator/routes/__init__.py:13-51): Resolves parent directories to specific repo subdirectories using therepofield from requests.
The updated troubleshooting text correctly describes this behavior.
Summary
- ✅ Documentation matches implementation
- ✅ Change is minimal and focused
- ✅ No issues found
— Authored by egg
This comment has been minimized.
This comment has been minimized.
| egg is addressing review feedback... |
|
egg feedback addressed. View run logs 1 previous review(s) hidden. |
Update documentation to reflect changes from #548 (commit 7409c68).
The orchestrator now supports parent directories containing multiple repositories, automatically scanning subdirectories for pipelines. Updated the troubleshooting section to clarify that
EGG_REPO_PATHcan be either a single git repository or a parent directory.Changes:
Context:
Commit 7409c68 removed
EGG_REPOS_DIRand implemented per-repo volume mounts with_resolve_pipeline()and_collect_all_pipelines()functions that handle parent directories containing multiple repos.Triggered by: #548
Authored-by: egg