]> git.ipfire.org Git - thirdparty/lxc.git/commitdiff
implement backend drivers and container clone API (v3)
authorSerge Hallyn <serge.hallyn@ubuntu.com>
Tue, 16 Apr 2013 13:07:05 +0000 (08:07 -0500)
committerSerge Hallyn <serge.hallyn@ubuntu.com>
Tue, 30 Apr 2013 14:12:42 +0000 (09:12 -0500)
1. commonize waitpid users to use a single helper.  We frequently want
to run something in a clean namespace, or fork off a script.  This
lets us keep the function doing fork:(1)exec(2)waitpid simpler.

2. start a blockdev backend implementation.  This will be used for
mounting, copying, and snapshotting container filesystems.

3. implement btrfs, lvm, directory, and overlayfs backends.

4. For overlayfs, support a new lxc.rootfs format of
'bdevtype:<extra>'.  This means you can now use overlayfs-based
containers without using lxc-start-ephemeral, by using
lxc.rootfs = overlayfs:/readonly-dir:writeable-dir

5. add a set of simple clone testcases

6. Write a new lxc_clone.c based on api clone.

Still to do (there's more, but off top of my head):

1. support zfs, aufs
2. have clone handle other mount entries (right now it only clones
the rootfs)
3. python, lua, and go bindings (not me :)
4. lxc-destroy: if lvm backing store, check for snapshots of it.
   (what about directories which have overlayfs clones?)

Changes since v2:
Initialize random generator when picking new macaddr (reported
  by caglar@10ur.org)
Fix wrong use of bitmask flags
On copy-clone of btrfs, create a subvolume
lxc_clone.c: respect the command line usage of the old script
lxc-clone(1): update documentation
Refuse to try changing backing stores expect to overlayfs, as
  it is not implemented (yet) anyway.

Signed-off-by: Serge Hallyn <serge.hallyn@ubuntu.com>
Conflicts:
src/lxc/utils.h

14 files changed:
configure.ac
doc/lxc-clone.sgml.in
src/lxc/Makefile.am
src/lxc/bdev.c [new file with mode: 0644]
src/lxc/bdev.h [new file with mode: 0644]
src/lxc/conf.c
src/lxc/lxc-clone-sh.in [moved from src/lxc/lxc-clone.in with 100% similarity]
src/lxc/lxc_clone.c [new file with mode: 0644]
src/lxc/lxccontainer.c
src/lxc/lxccontainer.h
src/lxc/utils.c
src/lxc/utils.h
src/tests/Makefile.am
src/tests/clonetest.c [new file with mode: 0644]

index eba19db7f3ebe718babdc7096c62e0608451d4c6..6b270387658f7afe8bb80b9b3062a58ccc4c5a64 100644 (file)
@@ -382,7 +382,7 @@ AC_CONFIG_FILES([
        src/lxc/lxc-checkconfig
        src/lxc/lxc-version
        src/lxc/lxc-create
-       src/lxc/lxc-clone
+       src/lxc/lxc-clone-sh
        src/lxc/lxc-shutdown
        src/lxc/lxc-start-ephemeral
        src/lxc/lxc-destroy
index 994f3cddb94c850ae63ed1424dbed87837620bf2..3ad7698804357723f9dde3395e600239a7c7a668 100644 (file)
@@ -50,13 +50,29 @@ Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
   <refsynopsisdiv>
     <cmdsynopsis>
       <command>lxc-clone</command>
+      <arg choice="opt">-s </arg>
+      <arg choice="opt">-K </arg>
+      <arg choice="opt">-M </arg>
+      <arg choice="opt">-H </arg>
+      <arg choice="opt">-B <replaceable>backingstore</replaceable></arg>
+      <arg choice="opt">-L <replaceable>fssize</replaceable></arg>
+      <arg choice="opt">-p <replaceable>lxcpath</replaceable></arg>
+      <arg choice="opt">-P <replaceable>newlxcpath</replaceable></arg>
       <arg choice="req">-o <replaceable>orig</replaceable></arg>
       <arg choice="req">-n <replaceable>new</replaceable></arg>
+    </cmdsynopsis>
+    <cmdsynopsis>
+      <command>lxc-clone</command>
       <arg choice="opt">-s </arg>
+      <arg choice="opt">-K </arg>
+      <arg choice="opt">-M </arg>
+      <arg choice="opt">-H </arg>
+      <arg choice="opt">-B <replaceable>backingstore</replaceable></arg>
       <arg choice="opt">-L <replaceable>fssize</replaceable></arg>
-      <arg choice="opt">-v <replaceable>vgname</replaceable></arg>
-      <arg choice="opt">-p <replaceable>lxc_lv_prefix</replaceable></arg>
-      <arg choice="opt">-t <replaceable>fstype</replaceable></arg>
+      <arg choice="opt">-p <replaceable>lxcpath</replaceable></arg>
+      <arg choice="opt">-P <replaceable>newlxcpath</replaceable></arg>
+      <arg choice="req">orig</arg>
+      <arg choice="req">new</arg>
     </cmdsynopsis>
   </refsynopsisdiv>
 
@@ -64,10 +80,29 @@ Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
     <title>Description</title>
 
     <para>
-      <command>lxc-clone</command> Creates a new container as a copy of an existing
-      container.  When the original container's rootfs is an LVM block device or
-      is on a btrfs filesystem, then a snapshotted clone can be created, taking up
-      very little initial disk space.
+      <command>lxc-clone</command> Creates a new container as a clone of an existing
+      container.  Two types of clones are supported: copy and snapshot.  A copy
+      clone copies the root filessytem from the original container to the new.  A
+      snapshot filesystem uses the backing store's snapshot functionality to create
+      a very small copy-on-write snapshot of the original container.  Snapshot
+      clones require the new container backing store to support snapshotting.  Currently
+      this includes only btrfs, lvm, overlayfs and zfs.  LVM devices do not support
+      snapshots fo snapshots.
+    </para>
+
+    <para>
+      The backing store of the new container will be the same type as the
+      original container,
+      with one exception:  overlayfs snapshots can be created of directory backed
+      containers.  This can be requested by using the <replaceable>-B overlayfs</replaceable>
+      arguments.
+    </para>
+
+    <para>
+      The names of the original and new container can be given (in that order)
+      after all options, or can be specified with the
+      <replaceable>-o</replaceable> and <replaceable>-n</replaceable> options,
+      respectively.
     </para>
 
   </refsect1>
@@ -80,33 +115,47 @@ Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
 
       <varlistentry>
        <term>
-         <option>-o, --orig <replaceable>orig</replaceable></option>
+         <option>-s, --snapshot</option>
        </term>
        <listitem>
          <para>
-           The name of the original container to clone.
+           The new container's rootfs should be a LVM or btrfs snapshot of the original.
          </para>
        </listitem>
       </varlistentry>
 
       <varlistentry>
        <term>
-         <option>-n, --new <replaceable>new</replaceable></option>
+         <option>-K, --keepname</option>
        </term>
        <listitem>
          <para>
-           The name of the new container to create.
+           Do not change the hostname of the container (in the root
+           filesystem).
          </para>
        </listitem>
       </varlistentry>
 
       <varlistentry>
        <term>
-         <option>-s, --snapshot</option>
+         <option>-M, --keepmac</option>
        </term>
        <listitem>
          <para>
-           The new container's rootfs should be a LVM or btrfs snapshot of the original.
+           Use the same MAC address as the original container, rather tahn
+           generating a new random one.
+         </para>
+       </listitem>
+      </varlistentry>
+
+      <varlistentry>
+       <term>
+         <option>-H, --copyhooks</option>
+       </term>
+       <listitem>
+         <para>
+           Copy all mount hooks into the new container's directory, and
+           update any lxcpaths and container names as needed.
          </para>
        </listitem>
       </varlistentry>
@@ -117,7 +166,7 @@ Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
        </term>
        <listitem>
          <para>
-           In the case of a LVM-backed container, a size for the new
+           In the case of a block device backed container, a size for the new
            block device.  By default, the new device will be made the
            same size as the original.
          </para>
@@ -126,43 +175,69 @@ Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
 
       <varlistentry>
        <term>
-         <option>-v, --vgname <replaceable>vgname</replaceable></option>
+         <option>-p, --lxcpath <replaceable>fssize</replaceable></option>
        </term>
        <listitem>
          <para>
-           For an LVM-backed container, the volume group name to use.  By
-           default it is 'lxc'.
+           The lxcpath of the original container.  By default, the system
+           wide configured lxcpath will be used.
          </para>
        </listitem>
       </varlistentry>
 
       <varlistentry>
        <term>
-         <option>-p, --lvprefix <replaceable>lxc_lv_prefix</replaceable></option>
+         <option>-P, --newpath <replaceable>fssize</replaceable></option>
        </term>
        <listitem>
          <para>
-           For an LVM-backed container, a string to prefix to the container name to
-           form the logical volume name.  For instance, specifying
-           <command>-n c1 -p lxc_</command> will cause the container rootfs to
-           be on a logical volume called <replaceable>lxc_c1</replaceable>.
+           The lxcpath for the new container.  By default the same lxcpath
+           as the original will be used.  Note that with btrfs snapshots,
+           changing lxcpaths may not be possible, as subvolume snapshots
+           must be in the same btrfs filesystem.
          </para>
        </listitem>
       </varlistentry>
 
       <varlistentry>
        <term>
-         <option>-t, --fstype <replaceable>fstype</replaceable></option>
+         <option>-B, --backingstore <replaceable>fssize</replaceable></option>
        </term>
        <listitem>
          <para>
-           For a non-snapshot LVM clone, the file system to use for the new
-           container.  Note this option is ignored when requesting a
-           snapshotted container.
+           Select a different backing store for the new container.  By
+           default the same as the original container's is used.  Note that
+           currently changing the backingstore is only supported for
+           overlayfs snapshots of directory backed containers.  Valid
+           backing stores include dir (directory), btrfs, lvm, zfs
+           and overlayfs.
+         </para>
+       </listitem>
+      </varlistentry>
+
+      <varlistentry>
+       <term>
+         <option>-o, --orig <replaceable>orig</replaceable></option>
+       </term>
+       <listitem>
+         <para>
+           The name of the original container to clone.
          </para>
        </listitem>
       </varlistentry>
 
+      <varlistentry>
+       <term>
+         <option>-n, --new <replaceable>new</replaceable></option>
+       </term>
+       <listitem>
+         <para>
+           The name of the new container to create.
+         </para>
+       </listitem>
+      </varlistentry>
+
+
     </variablelist>
 
   </refsect1>
index 1fa0fa88af09cd48576bb74dc80f8c45fa93cc1e..580b41d88797a5a6c7b9d5896177208780b6d5d4 100644 (file)
@@ -16,6 +16,7 @@ pkginclude_HEADERS = \
                attach.h \
                lxccontainer.h \
                lxclock.h \
+               bdev.h \
                version.h
 
 if IS_BIONIC
@@ -36,6 +37,7 @@ so_PROGRAMS = liblxc.so
 
 liblxc_so_SOURCES = \
        arguments.c arguments.h \
+       bdev.c bdev.h \
        commands.c commands.h \
        start.c start.h \
        stop.c \
@@ -122,7 +124,7 @@ bin_SCRIPTS = \
        lxc-checkconfig \
        lxc-version \
        lxc-create \
-       lxc-clone \
+       lxc-clone-sh \
        lxc-shutdown \
        lxc-destroy
 
@@ -148,6 +150,7 @@ bin_PROGRAMS = \
        lxc-unshare \
        lxc-stop \
        lxc-start \
+       lxc-clone \
        lxc-execute \
        lxc-monitor \
        lxc-monitord \
@@ -184,6 +187,7 @@ lxc_init_SOURCES = lxc_init.c
 lxc_monitor_SOURCES = lxc_monitor.c
 lxc_monitord_SOURCES = lxc_monitord.c
 lxc_restart_SOURCES = lxc_restart.c
+lxc_clone_SOURCES = lxc_clone.c
 lxc_start_SOURCES = lxc_start.c
 lxc_stop_SOURCES = lxc_stop.c
 lxc_unfreeze_SOURCES = lxc_unfreeze.c
diff --git a/src/lxc/bdev.c b/src/lxc/bdev.c
new file mode 100644 (file)
index 0000000..990e8c5
--- /dev/null
@@ -0,0 +1,1189 @@
+/*
+ * lxc: linux Container library
+ *
+ * (C) Copyright IBM Corp. 2007, 2008
+ *
+ * Authors:
+ * Daniel Lezcano <daniel.lezcano at free.fr>
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 2.1 of the License, or (at your option) any later version.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this library; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
+ */
+
+/*
+ * this is all just a first shot for experiment.  If we go this route, much
+ * shoudl change.  bdev should be a directory with per-bdev file.  Things which
+ * I'm doing by calling out to userspace should sometimes be done through
+ * libraries like liblvm2
+ */
+#define _GNU_SOURCE
+#include <stdio.h>
+#include <unistd.h>
+#include <errno.h>
+#include <sched.h>
+#include <sys/mount.h>
+#include <sys/wait.h>
+#include <libgen.h>
+#include "lxc.h"
+#include "config.h"
+#include "conf.h"
+#include "bdev.h"
+#include "log.h"
+#include "error.h"
+#include "utils.h"
+#include "namespace.h"
+#include "parse.h"
+
+lxc_log_define(bdev, lxc);
+
+/* Define unshare() if missing from the C library */
+/* this is also in attach.c and lxccontainer.c: commonize it in utils.c */
+#ifndef HAVE_UNSHARE
+static int unshare(int flags)
+{
+#ifdef __NR_unshare
+return syscall(__NR_unshare, flags);
+#else
+errno = ENOSYS;
+return -1;
+#endif
+}
+#endif
+
+static int do_rsync(const char *src, const char *dest)
+{
+       // call out to rsync
+       pid_t pid;
+       char *s;
+       size_t l;
+
+       pid = fork();
+       if (pid < 0)
+               return -1;
+       if (pid > 0)
+               return wait_for_pid(pid);
+       l = strlen(src) + 2;
+       s = malloc(l);
+       if (!s)
+               exit(1);
+       strcpy(s, src);
+       s[l-2] = '/';
+       s[l-1] = '\0';
+
+       return execlp("rsync", "rsync", "-a", s, dest, (char *)NULL);
+}
+
+static int blk_getsize(const char *path, unsigned long *size)
+{
+       int fd, ret;
+
+       fd = open(path, O_RDONLY);
+       if (!fd)
+               return -1;
+       ret = ioctl(fd, BLKGETSIZE64, size);
+       close(fd);
+       return ret;
+}
+
+/*
+ * These are copied from conf.c.  However as conf.c will be moved to using
+ * the callback system, they can be pulled from there eventually, so we
+ * don't need to pollute utils.c with these low level functions
+ */
+static int find_fstype_cb(char* buffer, void *data)
+{
+       struct cbarg {
+               const char *rootfs;
+               const char *target;
+               int mntopt;
+       } *cbarg = data;
+
+       char *fstype;
+
+       /* we don't try 'nodev' entries */
+       if (strstr(buffer, "nodev"))
+               return 0;
+
+       fstype = buffer;
+       fstype += lxc_char_left_gc(fstype, strlen(fstype));
+       fstype[lxc_char_right_gc(fstype, strlen(fstype))] = '\0';
+
+       DEBUG("trying to mount '%s'->'%s' with fstype '%s'",
+             cbarg->rootfs, cbarg->target, fstype);
+
+       if (mount(cbarg->rootfs, cbarg->target, fstype, cbarg->mntopt, NULL)) {
+               DEBUG("mount failed with error: %s", strerror(errno));
+               return 0;
+       }
+
+       INFO("mounted '%s' on '%s', with fstype '%s'",
+            cbarg->rootfs, cbarg->target, fstype);
+
+       return 1;
+}
+
+static int mount_unknow_fs(const char *rootfs, const char *target, int mntopt)
+{
+       int i;
+
+       struct cbarg {
+               const char *rootfs;
+               const char *target;
+               int mntopt;
+       } cbarg = {
+               .rootfs = rootfs,
+               .target = target,
+               .mntopt = mntopt,
+       };
+
+       /*
+        * find the filesystem type with brute force:
+        * first we check with /etc/filesystems, in case the modules
+        * are auto-loaded and fall back to the supported kernel fs
+        */
+       char *fsfile[] = {
+               "/etc/filesystems",
+               "/proc/filesystems",
+       };
+
+       for (i = 0; i < sizeof(fsfile)/sizeof(fsfile[0]); i++) {
+
+               int ret;
+
+               if (access(fsfile[i], F_OK))
+                       continue;
+
+               ret = lxc_file_for_each_line(fsfile[i], find_fstype_cb, &cbarg);
+               if (ret < 0) {
+                       ERROR("failed to parse '%s'", fsfile[i]);
+                       return -1;
+               }
+
+               if (ret)
+                       return 0;
+       }
+
+       ERROR("failed to determine fs type for '%s'", rootfs);
+       return -1;
+}
+
+static int do_mkfs(const char *path, const char *fstype)
+{
+       pid_t pid;
+
+       if ((pid = fork()) < 0) {
+               ERROR("error forking");
+               return -1;
+       }
+       if (pid > 0)
+               return wait_for_pid(pid);
+
+       return execlp("mkfs", "mkfs", "-t", fstype, path, NULL);
+}
+
+static char *linkderef(char *path, char *dest)
+{
+       struct stat sbuf;
+       ssize_t ret;
+
+       ret = stat(path, &sbuf);
+       if (ret < 0)
+               return NULL;
+       if (!S_ISLNK(sbuf.st_mode))
+               return path;
+       ret = readlink(path, dest, MAXPATHLEN);
+       if (ret < 0) {
+               SYSERROR("error reading link %s", path);
+               return NULL;
+       } else if (ret >= MAXPATHLEN) {
+               ERROR("link in %s too long", path);
+               return NULL;
+       }
+       dest[ret] = '\0';
+       return dest;
+}
+
+/*
+ * Given a bdev (presumably blockdev-based), detect the fstype
+ * by trying mounting (in a private mntns) it.
+ * @bdev: bdev to investigate
+ * @type: preallocated char* in which to write the fstype
+ * @len: length of passed in char*
+ * Returns length of fstype, of -1 on error
+ */
+static int detect_fs(struct bdev *bdev, char *type, int len)
+{
+       int  p[2], ret;
+       size_t linelen;
+       pid_t pid;
+       FILE *f;
+       char *sp1, *sp2, *sp3, *line = NULL;
+
+       if (!bdev || !bdev->src || !bdev->dest)
+               return -1;
+
+       if (pipe(p) < 0)
+               return -1;
+       if ((pid = fork()) < 0)
+               return -1;
+       if (pid > 0) {
+               int status;
+               close(p[1]);
+               memset(type, 0, len);
+               ret = read(p[0], type, len-1);
+               close(p[0]);
+               if (ret < 0) {
+                       SYSERROR("error reading from pipe");
+                       wait(&status);
+                       return -1;
+               } else if (ret == 0) {
+                       ERROR("child exited early - fstype not found");
+                       wait(&status);
+                       return -1;
+               }
+               wait(&status);
+               type[len-1] = '\0';
+               INFO("detected fstype %s for %s", type, bdev->src);
+               return ret;
+       }
+
+       if (unshare(CLONE_NEWNS) < 0)
+               exit(1);
+
+       ret = mount_unknow_fs(bdev->src, bdev->dest, 0);
+       if (ret < 0) {
+               ERROR("failed mounting %s onto %s to detect fstype", bdev->src, bdev->dest);
+               exit(1);
+       }
+       // if symlink, get the real dev name
+       char devpath[MAXPATHLEN];
+       char *l = linkderef(bdev->src, devpath);
+       if (!l)
+               exit(1);
+       f = fopen("/proc/self/mounts", "r");
+       if (!f)
+               exit(1);
+       while (getline(&line, &linelen, f) != -1) {
+               sp1 = index(line, ' ');
+               if (!sp1)
+                       exit(1);
+               *sp1 = '\0';
+               if (strcmp(line, l))
+                       continue;
+               sp2 = index(sp1+1, ' ');
+               if (!sp2)
+                       exit(1);
+               *sp2 = '\0';
+               sp3 = index(sp2+1, ' ');
+               if (!sp3)
+                       exit(1);
+               *sp3 = '\0';
+               sp2++;
+               if (write(p[1], sp2, strlen(sp2)) != strlen(sp2))
+                       exit(1);
+               exit(0);
+       }
+       exit(1);
+}
+
+struct bdev_type {
+       char *name;
+       struct bdev_ops *ops;
+};
+
+static int is_dir(const char *path)
+{
+       struct stat statbuf;
+       int ret = stat(path, &statbuf);
+       if (ret == 0 && S_ISDIR(statbuf.st_mode))
+               return 1;
+       return 0;
+}
+
+static int dir_detect(const char *path)
+{
+       if (strncmp(path, "dir:", 4) == 0)
+               return 1; // take their word for it
+       if (is_dir(path))
+               return 1;
+       return 0;
+}
+
+//
+// XXXXXXX plain directory bind mount ops
+//
+int dir_mount(struct bdev *bdev)
+{
+       if (strcmp(bdev->type, "dir"))
+               return -22;
+       if (!bdev->src || !bdev->dest)
+               return -22;
+       return mount(bdev->src, bdev->dest, "bind", MS_BIND | MS_REC, NULL);
+}
+
+int dir_umount(struct bdev *bdev)
+{
+       if (strcmp(bdev->type, "dir"))
+               return -22;
+       if (!bdev->src || !bdev->dest)
+               return -22;
+       return umount(bdev->dest);
+}
+
+/* the bulk of this needs to become a common helper */
+static char *dir_new_path(char *src, const char *oldname, const char *name,
+                       const char *oldpath, const char *lxcpath)
+{
+       char *ret, *p, *p2;
+       int l1, l2, nlen;
+
+       nlen = strlen(src) + 1;
+       l1 = strlen(oldpath);
+       p = src;
+       /* if src starts with oldpath, look for oldname only after
+        * that path */
+       if (strncmp(src, oldpath, l1) == 0) {
+               p += l1;
+               nlen += (strlen(lxcpath) - l1);
+       }
+       l2 = strlen(oldname);
+       while ((p = strstr(p, oldname)) != NULL) {
+               p += l2;
+               nlen += strlen(name) - l2;
+       }
+
+       ret = malloc(nlen);
+       if (!ret)
+               return NULL;
+
+       p = ret;
+       if (strncmp(src, oldpath, l1) == 0) {
+               p += sprintf(p, "%s", lxcpath);
+               src += l1;
+       }
+
+       while ((p2 = strstr(src, oldname)) != NULL) {
+               strncpy(p, src, p2-src); // copy text up to oldname
+               p += p2-src; // move target pointer (p)
+               p += sprintf(p, "%s", name); // print new name in place of oldname
+               src = p2 + l2;  // move src to end of oldname
+       }
+       sprintf(p, "%s", src);  // copy the rest of src
+       return ret;
+}
+
+/*
+ * for a simple directory bind mount, we substitute the old container
+ * name and paths for the new
+ */
+static int dir_clonepaths(struct bdev *orig, struct bdev *new, const char *oldname,
+               const char *cname, const char *oldpath, const char *lxcpath, int snap,
+               unsigned long newsize)
+{
+       if (snap) {
+               ERROR("directories cannot be snapshotted.  Try overlayfs.");
+               return -1;
+       }
+
+       if (strcmp(orig->type, "dir")) {
+               ERROR("Directory clone from %s backing store is not supported",
+                       orig->type);
+               return -1;
+       }
+
+       if (!orig->dest || !orig->src)
+               return -1;
+       if (orig->data) {
+               new->data = strdup(orig->data);
+               if (!new->data)
+                       return -1;
+       }
+
+       new->dest = dir_new_path(orig->dest, oldname, cname, oldpath, lxcpath);
+       if (!new->dest)
+               return -1;
+       new->src = dir_new_path(orig->src, oldname, cname, oldpath, lxcpath);
+       if (!new->src)
+               return -1;
+
+       return 0;
+}
+
+struct bdev_ops dir_ops = {
+       .detect = &dir_detect,
+       .mount = &dir_mount,
+       .umount = &dir_umount,
+       .clone_paths = &dir_clonepaths,
+};
+
+//
+// LVM ops
+//
+
+/*
+ * Look at /sys/dev/block/maj:min/dm/uuid.  If it contains the hardcoded LVM
+ * prefix "LVM-", then this is an lvm2 LV
+ */
+static int lvm_detect(const char *path)
+{
+       char devp[MAXPATHLEN], buf[4];
+       FILE *fout;
+       int ret;
+       struct stat statbuf;
+
+       if (strncmp(path, "lvm:", 4) == 0)
+               return 1; // take their word for it
+
+       ret = stat(path, &statbuf);
+       if (ret != 0)
+               return 0;
+       if (!S_ISBLK(statbuf.st_mode))
+               return 0;
+
+       ret = snprintf(devp, MAXPATHLEN, "/sys/dev/block/%d:%d/dm/uuid",
+                       major(statbuf.st_rdev), minor(statbuf.st_rdev));
+       if (ret < 0 || ret >= MAXPATHLEN) {
+               ERROR("lvm uuid pathname too long");
+               return 0;
+       }
+       fout = fopen(devp, "r");
+       if (!fout)
+               return 0;
+       ret = fread(buf, 1, 4, fout);
+       fclose(fout);
+       if (ret != 4 || strncmp(buf, "LVM-", 4) != 0)
+               return 0;
+       return 1;
+}
+
+static int lvm_mount(struct bdev *bdev)
+{
+       if (strcmp(bdev->type, "lvm"))
+               return -22;
+       if (!bdev->src || !bdev->dest)
+               return -22;
+       /* if we might pass in data sometime, then we'll have to enrich
+        * mount_unknow_fs */
+       return mount_unknow_fs(bdev->src, bdev->dest, 0);
+}
+
+static int lvm_umount(struct bdev *bdev)
+{
+       if (strcmp(bdev->type, "lvm"))
+               return -22;
+       if (!bdev->src || !bdev->dest)
+               return -22;
+       return umount(bdev->dest);
+}
+
+/*
+ * path must be '/dev/$vg/$lv', $vg must be an existing VG, and $lv must
+ * not yet exist.  This function will attempt to create /dev/$vg/$lv of
+ * size $size.
+ */
+static int lvm_create(const char *path, unsigned long size)
+{
+       int ret, pid;
+       char sz[24], *pathdup, *vg, *lv;
+
+       if ((pid = fork()) < 0) {
+               SYSERROR("failed fork");
+               return -1;
+       }
+       if (pid > 0)
+               return wait_for_pid(pid);
+
+       // lvcreate default size is in M, not bytes.
+       ret = snprintf(sz, 24, "%lu", size/1000000);
+       if (ret < 0 || ret >= 24)
+               exit(1);
+
+       pathdup = strdup(path);
+       if (!pathdup)
+               exit(1);
+       lv = rindex(pathdup, '/');
+       if (!lv) {
+               free(pathdup);
+               exit(1);
+       }
+       *lv = '\0';
+       lv++;
+       vg = rindex(pathdup, '/');
+       if (!vg)
+               exit(1);
+       vg++;
+       ret = execlp("lvcreate", "lvcreate", "-L", sz, vg, "-n", lv, (char *)NULL);
+       free(pathdup);
+       return ret;
+}
+
+static int lvm_snapshot(const char *orig, const char *path, unsigned long size)
+{
+       int ret, pid;
+       char sz[24], *pathdup, *lv;
+
+       if ((pid = fork()) < 0) {
+               SYSERROR("failed fork");
+               return -1;
+       }
+       if (pid > 0)
+               return wait_for_pid(pid);
+       // lvcreate default size is in M, not bytes.
+       ret = snprintf(sz, 24, "%lu", size/1000000);
+       if (ret < 0 || ret >= 24)
+               exit(1);
+
+       pathdup = strdup(path);
+       if (!pathdup)
+               exit(1);
+       lv = rindex(pathdup, '/');
+       if (!lv) {
+               free(pathdup);
+               exit(1);
+       }
+       *lv = '\0';
+       lv++;
+
+       ret = execlp("lvcreate", "lvcreate", "-s", "-L", sz, "-n", lv, orig, (char *)NULL);
+       free(pathdup);
+       return ret;
+}
+
+static int lvm_clonepaths(struct bdev *orig, struct bdev *new, const char *oldname,
+               const char *cname, const char *oldpath, const char *lxcpath, int snap,
+               unsigned long newsize)
+{
+       char fstype[100];
+       unsigned long size = newsize;
+
+       if (!orig->src || !orig->dest)
+               return -1;
+
+       if (strcmp(orig->type, "lvm")) {
+               ERROR("LVM clone from %s backing store is not supported",
+                       orig->type);
+               return -1;
+       }
+
+       if (orig->data) {
+               new->data = strdup(orig->data);
+               if (!new->data)
+                       return -1;
+       }
+       new->dest = dir_new_path(orig->dest, oldname, cname, oldpath, lxcpath);
+       if (!new->dest)
+               return -1;
+       if (mkdir_p(new->dest, 0755) < 0)
+               return -1;
+
+
+       new->src = dir_new_path(orig->src, oldname, cname, oldpath, lxcpath);
+       if (!new->src)
+               return -1;
+
+       if (!newsize && blk_getsize(orig->src, &size) < 0) {
+               ERROR("Error getting size of %s", orig->src);
+               return -1;
+       }
+       if (snap) {
+               if (lvm_snapshot(orig->src, new->src, size) < 0) {
+                       ERROR("could not create %s snapshot of %s", new->src, orig->src);
+                       return -1;
+               }
+       } else {
+               if (lvm_create(new->src, size) < 0) {
+                       ERROR("Error creating new lvm blockdev");
+                       return -1;
+               }
+               if (detect_fs(orig, fstype, 100) < 0) {
+                       ERROR("could not find fstype for %s", orig->src);
+                       return -1;
+               }
+               if (do_mkfs(new->src, fstype) < 0) {
+                       ERROR("Error creating filesystem type %s on %s", fstype,
+                               new->src);
+                       return -1;
+               }
+       }
+
+       return 0;
+}
+
+struct bdev_ops lvm_ops = {
+       .detect = &lvm_detect,
+       .mount = &lvm_mount,
+       .umount = &lvm_umount,
+       .clone_paths = &lvm_clonepaths,
+};
+
+//
+// btrfs ops
+//
+
+struct btrfs_ioctl_space_info {
+       unsigned long long flags;
+       unsigned long long total_bytes;
+       unsigned long long used_bytes;
+};
+
+struct btrfs_ioctl_space_args {
+       unsigned long long space_slots;
+       unsigned long long total_spaces;
+       struct btrfs_ioctl_space_info spaces[0];
+};
+
+#define BTRFS_IOCTL_MAGIC 0x94
+#define BTRFS_IOC_SUBVOL_GETFLAGS _IOR(BTRFS_IOCTL_MAGIC, 25, unsigned long long)
+#define BTRFS_IOC_SPACE_INFO _IOWR(BTRFS_IOCTL_MAGIC, 20, \
+                                    struct btrfs_ioctl_space_args)
+
+static int btrfs_detect(const char *path)
+{
+       struct stat st;
+       int fd, ret;
+       struct btrfs_ioctl_space_args sargs;
+
+       // make sure this is a btrfs filesystem
+       fd = open(path, O_RDONLY);
+       if (fd < 0)
+               return 0;
+       sargs.space_slots = 0;
+       sargs.total_spaces = 0;
+       ret = ioctl(fd, BTRFS_IOC_SPACE_INFO, &sargs);
+       close(fd);
+       if (ret < 0)
+               return 0;
+
+       // and make sure it's a subvolume.
+       ret = stat(path, &st);
+       if (ret < 0)
+               return 0;
+
+       if (st.st_ino == 256 && S_ISDIR(st.st_mode))
+               return 1;
+
+       return 0;
+}
+
+int btrfs_mount(struct bdev *bdev)
+{
+       if (strcmp(bdev->type, "btrfs"))
+               return -22;
+       if (!bdev->src || !bdev->dest)
+               return -22;
+       return mount(bdev->src, bdev->dest, "bind", MS_BIND | MS_REC, NULL);
+}
+
+int btrfs_umount(struct bdev *bdev)
+{
+       if (strcmp(bdev->type, "btrfs"))
+               return -22;
+       if (!bdev->src || !bdev->dest)
+               return -22;
+       return umount(bdev->dest);
+}
+
+#define BTRFS_SUBVOL_NAME_MAX 4039
+#define BTRFS_PATH_NAME_MAX 4087
+
+struct btrfs_ioctl_vol_args {
+       signed long long fd;
+       char name[BTRFS_PATH_NAME_MAX + 1];
+};
+
+#define BTRFS_IOCTL_MAGIC 0x94
+#define BTRFS_IOC_SUBVOL_CREATE_V2 _IOW(BTRFS_IOCTL_MAGIC, 24, \
+                                   struct btrfs_ioctl_vol_args_v2)
+#define BTRFS_IOC_SNAP_CREATE_V2 _IOW(BTRFS_IOCTL_MAGIC, 23, \
+                                   struct btrfs_ioctl_vol_args_v2)
+#define BTRFS_IOC_SUBVOL_CREATE _IOW(BTRFS_IOCTL_MAGIC, 14, \
+                                   struct btrfs_ioctl_vol_args)
+
+#define BTRFS_QGROUP_INHERIT_SET_LIMITS (1ULL << 0)
+
+struct btrfs_ioctl_vol_args_v2 {
+       signed long long fd;
+       unsigned long long transid;
+       unsigned long long flags;
+       union {
+               struct {
+                       unsigned long long size;
+                       //struct btrfs_qgroup_inherit *qgroup_inherit;
+                       void *qgroup_inherit;
+               };
+               unsigned long long unused[4];
+       };
+       char name[BTRFS_SUBVOL_NAME_MAX + 1];
+};
+
+static int btrfs_subvolume_create(const char *path)
+{
+       int ret, fd = -1;
+       struct btrfs_ioctl_vol_args  args;
+       char *p, *newfull = strdup(path);
+
+       if (!newfull) {
+               ERROR("Error: out of memory");
+               return -1;
+       }
+
+       p = rindex(newfull, '/');
+       if (!p) {
+               ERROR("bad path: %s", path);
+               return -1;
+       }
+       *p = '\0';
+
+       if ((fd = open(newfull, O_RDONLY)) < 0) {
+               ERROR("Error opening %s", newfull);
+               free(newfull);
+               return -1;
+       }
+
+       memset(&args, 0, sizeof(args));
+       strncpy(args.name, p+1, BTRFS_SUBVOL_NAME_MAX);
+       args.name[BTRFS_SUBVOL_NAME_MAX-1] = 0;
+       ret = ioctl(fd, BTRFS_IOC_SUBVOL_CREATE, &args);
+       INFO("btrfs: snapshot create ioctl returned %d", ret);
+{
+       FILE *f = fopen("/tmp/a", "a");
+       fprintf(f, "ioctl returned %d\n", ret);
+       fclose(f);
+}
+
+       free(newfull);
+       close(fd);
+       return ret;
+}
+
+static int btrfs_snapshot(const char *orig, const char *new)
+{
+       int fd = -1, fddst = -1, ret = -1;
+       struct btrfs_ioctl_vol_args_v2  args;
+       char *newdir, *newname, *newfull = NULL;
+
+       newfull = strdup(new);
+       if (!newfull) {
+               ERROR("Error: out of memory");
+               goto out;
+       }
+       // make sure the directory doesn't already exist
+       if (rmdir(newfull) < 0 && errno != -ENOENT) {
+               SYSERROR("Error removing empty new rootfs");
+               goto out;
+       }
+       newname = basename(newfull);
+       newdir = dirname(newfull);
+       fd = open(orig, O_RDONLY);
+       if (fd < 0) {
+               SYSERROR("Error opening original rootfs %s", orig);
+               goto out;
+       }
+       fddst = open(newdir, O_RDONLY);
+       if (fddst < 0) {
+               SYSERROR("Error opening new container dir %s", newdir);
+               goto out;
+       }
+
+       memset(&args, 0, sizeof(args));
+       args.fd = fd;
+       strncpy(args.name, newname, BTRFS_SUBVOL_NAME_MAX);
+       args.name[BTRFS_SUBVOL_NAME_MAX-1] = 0;
+       ret = ioctl(fddst, BTRFS_IOC_SNAP_CREATE_V2, &args);
+       INFO("btrfs: snapshot create ioctl returned %d", ret);
+
+out:
+       if (fddst != -1)
+               close(fddst);
+       if (fd != -1)
+               close(fd);
+       if (newfull)
+               free(newfull);
+       return ret;
+}
+
+static int btrfs_clonepaths(struct bdev *orig, struct bdev *new, const char *oldname,
+               const char *cname, const char *oldpath, const char *lxcpath, int snap,
+               unsigned long newsize)
+{
+       if (!orig->dest || !orig->src)
+               return -1;
+
+       if (strcmp(orig->type, "btrfs")) {
+               ERROR("btrfs cloen from %s backing store is not supported",
+                       orig->type);
+               return -1;
+       }
+
+       if ((new->dest = dir_new_path(orig->dest, oldname, cname, oldpath, lxcpath)) == NULL)
+               return -1;
+
+       if ((new->src = strdup(new->dest)) == NULL)
+               return -1;
+
+       if (orig->data && (new->data = strdup(orig->data)) == NULL)
+               return -1;
+
+       if (snap)
+               return btrfs_snapshot(orig->dest, new->dest);
+
+       if (rmdir(new->dest) < 0 && errno != -ENOENT) {
+               SYSERROR("removing %s\n", new->dest);
+               return -1;
+       }
+
+       return btrfs_subvolume_create(new->dest);
+}
+
+struct bdev_ops btrfs_ops = {
+       .detect = &btrfs_detect,
+       .mount = &btrfs_mount,
+       .umount = &btrfs_umount,
+       .clone_paths = &btrfs_clonepaths,
+};
+
+//
+// overlayfs ops
+//
+
+static int overlayfs_detect(const char *path)
+{
+       if (strncmp(path, "overlayfs:", 10) == 0)
+               return 1; // take their word for it
+       return 0;
+}
+
+//
+// XXXXXXX plain directory bind mount ops
+//
+int overlayfs_mount(struct bdev *bdev)
+{
+       char *options, *dup, *lower, *upper;
+       int len;
+       int ret;
+
+       if (strcmp(bdev->type, "overlayfs"))
+               return -22;
+       if (!bdev->src || !bdev->dest)
+               return -22;
+
+       //  separately mount it first
+       //  mount -t overlayfs -oupperdir=${upper},lowerdir=${lower} lower dest
+       dup = strdupa(bdev->src);
+       if (!(lower = index(dup, ':')))
+               return -22;
+       if (!(upper = index(++lower, ':')))
+               return -22;
+       *upper = '\0';
+       upper++;
+
+       // TODO We should check whether bdev->src is a blockdev, and if so
+       // but for now, only support overlays of a basic directory
+
+       len = strlen(lower) + strlen(upper) + strlen("upperdir=,lowerdir=") + 1;
+       options = alloca(len);
+       ret = snprintf(options, len, "upperdir=%s,lowerdir=%s", upper, lower);
+       if (ret < 0 || ret >= len)
+               return -1;
+       ret = mount(lower, bdev->dest, "overlayfs", MS_MGC_VAL, options);
+       if (ret < 0)
+               SYSERROR("overlayfs: error mounting %s onto %s options %s",
+                       lower, bdev->dest, options);
+       else
+               INFO("overlayfs: mounted %s onto %s options %s",
+                       lower, bdev->dest, options);
+       return ret;
+}
+
+int overlayfs_umount(struct bdev *bdev)
+{
+       if (strcmp(bdev->type, "overlayfs"))
+               return -22;
+       if (!bdev->src || !bdev->dest)
+               return -22;
+       return umount(bdev->dest);
+}
+
+static int overlayfs_clonepaths(struct bdev *orig, struct bdev *new, const char *oldname,
+               const char *cname, const char *oldpath, const char *lxcpath, int snap,
+               unsigned long newsize)
+{
+       if (!snap) {
+               ERROR("overlayfs is only for snapshot clones");
+               return -22;
+       }
+
+       if (!orig->src || !orig->dest)
+               return -1;
+
+       new->dest = dir_new_path(orig->dest, oldname, cname, oldpath, lxcpath);
+       if (!new->dest)
+               return -1;
+       if (mkdir_p(new->dest, 0755) < 0)
+               return -1;
+
+       if (strcmp(orig->type, "dir") == 0) {
+               char *delta;
+               int ret, len;
+               if (!snap)
+                       return -1;
+               // if we have /var/lib/lxc/c2/rootfs, then delta will be
+               //            /var/lib/lxc/c2/delta0
+               delta = strdup(new->dest);
+               if (!delta) {
+                       return -1;
+               }
+               if (strlen(delta) < 6) {
+                       free(delta);
+                       return -22;
+               }
+               strcpy(&delta[strlen(delta)-6], "delta0");
+               if ((ret = mkdir(delta, 0755)) < 0) {
+                       SYSERROR("error: mkdir %s", delta);
+                       free(delta);
+                       return -1;
+               }
+
+               // the src will be 'overlayfs:lowerdir:upperdir'
+               len = strlen(delta) + strlen(orig->src) + 12;
+               new->src = malloc(len);
+               if (!new->src) {
+                       free(delta);
+                       return -ENOMEM;
+               }
+               ret = snprintf(new->src, len, "overlayfs:%s:%s", orig->src, delta);
+               free(delta);
+               if (ret < 0 || ret >= len)
+                       return -ENOMEM;
+       } else if (strcmp(orig->type, "lvm") == 0) {
+               ERROR("overlayfs clone of lvm container is not yet supported");
+               // Note, supporting this will require overlayfs_mount supporting
+               // mounting of the underlay.  No big deal, just needs to be done.
+               return -1;
+       } else if (strcmp(orig->type, "overlayfs") == 0) {
+               // What exactly do we want to do here?
+               // I think we want to use the original lowerdir, with a
+               // private delta which is originally rsynced from the
+               // original delta
+               char *osrc, *odelta, *nsrc, *ndelta;
+               int len, ret;
+               if (!(osrc = strdup(orig->src)))
+                       return -22;
+               nsrc = index(osrc, ':') + 1;
+               if (nsrc != osrc + 10 || (odelta = index(nsrc, ':')) == NULL) {
+                       free(osrc);
+                       return -22;
+               }
+               *odelta = '\0';
+               odelta++;
+               ndelta = dir_new_path(odelta, oldname, cname, oldpath, lxcpath);
+               if (!ndelta) {
+                       free(osrc);
+                       return -ENOMEM;
+               }
+               if (do_rsync(odelta, ndelta) < 0) {
+                       ERROR("copying overlayfs delta");
+                       return -1;
+               }
+               len = strlen(nsrc) + strlen(ndelta) + 12;
+               new->src = malloc(len);
+               if (!new->src) {
+                       free(osrc);
+                       free(ndelta);
+                       return -ENOMEM;
+               }
+               ret = snprintf(new->src, len, "overlayfs:%s:%s", nsrc, ndelta);
+               free(osrc);
+               free(ndelta);
+               if (ret < 0 || ret >= len)
+                       return -ENOMEM;
+       }
+
+       return 0;
+}
+
+struct bdev_ops overlayfs_ops = {
+       .detect = &overlayfs_detect,
+       .mount = &overlayfs_mount,
+       .umount = &overlayfs_umount,
+       .clone_paths = &overlayfs_clonepaths,
+};
+
+struct bdev_type bdevs[] = {
+       {.name = "lvm", .ops = &lvm_ops,},
+       {.name = "btrfs", .ops = &btrfs_ops,},
+       {.name = "dir", .ops = &dir_ops,},
+       {.name = "overlayfs", .ops = &overlayfs_ops,},
+};
+
+static const size_t numbdevs = sizeof(bdevs) / sizeof(struct bdev_type);
+
+void bdev_put(struct bdev *bdev)
+{
+       if (bdev->data)
+               free(bdev->data);
+       if (bdev->src)
+               free(bdev->src);
+       if (bdev->dest)
+               free(bdev->dest);
+       free(bdev);
+}
+
+struct bdev *bdev_get(const char *type)
+{
+       int i;
+       struct bdev *bdev;
+
+       for (i=0; i<numbdevs; i++) {
+               if (strcmp(bdevs[i].name, type) == 0)
+                       break;
+       }
+       if (i == numbdevs)
+               return NULL;
+       bdev = malloc(sizeof(struct bdev));
+       if (!bdev)
+               return NULL;
+       memset(bdev, 0, sizeof(struct bdev));
+       bdev->ops = bdevs[i].ops;
+       bdev->type = bdevs[i].name;
+       return bdev;
+}
+
+struct bdev *bdev_init(const char *src, const char *dst, const char *data)
+{
+       int i;
+       struct bdev *bdev;
+
+       for (i=0; i<numbdevs; i++) {
+               int r;
+               r = bdevs[i].ops->detect(src);
+               if (r)
+                       break;
+       }
+       if (i == numbdevs)
+               return NULL;
+       bdev = malloc(sizeof(struct bdev));
+       if (!bdev)
+               return NULL;
+       memset(bdev, 0, sizeof(struct bdev));
+       bdev->ops = bdevs[i].ops;
+       bdev->type = bdevs[i].name;
+       if (data)
+               bdev->data = strdup(data);
+       if (src)
+               bdev->src = strdup(src);
+       if (dst)
+               bdev->dest = strdup(dst);
+
+       return bdev;
+}
+
+/*
+ * If we're not snaphotting, then bdev_copy becomes a simple case of mount
+ * the original, mount the new, and rsync the contents.
+ */
+struct bdev *bdev_copy(const char *src, const char *oldname, const char *cname,
+                       const char *oldpath, const char *lxcpath, const char *bdevtype,
+                       int snap, const char *bdevdata, unsigned long newsize)
+{
+       struct bdev *orig, *new;
+       pid_t pid;
+
+       /* if the container name doesn't show up in the rootfs path, then
+        * we don't know how to come up with a new name
+        */
+       if (strstr(src, oldname) == NULL) {
+               ERROR("original rootfs path %s doesn't include container name %s",
+                       src, oldname);
+               return NULL;
+       }
+
+       orig = bdev_init(src, NULL, NULL);
+       if (!orig) {
+               ERROR("failed to detect blockdev type for %s\n", src);
+               return NULL;
+       }
+
+       if (!orig->dest) {
+               int ret;
+               orig->dest = malloc(MAXPATHLEN);
+               if (!orig->dest) {
+                       ERROR("out of memory");
+                       bdev_put(orig);
+                       return NULL;
+               }
+               ret = snprintf(orig->dest, MAXPATHLEN, "%s/%s/rootfs", oldpath, oldname);
+               if (ret < 0 || ret >= MAXPATHLEN) {
+                       ERROR("rootfs path too long");
+                       bdev_put(orig);
+                       return NULL;
+               }
+       }
+
+       new = bdev_get(bdevtype ? bdevtype : orig->type);
+       if (!new) {
+               ERROR("no such block device type: %s", bdevtype ? bdevtype : orig->type);
+               bdev_put(orig);
+               return NULL;
+       }
+
+       if (new->ops->clone_paths(orig, new, oldname, cname, oldpath, lxcpath, snap, newsize) < 0) {
+               ERROR("failed getting pathnames for cloned storage: %s\n", src);
+               bdev_put(orig);
+               bdev_put(new);
+               return NULL;
+       }
+
+       pid = fork();
+       if (pid < 0) {
+               SYSERROR("fork");
+               bdev_put(orig);
+               bdev_put(new);
+               return NULL;
+       }
+
+       if (pid > 0) {
+               int ret = wait_for_pid(pid);
+               bdev_put(orig);
+               if (ret < 0) {
+                       bdev_put(new);
+                       return NULL;
+               }
+               return new;
+       }
+
+       if (unshare(CLONE_NEWNS) < 0) {
+               SYSERROR("unshare CLONE_NEWNS");
+               exit(1);
+       }
+       if (snap)
+               exit(0);
+
+       // If not a snapshot, copy the fs.
+       if (orig->ops->mount(orig) < 0) {
+               ERROR("failed mounting %s onto %s\n", src, orig->dest);
+               exit(1);
+       }
+       if (new->ops->mount(new) < 0) {
+               ERROR("failed mounting %s onto %s\n", new->src, new->dest);
+               exit(1);
+       }
+       if (do_rsync(orig->dest, new->dest) < 0) {
+               ERROR("rsyncing %s to %s\n", orig->src, new->src);
+               exit(1);
+       }
+       // don't bother umounting, ns exit will do that
+
+       exit(0);
+}
diff --git a/src/lxc/bdev.h b/src/lxc/bdev.h
new file mode 100644 (file)
index 0000000..131f158
--- /dev/null
@@ -0,0 +1,50 @@
+#ifndef __LXC_BDEV_H
+#define __LXC_BDEV_H
+/* blockdev operations for:
+ * dir, raw, btrfs, overlayfs, aufs, lvm, loop, zfs, btrfs
+ * someday: qemu-nbd, qcow2, qed
+ */
+
+#include "config.h"
+#include "lxccontainer.h"
+
+struct bdev;
+
+struct bdev_ops {
+       /* detect whether path is of this bdev type */
+       int (*detect)(const char *path);
+       // mount requires src and dest to be set.
+       int (*mount)(struct bdev *bdev);
+       int (*umount)(struct bdev *bdev);
+       /* given original mount, rename the paths for cloned container */
+       int (*clone_paths)(struct bdev *orig, struct bdev *new, const char *oldname,
+                       const char *cname, const char *oldpath, const char *lxcpath,
+                       int snap, unsigned long newsize);
+};
+
+struct bdev {
+       struct bdev_ops *ops;
+       char *type;
+       char *src;
+       char *dest;
+       char *data;
+};
+
+/*
+ * Instantiate a bdev object.  The src is used to determine which blockdev
+ * type this should be.  The dst and data are optional, and will be used
+ * in case of mount/umount.
+ *
+ * Optionally, src can be 'dir:/var/lib/lxc/c1' or 'lvm:/dev/lxc/c1'.  For
+ * other backing stores, this will allow additional options.  In particular,
+ * "overlayfs:/var/lib/lxc/canonical/rootfs:/var/lib/lxc/c1/delta" will mean
+ * use /var/lib/lxc/canonical/rootfs as lower dir, and /var/lib/lxc/c1/delta
+ * as the upper, writeable layer.
+ */
+struct bdev *bdev_init(const char *src, const char *dst, const char *data);
+
+struct bdev *bdev_copy(const char *src, const char *oldname, const char *cname,
+                       const char *oldpath, const char *lxcpath, const char *bdevtype,
+                       int snap, const char *bdevdata, unsigned long newsize);
+void bdev_put(struct bdev *bdev);
+#endif
index f8950500335c87359c507cff85978e1c587518b2..27b972755784d3291c2946cb2288663f6f4d4bf4 100644 (file)
@@ -64,6 +64,7 @@
 #include "log.h"
 #include "lxc.h"       /* for lxc_cgroup_set() */
 #include "caps.h"       /* for lxc_caps_last_cap() */
