summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAdrian C. (anrxc) <anrxc@sysphere.org>2010-06-03 21:16:34 (GMT)
committerAdrian C. (anrxc) <anrxc@sysphere.org>2010-06-03 21:58:12 (GMT)
commit70b07073fe2c03825c937ee225a9c506215e8509 (patch)
treec176dab3bd9e470959e9842cd7fb229a631a1286
parent1c6735d16bef0aa304530001280935a5e65d405f (diff)
downloadrybackup-70b07073fe2c03825c937ee225a9c506215e8509.tar.xz
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.
-rwxr-xr-xrybackup.py2
1 files changed, 1 insertions, 1 deletions
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")