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