[lxc-devel] [PATCH 3/4] templates: Consistent use of locking

[Stéphane Graber]

Move to per-template lock (except for oracle that's per-container).
Also ensure that the path used for the lock is relative to LOCALSTATEDIR.

Signed-off-by: Stéphane Graber <stgraber at ubuntu.com>
---
 templates/lxc-altlinux.in     | 4 ++--
 templates/lxc-debian.in       | 4 ++--
 templates/lxc-fedora.in       | 4 ++--
 templates/lxc-lenny.in        | 4 ++--
 templates/lxc-opensuse.in     | 4 ++--
 templates/lxc-oracle.in       | 4 ++--
 templates/lxc-ubuntu-cloud.in | 2 +-
 templates/lxc-ubuntu.in       | 4 ++--
 8 files changed, 15 insertions(+), 15 deletions(-)

diff --git a/templates/lxc-altlinux.in b/templates/lxc-altlinux.in
index eab473c..fac545c 100644
--- a/templates/lxc-altlinux.in
+++ b/templates/lxc-altlinux.in
@@ -228,7 +228,7 @@ install_altlinux()
             return 1
         fi
         return 0
-    ) 200>@LOCALSTATEDIR@/lock/subsys/lxc
+    ) 200>@LOCALSTATEDIR@/lock/subsys/lxc-altlinux
 
     return $?
 }
@@ -325,7 +325,7 @@ clean()
         echo -n "Purging the download cache for ALTLinux-$release..."
         rm --preserve-root --one-file-system -rf $cache && echo "Done." || exit 1
         exit 0
-    ) 200>@LOCALSTATEDIR@/lock/subsys/lxc
+    ) 200>@LOCALSTATEDIR@/lock/subsys/lxc-altlinux
 }
 
 usage()
diff --git a/templates/lxc-debian.in b/templates/lxc-debian.in
index 6b5f2f9..7bbc46b 100644
--- a/templates/lxc-debian.in
+++ b/templates/lxc-debian.in
@@ -191,7 +191,7 @@ install_debian()
 
         return 0
 
-        ) 200>@LOCALSTATEDIR@/lock/subsys/lxc
+        ) 200>@LOCALSTATEDIR@/lock/subsys/lxc-debian
 
     return $?
 }
@@ -261,7 +261,7 @@ clean()
         rm --preserve-root --one-file-system -rf $cache && echo "Done." || exit 1
         exit 0
 
-    ) 200>@LOCALSTATEDIR@/lock/subsys/lxc
+    ) 200>@LOCALSTATEDIR@/lock/subsys/lxc-debian
 }
 
 usage()
diff --git a/templates/lxc-fedora.in b/templates/lxc-fedora.in
index bc4b264..366e776 100644
--- a/templates/lxc-fedora.in
+++ b/templates/lxc-fedora.in
@@ -233,7 +233,7 @@ install_fedora()
         fi
 
         return 0
-    ) 200>@LOCALSTATEDIR@/lock/subsys/lxc
+    ) 200>@LOCALSTATEDIR@/lock/subsys/lxc-fedora
 
     return $?
 }
@@ -301,7 +301,7 @@ clean()
         echo -n "Purging the download cache for Fedora-$release..."
         rm --preserve-root --one-file-system -rf $cache && echo "Done." || exit 1
         exit 0
-    ) 200>@LOCALSTATEDIR@/lock/subsys/lxc
+    ) 200>@LOCALSTATEDIR@/lock/subsys/lxc-fedora
 }
 
 usage()
diff --git a/templates/lxc-lenny.in b/templates/lxc-lenny.in
index afc9321..cb93853 100644
--- a/templates/lxc-lenny.in
+++ b/templates/lxc-lenny.in
@@ -166,7 +166,7 @@ install_debian()
             return 1
         fi
         return 0