+#include "bdev.h"
 
 #if HAVE_APPARMOR
 #include <apparmor.h>
@@ -590,8 +591,8 @@ int pin_rootfs(const char *rootfs)
                return -2;
 
        if (!realpath(rootfs, absrootfs)) {
-               SYSERROR("failed to get real path for '%s'", rootfs);
-               return -1;
+               INFO("failed to get real path for '%s', not pinning", rootfs);
+               return -2;
        }
 
        if (access(absrootfs, F_OK)) {
@@ -1163,6 +1164,12 @@ static int setup_rootfs(struct lxc_conf *conf)
                }
        }
 
+       // First try mounting rootfs using a bdev
+       struct bdev *bdev = bdev_init(rootfs->path, rootfs->mount, NULL);
+       if (bdev && bdev->ops->mount(bdev) == 0) {
+               DEBUG("mounted '%s' on '%s'", rootfs->path, rootfs->mount);
+               return 0;
+       }
        if (mount_rootfs(rootfs->path, rootfs->mount)) {
                ERROR("failed to mount rootfs");
                return -1;
similarity index 100%
rename from src/lxc/lxc-clone.in
rename to src/lxc/lxc-clone-sh.in
diff --git a/src/lxc/lxc_clone.c b/src/lxc/lxc_clone.c
new file mode 100644 (file)
index 0000000..b9c7a6e
--- /dev/null
@@ -0,0 +1,138 @@
+#include <unistd.h>
+#include <getopt.h>
+#include <signal.h>
+#include <stdio.h>
+#include <sys/types.h>
+#include <sys/wait.h>
+#include <stdlib.h>
+#include <errno.h>
+
+#include "log.h"
+#include "config.h"
+#include "lxc.h"
+#include "conf.h"
+#include "state.h"
+#include "lxccontainer.h"
+
+lxc_log_define(lxc_clone, lxc);
+
+void usage(char *me)
+{
+       printf("Usage: %s [-s] [-B backingstore] [-L size] [-K] [-M] [-H]\n", me);
+       printf("          [-p lxcpath] [-P newlxcpath] orig new\n");
+       printf("\n");
+       printf("  -s: snapshot rather than copy\n");
+       printf("  -B: use specified new backingstore.  Default is the same as\n");
+       printf("      the original.  Options include btrfs, lvm, overlayfs, \n");
+       printf("      dir\n");
+       printf("  -L: for blockdev-backed backingstore, use specified size\n");
+       printf("  -K: Keep name - do not change the container name\n");
+       printf("  -M: Keep macaddr - do not choose a random new mac address\n");
+       printf("  -H: copy Hooks - copy mount hooks into container directory\n");
+       printf("      and substitute container names and lxcpaths\n");
+       printf("  -p: use container orig from custom lxcpath\n");
+       printf("  -P: create container new in custom lxcpath\n");
+       exit(1);
+}
+
+static struct option options[] = {
+       { "snapshot", no_argument, 0, 's'},
+       { "backingstore", required_argument, 0, 'B'},
+       { "size", required_argument, 0, 'L'},
+       { "orig", required_argument, 0, 'o'},
+       { "new", required_argument, 0, 'n'},
+       { "vgname", required_argument, 0, 'v'},
+       { "keepname", no_argument, 0, 'K'},
+       { "keepmac", no_argument, 0, 'M'},
+       { "copyhooks", no_argument, 0, 'H'},  // should this be default?
+       { "lxcpath", required_argument, 0, 'p'},
+       { "newpath", required_argument, 0, 'P'},
+       { "fstype", required_argument, 0, 't'},
+       { "help", no_argument, 0, 'h'},
+       { 0, 0, 0, 0 },
+};
+
+int main(int argc, char *argv[])
+{
+       struct lxc_container *c1 = NULL, *c2 = NULL;
+       int snapshot = 0, keepname = 0, keepmac = 0, copyhooks = 0;
+       int flags = 0, option_index;
+       long newsize = 0;
+       char *bdevtype = NULL, *lxcpath = NULL, *newpath = NULL, *fstype = NULL;
+       char *orig = NULL, *new = NULL, *vgname = NULL;
+       char c;
+
+       if (argc < 3)
+               usage(argv[0]);
+
+       while (1) {
+               c = getopt_long(argc, argv, "sB:L:o:n:v:KMHp:P:t:h", options, &option_index);
+               if (c == -1)
+                       break;
+               switch (c) {
+               case 's': snapshot = 1; break;
+               case 'B': bdevtype = optarg; break;
+               case 'L': newsize = atol(optarg); break;
+               case 'o': orig = optarg; break;
+               case 'n': new = optarg; break;
+               case 'v': vgname = optarg; break;
+               case 'K': keepname = 1; break;
+               case 'M': keepmac = 1; break;
+               case 'H': copyhooks = 1; break;
+               case 'p': lxcpath = optarg; break;
+               case 'P': newpath = optarg; break;
+               case 't': fstype = optarg; break;
+               case 'h': usage(argv[0]);
+               default: break;
+               }
+       }
+       if (optind == argc-2 && !orig)
+               orig = argv[optind++];
+       if (optind == argc-1 && !new)
+               new = argv[optind++];
+       if (optind < argc) {
+               printf("%d extraneous arguments\n", argc-optind);
+               usage(argv[0]);
+       }
+       if (!new || !orig) {
+               printf("Error: you must provide orig and new names\n");
+               usage(argv[0]);
+       }
+
+       if (snapshot)  flags |= LXC_CLONE_SNAPSHOT;
+       if (keepname)  flags |= LXC_CLONE_KEEPNAME;
+       if (keepmac)   flags |= LXC_CLONE_KEEPMACADDR;
+       if (copyhooks) flags |= LXC_CLONE_COPYHOOKS;
+
+       // vgname and fstype could be supported by sending them through the
+       // bdevdata.  However, they currently are not yet.  I'm not convinced
+       // they are worthwhile.
+       if (vgname) {
+               printf("Error: vgname not supported\n");
+               usage(argv[0]);
+       }
+       if (fstype) {
+               printf("Error: fstype not supported\n");
+               usage(argv[0]);
+       }
+
+       c1 = lxc_container_new(orig, lxcpath);
+       if (!c1)
+               exit(1);
+       if (!c1->is_defined(c1)) {
+               fprintf(stderr, "Error: container %s is not defined\n", orig);
+               lxc_container_put(c1);
+               exit(1);
+       }
+       c2 = c1->clone(c1, new, newpath, flags, bdevtype, NULL, newsize);
+       if (c2 == NULL) {
+               lxc_container_put(c1);
+               fprintf(stderr, "clone failed\n");
+               exit(1);
+       }
+       printf("Created container %s as %s of %s\n", new,
+               snapshot ? "snapshot" : "copy", orig);
+       lxc_container_put(c1);
+       lxc_container_put(c2);
+       return(0);
+}
index 740a8d99eed7133c3b03624761a844690a261aef..312cc99e48b85ce675d3bedbee508117017db764 100644 (file)
  * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
  */
 
+#define _GNU_SOURCE
+#include <unistd.h>
+#include <sys/types.h>
+#include <sys/wait.h>
+#include <errno.h>
+#include <sched.h>
+#include "config.h"
 #include "lxc.h"
 #include "state.h"
 #include "lxccontainer.h"
 #include "conf.h"
-#include "config.h"
 #include "confile.h"
 #include "cgroup.h"
 #include "commands.h"
 #include "version.h"
 #include "log.h"
-#include <unistd.h>
-#include <sys/types.h>
-#include <sys/wait.h>
-#include <errno.h>
+#include "bdev.h"
 #include <lxc/utils.h>
 #include <lxc/monitor.h>
 
+/* Define unshare() if missing from the C library */
+/* this is also in attach.c and lxccontainer.c: commonize it in utils.c */
+#ifndef HAVE_UNSHARE
+static int unshare(int flags)
+{
+#ifdef __NR_unshare
+       return syscall(__NR_unshare, flags);
+#else
+       errno = ENOSYS;
+       return -1;
+#endif
+}
+#else
+int unshare(int);
+#endif
+
 lxc_log_define(lxc_container, lxc);
 
 /* LOCKING
@@ -536,10 +555,8 @@ static bool lxcapi_create(struct lxc_container *c, char *t, char *const argv[])
 {
        bool bret = false;
        pid_t pid;
-       int ret, status;
-       char *tpath = NULL;
-       int len, nargs = 0;
-       char **newargv;
+       char *tpath = NULL, **newargv;
+       int ret, len, nargs = 0;
 
        if (!c)
                return false;
@@ -566,7 +583,7 @@ static bool lxcapi_create(struct lxc_container *c, char *t, char *const argv[])
                goto out;
 
        /* we're going to fork.  but since we'll wait for our child, we
-          don't need to lxc_container_get */
+        * don't need to lxc_container_get */
 
        if (lxclock(c->slock, 0)) {
                ERROR("failed to grab global container lock for %s\n", c->name);
@@ -637,26 +654,8 @@ static bool lxcapi_create(struct lxc_container *c, char *t, char *const argv[])
                exit(1);
        }
 
-again:
-       ret = waitpid(pid, &status, 0);
-       if (ret == -1) {
-               if (errno == -EINTR)
-                       goto again;
-               SYSERROR("waitpid failed");
-               goto out_unlock;
-       }
-       if (ret != pid)
-               goto again;
-       if (!WIFEXITED(status))  { // did not exit normally
-               // we could set an error code and string inside the
-               // container_struct here if we like
-               ERROR("container creation template exited abnormally\n");
-               goto out_unlock;
-       }
-
-       if (WEXITSTATUS(status) != 0) {
-               ERROR("container creation template for %s exited with %d\n",
-                     c->name, WEXITSTATUS(status));
+       if (wait_for_pid(pid) != 0) {
+               ERROR("container creation template for %s failed\n", c->name);
                goto out_unlock;
        }
 
@@ -822,7 +821,6 @@ static bool lxcapi_save_config(struct lxc_container *c, const char *alt_file)
 static bool lxcapi_destroy(struct lxc_container *c)
 {
        pid_t pid;
-       int ret, status;
 
        if (!c)
                return false;
@@ -840,23 +838,12 @@ static bool lxcapi_destroy(struct lxc_container *c)
                exit(1);
        }
 
-again:
-       ret = waitpid(pid, &status, 0);
-       if (ret == -1) {
-               if (errno == -EINTR)
-                       goto again;
-               perror("waitpid");
-               return false;
-       }
-       if (ret != pid)
-               goto again;
-       if (!WIFEXITED(status))  { // did not exit normally
-               // we could set an error code and string inside the
-               // container_struct here if we like
+       if (wait_for_pid(pid) < 0) {
+               ERROR("Error destroying container %s", c->name);
                return false;
        }
 
-       return WEXITSTATUS(status) == 0;
+       return true;
 }
 
 static bool lxcapi_set_config_item(struct lxc_container *c, const char *key, const char *v)
@@ -1027,6 +1014,518 @@ const char *lxc_get_version(void)
        return lxc_version();
 }
 
+static int copy_file(char *old, char *new)
+{
+       int in, out;
+       ssize_t len, ret;
+       char buf[8096];
+       struct stat sbuf;
+
+       if (file_exists(new)) {
+               ERROR("copy destination %s exists", new);
+               return -1;
+       }
+       ret = stat(old, &sbuf);
+       if (ret < 0) {
+               SYSERROR("stat'ing %s", old);
+               return -1;
+       }
+
+       in = open(old, O_RDONLY);
+       if (in < 0) {
+               SYSERROR("opening original file %s", old);
+               return -1;
+       }
+       out = open(new, O_CREAT | O_EXCL | O_WRONLY, 0644);
+       if (out < 0) {
+               SYSERROR("opening new file %s", new);
+               close(in);
+               return -1;
+       }
+
+       while (1) {
+               len = read(in, buf, 8096);
+               if (len < 0) {
+                       SYSERROR("reading old file %s", old);
+                       goto err;
+               }
+               if (len == 0)
+                       break;
+               ret = write(out, buf, len);
+               if (ret < len) {  // should we retry?
+                       SYSERROR("write to new file %s was interrupted", new);
+                       goto err;
+               }
+       }
+       close(in);
+       close(out);
+
+       // we set mode, but not owner/group
+       ret = chmod(new, sbuf.st_mode);
+       if (ret) {
+               SYSERROR("setting mode on %s", new);
+               return -1;
+       }
+
+       return 0;
+
+err:
+       close(in);
+       close(out);
+       return -1;
+}
+
+/*
+ * we're being passed result of two strstrs(x, y).  We want to write
+ * all data up to the first found string, or to end of the string if
+ * neither string was found.
+ * This function will return the earliest found string if any, or else
+ * NULL
+ */
+static const char *lowest_nonnull(const char *p1, const char *p2)
+{
+       if (!p1)
+               return p2;
+       if (!p2)
+               return p1;
+       return p1 < p2 ? p1 : p2;
+}
+
+static int is_word_sep(char c)
+{
+       switch(c) {
+       case '\0':
+       case '\n':
+       case '\r':
+       case '\t':
+       case ' ':
+       case '=':
+       case '/':
+               return 1;
+       default: return 0;
+       }
+}
+
+static const char *find_first_wholeword(const char *p, const char *word)
+{
+       if (!p)
+               return NULL;
+
+       while ((p = strstr(p, word)) != NULL) {
+               if (is_word_sep(*(p-1)) && is_word_sep(p[strlen(word)]))
+                       return p;
+               p++;
+       }
+       return NULL;
+}
+
+static int update_name_and_paths(const char *path, struct lxc_container *oldc,
+               const char *newname, const char *newpath)
+{
+       FILE *f;
+       size_t flen;
+       char *contents;
+       const char *p0, *p1, *p2, *end;
+       const char *oldpath = oldc->get_config_path(oldc);
+       const char *oldname = oldc->name;
+
+       f = fopen(path, "r");
+       if (!f) {
+               SYSERROR("opening old config");
+               return -1;
+       }
+       if (fseek(f, 0, SEEK_END) < 0) {
+               SYSERROR("seeking to end of old config");
+               fclose(f);
+               return -1;
+       }
+       flen = ftell(f);
+       if (flen < 0) {
+               fclose(f);
+               SYSERROR("telling size of old config");
+               return -1;
+       }
+       if (fseek(f, 0, SEEK_SET) < 0) {
+               fclose(f);
+               SYSERROR("rewinding old config");
+               return -1;
+       }
+       contents = malloc(flen);
+       if (!contents) {
+               SYSERROR("out of memory");
+               fclose(f);
+       }
+       if (fread(contents, 1, flen, f) != flen) {
+               free(contents);
+               fclose(f);
+               SYSERROR("reading old config");
+               return -1;
+       }
+       if (fclose(f) < 0) {
+               free(contents);
+               SYSERROR("closing old config");
+               return -1;
+       }
+
+       f = fopen(path, "w");
+       if (!f) {
+               SYSERROR("reopening config");
+               free(contents);
+               return -1;
+       }
+
+       p0 = contents;
+       end = contents + flen;
+       while (1) {
+               p1 = find_first_wholeword(p0, oldpath);
+               p2 = find_first_wholeword(p0, oldname);
+               if (!p1 && !p2) {
+                       // write the rest and be done
+                       if (fwrite(p0, 1, (end-p0), f) != (end-p0)) {
+                               SYSERROR("writing new config");
+                               free(contents);
+                               fclose(f);
+                               return -1;
+                       }
+                       free(contents);
+                       fclose(f);
+                       // success
+                       return 0;
+               } else {
+                       const char *p = lowest_nonnull(p1, p2);
+                       const char *new = (p == p2) ? newname : newpath;
+                       if (fwrite(p0, 1, (p-p0), f) != (p-p0)) {
+                               SYSERROR("writing new config");
+                               free(contents);
+                               fclose(f);
+                               return -1;
+                       }
+                       p0 = p;
+                       // now write the newpath or newname
+                       if (fwrite(new, 1, strlen(new), f) != strlen(new)) {
+                               SYSERROR("writing new name or path in new config");
+                               free(contents);
+                               fclose(f);
+                               return -1;
+                       }
+                       p0 += (p == p2) ? strlen(oldname) : strlen(oldpath);
+               }
+       }
+}
+
+static int copyhooks(struct lxc_container *oldc, struct lxc_container *c)
+{
+       int i;
+       int ret;
+       struct lxc_list *it;
+
+       for (i=0; i<NUM_LXC_HOOKS; i++) {
+               lxc_list_for_each(it, &c->lxc_conf->hooks[i]) {
+                       char *hookname = it->elem;
+                       char *fname = rindex(hookname, '/');
+                       char tmppath[MAXPATHLEN];
+                       if (!fname) // relative path - we don't support, but maybe we should
+                               return 0;
+                       // copy the script, and change the entry in confile
+                       ret = snprintf(tmppath, MAXPATHLEN, "%s/%s/%s",
+                                       c->config_path, c->name, fname+1);
+                       if (ret < 0 || ret >= MAXPATHLEN)
+                               return -1;
+                       ret = copy_file(it->elem, tmppath);
+                       if (ret < 0)
+                               return -1;
+                       free(it->elem);
+                       it->elem = strdup(tmppath);
+                       if (!it->elem) {
+                               ERROR("out of memory copying hook path");
+                               return -1;
+                       }
+                       update_name_and_paths(it->elem, oldc, c->name, c->get_config_path(c));
+               }
+       }
+
+       c->save_config(c, NULL);
+       return 0;
+}
+
+static void new_hwaddr(char *hwaddr)
+{
+       FILE *f = fopen("/dev/urandom", "r");
+       if (f) {
+               unsigned int seed;
+               int ret = fread(&seed, sizeof(seed), 1, f);
+               if (ret != 1)
+                       seed = time(NULL);
+               fclose(f);
+               srand(seed);
+       } else
+               srand(time(NULL));
+       snprintf(hwaddr, 18, "00:16:3e:%02x:%02x:%02x",
+                       rand() % 255, rand() % 255, rand() % 255);
+}
+
+static void network_new_hwaddrs(struct lxc_container *c)
+{
+       struct lxc_list *it;
+
+       lxc_list_for_each(it, &c->lxc_conf->network) {
+               struct lxc_netdev *n = it->elem;
+               if (n->hwaddr)
+                       new_hwaddr(n->hwaddr);
+       }
+}
+
+static int copy_fstab(struct lxc_container *oldc, struct lxc_container *c)
+{
+       char newpath[MAXPATHLEN];
+       char *oldpath = oldc->lxc_conf->fstab;
+       int ret;
+
+       if (!oldpath)
+               return 0;
+
+       char *p = rindex(oldpath, '/');
+       if (!p)
+               return -1;
+       ret = snprintf(newpath, MAXPATHLEN, "%s/%s%s",
+                       c->config_path, c->name, p);
+       if (ret < 0 || ret >= MAXPATHLEN) {
+               ERROR("error printing new path for %s", oldpath);
+               return -1;
+       }
+       if (file_exists(newpath)) {
+               ERROR("error: fstab file %s exists", newpath);
+               return -1;
+       }
+
+       if (copy_file(oldpath, newpath) < 0) {
+               ERROR("error: copying %s to %s", oldpath, newpath);
+               return -1;
+       }
+       free(c->lxc_conf->fstab);
+       c->lxc_conf->fstab = strdup(newpath);
+       if (!c->lxc_conf->fstab) {
+               ERROR("error: allocating pathname");
+               return -1;
+       }
+
+       return 0;
+}
+
+static int copy_storage(struct lxc_container *c0, struct lxc_container *c,
+               const char *newtype, int flags, const char *bdevdata, unsigned long newsize)
+{
+       struct bdev *bdev;
+
+       bdev = bdev_copy(c0->lxc_conf->rootfs.path, c0->name, c->name,
+                       c0->config_path, c->config_path, newtype, !!(flags & LXC_CLONE_SNAPSHOT),
+                       bdevdata, newsize);
+       if (!bdev) {
+               ERROR("error copying storage");
+               return -1;
+       }
+       free(c->lxc_conf->rootfs.path);
+       c->lxc_conf->rootfs.path = strdup(bdev->src);
+       bdev_put(bdev);
+       if (!c->lxc_conf->rootfs.path)
+               return -1;
+       // here we could also update all lxc.mount.entries or even
+       // items in the lxc.mount fstab list.  As discussed on m-l,
+       // we could do either any source paths starting with the
+       // lxcpath/oldname, or simply anythign which is not a virtual
+       // fs or a bind mount.
+       return 0;
+}
+
+static int clone_update_rootfs(struct lxc_container *c, int flags)
+{
+       int ret = -1;
+       char path[MAXPATHLEN];
+       struct bdev *bdev;
+       FILE *fout;
+       pid_t pid;
+
+       if (flags & LXC_CLONE_KEEPNAME)
+               return 0;
+
+       /* update hostname in rootfs */
+       /* we're going to mount, so run in a clean namespace to simplify cleanup */
+
+       pid = fork();
+       if (pid < 0)
+               return -1;
+       if (pid > 0)
+               return wait_for_pid(pid);
+
+       if (unshare(CLONE_NEWNS) < 0) {
+               ERROR("error unsharing mounts");
+               exit(1);
+       }
+       bdev = bdev_init(c->lxc_conf->rootfs.path, c->lxc_conf->rootfs.mount, NULL);
+       if (!bdev)
+               exit(1);
+       if (bdev->ops->mount(bdev) < 0)
+               exit(1);
+       ret = snprintf(path, MAXPATHLEN, "%s/etc/hostname", bdev->dest);
+       if (ret < 0 || ret >= MAXPATHLEN)
+               exit(1);
+       if (!(fout = fopen(path, "w"))) {
+               SYSERROR("unable to open %s: ignoring\n", path);
+               exit(0);
+       }
+       if (fprintf(fout, "%s", c->name) < 0)
+               exit(1);
+       if (fclose(fout) < 0)
+               exit(1);
+       exit(0);
+}
+
+/*
+ * We want to support:
+sudo lxc-clone -o o1 -n n1 -s -L|-fssize fssize -v|--vgname vgname \
+        -p|--lvprefix lvprefix -t|--fstype fstype  -B backingstore
+
+-s [ implies overlayfs]
+-s -B overlayfs
+-s -B aufs
+
+only rootfs gets converted (copied/snapshotted) on clone.
+*/
+
+static int create_file_dirname(char *path)
+{
+       char *p = rindex(path, '/');
+       int ret;
+
+       if (!p)
+               return -1;
+       *p = '\0';
+       ret = mkdir(path, 0755);
+       if (ret && errno != EEXIST)
+               SYSERROR("creating container path %s\n", path);
+       *p = '/';
+       return ret;
+}
+
+struct lxc_container *lxcapi_clone(struct lxc_container *c, const char *newname,
+               const char *lxcpath, int flags,
+               const char *bdevtype, const char *bdevdata, unsigned long newsize)
+{
+       struct lxc_container *c2 = NULL;
+       char newpath[MAXPATHLEN];
+       int ret;
+       const char *n, *l;
+       FILE *fout;
+
+       if (!c || !c->is_defined(c))
+               return NULL;
+
+       if (lxclock(c->privlock, 0))
+               return NULL;
+
+       if (c->is_running(c)) {
+               ERROR("error: Original container (%s) is running", c->name);
+               goto out;
+       }
+
+       // Make sure the container doesn't yet exist.
+       n = newname ? newname : c->name;
+       l = lxcpath ? lxcpath : c->get_config_path(c);
+       ret = snprintf(newpath, MAXPATHLEN, "%s/%s/config", l, n);
+       if (ret < 0  || ret >= MAXPATHLEN) {
+               SYSERROR("clone: failed making config pathname");
+               goto out;
+       }
+       if (file_exists(newpath)) {
+               ERROR("error: clone: %s exists", newpath);
+               goto out;
+       }
+
+       if (create_file_dirname(newpath) < 0) {
+               ERROR("Error creating container dir for %s", newpath);
+               goto out;
+       }
+
+       // copy the configuration, tweak it as needed,
+       fout = fopen(newpath, "w");
+       if (!fout) {
+               SYSERROR("open %s", newpath);
+               goto out;
+       }
+       write_config(fout, c->lxc_conf);
+       fclose(fout);
+
+       if (update_name_and_paths(newpath, c, n, l) < 0) {
+               ERROR("Error updating name in cloned config");
+               goto out;
+       }
+
+       sprintf(newpath, "%s/%s/rootfs", l, n);
+       if (mkdir(newpath, 0755) < 0) {
+               SYSERROR("error creating %s", newpath);
+               goto out;
+       }
+
+       c2 = lxc_container_new(n, l);
+       if (!c) {
+               ERROR("clone: failed to create new container (%s %s)", n, l);
+               goto out;
+       }
+
+       // copy hooks if requested
+       if (flags & LXC_CLONE_COPYHOOKS) {
+               ret = copyhooks(c, c2);
+               if (ret < 0) {
+                       ERROR("error copying hooks");
+                       c2->destroy(c2);
+                       lxc_container_put(c2);
+                       goto out;
+               }
+       }
+
+       if (copy_fstab(c, c2) < 0) {
+               ERROR("error copying fstab");
+               c2->destroy(c2);
+               lxc_container_put(c2);
+               goto out;
+       }
+
+       // update macaddrs
+       if (!(flags & LXC_CLONE_KEEPMACADDR))
+               network_new_hwaddrs(c2);
+
+       // copy/snapshot rootfs's
+       ret = copy_storage(c, c2, bdevtype, flags, bdevdata, newsize);
+       if (ret < 0) {
+               c2->destroy(c2);
+               lxc_container_put(c2);
+               goto out;
+       }
+
+       if (!c2->save_config(c2, NULL)) {
+               c2->destroy(c2);
+               lxc_container_put(c2);
+               goto out;
+       }
+
+       if (clone_update_rootfs(c2, flags) < 0) {
+               //c2->destroy(c2);
+               lxc_container_put(c2);
+               goto out;
+       }
+
+       // TODO: update c's lxc.snapshot = count
+       lxcunlock(c->privlock);
+       return c2;
+
+out:
+       lxcunlock(c->privlock);
+       if (c2)
+               lxc_container_put(c2);
+
+       return NULL;
+}
+
 struct lxc_container *lxc_container_new(const char *name, const char *configpath)
 {
        struct lxc_container *c;
@@ -1103,6 +1602,7 @@ struct lxc_container *lxc_container_new(const char *name, const char *configpath
        c->set_cgroup_item = lxcapi_set_cgroup_item;
        c->get_config_path = lxcapi_get_config_path;
        c->set_config_path = lxcapi_set_config_path;
+       c->clone = lxcapi_clone;
 
        /* we'll allow the caller to update these later */
        if (lxc_log_init(NULL, "none", NULL, "lxc_container", 0, c->config_path)) {
index de9854c905802bd87b4cacbccb20ae89b3e43871..a4be7537c242ab81987593c6119dd11a7fc60513 100644 (file)
@@ -1,9 +1,17 @@
+#ifndef __LXC_CONTAINER_H
+#define __LXC_CONTAINER_H
 #include "lxclock.h"
 #include <stdlib.h>
 #include <malloc.h>
 
 #include <stdbool.h>
 
+#define LXC_CLONE_KEEPNAME        (1 << 0)
+#define LXC_CLONE_COPYHOOKS       (1 << 1)
+#define LXC_CLONE_KEEPMACADDR     (1 << 2)
+#define LXC_CLONE_SNAPSHOT        (1 << 3)
+#define LXC_CLONE_MAXFLAGS        (1 << 4)
+
 struct lxc_container {
        // private fields
        char *name;
@@ -72,6 +80,33 @@ struct lxc_container {
        const char *(*get_config_path)(struct lxc_container *c);
        bool (*set_config_path)(struct lxc_container *c, const char *path);
 
+       /*
+        * @c: the original container
+        * @newname: new name for the container.  If NULL, the same name is used, and
+        *  a new lxcpath MUST be specified.
+        * @lxcpath: lxcpath in which to create the new container.  If NULL, then the
+        *  original container's lxcpath will be used.  (Shoudl we use the default
+        *  instead?)
+        * @flags: additional flags to modify cloning behavior.
+        *  LXC_CLONE_KEEPNAME: don't edit the rootfs to change the hostname.
+        *  LXC_CLONE_COPYHOOKS: copy all hooks into the container dir
+        *  LXC_CLONE_KEEPMACADDR: don't change the mac address on network interfaces.
+        *  LXC_CLONE_SNAPSHOT: snapshot the original filesystem(s).  If @devtype was not
+        *   specified, then do so with the native bdevtype if possible, else use an
+        *   overlayfs.
+        * @bdevtype: optionally force the cloned bdevtype to a specified plugin.  By
+        *  default the original  is used (subject to snapshot requirements).
+        * @bdevdata: information about how to create the new storage (i.e. fstype and
+        *  fsdata)
+        * @newsize: in case of a block device backing store, an optional size.  If 0,
+        *  then the original backing store's size will be used if possible.  Note this
+        *  only applies to the rootfs.  For any other filesystems, the original size
+        *  will be duplicated.
+        */
+       struct lxc_container *(*clone)(struct lxc_container *c, const char *newname,
+               const char *lxcpath, int flags, const char *bdevtype,
+               const char *bdevdata, unsigned long newsize);
+
 #if 0
        bool (*commit_cgroups)(struct lxc_container *c);
        bool (*reread_cgroups)(struct lxc_container *c);
@@ -93,3 +128,4 @@ const char *lxc_get_version(void);
 char ** lxc_get_valid_keys();
 char ** lxc_get_valid_values(char *key);
 #endif
+#endif
index 6a154f916316dc2691ad3ea563e55fe8175eb906..2fa2b0cc86046cb5d2248ca363dbb559198c3df4 100644 (file)
@@ -34,6 +34,8 @@
 #include <dirent.h>
 #include <fcntl.h>
 #include <libgen.h>
+#include <sys/types.h>
+#include <sys/wait.h>
 
 #include "log.h"
 
@@ -172,3 +174,21 @@ out:
                fclose(fin);
        return default_lxcpath;
 }
+
+int wait_for_pid(pid_t pid)
+{
+       int status, ret;
+
+again:
+       ret = waitpid(pid, &status, 0);
+       if (ret == -1) {
+               if (errno == -EINTR)
+                       goto again;
+               return -1;
+       }
+       if (ret != pid)
+               goto again;
+       if (!WIFEXITED(status) || WEXITSTATUS(status) != 0)
+               return -1;
+       return 0;
+}
index 8e6a7489c17cd7f661bef46fc63ad9825df08f86..ee463a7f8b1deb6469a44cfed9683694a596bdf5 100644 (file)
@@ -58,4 +58,9 @@ extern int __build_bug_on_failed;
        } while(0)
 #endif
 
+/*
+ * wait on a child we forked
+ */
+extern int wait_for_pid(pid_t pid);
+
 #endif
index 4cbeeb3feafa6fd7acef35a75b8e365b7244746a..c0ce6484a52e2d402e624d689e3a4c55a419df60 100644 (file)
@@ -13,6 +13,7 @@ lxc_test_get_item_SOURCES = get_item.c
 lxc_test_getkeys_SOURCES = getkeys.c
 lxc_test_lxcpath_SOURCES = lxcpath.c
 lxc_test_cgpath_SOURCES = cgpath.c
+lxc_test_clonetest_SOURCES = clonetest.c
 
 AM_CFLAGS=-I$(top_srcdir)/src \
        -DLXCROOTFSMOUNT=\"$(LXCROOTFSMOUNT)\" \
@@ -23,7 +24,7 @@ AM_CFLAGS=-I$(top_srcdir)/src \
 bin_PROGRAMS = lxc-test-containertests lxc-test-locktests lxc-test-startone \
        lxc-test-destroytest lxc-test-saveconfig lxc-test-createtest \
        lxc-test-shutdowntest lxc-test-get_item lxc-test-getkeys lxc-test-lxcpath \
-       lxc-test-cgpath
+       lxc-test-cgpath lxc-test-clonetest
 
 endif
 
@@ -38,4 +39,5 @@ EXTRA_DIST = \
        lxcpath.c \
        saveconfig.c \
        shutdowntest.c \
+       clonetest.c \
        startone.c
diff --git a/src/tests/clonetest.c b/src/tests/clonetest.c
new file mode 100644 (file)
index 0000000..fcb5ea6
--- /dev/null
@@ -0,0 +1,178 @@
+/* liblxcapi
+ *
+ * Copyright © 2012 Serge Hallyn <serge.hallyn@ubuntu.com>.
+ * Copyright © 2012 Canonical Ltd.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2, as
+ * published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License along
+ * with this program; if not, write to the Free Software Foundation, Inc.,
+ * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
+ */
+#include "../lxc/lxccontainer.h"
+
+#include <unistd.h>
+#include <signal.h>
+#include <stdio.h>
+#include <sys/types.h>
+#include <sys/wait.h>
+#include <stdlib.h>
+#include <errno.h>
+
+#define MYNAME "clonetest1"
+#define MYNAME2 "clonetest2"
+
+int main(int argc, char *argv[])
+{
+       struct lxc_container *c = NULL, *c2 = NULL, *c3 = NULL;
+       int ret = 1;
+
+       c = lxc_container_new(MYNAME, NULL);
+       c2 = lxc_container_new(MYNAME2, NULL);
+       if (c) {
+               c->destroy(c);
+               lxc_container_put(c);
+               c = NULL;
+       }
+       if (c2) {
+               c2->destroy(c2);
+               lxc_container_put(c2);
+               c2 = NULL;
+       }
+
+       if ((c = lxc_container_new(MYNAME, NULL)) == NULL) {
+               fprintf(stderr, "%d: error opening lxc_container %s\n", __LINE__, MYNAME);
+               ret = 1;
+               goto out;
+       }
+       c->save_config(c, NULL);
+       if (!c->createl(c, "ubuntu", NULL)) {
+               fprintf(stderr, "%d: failed to create a container\n", __LINE__);
+               goto out;
+       }
+       c->load_config(c, NULL);
+
+       if (!c->is_defined(c)) {
+               fprintf(stderr, "%d: %s thought it was not defined\n", __LINE__, MYNAME);
+               goto out;
+       }
+
+       c2 = c->clone(c, MYNAME2, NULL, 0, NULL, NULL, 0);
+       if (!c2) {
+               fprintf(stderr, "%d: %s clone returned NULL\n", __LINE__, MYNAME2);
+               goto out;
+       }
+
+       if (!c2->is_defined(c)) {
+               fprintf(stderr, "%d: %s not defined after clone\n", __LINE__, MYNAME2);
+               goto out;
+       }
+
+       fprintf(stderr, "directory backing store tests passed\n");
+
+       // now test with lvm
+       // Only do this if clonetestlvm1 exists - user has to set this up
+       // in advance
+       //c2->destroy(c2);
+       lxc_container_put(c2);
+       //c->destroy(c);
+       lxc_container_put(c);
+       c = NULL;
+
+       c2 = lxc_container_new("clonetestlvm2", NULL);
+       if (c2) {
+               if (c2->is_defined(c2))
+                       c2->destroy(c2);
+               lxc_container_put(c2);
+       }
+       c2 = lxc_container_new("clonetest-o1", NULL);
+       if (c2) {
+               if (c2->is_defined(c2))
+                       c2->destroy(c2);
+               lxc_container_put(c2);
+       }
+       c2 = lxc_container_new("clonetest-o2", NULL);
+       if (c2) {
+               if (c2->is_defined(c2))
+                       c2->destroy(c2);
+               lxc_container_put(c2);
+       }
+       c2 = NULL;
+
+       // lvm-copied
+       c = lxc_container_new("clonetestlvm1", NULL);
+       if (!c) {
+               fprintf(stderr, "failed loading clonetestlvm1\n");
+               goto out;
+       }
+       if (!c->is_defined(c)) {
+               fprintf(stderr, "clonetestlvm1 does not exist, skipping lvm tests\n");
+               ret = 0;
+               goto out;
+       }
+
+       if ((c2 = c->clone(c, "clonetestlvm2", NULL, 0, NULL, NULL, 0)) == NULL) {
+               fprintf(stderr, "lvm clone failed\n");
+               goto out;
+       }
+
+       lxc_container_put(c2);
+
+       // lvm-snapshot
+       c2 = lxc_container_new("clonetestlvm3", NULL);
+       if (c2) {
+               if (c2->is_defined(c2))
+                       c2->destroy(c2);
+               lxc_container_put(c2);
+               c2 = NULL;
+       }
+
+       if ((c2 = c->clone(c, "clonetestlvm3", NULL, LXC_CLONE_SNAPSHOT, NULL, NULL, 0)) == NULL) {
+               fprintf(stderr, "lvm clone failed\n");
+               goto out;
+       }
+       lxc_container_put(c2);
+       lxc_container_put(c);
+       c = c2 = NULL;
+
+       if ((c = lxc_container_new(MYNAME, NULL)) == NULL) {
+               fprintf(stderr, "error opening original container for overlay test\n");
+               goto out;
+       }
+
+       // Now create an overlayfs clone of a dir-backed container
+       if ((c2 = c->clone(c, "clonetest-o1", NULL, LXC_CLONE_SNAPSHOT, "overlayfs", NULL, 0)) == NULL) {
+               fprintf(stderr, "overlayfs clone of dir failed\n");
+               goto out;
+       }
+
+       // Now create an overlayfs clone of the overlayfs clone
+       if ((c3 = c2->clone(c2, "clonetest-o2", NULL, LXC_CLONE_SNAPSHOT, "overlayfs", NULL, 0)) == NULL) {
+               fprintf(stderr, "overlayfs clone of overlayfs failed\n");
+               goto out;
+       }
+
+       fprintf(stderr, "all clone tests passed for %s\n", c->name);
+       ret = 0;
+
+out:
+       if (c3) {
+               lxc_container_put(c3);
+       }
+       if (c2) {
+               //c2->destroy(c2); // keep around to verify manuall
+               lxc_container_put(c2);
+       }
+       if (c) {
+               //c->destroy(c);
+               lxc_container_put(c);
+       }
+       exit(ret);
+}