Re: Review Request 26898: Add hostname option to docker info

2014-10-31 Thread Benjamin Hindman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26898/#review59348 --- Ship it! Ship It! - Benjamin Hindman On Oct. 29, 2014, 12:24

Re: Review Request 26898: Add hostname option to docker info

2014-10-28 Thread Timothy Chen
On Oct. 26, 2014, 12:23 a.m., Benjamin Hindman wrote: include/mesos/mesos.proto, lines 889-890 https://reviews.apache.org/r/26898/diff/1/?file=724989#file724989line889 Shouldn't this be a generic ContainerInfo field? hmm, didn't think about that, but I think you're right it's

Re: Review Request 26898: Add hostname option to docker info

2014-10-28 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26898/ --- (Updated Oct. 29, 2014, 12:24 a.m.) Review request for mesos and Benjamin

Re: Review Request 26898: Add hostname option to docker info

2014-10-25 Thread Benjamin Hindman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26898/#review58539 --- include/mesos/mesos.proto

Review Request 26898: Add hostname option to docker info

2014-10-17 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26898/ --- Review request for mesos and Benjamin Hindman. Repository: mesos-git