----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44366/#review122039 -----------------------------------------------------------
Patch looks great! Reviews applied: [44360, 44361, 44363, 44364, 44365, 44366] Passed command: export OS='ubuntu:14.04' CONFIGURATION='--verbose' COMPILER='gcc' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; ./support/docker_build.sh - Mesos ReviewBot On March 4, 2016, 1:11 a.m., Kevin Klues wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/44366/ > ----------------------------------------------------------- > > (Updated March 4, 2016, 1:11 a.m.) > > > Review request for mesos, Ben Mahler and Rob Todd. > > > Bugs: MESOS-4865 > https://issues.apache.org/jira/browse/MESOS-4865 > > > Repository: mesos > > > Description > ------- > > Currently we can either infer the number of GPUs to offer as resources > from the --nvidia_gpus flag, or pass the amount explicity as a "gpus" > resource. In the future, we will generalize this via autodiscovery of > gpus and support for gpus types other than Nvidia. > > > Diffs > ----- > > include/mesos/resources.hpp 388c4f383b77be490cb2debb64a734c6d6e4a176 > include/mesos/v1/resources.hpp 64ad8bf08230aeaa173325364c91b765f091210e > src/common/resources.cpp cf0707209143084d86aff5e4427846f461479c54 > src/slave/constants.hpp bcbb1401aa8f9f04c4f9256bb4f560e18d8994e0 > src/slave/constants.cpp 0f0d8e4b079d136d250f83cfc768de8c98b8bee2 > src/slave/containerizer/containerizer.cpp > f6fc7863d0c215611f170dc0c89aa229407b5137 > src/v1/resources.cpp 296c4c2392384a2fcc4f2c1843980ff97e71114d > > Diff: https://reviews.apache.org/r/44366/diff/ > > > Testing > ------- > > > Thanks, > > Kevin Klues > >