> 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?

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)
```


- Kevin


-----------------------------------------------------------
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
> 
>

Reply via email to