-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/51879/
-----------------------------------------------------------
(Updated May 24, 2017, 7:56 p.m.)
Review request for mesos and Jiang Yan Xu.
Bugs: MESOS-6062
https://issues.apache.org/jira/browse/MESOS-6062
Repository: mesos
Description
-------
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. Note
that auto-detection of resources is allowed for all known resource
types represented in JSON format only. Auto-detection is not done when
the resources are represented in text format.
With this change, JSON representation for disk resources that do not
specify any value 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 still results in an invalid resource.
Diffs (updated)
-----
include/mesos/resources.hpp 74583e3c784f22b45cce81f07c5a69b339e1684d
include/mesos/v1/resources.hpp 132ef3eb03d335774ba13ecb39045128d0476954
src/common/resources.cpp 1d07f1a049ba3bb3f5d78f05031f33ba97e07e8c
src/slave/containerizer/containerizer.cpp
15ae0b33a5ea8f73a9a84081d9c310c1d59c3141
src/slave/containerizer/mesos/isolators/gpu/allocator.cpp
c288ad634b856702483b9751f41445308babd0c9
src/tests/persistent_volume_endpoints_tests.cpp
8c54372b7f6d94f0335561086f2a8cb90373e285
src/tests/reservation_endpoints_tests.cpp
8c195eb7d788fbca8722d66d81c77d399702160a
src/v1/resources.cpp 1bb5d0741c9f9ea59f34e92159a335661019444d
Diff: https://reviews.apache.org/r/51879/diff/13/
Changes: https://reviews.apache.org/r/51879/diff/12-13/
Testing
-------
Tests passed.
Thanks,
Anindya Sinha