Re: Review Request 44486: Exposing DSL defined variables to shell health checkers

2016-03-11 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44486/#review123202 --- Ship it! Ship It! - John Sirois On March 10, 2016, 11:59 a.m

Re: Review Request 44486: Exposing DSL defined variables to shell health checkers

2016-03-10 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44486/#review123018 --- Ship it! Ship It! - Bill Farner On March 10, 2016, 10:59 a.m

Re: Review Request 44486: Exposing DSL defined variables to shell health checkers

2016-03-10 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44486/#review122968 --- Ship it! Master (31a538f) is green with this patch. ./build-s

Re: Review Request 44486: Exposing DSL defined variables to shell health checkers

2016-03-10 Thread Dmitriy Shirchenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44486/ --- (Updated March 10, 2016, 6:59 p.m.) Review request for Aurora, John Sirois, Bil

Re: Review Request 44486: Exposing DSL defined variables to shell health checkers

2016-03-10 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44486/#review122943 --- Fix it, then Ship it! Modulo a missing test, this LGTM. I will

Re: Review Request 44486: Exposing DSL defined variables to shell health checkers

2016-03-08 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44486/#review122688 --- Ship it! Master (bab21ed) is green with this patch. ./build-s

Re: Review Request 44486: Exposing DSL defined variables to shell health checkers

2016-03-08 Thread Dmitriy Shirchenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44486/#review122683 --- @ReviewBot retry - Dmitriy Shirchenko On March 9, 2016, 5:34 a.

Re: Review Request 44486: Exposing DSL defined variables to shell health checkers

2016-03-08 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44486/#review122681 --- Master (bab21ed) is red with this patch. ./build-support/jenkins

Re: Review Request 44486: Exposing DSL defined variables to shell health checkers

2016-03-08 Thread Dmitriy Shirchenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44486/ --- (Updated March 9, 2016, 5:34 a.m.) Review request for Aurora, John Sirois, Bill

Re: Review Request 44486: Exposing DSL defined variables to shell health checkers

2016-03-08 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44486/#review122678 --- Master (bab21ed) is red with this patch. ./build-support/jenkins

Re: Review Request 44486: Exposing DSL defined variables to shell health checkers

2016-03-08 Thread Dmitriy Shirchenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44486/ --- (Updated March 9, 2016, 5:11 a.m.) Review request for Aurora, John Sirois, Bill

Re: Review Request 44486: Exposing DSL defined variables to shell health checkers

2016-03-08 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44486/#review122671 --- Master (bab21ed) is red with this patch. ./build-support/jenkins

Re: Review Request 44486: Exposing DSL defined variables to shell health checkers

2016-03-08 Thread Dmitriy Shirchenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44486/ --- (Updated March 9, 2016, 4:24 a.m.) Review request for Aurora, John Sirois, Bill