From 70b07073fe2c03825c937ee225a9c506215e8509 Mon Sep 17 00:00:00 2001 From: "Adrian C. (anrxc)" Date: Thu, 3 Jun 2010 23:16:34 +0200 Subject: rybackup: clarify unmounting reasons Backup destination should only be writeable during backup, rybackup unmounts (or it could at least remount as ro). Autofs is nice, works good with NFS, but backup export should not be handled by it - so there is no option to skip mount() all together. At least not in the script as it is distributed. --- rybackup.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/rybackup.py b/rybackup.py index 4e734be..a5ddf91 100755 --- a/rybackup.py +++ b/rybackup.py @@ -142,7 +142,7 @@ def mount(action): rmt['dev'], bak['dst']), shell=True) != 0: raise SystemExit("Error: could not mount destination") elif action == "umount": - # Unmount the snapshot mountpoint + # Unmount the snapshot mountpoint (should never be writeable) if call("%s %s" % (bin[action], bak['dst']), shell=True) != 0: raise SystemExit("Error: could not unmount destination") -- cgit v1.2.3