fix orion build issues#2151
Conversation
When a CL under /project/<crate> resolves repo_path to /, keep cl_path and prefix change paths so Orion can discover impacted targets.
Keep unsigned CLA visible in the merge box as a reminder, but do not gate Add to Queue or ensure_cl_mergeable. Mark path_check_configs.cla_sign as not required so existing environments pick this up on migrate.
Constrain the /oc log panel so live SSE output can scroll, and add an admin View logs action that streams scheduler logs by the client's domain.
Add Pipeline scripts for mono-engine, mega-ui, web-sync-server, and orion-server that checkout GitHub and push short-SHA/latest tags to Harbor.
Merge failed with "CL ref not found" when the mega_cl row and refs/cl/* got out of sync (empty pack, web-edit updates, or orphaned link). Ensure the canonical CL ref exists (adopting matching orphans), keep push/edit paths writing save_or_update_cl_ref, and cover the heal path with unit tests.
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 267e72ef21
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| }; | ||
|
|
||
| let change_paths: Vec<&str> = changes.iter().map(|c| c.get().as_str()).collect(); | ||
| let inferred_cl_path = crate::antares::infer_cl_path_from_changes(&repo, &change_paths); |
There was a problem hiding this comment.
For monorepo-root builds, this derives the CL overlay prefix from the build change paths, but those paths can all live under a nested directory. For example a CL under /project/dagrs-derive that only changes src/lib.rs produces the build path project/dagrs-derive/src/lib.rs, so infer_cl_path_from_changes returns project/dagrs-derive/src; when /api/v1/cl/{link}/files-list later returns the CL-relative src/lib.rs, the overlay is written to project/dagrs-derive/src/src/lib.rs and the build runs without the actual edit.
Useful? React with 👍 / 👎.
2 participants
No description provided.