diff options
author | Adrian C. (anrxc) <anrxc@sysphere.org> | 2010-06-03 23:16:34 +0200 |
---|---|---|
committer | Adrian C. (anrxc) <anrxc@sysphere.org> | 2010-06-03 23:58:12 +0200 |
commit | 70b07073fe2c03825c937ee225a9c506215e8509 (patch) | |
tree | c176dab3bd9e470959e9842cd7fb229a631a1286 | |
parent | 1c6735d16bef0aa304530001280935a5e65d405f (diff) | |
download | rybackup-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-x | rybackup.py | 2 |
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") |