----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66537/#review203817 -----------------------------------------------------------
Can you include an end-to-end test? https://github.com/apache/aurora/blob/master/src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh api/src/main/thrift/org/apache/aurora/gen/api.thrift Lines 191 (patched) <https://reviews.apache.org/r/66537/#comment286149> Should we just make this a string so that future network types supported by Docker does not need an Aurora update? Since Mesos's `DockerInfo` already type checks via enums this is probably leaking abstraction and is not needed? api/src/main/thrift/org/apache/aurora/gen/api.thrift Lines 238 (patched) <https://reviews.apache.org/r/66537/#comment286130> nit - s/Hub/registry/ src/main/java/org/apache/aurora/scheduler/mesos/MesosTaskFactory.java Lines 221-227 (patched) <https://reviews.apache.org/r/66537/#comment286147> Not an expert on how Docker entry points work. The Mesos documentation on invoking a Docker entry-point seems to differ here? Or I am not understanding this. http://mesos.apache.org/documentation/latest/docker-containerizer/#commandinfo-to-run-docker-images src/main/java/org/apache/aurora/scheduler/mesos/MesosTaskFactory.java Lines 293 (patched) <https://reviews.apache.org/r/66537/#comment286148> Should this be resolved when interpolating the pystachio schema in the client? What about the other templated namespaces? http://aurora.apache.org/documentation/latest/reference/configuration/#template-namespaces - Santhosh Kumar Shanmugham On May 2, 2018, 6:06 p.m., Steve Salevan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/66537/ > ----------------------------------------------------------- > > (Updated May 2, 2018, 6:06 p.m.) > > > Review request for Aurora, Renan DelValle and Stephan Erb. > > > Bugs: AURORA-1982 > https://issues.apache.org/jira/browse/AURORA-1982 > > > Repository: aurora > > > Description > ------- > > Hey there! > > Over here at Spine we've added a few enhancements to Aurora's Docker > functionality to support several of our use cases. We'd like to submit these > back up to the upstream to support the great work y'all are doing in this > space. > > Here's what's included in this RB: > > * Support for the force_pull flag to help ensure container freshness > * Overrides for a Docker --entrypoint which can be specified on the job > * Support for alternative Docker networks (defaults to HOST) > * Support for user Docker networks > > We currently use Aurora to schedule Docker containers without the use of > Thermos, so we've added support for server-side templating of common Thermos > variables into Docker executor's parameters for this purpose. > > This change modifies Aurora's api.thrift with several new optional fields, > and all added code handles their absence gracefully, so no backfills have > been added. We've threaded these schema changes through to the Python Aurora > client alongside support for the Mesos Fetcher URIs already supported > server-side. > > Let me know what you think and thanks! > > > Diffs > ----- > > .gitignore 9ce74ebbbc57b77d912eaa573a8fb18ed4aa3c15 > api/src/main/thrift/org/apache/aurora/gen/api.thrift > ef754e32172e7490a47a13e7b526f243ffa3efeb > > src/main/java/org/apache/aurora/scheduler/configuration/ConfigurationManager.java > 4073229b74d0e0e7fd31552bd96894ceb8a0971a > src/main/java/org/apache/aurora/scheduler/mesos/MesosTaskFactory.java > bcb2bbf882f43d813dd26c746d806e78bae6bcf3 > src/main/python/apache/aurora/config/schema/base.py > a629bcd1261e5959da0a8458a55545d4e2c2a7a5 > src/main/python/apache/aurora/config/thrift.py > 6d2dde6e964daa68bf6f0e5bbbffecc5bd8c0431 > src/test/java/org/apache/aurora/scheduler/base/TaskTestUtil.java > 778148a7c033cba9004954cabc33a2b1d003dccf > > src/test/java/org/apache/aurora/scheduler/configuration/ConfigurationManagerTest.java > 749ffeac6cb851f32bba7606390203d7a046a0e6 > > src/test/java/org/apache/aurora/scheduler/mesos/MesosTaskFactoryImplTest.java > 686087ef858b8a5a8e956d82a7bd692f7be28b12 > src/test/java/org/apache/aurora/scheduler/thrift/ThriftIT.java > 40851c419e4d62e6545959eebc0ce144fdecc697 > src/test/python/apache/aurora/client/cli/test_inspect.py > e4f43d0573c7862adc9bc679f4cea40cc76eac38 > src/test/python/apache/aurora/config/test_thrift.py > 8e1d0e177959af12b97bdd1cd47845b72bc12fe1 > > src/test/resources/org/apache/aurora/scheduler/storage/durability/goldens/current/saveCronJob > 88e1c36a1aa2d192b95963f7aa36e243a447e4af > > src/test/resources/org/apache/aurora/scheduler/storage/durability/goldens/current/saveJobUpdate > 32fdcdacde58345cdd6c4b449b82c0c90c2b2aae > > src/test/resources/org/apache/aurora/scheduler/storage/durability/goldens/current/saveTasks > 4323031ec6bd128576c2a43ebc11f04a9f046e2f > > > Diff: https://reviews.apache.org/r/66537/diff/4/ > > > Testing > ------- > > Unit tests for all new functionality have been added and run cleanly > alongside checkstyle/static analysis tooling. > > Operational/functional testing completed within Spine infrastructure. > > > Thanks, > > Steve Salevan > >
