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

Review request for Aurora.


Repository: aurora


Description
-------

What was done:
==============
Added support for custom executors in the Scheduler via a config file. 
Removed command line arguments that were moved over to the config file.

Future:
=======
Extending the client to support custom executors.

Caveats:
========
This contains initial config file with support for thermos and limited support 
for the mesos commandline executor. Mesos-command line executor needs support 
from the client side in order to function at a better capacity. 

Currently, this uses the current client to launch both tasks, meaning as long 
as the client sends a thrift call, the scheduler will schedule a task, be it a 
mesos-command task with a preconfigured command temporarily set in the config 
file or a custom executor task. 

*Support for custom executors in the client must be added in order to fully 
utilize this feature.*


Diffs
-----

  build.gradle 78159932388046883494dda97b4a86c06772d26d 
  examples/vagrant/upstart/aurora-scheduler.conf 
1c2390f0248e91e65a548e67f6af1be8d2526b0a 
  gradle/wrapper/gradle-wrapper.jar 085a1cdc27db1185342f15a00441734e74fe3735 
  gradle/wrapper/gradle-wrapper.properties 
e713793f7d63e65ea68e091b959ba6a68352b48f 
  gradlew dd29ec5eae9a34d216475d6e5b5d8c810885aa76 
  src/dist/etc/executors.json PRE-CREATION 
  src/main/java/org/apache/aurora/scheduler/app/SchedulerMain.java 
554a380bdb4ef69561259cdbfbc361694041571e 
  
src/main/java/org/apache/aurora/scheduler/configuration/ExecutorSettingsLoader.java
 PRE-CREATION 
  src/main/java/org/apache/aurora/scheduler/mesos/ExecutorSettings.java 
325f55640648151ae19e0c18c6961aeff10bfac3 
  src/main/java/org/apache/aurora/scheduler/mesos/MesosTaskFactory.java 
c0d165ad34e46653dad95918e0058ebd3f2ee57f 

Diff: https://reviews.apache.org/r/36289/diff/


Testing
-------

./gradlew test
895 tests completed, 44 failed, 3 skipped


Thanks,

Renan DelValle

Reply via email to