diff options
author | 2012-02-16 14:13:26 -0600 | |
---|---|---|
committer | 2012-02-26 10:44:41 +0100 | |
commit | bb59e078091c7284cbb852c50606754ffaefafef (patch) | |
tree | 41e41436a8b125be02c01c31e6e4154fd66b91cc | |
parent | add option to close inherited fds (diff) | |
download | lxc-bb59e078091c7284cbb852c50606754ffaefafef.tar.gz lxc-bb59e078091c7284cbb852c50606754ffaefafef.tar.bz2 lxc-bb59e078091c7284cbb852c50606754ffaefafef.zip |
lxc-ubuntu: fix obscure arguments
1. --path is meant to be passed by lxc-create, but should not be passed
in by users. Don't advertise it in --help.
2. --clean syntax ends up not making much sense. Get rid of it, and
add '--flush-cache' option instead.
Signed-off-by: Serge Hallyn <serge@hallyn.com>
Signed-off-by: Daniel Lezcano <dlezcano@fr.ibm.com>
-rw-r--r-- | templates/lxc-ubuntu.in | 51 |
1 files changed, 13 insertions, 38 deletions
diff --git a/templates/lxc-ubuntu.in b/templates/lxc-ubuntu.in index ffeb871..26db686 100644 --- a/templates/lxc-ubuntu.in +++ b/templates/lxc-ubuntu.in @@ -227,6 +227,7 @@ install_ubuntu() { rootfs=$1 release=$2 + flushcache=$3 cache="/var/cache/lxc/$release" mkdir -p /var/lock/subsys/ ( @@ -237,6 +238,12 @@ install_ubuntu() fi + if [ $flushcache -eq 1 ]; then + echo "Flushing cache..." + rm -rf "$cache/partial-$arch" + rm -rf "$cache/rootfs-$arch" + fi + echo "Checking cache download in $cache/rootfs-$arch ... " if [ ! -e "$cache/rootfs-$arch" ]; then download_ubuntu $cache $arch $release @@ -491,34 +498,11 @@ do_bindhome() echo "$h $rootfs/$h none bind 0 0" >> $path/fstab } -clean() -{ - release=$1 - cache="/var/cache/lxc/$release" - - if [ ! -e $cache ]; then - exit 0 - fi - - # lock, so we won't purge while someone is creating a repository - ( - flock -n -x 200 - if [ $? != 0 ]; then - echo "Cache repository is busy." - exit 1 - fi - - echo -n "Purging the download cache..." - rm --preserve-root --one-file-system -rf $cache && echo "Done." || exit 1 - exit 0 - - ) 200>/var/lock/subsys/lxc -} - usage() { cat <<EOF -$1 -h|--help -p|--path=<path> --clean [-a|--arch] [-b|--bindhome <user>] [--trim] [-r|--release] +$1 -h|--help [-a|--arch] [-b|--bindhome <user>] [--trim] + [-F | --flush-cache] [-r|--release <release>] release: lucid | maverick | natty | oneiric | precise trim: make a minimal (faster, but not upgrade-safe) container bindhome: bind <user>'s home into the container @@ -527,7 +511,7 @@ EOF return 0 } -options=$(getopt -o a:b:hp:r:xn:c -l arch:,bindhome:,help,path:,release:,trim,name:,clean -- "$@") +options=$(getopt -o a:b:hp:r:xn:F -l arch:,bindhome:,help,path:,release:,trim,name:,flush-cache -- "$@") if [ $? -ne 0 ]; then usage $(basename $0) exit 1 @@ -565,13 +549,14 @@ fi trim_container=0 hostarch=$arch +flushcache=0 while true do case "$1" in -h|--help) usage $0 && exit 0;; -p|--path) path=$2; shift 2;; -n|--name) name=$2; shift 2;; - -c|--clean) clean=$2; shift 2;; + -F|--flush-cache) flushcache=1; shift 1;; -r|--release) release=$2; shift 2;; -b|--bindhome) bindhome=$2; shift 2;; -a|--arch) arch=$2; shift 2;; @@ -592,11 +577,6 @@ if [ "$arch" == "i686" ]; then arch=i386 fi -if [ ! -z "$clean" -a -z "$path" ]; then - clean || exit 1 - exit 0 -fi - if [ $hostarch = "i386" -a $arch = "amd64" ]; then echo "can't create amd64 container on i386" exit 1 @@ -620,7 +600,7 @@ fi rootfs=$path/rootfs -install_ubuntu $rootfs $release +install_ubuntu $rootfs $release $flushcache if [ $? -ne 0 ]; then echo "failed to install ubuntu $release" exit 1 @@ -643,11 +623,6 @@ if [ ! -z $bindhome ]; then do_bindhome $rootfs $bindhome fi -if [ ! -z $clean ]; then - clean $release || exit 1 - exit 0 -fi - echo "" echo "##" echo "# The default user is 'ubuntu' with password 'ubuntu'!" |