----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44366/#review125994 -----------------------------------------------------------
Ship it! Went over this with kevin, we'll be making some minor adjustments to the comments. src/slave/containerizer/containerizer.cpp (line 105) <https://reviews.apache.org/r/44366/#comment188912> how about "millis" to try to make the intent of the modulo logic a bit more clear? src/slave/containerizer/containerizer.cpp (lines 117 - 119) <https://reviews.apache.org/r/44366/#comment188913> We would probably need this TODO in all of our nvidia gpu blocks :) - Ben Mahler On March 14, 2016, 7:39 a.m., Kevin Klues wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/44366/ > ----------------------------------------------------------- > > (Updated March 14, 2016, 7:39 a.m.) > > > Review request for mesos, Ben Mahler, Rob Todd, and Vikrama Ditya. > > > Bugs: MESOS-4865 > https://issues.apache.org/jira/browse/MESOS-4865 > > > Repository: mesos > > > Description > ------- > > Currently, we enforce that the number of GPUs specified in the 'gpus' > resource parameter equal the number of GPUs passed in via the > --nvidia_gpu_devices flag. In the future, we will generalize this via > autodiscovery of GPUs and support for GPU types other than Nvidia. > > > Diffs > ----- > > include/mesos/resources.hpp bb343ad852576a75615a93ef850b413bf77698e0 > include/mesos/v1/resources.hpp 719110fbbf39f1755460ac0b32e3893656054a4e > src/common/resources.cpp 1f23dc83f7330c305a836d698f114b7eaf3d7ba1 > src/slave/containerizer/containerizer.cpp > f6fc7863d0c215611f170dc0c89aa229407b5137 > src/v1/resources.cpp c6f125ec317e2da537a6456f5cff2da0a48701d8 > > Diff: https://reviews.apache.org/r/44366/diff/ > > > Testing > ------- > > ./bin/mesos-slave.sh --master=127.0.0.1:5050 --resources="gpus:string" > Failed to determine slave resources: Bad type for resource gpus value string > type TEXT > > ./bin/mesos-slave.sh --master=127.0.0.1:5050 --resources="gpus:4.9" > Failed to determine slave resources: The `gpus` resource must specified as an > unsigned integer > > ./bin/mesos-slave.sh --master=127.0.0.1:5050 --resources="gpus:4.0" > Failed to determine slave resources: When specifying the `gpus` resource, you > must also specify a list of GPUs via the `--nvidia_gpu_devices` flag > > ./bin/mesos-slave.sh --master=127.0.0.1:5050 --resources="gpus:4.0" > --nvidia_gpu_devices=1,2,3 > Failed to determine slave resources: The number of GPUs passed in the > `--nvidia_gpu_devices` flag must match the number of GPUs specified in the > `gpus` resource > > ./bin/mesos-slave.sh --master=127.0.0.1:5050 --resources="gpus:4.0" > --nvidia_gpu_devices=1,2,3,4 > SUCCESS > > **NOTE**: I didn't set the `--isolation` flag here, so the agent actually > started up correctly (i.e. it didn't error out saying that the Nvidia GPU > isolator is currently unsupported). This is the correct behaviour, and it > properly exercises the code added in this patch. > > > Thanks, > > Kevin Klues > >