Skip to content

Commit

Permalink
lxc-start-ephemeral: handle the overlayfs workdir option (v2)
Browse files Browse the repository at this point in the history
We fixed this some time ago for basic lxc-start, but never did
lxc-start-ephemeral.

Since the lxc-start patches were pushed, Miklos has given us a
way to detect whether we need the workdir= option.  So the
bdev.c code could be simplified to check for "overlay\n" in
/proc/filesystems just as lxc-start-ephemeral does.  This
patch doesn't do that.

Changelog (v2):
	1. use 'overlay' fstype for new overlay upstream module
	2. avoid using unneeded readlines().

Signed-off-by: Serge Hallyn <serge.hallyn@ubuntu.com>
Acked-by: Stéphane Graber <stgraber@ubuntu.com>
  • Loading branch information
hallyn authored and stgraber committed Aug 28, 2015
1 parent 4633d67 commit 47ee663
Showing 1 changed file with 22 additions and 1 deletion.
23 changes: 22 additions & 1 deletion src/lxc/lxc-start-ephemeral.in
Expand Up @@ -200,6 +200,14 @@ if orig.get_config_item("lxc.mount"):
# Setup an overlay for anything remaining
overlay_dirs += [(fields[0], dest_mount)]

# do we have the new overlay fs which requires workdir, or the older
# overlayfs which does not?
have_new_overlay = False
with open("/proc/filesystems", "r") as fd:
for line in fd:
if line == "nodev\toverlay\n":
have_new_overlay = True

# Generate pre-mount script
with open(os.path.join(dest_path, "pre-mount"), "w+") as fd:
os.fchmod(fd.fileno(), 0o755)
Expand All @@ -212,13 +220,26 @@ LXC_NAME="%s"
count = 0
for entry in overlay_dirs:
target = "%s/delta%s" % (dest_path, count)
workdir = "%s/work%s" % (dest_path, count)
fd.write("mkdir -p %s %s\n" % (target, entry[1]))
if have_new_overlay:
fd.write("mkdir -p %s\n" % workdir)

if args.storage_type == "tmpfs":
fd.write("mount -n -t tmpfs -o mode=0755 none %s\n" % (target))
if have_new_overlay:
fd.write("mount -n -t tmpfs -o mode=0755 none %s\n" % (workdir))

if args.union_type == "overlayfs":
fd.write("mount -n -t overlayfs"
if have_new_overlay:
fd.write("mount -n -t overlay"
" -oupperdir=%s,lowerdir=%s,workdir=%s none %s\n" % (
target,
entry[0],
workdir,
entry[1]))
else:
fd.write("mount -n -t overlayfs"
" -oupperdir=%s,lowerdir=%s none %s\n" % (
target,
entry[0],
Expand Down

0 comments on commit 47ee663

Please sign in to comment.