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/03/01 22:15:54 UTC
mesos git commit: Added docker runtime isolator check if image
providers include docker.
Repository: mesos
Updated Branches:
refs/heads/master 4194e395d -> c7dea5888
Added docker runtime isolator check if image providers include docker.
Review: https://reviews.apache.org/r/44221/
Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/c7dea588
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/c7dea588
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/c7dea588
Branch: refs/heads/master
Commit: c7dea5888b4f5194e7d1c6abe0f379ab229e3039
Parents: 4194e39
Author: Gilbert Song <so...@gmail.com>
Authored: Tue Mar 1 13:15:23 2016 -0800
Committer: Jie Yu <yu...@gmail.com>
Committed: Tue Mar 1 13:15:23 2016 -0800
----------------------------------------------------------------------
src/slave/containerizer/mesos/provisioner/store.cpp | 8 ++++++++
1 file changed, 8 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/mesos/blob/c7dea588/src/slave/containerizer/mesos/provisioner/store.cpp
----------------------------------------------------------------------
diff --git a/src/slave/containerizer/mesos/provisioner/store.cpp b/src/slave/containerizer/mesos/provisioner/store.cpp
index 40bc00c..8324269 100644
--- a/src/slave/containerizer/mesos/provisioner/store.cpp
+++ b/src/slave/containerizer/mesos/provisioner/store.cpp
@@ -69,6 +69,14 @@ Try<hashmap<Image::Type, Owned<Store>>> Store::create(const Flags& flags)
stores.put(imageType, store.get());
}
+ if (stores.contains(Image::DOCKER) &&
+ !strings::contains(flags.isolation, "docker/runtime")) {
+ EXIT(1)
+ << "Docker runtime isolator has to be specified if 'DOCKER' is included "
+ << "in 'image_providers'. Please add 'docker/runtime' to '--isolation' "
+ << "flags";
+ }
+
return stores;
}