> On Aug. 26, 2016, 12:31 a.m., Jie Yu wrote: > > src/tests/containerizer/isolator_tests.cpp, lines 215-218 > > <https://reviews.apache.org/r/51405/diff/2/?file=1486148#file1486148line215> > > > > Can you call CreateSlaveFlags instead? > > Kevin Klues wrote: > I tried doing this and it gave me the error: > ``` > ../../src/tests/containerizer/isolator_tests.cpp: In member function > ‘virtual void > mesos::internal::tests::CpuIsolatorTest_UserCpuUsage_Test<gtest_TypeParam_>::TestBody()’: > ../../src/tests/containerizer/isolator_tests.cpp:212:41: error: there are > no arguments to ‘CreateSlaveFlags’ that depend on a template parameter, so a > declaration of ‘CreateSlaveFlags’ must be available [-fpermissive] > slave::Flags flags = CreateSlaveFlags(); > ^ > ../../src/tests/containerizer/isolator_tests.cpp:212:41: note: (if you > use ‘-fpermissive’, G++ will accept your code, but allowing the use of an > undeclared name is deprecated) > ```
Try `this->CreateSlaveFlags()` - Jie ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51405/#review146896 ----------------------------------------------------------- On Aug. 25, 2016, 11:28 p.m., Kevin Klues wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51405/ > ----------------------------------------------------------- > > (Updated Aug. 25, 2016, 11:28 p.m.) > > > Review request for mesos and Jie Yu. > > > Bugs: MESOS-6050 > https://issues.apache.org/jira/browse/MESOS-6050 > > > Repository: mesos > > > Description > ------- > > Updated a few more tests to create a temporary 'runtime_dir'. > > > Diffs > ----- > > src/tests/containerizer/isolator_tests.cpp > 2725eb0f76f4e2668381c0d6686a99649f4f9f25 > src/tests/containerizer/mesos_containerizer_tests.cpp > c0dda11c18a770f594b9dcf2e506ec624706dbbd > > Diff: https://reviews.apache.org/r/51405/diff/ > > > Testing > ------- > > > Thanks, > > Kevin Klues > >
