Closed anti-social closed 2 years ago
I've also merged github-actions support, so please rebase on top of that, so we see tests here.
@tailhook Could you review it
.owner(CopyOwnerPolicy::Set(..
is a bit verbose, but since used only single time looks fine.
Also Preserve
should be removed instead of commented out.
The failing test is the flaky one, right?
Other than that it looks good to me.
Also updated Lock
. It is now possible to clean up lock file.
Merged. Thanks!
Okay. So all looks good so far.
I think we should proceed with copying files for now and maybe try to optimize (i.e. have a mountpoint that crosses both cache-dir and storage-dir) later. With the amount of file operations we have to make a container it should not make much difference.