Re: Review Request 30263: Added test for CRAM-MD5 support of SASL within configuration phase.

2015-01-25 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30263/#review69589 --- Patch looks great! Reviews applied: [30263] All tests passed. - M

Review Request 30263: Added test for CRAM-MD5 support of SASL within configuration phase.

2015-01-25 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30263/ --- Review request for mesos and Cody Maloney. Bugs: MESOS-2165 https://issues.

GPU computing resource add into Mesos

2015-01-25 Thread Chester Kuo
Hi All I'd like to extend and add new feature into Mesos to support GPU resource allocation, so we can put OpenCL application/framework on top of Mesos and make it write once run across cluster. Why choose OpenCL, due to it was widely supported by Intel , Nvidia, AMD, Qualcomm GPGPU, so we may e

Seeking nominations for #MesosCon 2015 keynote speakers

2015-01-25 Thread Dave Lester
Hi All, The #MesosCon[1] program committee is actively seeking nominations for keynote speakers. This year's #MesosCon will take place in Seattle, August 20-21, 2015. If you have any recommendations, please share them by filling out the following form: https://docs.google.com/forms/d/15zG-S7h_Se

Re: Review Request 30173: Created fetcher cache tests.

2015-01-25 Thread Bernd Mathiske
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30173/ --- (Updated Jan. 25, 2015, 9:47 a.m.) Review request for mesos, Adam B, Benjamin H

Re: Review Request 30037: Introduced fetcher cache actions

2015-01-25 Thread Bernd Mathiske
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30037/ --- (Updated Jan. 25, 2015, 9:44 a.m.) Review request for mesos, Adam B, Benjamin H

Re: Review Request 30036: Introduced caching fields to command URI and fetcher parameter protobufs

2015-01-25 Thread Bernd Mathiske
> On Jan. 23, 2015, 4:16 p.m., Adam B wrote: > > src/slave/containerizer/fetcher.cpp, line 253 > > > > > > Do we also need to check for empty-string user here, or valid path > > name, or valid user? Or is that validat

Re: Review Request 30039: Enabled fetcher cache actions in mesos fetcher program

2015-01-25 Thread Bernd Mathiske
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30039/ --- (Updated Jan. 25, 2015, 9:21 a.m.) Review request for mesos, Adam B, Benjamin H

Re: mesos-dns

2015-01-25 Thread Christos Kozyrakis
Hi Tom, the protobuf changes for service discovery will hopefully make it to the 0.22 release and we will update the A and SRV record generator to take advantage of them. We still have to decide what naming schemes to support based on this information. I suspect that we will have to support a few

Re: Review Request 30173: Created fetcher cache tests.

2015-01-25 Thread Bernd Mathiske
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30173/ --- (Updated Jan. 25, 2015, 8:49 a.m.) Review request for mesos, Adam B, Benjamin H

Re: Review Request 30173: Created fetcher cache tests.

2015-01-25 Thread Bernd Mathiske
> On Jan. 22, 2015, 11:19 a.m., Timothy Chen wrote: > > src/tests/mesos.cpp, line 614 > > > > > > If you call pop with an empty task status queue, you actually remove > > the temporaroy holder in your constructor. > >

Re: Review Request 30006: Enabled concurrent downloading into the fetcher cache

2015-01-25 Thread Bernd Mathiske
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30006/ --- (Updated Jan. 25, 2015, 5:36 a.m.) Review request for mesos, Adam B, Benjamin H

Re: Review Request 30006: Enabled concurrent downloading into the fetcher cache

2015-01-25 Thread Bernd Mathiske
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30006/ --- (Updated Jan. 25, 2015, 4:16 a.m.) Review request for mesos, Adam B, Benjamin H

Re: Review Request 30006: Enabled concurrent downloading into the fetcher cache

2015-01-25 Thread Bernd Mathiske
> On Jan. 21, 2015, 11:45 a.m., Timothy Chen wrote: > > src/slave/containerizer/fetcher.cpp, line 222 > > > > > > Can't you directly bind(&collect, pendingDownloads)? > > Bernd Mathiske wrote: > Don't know how to

Re: Review Request 30037: Introduced fetcher cache actions

2015-01-25 Thread Bernd Mathiske
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30037/ --- (Updated Jan. 25, 2015, 3:57 a.m.) Review request for mesos, Adam B, Benjamin H

Re: Review Request 30037: Introduced fetcher cache actions

2015-01-25 Thread Bernd Mathiske
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30037/#review69566 --- src/slave/containerizer/fetcher.cpp

Re: Review Request 30037: Introduced fetcher cache actions

2015-01-25 Thread Bernd Mathiske
> On Jan. 23, 2015, 5:28 p.m., Timothy Chen wrote: > > src/slave/containerizer/fetcher.cpp, line 443 > > > > > > Why only return a extension when you have a basename? There is an explanation a few lines further down n

Re: Review Request 30037: Introduced fetcher cache actions

2015-01-25 Thread Bernd Mathiske
> On Jan. 23, 2015, 5:01 p.m., Adam B wrote: > > include/mesos/fetcher/fetcher.proto, lines 37-38 > > > > > > Maybe it would be better to call this BYPASS_CACHE, and then the other > > two could be combined into USE_CA

Re: Review Request 30036: Introduced caching fields to command URI and fetcher parameter protobufs

2015-01-25 Thread Bernd Mathiske
> On Jan. 23, 2015, 4:16 p.m., Adam B wrote: > > Changing protobufs is tricky when considering backwards-compatibility. > > Looks like include/mesos/fetcher/fetcher.proto is part of the public API, > > it's only used by containerizers, but not just the mesos/docker > > containerizers, but also

Re: Review Request 30036: Introduced caching fields to command URI and fetcher parameter protobufs

2015-01-25 Thread Bernd Mathiske
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30036/ --- (Updated Jan. 25, 2015, 3:27 a.m.) Review request for mesos, Adam B, Benjamin H

Re: Review Request 30036: Introduced caching fields to command URI and fetcher parameter protobufs

2015-01-25 Thread Bernd Mathiske
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30036/ --- (Updated Jan. 25, 2015, 3:26 a.m.) Review request for mesos, Adam B, Benjamin H

Re: mesos-dns

2015-01-25 Thread Tom Arnfeld
Thanks for sharing! Does this project by any chance utilise the new (not sure if merged or released) service discovery protobufs in Mesos? -- Tom Arnfeld Developer // DueDil (+44) 7525940046 25 Christopher Street, London, EC2A 2BS On Sat, Jan 24, 2015 at 5:20 PM, Christos Kozyrakis wro

Re: Review Request 29507: Added Configurable Slave Ping Timeouts

2015-01-25 Thread Adam B
> On Jan. 7, 2015, 2:50 p.m., Ben Mahler wrote: > > src/master/flags.hpp, lines 361-373 > > > > > > Do you mind changing the names now that they're merely defaults? > > > > s/SLAVE_PING_TIMEOUT/DEFAULT_SLAVE_P

Re: Review Request 30034: Unified fetcher testing, removed stderr/out redirection

2015-01-25 Thread Bernd Mathiske
> On Jan. 23, 2015, 3:31 p.m., Adam B wrote: > > src/tests/fetcher_tests.cpp, line 269 > > > > > > How do we get verbose fetcher output now? If it's a libprocess actor, > > can we just set the GTEST_v environment vari