summaryrefslogtreecommitdiff
path: root/rybackup.py
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 /rybackup.py
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.
Diffstat (limited to 'rybackup.py')
-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")