Re: Review Request 30187: Remove support for cluster metadata in YAML format.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30187/#review70419 --- Ship it! Ship It! - Brian Wickman On Jan. 28, 2015, 8:26 p.m., B

Re: Review Request 30187: Remove support for cluster metadata in YAML format.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30187/#review70281 --- Brian - ping? - Bill Farner On Jan. 28, 2015, 8:26 p.m., Bill Far

Re: Review Request 30187: Remove support for cluster metadata in YAML format.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30187/#review70079 --- Ship it! Master (5902574) is green with this patch. ./build-suppo

Re: Review Request 30187: Remove support for cluster metadata in YAML format.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30187/ --- (Updated Jan. 28, 2015, 8:26 p.m.) Review request for Aurora, Brian Wickman and

Re: Review Request 30187: Remove support for cluster metadata in YAML format.

> On Jan. 26, 2015, 8:40 p.m., Brian Wickman wrote: > > src/main/python/apache/aurora/common/clusters.py, line 42 > > > > > > wait -- the # noqa is necessary, otherwise checkstyle will fail. Reverted - Bill ---

Re: Review Request 30187: Remove support for cluster metadata in YAML format.

> On Jan. 26, 2015, 8:16 p.m., Kevin Sweeney wrote: > > src/main/python/apache/aurora/common/clusters.py, line 42 > > > > > > For the purposes of sheparding this review along would you consider > > moving this to anoth

Re: Review Request 30187: Remove support for cluster metadata in YAML format.

> On Jan. 26, 2015, 12:16 p.m., Kevin Sweeney wrote: > > src/main/python/apache/aurora/common/clusters.py, line 42 > > > > > > For the purposes of sheparding this review along would you consider > > moving this to anot

Re: Review Request 30187: Remove support for cluster metadata in YAML format.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30187/#review69668 --- src/main/python/apache/aurora/common/clusters.py

Re: Review Request 30187: Remove support for cluster metadata in YAML format.

> On Jan. 26, 2015, 8:16 p.m., Kevin Sweeney wrote: > > src/main/python/apache/aurora/common/clusters.py, line 42 > > > > > > For the purposes of sheparding this review along would you consider > > moving this to anoth

Re: Review Request 30187: Remove support for cluster metadata in YAML format.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30187/#review69663 --- src/main/python/apache/aurora/common/clusters.py

Re: Review Request 30187: Remove support for cluster metadata in YAML format.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30187/#review69478 --- Master (e3b8905) is red with this patch. ./build-support/jenkins/b

Re: Review Request 30187: Remove support for cluster metadata in YAML format.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30187/#review69469 --- @ReviewBot retry - Bill Farner On Jan. 22, 2015, 9:09 p.m., Bill

Re: Review Request 30187: Remove support for cluster metadata in YAML format.

> On Jan. 22, 2015, 9:30 p.m., Brian Wickman wrote: > > I think we should leave the yaml code but drop the PyYAML dependency from > > the client requirements. (And inject it into the test -- possibly one test > > with and one test without to make sure the try/except also functions > > correct

Re: Review Request 30187: Remove support for cluster metadata in YAML format.

> On Jan. 22, 2015, 9:30 p.m., Brian Wickman wrote: > > I think we should leave the yaml code but drop the PyYAML dependency from > > the client requirements. (And inject it into the test -- possibly one test > > with and one test without to make sure the try/except also functions > > correct

Re: Review Request 30187: Remove support for cluster metadata in YAML format.

> On Jan. 22, 2015, 1:30 p.m., Brian Wickman wrote: > > I think we should leave the yaml code but drop the PyYAML dependency from > > the client requirements. (And inject it into the test -- possibly one test > > with and one test without to make sure the try/except also functions > > correct

Re: Review Request 30187: Remove support for cluster metadata in YAML format.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30187/#review69270 --- Master (116ee2d) is red with this patch. ./build-support/jenkins/b

Re: Review Request 30187: Remove support for cluster metadata in YAML format.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30187/#review69268 --- Ship it! Ship It! - Zameer Manji On Jan. 22, 2015, 1:09 p.m., Bi

Re: Review Request 30187: Remove support for cluster metadata in YAML format.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30187/#review69269 --- I think we should leave the yaml code but drop the PyYAML dependency

Review Request 30187: Remove support for cluster metadata in YAML format.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30187/ --- Review request for Aurora, Brian Wickman and Zameer Manji. Bugs: AURORA-1029