GNOME Bugzilla – Bug 709115
cleaning up tmpdir on transaction completion breaks concurrency
Last modified: 2013-10-02 22:18:26 UTC
gnome-continuous has multiple processes; the test processes just use ostree_repo_transaction_set_refspec(), but this will end up cleaning out the tmpdir, deleting any temporary files used by the commit process from the build. We need a more intelligent way to clean up the tmpdir. Cleaning out files older than a day is a traditional way to do it, but is still theoretically racy. I think we need to have some way for multiple processes to dynamically synchronize. Sqlite for example uses a mmapable file.
Created attachment 256259 [details] [review] repo: Only delete temp files older than a day This is somewhat lame, but to do better we need a reliable multiprocess synchronization mechanism.
Attachment 256259 [details] pushed as f9379b0 - repo: Only delete temp files older than a day