> On Jan. 12, 2015, 11:33 p.m., Kevin Sweeney wrote:
> > src/main/python/apache/aurora/executor/common/resource_manager.py, lines 
> > 24-25
> > <https://reviews.apache.org/r/29828/diff/1/?file=818231#file818231line24>
> >
> >     Mind adding a TODO to drop the cpu and ram portions of this? Those are 
> > enforced by mesos.
> 
> Brian Wickman wrote:
>     They're here for cpu_percent and ram_percent to be exported.  Does the 
> mesos slave export usage stats relative to reservation?
> 
> Kevin Sweeney wrote:
>     It doesn't, though it does export the raw metrics that make it possible. 
> See https://issues.apache.org/jira/browse/MESOS-1458 for a discussion on why 
> friendlier numbers are not exported.

OK then I'm inclined to keep them here for now.  Will add a TODO referencing 
MESOS-1458 which we can then delete if MESOS-1458 is closed wontfix.


- Brian


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29828/#review67743
-----------------------------------------------------------


On Jan. 12, 2015, 11:40 p.m., Brian Wickman wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/29828/
> -----------------------------------------------------------
> 
> (Updated Jan. 12, 2015, 11:40 p.m.)
> 
> 
> Review request for Aurora, Joshua Cohen and Kevin Sweeney.
> 
> 
> Bugs: AURORA-1002
>     https://issues.apache.org/jira/browse/AURORA-1002
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> Patch ResourceManager into OSS Aurora.
> 
> 
> Diffs
> -----
> 
>   src/main/python/apache/aurora/executor/bin/BUILD 
> 0434c7ba480a80a9722e626775cf5c3adbc3e68e 
>   src/main/python/apache/aurora/executor/bin/thermos_executor_main.py 
> 9df9b4b79c0c7d29c5088409bf15c0d32a621df0 
>   src/main/python/apache/aurora/executor/common/BUILD 
> 142ec0d42b2cb31dcef1e32768e9ea5286355913 
>   src/main/python/apache/aurora/executor/common/resource_manager.py 
> PRE-CREATION 
>   src/test/python/apache/aurora/executor/common/BUILD 
> 2bf6b2df4761acaa88c38868f03e686ec6b42ab7 
>   src/test/python/apache/aurora/executor/common/test_resource_manager.py 
> PRE-CREATION 
>   
> src/test/python/apache/aurora/executor/common/test_resource_manager_integration.py
>  PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/29828/diff/
> 
> 
> Testing
> -------
> 
> Ran e2e v1 tests which broke, but not because of the RM wiring.  Running e2e2 
> tests now.
> 
> 
> Thanks,
> 
> Brian Wickman
> 
>

Reply via email to