Skip to content

Fix file already exist error while running substitute in Docker - #2885

Merged
k311093 merged 1 commit into
google:mainfrom
k311093:subst
Jul 31, 2026
Merged

Fix file already exist error while running substitute in Docker#2885
k311093 merged 1 commit into
google:mainfrom
k311093:subst

Conversation

@k311093

@k311093 k311093 commented Jul 22, 2026

Copy link
Copy Markdown
Collaborator

In the OverlayFS, even if unlink() is succeeded, looks like there is a delay between unlink call and the file is actually disappears.

Change to use rename call which ensures atomic operation for substitution.

@k311093
k311093 force-pushed the subst branch 2 times, most recently from 45c7d89 to f2f06f2 Compare July 29, 2026 04:22
@k311093 k311093 added the kokoro:run Run e2e tests. label Jul 29, 2026
@GoogleCuttlefishTesterBot GoogleCuttlefishTesterBot removed the kokoro:run Run e2e tests. label Jul 29, 2026
@k311093
k311093 marked this pull request as ready for review July 29, 2026 06:07
@k311093
k311093 requested review from Databean and ikicha July 29, 2026 06:07
Comment thread base/cvd/cuttlefish/host/commands/cvd/fetch/substitute.cc Outdated
In the OverlayFS, even if unlink() is succeeded, looks like there is
a delay between unlink call and the file is actually disappears.

Change to use rename call which ensures atomic operation for
substitution.
@k311093 k311093 added the kokoro:run Run e2e tests. label Jul 30, 2026
@GoogleCuttlefishTesterBot GoogleCuttlefishTesterBot removed the kokoro:run Run e2e tests. label Jul 30, 2026
@k311093
k311093 added this pull request to the merge queue Jul 31, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 31, 2026
@k311093
k311093 added this pull request to the merge queue Jul 31, 2026
Merged via the queue into google:main with commit 600c989 Jul 31, 2026
46 checks passed
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