> On Feb. 29, 2020, 1:57 a.m., Greg Mann wrote: > > src/slave/slave.cpp > > Lines 3262 (patched) > > <https://reviews.apache.org/r/71858/diff/10/?file=2212019#file2212019line3262> > > > > We have validation which ensures that executor CPU and mem are always > > greater than or equal to the minimum values, so I think this could be a > > `CHECK()`? > > > > > > https://github.com/apache/mesos/blob/998aee66bfedd1fe15bb1e1fc43a637fe91662a5/src/master/validation.cpp#L1842-L1859
Unfortunately, in `task::internal::validateExecutor()`, if there is no CPU or memory in executor resources, we will just log a warning message but not return an error. https://github.com/apache/mesos/blob/1.9.0/src/master/validation.cpp#L1638-L1648 - Qian ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/71858/#review219697 ----------------------------------------------------------- On Feb. 25, 2020, 9:46 a.m., Qian Zhang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/71858/ > ----------------------------------------------------------- > > (Updated Feb. 25, 2020, 9:46 a.m.) > > > Review request for mesos, Andrei Budnik and Greg Mann. > > > Bugs: MESOS-10046 > https://issues.apache.org/jira/browse/MESOS-10046 > > > Repository: mesos > > > Description > ------- > > Set resource limits when launching executor container. > > > Diffs > ----- > > src/slave/slave.hpp 03279db082bdba23dbfeb2d93081a908e609aec2 > src/slave/slave.cpp cce275a504effae7a6b71dd333ce8a300d1ce5be > > > Diff: https://reviews.apache.org/r/71858/diff/10/ > > > Testing > ------- > > > Thanks, > > Qian Zhang > >
