Re: Review Request 48796: AURORA-1458 Add tier into the UI "show config" summary

2016-06-21 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48796/#review138871 --- Ship it! Ship It! - Maxim Khutornenko On June 16, 2016, 4:21

Review Request 49048: AURORA-1710 Make 'tier' required and remove support for 'production' flag in Job configuration - CLI changes

2016-06-21 Thread Mehrdad Nurolahzade
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49048/ --- Review request for Aurora, Joshua Cohen and Maxim Khutornenko. Repository: auro

Re: Review Request 49048: AURORA-1710 Make 'tier' required and remove support for 'production' flag in Job configuration - CLI changes

2016-06-21 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49048/#review138964 --- src/main/python/apache/aurora/config/thrift.py (lines 231 - 258)

Re: Review Request 49048: AURORA-1710 Make 'tier' required and remove support for 'production' flag in Job configuration - CLI changes

2016-06-21 Thread Mehrdad Nurolahzade
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49048/ --- (Updated June 21, 2016, 4:09 p.m.) Review request for Aurora, Joshua Cohen and

Re: Review Request 49048: AURORA-1710 Make 'tier' required and remove support for 'production' flag in Job configuration - CLI changes

2016-06-21 Thread Mehrdad Nurolahzade
> On June 21, 2016, 4:02 p.m., Maxim Khutornenko wrote: > > src/main/python/apache/aurora/config/thrift.py, lines 231-258 > > > > > > A better place for this would be `get_job_config()` in > > apache/aurora/client/c

Re: Review Request 49048: AURORA-1710 Make 'tier' required and remove support for 'production' flag in Job configuration - CLI changes

2016-06-21 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49048/#review138975 --- Ship it! Master (73dd2a8) is green with this patch. ./build-s