Skip to content

mirror: clean up stage when retrying#43519

Merged
alalazo merged 1 commit intospack:developfrom
michaelkuhn:mirror-retry-fix
Oct 23, 2025
Merged

mirror: clean up stage when retrying#43519
alalazo merged 1 commit intospack:developfrom
michaelkuhn:mirror-retry-fix

Conversation

@michaelkuhn
Copy link
Copy Markdown
Member

If we do not clean up the stage, retries might use a corrupted stage:

  1. Cloning a Git repository succeeds
  2. Disk space runs out while copying the repository to the stage
  3. Since the stage exists, a retry will simply reuse it without noticing that it is missing files due to the previous error

@spackbot-app spackbot-app bot added the core PR affects Spack core functionality label Apr 5, 2024
@eugeneswalker
Copy link
Copy Markdown
Contributor

Thanks for doing this.

If we do not clean up the stage, retries might use a corrupted stage:
1. Cloning a Git repository succeeds
2. Disk space runs out while copying the repository to the stage
3. Since the stage exists, a retry will simply reuse it without noticing
   that it is missing files due to the previous error

Signed-off-by: Michael Kuhn <[email protected]>
@michaelkuhn
Copy link
Copy Markdown
Member Author

@alalazo ping?

@alalazo alalazo merged commit 61b87b1 into spack:develop Oct 23, 2025
32 of 33 checks passed
kshea21 pushed a commit to kshea21/spack that referenced this pull request Nov 4, 2025
If we do not clean up the stage, retries might use a corrupted stage:
1. Cloning a Git repository succeeds
2. Disk space runs out while copying the repository to the stage
3. Since the stage exists, a retry will simply reuse it without noticing
   that it is missing files due to the previous error

Signed-off-by: Michael Kuhn <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core PR affects Spack core functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants