mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Gilbert Song <>
Subject Re: Review Request 54212: Fixed overlay backend provisioning multi images symlink.
Date Tue, 24 Jan 2017 20:34:39 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated Jan. 24, 2017, 12:34 p.m.)

Review request for mesos, Avinash sridharan, Artem Harutyunyan, Jie Yu, Qian Zhang, and Zhitao

Bugs: MESOS-6653

Repository: mesos

Description (updated)

Since the fix of MESOS-6000, symlinks are used in overlayfs
backend to shorten the arguments when mounting the rootfs.
E.g., '.../backends/overlay/links' is the symlink created
for a provisioned image. It becomes problematic if a
container image is specified while some image volumes are
specified for the same container. An unique symlink is
needed for each image to be provisioned.

Please note that changing the symlinks directory would
still be backward compatible for legacy containers, since
the container backend directory will be removed anyway in

Diffs (updated)

  src/slave/containerizer/mesos/provisioner/backends/overlay.hpp 4d591c5f988d87e0e905f973df5ab15a3386d676

  src/slave/containerizer/mesos/provisioner/backends/overlay.cpp a6e6c8b84adadc24b915b45533d1789f8de758dd



make check

Verified that the unit test `ROOT_ImageInVolumeWithRootFilesystem` passed.


Gilbert Song

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message