[orion-server] feat: add buck2 log segment function , add tasks interface and so on...#1345
Conversation
Signed-off-by: Xiaoyang Han <lux1an@qq.com>
Signed-off-by: Xiaoyang Han <lux1an@qq.com>
Signed-off-by: Xiaoyang Han <lux1an@qq.com>
Signed-off-by: Xiaoyang Han <lux1an@qq.com>
Signed-off-by: Xiaoyang Han <lux1an@qq.com>
Signed-off-by: Xiaoyang Han <lux1an@qq.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Signed-off-by: Lux1an <lux1an@qq.com>
Signed-off-by: Xiaoyang Han <lux1an@qq.com>
There was a problem hiding this comment.
Pull Request Overview
This PR adds new functionality to the orion-server including a buck2 log segment function and tasks interface improvements. The changes focus on enhancing database initialization, API endpoints, and dependency configurations.
- Updates dependencies to include TLS support for tokio-tungstenite and reqwest
- Modifies the tasks API endpoint to filter by MR number instead of returning all tasks
- Adds database initialization through Docker Compose with schema creation
Reviewed Changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| orion/Cargo.toml | Adds TLS features to networking dependencies |
| orion-server/src/server.rs | Updates OpenAPI documentation for new task interface |
| orion-server/src/api.rs | Modifies tasks endpoint to filter by MR parameter |
| orion-server/docker-compose.yml | Adds database initialization service with schema setup |
| orion-server/db/schema.sql | Creates database schema with builds table and indexes |
| ceres/src/api_service/mono_api_service.rs | Minor formatting and import organization changes |
Comments suppressed due to low confidence (1)
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| @@ -812,6 +812,9 @@ impl TaskInfoDTO { | |||
| #[utoipa::path( | |||
| get, | |||
| path = "/tasks", | |||
There was a problem hiding this comment.
The OpenAPI path documentation shows "/tasks" but the actual route is "/tasks/{mr}". The path should be updated to "/tasks/{mr}" to match the implementation.
| path = "/tasks", | |
| path = "/tasks/{mr}", |
| use super::*; | ||
| use crate::model::mr::MrDiffFile; | ||
|
|
||
| use crate::model::mr::{MrDiffFile, MrPageInfo}; |
There was a problem hiding this comment.
The import adds MrPageInfo but it doesn't appear to be used in the visible test code. If it's not used elsewhere in the test module, this import should be removed.
| use crate::model::mr::{MrDiffFile, MrPageInfo}; | |
| use crate::model::mr::MrDiffFile; |
Signed-off-by: Han Xiaoyang <lux1an@qq.com>
No description provided.