----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62692/ -----------------------------------------------------------
(Updated Sept. 30, 2017, 3:04 p.m.) Review request for Aurora and Stephan Erb. Changes ------- Fixed code style Repository: aurora Description ------- Moves the job environment validation to the scheduler, which can be enabled with the scheduler require_predefined_environments flag. This allows to have a consistent behavior when using the CLI and the API. In order to preserve backward compatibility, the validation is kept in the CLI and for the API it needs to be manually enabled in the scheduler. Diffs (updated) ----- src/main/java/org/apache/aurora/scheduler/app/AppModule.java 081dff2bda626f01ba222628b8d7d8afebbb0004 src/main/java/org/apache/aurora/scheduler/base/TaskTestUtil.java 186fa1b3a4780c0536fb486d50a33133258110cd src/main/java/org/apache/aurora/scheduler/configuration/ConfigurationManager.java 754fde0fdc976b673d78ae15d8ccd8c85b792373 src/main/python/apache/aurora/client/config.py 70c2c980309e18de576b251087cdfea00ac06b75 src/test/java/org/apache/aurora/scheduler/configuration/ConfigurationManagerTest.java 50d7499f4332a3feb0e2301cb707f2cea6bb2e98 src/test/java/org/apache/aurora/scheduler/thrift/ThriftIT.java 6b4b17f8dafd5c2d751dcda3080b476335f8aec0 Diff: https://reviews.apache.org/r/62692/diff/2/ Changes: https://reviews.apache.org/r/62692/diff/1-2/ Testing ------- Thanks, Mauricio Garavaglia