----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26333/ -----------------------------------------------------------
(Updated Oct. 4, 2014, 12:46 a.m.) Review request for Aurora, Joe Smith and Kevin Sweeney. Changes ------- Address kevin's feedback. Integrate runner exit status into compute_status so that statuses are actually propagated to mesos task statuses. Bugs: AURORA-175 https://issues.apache.org/jira/browse/AURORA-175 Repository: aurora Description ------- Adds an exit status contract between the ThermosTaskRunner and the thermos_runner.pex so that we can discriminate between losses / failures. Diffs (updated) ----- src/main/python/apache/aurora/executor/BUILD 5f0ef7c12479fe2bbdebc3dacdd6ce5aa4910413 src/main/python/apache/aurora/executor/thermos_runner.py 396753e0f95ce1342dba05ea1897a6842e6547df src/main/python/apache/aurora/executor/thermos_statuses.py PRE-CREATION src/main/python/apache/aurora/executor/thermos_task_runner.py 7b346e253677ee9b42c57782f7f67ff63b6a0083 src/test/python/apache/aurora/executor/test_thermos_executor.py f6ca4dfd0fd262361709361c48c93799837e0a54 src/test/python/apache/aurora/executor/test_thermos_task_runner.py 8e4042b29dce8a815c5995b9e2701c95fe3f034b Diff: https://reviews.apache.org/r/26333/diff/ Testing ------- Added tests. Refers to Process.UnknownUserError which is added in https://reviews.apache.org/r/25972/ but not available here. Thanks, Brian Wickman