-    ) 200>@LOCALSTATEDIR@/lock/subsys/lxc
+    ) 200>@LOCALSTATEDIR@/lock/subsys/lxc-lenny
 
     return $?
 }
@@ -234,7 +234,7 @@ clean()
         echo -n "Purging the download cache..."
         rm --preserve-root --one-file-system -rf $cache && echo "Done." || exit 1
         exit 0
-    ) 200>@LOCALSTATEDIR@/lock/subsys/lxc
+    ) 200>@LOCALSTATEDIR@/lock/subsys/lxc-lenny
 }
 
 usage()
diff --git a/templates/lxc-opensuse.in b/templates/lxc-opensuse.in
index e0f8e00..65fb7b0 100644
--- a/templates/lxc-opensuse.in
+++ b/templates/lxc-opensuse.in
@@ -242,7 +242,7 @@ install_opensuse()
         fi
 
         return 0
-    ) 200>@LOCALSTATEDIR@/lock/subsys/lxc
+    ) 200>@LOCALSTATEDIR@/lock/subsys/lxc-opensuse
 
     return $?
 }
@@ -314,7 +314,7 @@ clean()
         echo -n "Purging the download cache..."
         rm --preserve-root --one-file-system -rf $cache && echo "Done." || exit 1
         exit 0
-    ) 200>@LOCALSTATEDIR@/lock/subsys/lxc
+    ) 200>@LOCALSTATEDIR@/lock/subsys/lxc-opensuse
 }
 
 usage()
diff --git a/templates/lxc-oracle.in b/templates/lxc-oracle.in
index 90b2bad..74880bc 100644
--- a/templates/lxc-oracle.in
+++ b/templates/lxc-oracle.in
@@ -337,7 +337,7 @@ container_rootfs_create()
         fi
     done
 
-    mkdir -p /var/lock/subsys/
+    mkdir -p @LOCALSTATEDIR@/lock/subsys/lxc
     (
         flock -x 200
         if [ $? -ne 0 ]; then
@@ -433,7 +433,7 @@ container_rootfs_create()
         echo "Rebuilding rpm database"
         rm -f $container_rootfs/var/lib/rpm/__db*
         chroot $container_rootfs rpm --rebuilddb >/dev/null 2>&1
-    ) 200>/var/lock/subsys/lxc-oracle-$name
+    ) 200>@LOCALSTATEDIR@/lock/subsys/lxc-oracle-$name
 }
 
 usage()
diff --git a/templates/lxc-ubuntu-cloud.in b/templates/lxc-ubuntu-cloud.in
index 4ef5f8d..de9f3c3 100644
--- a/templates/lxc-ubuntu-cloud.in
+++ b/templates/lxc-ubuntu-cloud.in
@@ -395,7 +395,7 @@ EOF
         echo "If you do not have a meta-data service, this container will likely be useless."
 
     fi
-) 200>@LOCALSTATEDIR@/lock/subsys/lxc-ubucloud
+) 200>@LOCALSTATEDIR@/lock/subsys/lxc-ubuntu-cloud
 
 copy_configuration $path $rootfs $name $arch $release
 
diff --git a/templates/lxc-ubuntu.in b/templates/lxc-ubuntu.in
index 19de991..0ed8808 100644
--- a/templates/lxc-ubuntu.in
+++ b/templates/lxc-ubuntu.in
@@ -259,7 +259,7 @@ install_ubuntu()
     release=$2
     flushcache=$3
     cache="@LOCALSTATEDIR@/cache/lxc/$release"
-    mkdir -p /var/lock/subsys/
+    mkdir -p @LOCALSTATEDIR@/lock/subsys/
 
     (
         flock -x 200
@@ -293,7 +293,7 @@ install_ubuntu()
 
         return 0
 
-    ) 200>/var/lock/subsys/lxc
+    ) 200>@LOCALSTATEDIR@/lock/subsys/lxc-ubuntu
 
     return $?
 }
-- 
1.8.0