----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51879/ -----------------------------------------------------------
(Updated Sept. 26, 2016, 6:51 a.m.) Review request for mesos and Jiang Yan Xu. Changes ------- Added support for auto detection when value is not specified in `flags.resources` for all known resource types (except gpus). Summary (updated) ----------------- Autodetect value of resource when not specified in static resources. Bugs: MESOS-6062 https://issues.apache.org/jira/browse/MESOS-6062 Repository: mesos Description (updated) ------- When static resources indicate resources with a positive size, we use that for the resources on the agent. However, --resources can include resources with no size, which indicates that mesos agent determine the size of those resources from the agent and uses that information. For disk resources, this is not allowed for PATH disks since PATH disks can be carved into smaller chunks and we cannot assume that the whole physical disk is available to the PATH disk. With this change, JSON or textual representation for disk resources that specify scalar value of 0 would not result in an error, but those resources will not be accounted for until a valid size is determined for such resources. A scalar value of -1 in JSON or textual formats still results in an invalid resource. Diffs (updated) ----- include/mesos/resources.hpp 3ef8cacee529addc745b4aeb6398d7606c61b749 include/mesos/v1/resources.hpp ef56b4960b103a3efd916fab64796aa334ba44c6 src/common/resources.cpp 0774ff0669e831494d5b12b88e19dfa0a4a3f757 src/slave/containerizer/containerizer.cpp d46882baa904fd439bffb23c324828b777228f1c src/v1/resources.cpp 62a644ebbd13cfc0862bd118ba16c43e0f6aaf90 Diff: https://reviews.apache.org/r/51879/diff/ Testing ------- Tests passed. Thanks, Anindya Sinha