-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
exp run: cannot clean up temp directory runs on Linux + NFS #7458
Comments
This is the same underlying problem as #5641. I looked into this some more and the existing workaround probably won't work for us now that we use a lot more pygit functionality. scmrepo will need some updates to make the behavior safer when we are using mixed git backends. |
Thanks for the response. Can you think of any temporary workaround for me to use? |
Returning back to a previous version might work, but #5641 was solved long time ago, not recommend to roll back to such an old version |
Is this bug caused by a change in a dependency? Or by DVC development? Do you have an idea of which change caused the bug?
Would not using mixed git backends help? |
There's not really a quick workaround for this issue right now, it's due to a lot of DVC changes that have been added in the last year since #5641 was addressed.
The short answer is "yes", but the long answer is "it's more complicated than that", as we need different functionality from each of the backends we use (since each backend does not support the same featureset). Since this is a regression, this issue is something that we will prioritize and hopefully will have a fix for it within the next couple of weeks. |
Ok gotcha. Is there anything I can do to help? |
I haven't tried any others, but it looks like at least |
|
@gregstarr can you please test the linked PR and see if it resolves your issue? To test it you will need to set up a new virtualenv and then install DVC + the PR version of scmrepo via pip:
|
Yes I believe I can get to this today. |
@pmrowla Is there a specific version of dvc and scmrepo I should be using? I notice that dvc main requires scmrepo 0.0.14. When I install scmrepo with the command you sent, it installs scmrepo 0.0.13 and pip complains because my version of dvc (2.9.5) requires 0.0.7. I'm pretty sure it didn't mess up the installation of scmrepo but I just wanted to check. |
pip list:
Yes I think this worked:
|
Your version of DVC has a pinned scmrepo version of 0.0.7, but as you noted pip will still allow you to force installation of the more recent scmrepo manually.
Thanks, this fix should be available soon then |
Hello,
I am having this issue still. When I check the pack directories, one is empty and one has files in it:
Below is the debug output:
Originally posted by @gregstarr in #5641 (comment)
The text was updated successfully, but these errors were encountered: