You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by ji...@apache.org on 2016/06/22 16:56:54 UTC
[3/3] mesos git commit: Used non recursive chown for persistent
volumes.
Used non recursive chown for persistent volumes.
Review: https://reviews.apache.org/r/49045
Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/0eb33980
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/0eb33980
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/0eb33980
Branch: refs/heads/master
Commit: 0eb33980a469f18d1bf4faecde1725651c563ee2
Parents: e01a6b1
Author: Jie Yu <yu...@gmail.com>
Authored: Tue Jun 21 13:45:35 2016 -0700
Committer: Jie Yu <yu...@gmail.com>
Committed: Wed Jun 22 09:56:44 2016 -0700
----------------------------------------------------------------------
src/slave/containerizer/docker.cpp | 2 +-
src/slave/containerizer/mesos/isolators/filesystem/linux.cpp | 2 +-
src/slave/containerizer/mesos/isolators/filesystem/posix.cpp | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/mesos/blob/0eb33980/src/slave/containerizer/docker.cpp
----------------------------------------------------------------------
diff --git a/src/slave/containerizer/docker.cpp b/src/slave/containerizer/docker.cpp
index a87a87a..514268f 100644
--- a/src/slave/containerizer/docker.cpp
+++ b/src/slave/containerizer/docker.cpp
@@ -518,7 +518,7 @@ Try<Nothing> DockerContainerizerProcess::updatePersistentVolumes(
<< source << "' with uid " << s.st_uid
<< " and gid " << s.st_gid;
- Try<Nothing> chown = os::chown(s.st_uid, s.st_gid, source, true);
+ Try<Nothing> chown = os::chown(s.st_uid, s.st_gid, source, false);
if (chown.isError()) {
return Error(
"Failed to change the ownership of the persistent volume at '" +
http://git-wip-us.apache.org/repos/asf/mesos/blob/0eb33980/src/slave/containerizer/mesos/isolators/filesystem/linux.cpp
----------------------------------------------------------------------
diff --git a/src/slave/containerizer/mesos/isolators/filesystem/linux.cpp b/src/slave/containerizer/mesos/isolators/filesystem/linux.cpp
index 37b0239..adacde1 100644
--- a/src/slave/containerizer/mesos/isolators/filesystem/linux.cpp
+++ b/src/slave/containerizer/mesos/isolators/filesystem/linux.cpp
@@ -562,7 +562,7 @@ Future<Nothing> LinuxFilesystemIsolatorProcess::update(
<< source << "' with uid " << s.st_uid
<< " and gid " << s.st_gid;
- Try<Nothing> chown = os::chown(s.st_uid, s.st_gid, source, true);
+ Try<Nothing> chown = os::chown(s.st_uid, s.st_gid, source, false);
if (chown.isError()) {
return Failure(
"Failed to change the ownership of the persistent volume at '" +
http://git-wip-us.apache.org/repos/asf/mesos/blob/0eb33980/src/slave/containerizer/mesos/isolators/filesystem/posix.cpp
----------------------------------------------------------------------
diff --git a/src/slave/containerizer/mesos/isolators/filesystem/posix.cpp b/src/slave/containerizer/mesos/isolators/filesystem/posix.cpp
index d45166f..0809e8e 100644
--- a/src/slave/containerizer/mesos/isolators/filesystem/posix.cpp
+++ b/src/slave/containerizer/mesos/isolators/filesystem/posix.cpp
@@ -195,7 +195,7 @@ Future<Nothing> PosixFilesystemIsolatorProcess::update(
<< original << "' with uid " << s.st_uid
<< " and gid " << s.st_gid;
- Try<Nothing> chown = os::chown(s.st_uid, s.st_gid, original, true);
+ Try<Nothing> chown = os::chown(s.st_uid, s.st_gid, original, false);
if (chown.isError()) {
return Failure(
"Failed to change the ownership of the persistent volume at '" +