Skip to content

[scorpio]:Fix the incorrect placement of the MR layer#1259

Merged
genedna merged 3 commits into
gitmono-dev:mainfrom
Ivanbeethoven:main
Jul 25, 2025
Merged

[scorpio]:Fix the incorrect placement of the MR layer#1259
genedna merged 3 commits into
gitmono-dev:mainfrom
Ivanbeethoven:main

Conversation

@Ivanbeethoven

Copy link
Copy Markdown
Collaborator

No description provided.

Signed-off-by: Han Xiaoyang <lux1an@qq.com>
Copilot AI review requested due to automatic review settings July 25, 2025 09:24
@vercel

vercel Bot commented Jul 25, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
mega ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 25, 2025 9:27am

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR fixes the incorrect placement of the MR (merge request) layer handling in the mount handler function. The fix ensures that the MR layer is built after fetching the dictionary node information and before the overlay mount operation.

  • Moves the MR layer building logic to occur after fetching the directory node info
  • Fixes path construction by using a reference to store_path instead of moving it
  • Improves error handling for the overlay mount operation

Comment thread scorpio/src/daemon/mod.rs Outdated
Comment thread scorpio/src/daemon/mod.rs Outdated
Ivanbeethoven and others added 2 commits July 25, 2025 17:25
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Lux1an <lux1an@qq.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Lux1an <lux1an@qq.com>
@genedna
genedna added this pull request to the merge queue Jul 25, 2025
Merged via the queue into gitmono-dev:main with commit 4f3fc16 Jul 25, 2025
33 of 36 checks passed
@Ivanbeethoven

Copy link
Copy Markdown
Collaborator Author

#1182

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.

3 participants