-
Notifications
You must be signed in to change notification settings - Fork 5.2k
Fix AppHost Bundle Tests I/O Failures #120673
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
@dotnet-policy-service agree |
1 similar comment
@dotnet-policy-service agree |
Aside from the implementation problems, I don't think this addresses the problem. The log seems to suggest we have some problem in extraction, but this is just about deleting assets after the test is done running. |
Got it! I thought maybe this would fix it. |
Summary
This PR addresses intermittent I/O failures in AppHost.Bundle.Tests.BundledAppWithSubDirs and updates the package release notes URL to point to the correct GitHub location.
Tried fixing #120653
Problem
- Temporary file system race conditions
- Directory deletion failures during cleanup
- File creation failures with long path names
Solution
Changes
- Modified DeleteExtractionDirectory() to retry failed directory deletions
- Modified AddLongNameContent() to retry failed directory/file creation operations
- Both methods now handle IOException gracefully with retry logic
Testing