Re: Review Request 27670: Bug fix: Check for non-zero status code and hadoop client not found

2014-11-06 Thread Mesos ReviewBot

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27670/#review60134
---


Patch looks great!

Reviews applied: [27670]

All tests passed.

- Mesos ReviewBot


On Nov. 6, 2014, 7:07 a.m., Ankur Chauhan wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27670/
 ---
 
 (Updated Nov. 6, 2014, 7:07 a.m.)
 
 
 Review request for mesos, Benjamin Hindman and Vinod Kone.
 
 
 Repository: mesos-git
 
 
 Description
 ---
 
 @benh noticed that the fetcher would detect the condition where hadoop 
 returns a success in case of a success non zero value
 
 
 Diffs
 -
 
   src/launcher/fetcher.cpp bd95928bc3191970330e839bcf41e343d5142c54 
 
 Diff: https://reviews.apache.org/r/27670/diff/
 
 
 Testing
 ---
 
 make check
 sudo bin/mesos-tests.sh --verbose
 support/mesos-style.py
 
 
 Thanks,
 
 Ankur Chauhan
 




Re: Review Request 27483: Fetcher uses hadoop to fetch URIs regardless of the url scheme.

2014-11-06 Thread Mesos ReviewBot

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27483/#review60136
---


Patch looks great!

Reviews applied: [27483]

All tests passed.

- Mesos ReviewBot


On Nov. 6, 2014, 4:12 a.m., Ankur Chauhan wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27483/
 ---
 
 (Updated Nov. 6, 2014, 4:12 a.m.)
 
 
 Review request for mesos, Timothy Chen, Timothy St. Clair, and Vinod Kone.
 
 
 Bugs: MESOS-1711
 https://issues.apache.org/jira/browse/MESOS-1711
 
 
 Repository: mesos-git
 
 
 Description
 ---
 
 Previously, the fetcher used a hardcoded list of schemes to determine what 
 URIs could be fetched by hadoop (if available). This is now changed such that 
 we first check if hadoop can fetch them for us and then we fallback to the 
 os::net and then a local copy method (same as it used to be). This allows 
 users to fetch artifacts from arbitrary filesystems as long as hadoop is 
 correctly configured (in core-site.xml).
 
 
 Diffs
 -
 
   src/launcher/fetcher.cpp bd95928bc3191970330e839bcf41e343d5142c54 
 
 Diff: https://reviews.apache.org/r/27483/diff/
 
 
 Testing
 ---
 
 make check
 sudo bin/mesos-tests.sh --verbose
 support/mesos-style.py
 
 
 Thanks,
 
 Ankur Chauhan
 




Re: Review Request 27483: Fetcher uses hadoop to fetch URIs regardless of the url scheme.

2014-11-06 Thread Mesos ReviewBot

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27483/#review60139
---


Patch looks great!

Reviews applied: [27483]

All tests passed.

- Mesos ReviewBot


On Nov. 6, 2014, 4:12 a.m., Ankur Chauhan wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27483/
 ---
 
 (Updated Nov. 6, 2014, 4:12 a.m.)
 
 
 Review request for mesos, Timothy Chen, Timothy St. Clair, and Vinod Kone.
 
 
 Bugs: MESOS-1711
 https://issues.apache.org/jira/browse/MESOS-1711
 
 
 Repository: mesos-git
 
 
 Description
 ---
 
 Previously, the fetcher used a hardcoded list of schemes to determine what 
 URIs could be fetched by hadoop (if available). This is now changed such that 
 we first check if hadoop can fetch them for us and then we fallback to the 
 os::net and then a local copy method (same as it used to be). This allows 
 users to fetch artifacts from arbitrary filesystems as long as hadoop is 
 correctly configured (in core-site.xml).
 
 
 Diffs
 -
 
   src/launcher/fetcher.cpp bd95928bc3191970330e839bcf41e343d5142c54 
 
 Diff: https://reviews.apache.org/r/27483/diff/
 
 
 Testing
 ---
 
 make check
 sudo bin/mesos-tests.sh --verbose
 support/mesos-style.py
 
 
 Thanks,
 
 Ankur Chauhan
 




Re: Review Request 27493: Added CRAM-MD5 Test Authenticatee Module.

2014-11-06 Thread Till Toenshoff

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27493/
---

(Updated Nov. 6, 2014, 10:33 a.m.)


Review request for mesos, Adam B, Kapil Arya, and Vinod Kone.


Changes
---

Addressed comments. Thanks for your thorough and patient reviews Adam! 


Bugs: MESOS-2001
https://issues.apache.org/jira/browse/MESOS-2001


Repository: mesos-git


Description
---

see summary.


Diffs (updated)
-

  src/Makefile.am 9ab3b9c 
  src/authentication/authenticatee.hpp PRE-CREATION 
  src/authentication/cram_md5/authenticatee.hpp 3088a77 
  src/examples/test_authentication_modules.cpp PRE-CREATION 
  src/examples/test_authenticator_module.cpp 4398d71 
  src/module/authenticatee.hpp PRE-CREATION 
  src/module/manager.cpp 0d71e33 

Diff: https://reviews.apache.org/r/27493/diff/


Testing
---

make check

Note: All three CRAM-MD5 Authenticatee patches need to get applied before 
running make check! 


Thanks,

Till Toenshoff



Re: Review Request 27493: Added CRAM-MD5 Test Authenticatee Module.

2014-11-06 Thread Till Toenshoff


 On Nov. 6, 2014, 6 a.m., Adam B wrote:
  src/authentication/cram_md5/authenticatee.hpp, lines 78-79
  https://reviews.apache.org/r/27493/diff/1/?file=746988#file746988line78
 
  Why the change in wrapping? I think it would still fit the other way, 
  and if you are bringing credential down to the next line, it might be 
  better if you give the UPID its own line too.

True, messed that up -- will add some clang-format love.


 On Nov. 6, 2014, 6 a.m., Adam B wrote:
  src/examples/test_authentication_module.cpp, line 1
  https://reviews.apache.org/r/27493/diff/1/?file=746989#file746989line1
 
  Maybe this file should be test_authentication_modules.cpp (plural) 
  since it contains multiple modules?

Totally, yes - we should also make Kapil aware of this change as the isolator 
modules should in fact also use this plural.


 On Nov. 6, 2014, 6 a.m., Adam B wrote:
  src/authentication/cram_md5/authenticatee.hpp, line 418
  https://reviews.apache.org/r/27493/diff/1/?file=746988#file746988line418
 
  requires a credential (singular)?

Yes, thank you.


 On Nov. 6, 2014, 6 a.m., Adam B wrote:
  src/authentication/cram_md5/authenticatee.hpp, line 80
  https://reviews.apache.org/r/27493/diff/1/?file=746988#file746988line80
 
  s/authenticatee/crammd5_authenticatee/

Initially I thought about doing so -- then I (falsely) turned it down as I had 
in mind that we would never have multiple, distinct authenticatee mechanism 
processes running. Certainly not true given that we plan to allow framework- 
and slave-authentication to freely choose their individual mechanisms. Thanks 
for this comment.


- Till


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27493/#review60124
---


On Nov. 5, 2014, 4:24 p.m., Till Toenshoff wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27493/
 ---
 
 (Updated Nov. 5, 2014, 4:24 p.m.)
 
 
 Review request for mesos, Adam B, Kapil Arya, and Vinod Kone.
 
 
 Bugs: MESOS-2001
 https://issues.apache.org/jira/browse/MESOS-2001
 
 
 Repository: mesos-git
 
 
 Description
 ---
 
 see summary.
 
 
 Diffs
 -
 
   src/Makefile.am 2d72a70 
   src/authentication/authenticatee.hpp PRE-CREATION 
   src/authentication/cram_md5/authenticatee.hpp PRE-CREATION 
   src/examples/test_authentication_module.cpp PRE-CREATION 
   src/examples/test_authenticator_module.cpp PRE-CREATION 
   src/module/authenticatee.hpp PRE-CREATION 
   src/module/manager.cpp 7a6c884 
 
 Diff: https://reviews.apache.org/r/27493/diff/
 
 
 Testing
 ---
 
 make check
 
 Note: All three CRAM-MD5 Authenticatee patches need to get applied before 
 running make check! 
 
 
 Thanks,
 
 Till Toenshoff
 




Review Request 27675: Updated upgrades.md to reflect changes on AuthenticationStartMessage.

2014-11-06 Thread Till Toenshoff

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27675/
---

Review request for mesos, Adam B and Vinod Kone.


Bugs: MESOS-2001
https://issues.apache.org/jira/browse/MESOS-2001


Repository: mesos-git


Description
---

see summary.


Diffs
-

  docs/upgrades.md 3aff335 

Diff: https://reviews.apache.org/r/27675/diff/


Testing
---


Thanks,

Till Toenshoff



Re: Review Request 27483: Fetcher uses hadoop to fetch URIs regardless of the url scheme.

2014-11-06 Thread Mesos ReviewBot

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27483/#review60142
---


Patch looks great!

Reviews applied: [27483]

All tests passed.

- Mesos ReviewBot


On Nov. 6, 2014, 4:12 a.m., Ankur Chauhan wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27483/
 ---
 
 (Updated Nov. 6, 2014, 4:12 a.m.)
 
 
 Review request for mesos, Timothy Chen, Timothy St. Clair, and Vinod Kone.
 
 
 Bugs: MESOS-1711
 https://issues.apache.org/jira/browse/MESOS-1711
 
 
 Repository: mesos-git
 
 
 Description
 ---
 
 Previously, the fetcher used a hardcoded list of schemes to determine what 
 URIs could be fetched by hadoop (if available). This is now changed such that 
 we first check if hadoop can fetch them for us and then we fallback to the 
 os::net and then a local copy method (same as it used to be). This allows 
 users to fetch artifacts from arbitrary filesystems as long as hadoop is 
 correctly configured (in core-site.xml).
 
 
 Diffs
 -
 
   src/launcher/fetcher.cpp bd95928bc3191970330e839bcf41e343d5142c54 
 
 Diff: https://reviews.apache.org/r/27483/diff/
 
 
 Testing
 ---
 
 make check
 sudo bin/mesos-tests.sh --verbose
 support/mesos-style.py
 
 
 Thanks,
 
 Ankur Chauhan
 




Re: Review Request 27494: Integrated CRAM-MD5 Test Authenticatee module into slave.

2014-11-06 Thread Till Toenshoff


 On Nov. 6, 2014, 6:30 a.m., Adam B wrote:
  src/messages/messages.proto, line 375
  https://reviews.apache.org/r/27494/diff/3/?file=750907#file750907line375
 
  A little more clarity on this change: 'string' and 'bytes' are exactly 
  the same on the wire, and both are represented as std::string in C++, so 
  that shouldn't be a problem.
  https://www.mail-archive.com/protobuf@googlegroups.com/msg01478.html
  Java and Python represent UTF-8 strings ('string') differently from 
  byte arrays ('bytes'), but the framework scheduler would send this message 
  from the C++ libmesos authenticatee.
  Some pure language bindings may have to change the datatype used to 
  support framework authentication after this change. This may need to be 
  documented in upgrade notes.

Awesome comment. I have just proposed a new RR that intends to cover the 
documentation of this change; https://reviews.apache.org/r/27675/


 On Nov. 6, 2014, 6:30 a.m., Adam B wrote:
  src/slave/slave.cpp, lines 620-621
  https://reviews.apache.org/r/27494/diff/3/?file=750914#file750914line620
 
  This feels a little awkward. Now, we try to authenticate if 
  --authenticatee is set (to non-default) or --credential is set; otherwise 
  register directly?
  Maybe it's time to have an explicit --authenticate flag, so 
  --credential isn't both a path to a credential file and a switch to enable 
  authentication. Thoughts?
  
  Or maybe just check to see if there's a viable authenticatee, and if 
  not, try to register without authenticating. The master/authenticator would 
  reject an unauthenticated slave/framework if the master has enabled authn, 
  so the authenticatee doesn't have to be so strict about exiting instead of 
  registering without authentication.

Entirely true as written Adam. I had the same feelings when implementing it 
this way but hesitated to revise the existing logic too much. My first hunch is 
to follow your second option - but let's see what others say about that. 

I will put up a comment on https://issues.apache.org/jira/browse/MESOS-2040 to 
open up a quick discussion, inviting everyone for input.


- Till


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27494/#review60125
---


On Nov. 6, 2014, 12:19 a.m., Till Toenshoff wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27494/
 ---
 
 (Updated Nov. 6, 2014, 12:19 a.m.)
 
 
 Review request for mesos, Adam B and Vinod Kone.
 
 
 Bugs: MESOS-2001
 https://issues.apache.org/jira/browse/MESOS-2001
 
 
 Repository: mesos-git
 
 
 Description
 ---
 
 Also fixes messages.proto to use a raw bytestream instead of a string for 
 AuthenticationStartMessage as non CRAM-MD5 authentication may transmit binary 
 data.
 
 Note that this patch covers modularized slave authentication only. Framework 
 authentication currently is currently covered by the default (built-in) 
 implementation. There will be a subsequent patch for modularized framework 
 authentication. 
 
 
 Diffs
 -
 
   src/messages/messages.proto de0e2a2 
   src/sched/sched.cpp e5f828d 
   src/scheduler/scheduler.cpp c74187c 
   src/slave/constants.hpp 701dd89 
   src/slave/constants.cpp d6ad78c 
   src/slave/flags.hpp efbd35d 
   src/slave/slave.hpp 5b082fc 
   src/slave/slave.cpp dbfd1a8 
 
 Diff: https://reviews.apache.org/r/27494/diff/
 
 
 Testing
 ---
 
 make check
 
 NOTE all three CRAM-MD5 authenticatee module related RRs need to get applied 
 before running make check.
 
 
 Thanks,
 
 Till Toenshoff
 




Re: Review Request 27494: Integrated CRAM-MD5 Test Authenticatee module into slave.

2014-11-06 Thread Till Toenshoff

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27494/
---

(Updated Nov. 6, 2014, 11:21 a.m.)


Review request for mesos, Adam B and Vinod Kone.


Bugs: MESOS-2001
https://issues.apache.org/jira/browse/MESOS-2001


Repository: mesos-git


Description (updated)
---

Also fixes messages.proto to use a raw bytestream instead of a string for 
AuthenticationStartMessage as non CRAM-MD5 authentication may transmit binary 
data.
Note that the change of AuthenticationStartMessage does basically have no 
impact on C++ based proto code other than the prevention of a warning due to 
non-UTF8 characters being encoded. That does in fact occur when using non 
CRAM-MD5 based SASL authentication mechanisms.

Note that this patch covers modularized slave authentication only. Framework 
authentication currently is currently covered by the default (built-in) 
implementation. There will be a subsequent patch for modularized framework 
authentication. 


Diffs
-

  src/messages/messages.proto de0e2a2 
  src/sched/sched.cpp e5f828d 
  src/scheduler/scheduler.cpp c74187c 
  src/slave/constants.hpp 701dd89 
  src/slave/constants.cpp d6ad78c 
  src/slave/flags.hpp efbd35d 
  src/slave/slave.hpp 5b082fc 
  src/slave/slave.cpp dbfd1a8 

Diff: https://reviews.apache.org/r/27494/diff/


Testing
---

make check

NOTE all three CRAM-MD5 authenticatee module related RRs need to get applied 
before running make check.


Thanks,

Till Toenshoff



Re: Review Request 27483: Fetcher uses hadoop to fetch URIs regardless of the url scheme.

2014-11-06 Thread Mesos ReviewBot

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27483/#review60147
---


Patch looks great!

Reviews applied: [27483]

All tests passed.

- Mesos ReviewBot


On Nov. 6, 2014, 4:12 a.m., Ankur Chauhan wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27483/
 ---
 
 (Updated Nov. 6, 2014, 4:12 a.m.)
 
 
 Review request for mesos, Timothy Chen, Timothy St. Clair, and Vinod Kone.
 
 
 Bugs: MESOS-1711
 https://issues.apache.org/jira/browse/MESOS-1711
 
 
 Repository: mesos-git
 
 
 Description
 ---
 
 Previously, the fetcher used a hardcoded list of schemes to determine what 
 URIs could be fetched by hadoop (if available). This is now changed such that 
 we first check if hadoop can fetch them for us and then we fallback to the 
 os::net and then a local copy method (same as it used to be). This allows 
 users to fetch artifacts from arbitrary filesystems as long as hadoop is 
 correctly configured (in core-site.xml).
 
 
 Diffs
 -
 
   src/launcher/fetcher.cpp bd95928bc3191970330e839bcf41e343d5142c54 
 
 Diff: https://reviews.apache.org/r/27483/diff/
 
 
 Testing
 ---
 
 make check
 sudo bin/mesos-tests.sh --verbose
 support/mesos-style.py
 
 
 Thanks,
 
 Ankur Chauhan
 




Re: Review Request 27483: Fetcher uses hadoop to fetch URIs regardless of the url scheme.

2014-11-06 Thread Mesos ReviewBot

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27483/#review60159
---


Patch looks great!

Reviews applied: [27483]

All tests passed.

- Mesos ReviewBot


On Nov. 6, 2014, 4:12 a.m., Ankur Chauhan wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27483/
 ---
 
 (Updated Nov. 6, 2014, 4:12 a.m.)
 
 
 Review request for mesos, Timothy Chen, Timothy St. Clair, and Vinod Kone.
 
 
 Bugs: MESOS-1711
 https://issues.apache.org/jira/browse/MESOS-1711
 
 
 Repository: mesos-git
 
 
 Description
 ---
 
 Previously, the fetcher used a hardcoded list of schemes to determine what 
 URIs could be fetched by hadoop (if available). This is now changed such that 
 we first check if hadoop can fetch them for us and then we fallback to the 
 os::net and then a local copy method (same as it used to be). This allows 
 users to fetch artifacts from arbitrary filesystems as long as hadoop is 
 correctly configured (in core-site.xml).
 
 
 Diffs
 -
 
   src/launcher/fetcher.cpp bd95928bc3191970330e839bcf41e343d5142c54 
 
 Diff: https://reviews.apache.org/r/27483/diff/
 
 
 Testing
 ---
 
 make check
 sudo bin/mesos-tests.sh --verbose
 support/mesos-style.py
 
 
 Thanks,
 
 Ankur Chauhan
 




Re: Unable to install Mesos on Ubuntu 14.04. Error during 'make'

2014-11-06 Thread Sweta Rani
Joris

Done with above commands, but now it gives following error:
WARNING: Logging before InitGoogleLogging() is written to STDERR
F1106 18:50:48.117947  9706 process.cpp:1532] Failed to initialize, bind
127.0.0.1:5050: Address already in use [98]
*** Check failure stack trace: ***

I tried to solve it by
http://stackoverflow.com/questions/10745878/ubuntu-error-with-apache-98address-already-in-use
 but it didnt work for me.


*Sweta Rani*

On Wed, Nov 5, 2014 at 3:50 AM, Joris Van Remoortere jo...@mesosphere.io
wrote:

 You need to make the directory and set the permissions to the user you are
 running the master as.

 For example:

 sudo mkdir /var/lib/mesos
 sudo chown jmlvanre /var/lib/mesos

 [jmlvanre] run mesos master...

 On Tue, Nov 4, 2014 at 2:07 PM, Sweta Rani swetarani3...@gmail.com
 wrote:

  Thanks for your help. I have installed mesos but  unable to start mesos
  master by the command
  ./bin/mesos-master.sh --ip=127.0.0.1 --work_dir=/var/lib/mesos.
 
  The file exists as mesos-master.sh.in in the bin folder.
 
  It reports:
  Failed to create work directory '/var/lib/mesos': Permission denied
 
  *Sweta Rani*
 
  On Tue, Nov 4, 2014 at 2:50 PM, Adam Bordelon a...@mesosphere.io
 wrote:
 
   As an alternate workaround, if you don't need the Java support (you're
  not
   running a JVM framework like Spark/Hadoop/Marathon), then you could use
   `configure --disable-java` to skip all the maven/pom build steps.
  
   By disabling Java, you could at least check that the rest of Mesos
 builds
   and runs fine, which would still work with C++, Python, or Go
 frameworks,
   or even any Java frameworks using the pure Java bindings (jesos).
  
   On Mon, Nov 3, 2014 at 2:01 PM, Joris Van Remoortere 
  jo...@mesosphere.io
   wrote:
  
Did you verify that the settings.xml file was parsed using mvn -X?
   
Specifically after the first few lines you will see something along
 the
lines of:
[DEBUG] Reading global settings from
 /usr/share/maven/conf/settings.xml
[DEBUG] Reading user settings from /home/abc/.m2/settings.xml
[DEBUG] Using local repository at /home/abc/.m2/repository
   
Make sure your settings file is in the right location.
   
On Mon, Nov 3, 2014 at 11:51 AM, Sweta Rani swetarani3...@gmail.com
 
wrote:
   
 I use campus proxy configuration for internet. But I am unable to
 run
make
 in installing Mesos on my Ubuntu -14.04, as it ends up with
 ERROR(1).
Upon
 searching, I created settings.xml file inside .m2 with the required
 configuration but failed again. Please help me out, as it is
 greatly
 hindering me from my work.

 On Sat Nov 01 2014 at 3:38:57 AM Joris Van Remoortere 
jo...@mesosphere.io
 
 wrote:

  I think this suggests you're fetching through a proxy:
  *Proxy request sent, awaiting response... 200 OK*
 
  When I wget this is my output:
   wget http://repo.maven.apache.org/maven2/org/apache/apache/11/
  apache-11.pom
  --2014-10-31
  

   
  
 
 http://repo.maven.apache.org/maven2/org/apache/apache/11/apache-11.pom--2014-10-31
 
  15:05:32--
 
   http://repo.maven.apache.org/maven2/org/apache/apache/11/apache-11.pom
  Resolving repo.maven.apache.org (repo.maven.apache.org)...
23.235.47.215
  Connecting to repo.maven.apache.org
  (repo.maven.apache.org)|23.235.47.215|:80...
  connected.
  *HTTP request sent, awaiting response... 200 OK*
 
  I think you might need to set up your proxy along these lines:
  http://maven.apache.org/guides/mini/guide-proxies.html ... but I
  am
not
  familiar with Maven.
 
  Joris
 
 
  On Fri, Oct 31, 2014 at 2:36 PM, Sweta Rani 
  swetarani3...@gmail.com
   
  wrote:
 
   Tried wget for all the given https, it was successful but still
   after
  that
   make was not successful and make check says YOU HAVE 3
 DISABLED
TESTS
  
   proxima@Centauri:~/mesos/build$ wget
  
   
 http://repo.maven.apache.org/maven2/org/apache/apache/11/apache-11.pom
   --2014-11-01 02:55:51--
  
   
 http://repo.maven.apache.org/maven2/org/apache/apache/11/apache-11.pom
   Connecting to 10.3.100.207:8080... connected.
   Proxy request sent, awaiting response... 200 OK
   Length: 14811 (14K) [text/xml]
   Saving to: ‘apache-11.pom’
  
*Joris Van Remoortere*
  
   On Sat Nov 01 2014 at 2:36:27 AM Joris Van Remoortere 
  jo...@mesosphere.io
   
   wrote:
  
That seems like you might have a connectivity / firewall
 issue.
Can you try just doing a wget on that artifact?
   
On Fri, Oct 31, 2014 at 1:56 PM, Sweta Rani 
swetarani3...@gmail.com
 
wrote:
   
 I tried it again but failed with following errors :

 make[1]: Entering directory `/home/proxima/mesos/build/src'
 Building mesos-0.21.0.jar ...
 

Re: Unable to install Mesos on Ubuntu 14.04. Error during 'make'

2014-11-06 Thread Alex Rukletsov
Sweta,

have you tried another port?

On Thu, Nov 6, 2014 at 2:24 PM, Sweta Rani swetarani3...@gmail.com wrote:

 Joris

 Done with above commands, but now it gives following error:
 WARNING: Logging before InitGoogleLogging() is written to STDERR
 F1106 18:50:48.117947  9706 process.cpp:1532] Failed to initialize, bind
 127.0.0.1:5050: Address already in use [98]
 *** Check failure stack trace: ***

 I tried to solve it by

 http://stackoverflow.com/questions/10745878/ubuntu-error-with-apache-98address-already-in-use
  but it didnt work for me.


 *Sweta Rani*

 On Wed, Nov 5, 2014 at 3:50 AM, Joris Van Remoortere jo...@mesosphere.io
 wrote:

  You need to make the directory and set the permissions to the user you
 are
  running the master as.
 
  For example:
 
  sudo mkdir /var/lib/mesos
  sudo chown jmlvanre /var/lib/mesos
 
  [jmlvanre] run mesos master...
 
  On Tue, Nov 4, 2014 at 2:07 PM, Sweta Rani swetarani3...@gmail.com
  wrote:
 
   Thanks for your help. I have installed mesos but  unable to start mesos
   master by the command
   ./bin/mesos-master.sh --ip=127.0.0.1 --work_dir=/var/lib/mesos.
  
   The file exists as mesos-master.sh.in in the bin folder.
  
   It reports:
   Failed to create work directory '/var/lib/mesos': Permission denied
  
   *Sweta Rani*
  
   On Tue, Nov 4, 2014 at 2:50 PM, Adam Bordelon a...@mesosphere.io
  wrote:
  
As an alternate workaround, if you don't need the Java support
 (you're
   not
running a JVM framework like Spark/Hadoop/Marathon), then you could
 use
`configure --disable-java` to skip all the maven/pom build steps.
   
By disabling Java, you could at least check that the rest of Mesos
  builds
and runs fine, which would still work with C++, Python, or Go
  frameworks,
or even any Java frameworks using the pure Java bindings (jesos).
   
On Mon, Nov 3, 2014 at 2:01 PM, Joris Van Remoortere 
   jo...@mesosphere.io
wrote:
   
 Did you verify that the settings.xml file was parsed using mvn -X?

 Specifically after the first few lines you will see something along
  the
 lines of:
 [DEBUG] Reading global settings from
  /usr/share/maven/conf/settings.xml
 [DEBUG] Reading user settings from /home/abc/.m2/settings.xml
 [DEBUG] Using local repository at /home/abc/.m2/repository

 Make sure your settings file is in the right location.

 On Mon, Nov 3, 2014 at 11:51 AM, Sweta Rani 
 swetarani3...@gmail.com
  
 wrote:

  I use campus proxy configuration for internet. But I am unable to
  run
 make
  in installing Mesos on my Ubuntu -14.04, as it ends up with
  ERROR(1).
 Upon
  searching, I created settings.xml file inside .m2 with the
 required
  configuration but failed again. Please help me out, as it is
  greatly
  hindering me from my work.
 
  On Sat Nov 01 2014 at 3:38:57 AM Joris Van Remoortere 
 jo...@mesosphere.io
  
  wrote:
 
   I think this suggests you're fetching through a proxy:
   *Proxy request sent, awaiting response... 200 OK*
  
   When I wget this is my output:
wget
 http://repo.maven.apache.org/maven2/org/apache/apache/11/
   apache-11.pom
   --2014-10-31
   
 

   
  
 
 http://repo.maven.apache.org/maven2/org/apache/apache/11/apache-11.pom--2014-10-31
  
   15:05:32--
  
   
 http://repo.maven.apache.org/maven2/org/apache/apache/11/apache-11.pom
   Resolving repo.maven.apache.org (repo.maven.apache.org)...
 23.235.47.215
   Connecting to repo.maven.apache.org
   (repo.maven.apache.org)|23.235.47.215|:80...
   connected.
   *HTTP request sent, awaiting response... 200 OK*
  
   I think you might need to set up your proxy along these lines:
   http://maven.apache.org/guides/mini/guide-proxies.html ...
 but I
   am
 not
   familiar with Maven.
  
   Joris
  
  
   On Fri, Oct 31, 2014 at 2:36 PM, Sweta Rani 
   swetarani3...@gmail.com

   wrote:
  
Tried wget for all the given https, it was successful but
 still
after
   that
make was not successful and make check says YOU HAVE 3
  DISABLED
 TESTS
   
proxima@Centauri:~/mesos/build$ wget
   

  http://repo.maven.apache.org/maven2/org/apache/apache/11/apache-11.pom
--2014-11-01 02:55:51--
   

  http://repo.maven.apache.org/maven2/org/apache/apache/11/apache-11.pom
Connecting to 10.3.100.207:8080... connected.
Proxy request sent, awaiting response... 200 OK
Length: 14811 (14K) [text/xml]
Saving to: ‘apache-11.pom’
   
 *Joris Van Remoortere*
   
On Sat Nov 01 2014 at 2:36:27 AM Joris Van Remoortere 
   jo...@mesosphere.io

wrote:
   
 That seems like you might have a connectivity / firewall
  issue.
 Can you try just doing a wget on that artifact?

 On Fri, 

Re: Review Request 27651: Updated CRAMMD5Authentication tests to additionally run against the test authenticatee module.

2014-11-06 Thread Till Toenshoff


 On Nov. 6, 2014, 6:47 a.m., Adam B wrote:
  src/tests/cram_md5_authentication_tests.cpp, line 60
  https://reviews.apache.org/r/27651/diff/1/?file=750935#file750935line60
 
  Not sure why you went from 'T' to 'S' instead of 'U', but it doesn't 
  bother me.

Cause I am a rebel ;) ... no, will adjust it towards the more common variant.


- Till


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27651/#review60128
---


On Nov. 6, 2014, 12:26 a.m., Till Toenshoff wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27651/
 ---
 
 (Updated Nov. 6, 2014, 12:26 a.m.)
 
 
 Review request for mesos, Adam B and Vinod Kone.
 
 
 Bugs: MESOS-2001
 https://issues.apache.org/jira/browse/MESOS-2001
 
 
 Repository: mesos-git
 
 
 Description
 ---
 
 CRAMMD5Authentication tests now cover all default and modularized 
 authenticator/authenticatee combinations.
 Also adds a test for missing credentials for the authenticatee.
 
 
 Diffs
 -
 
   src/authentication/cram_md5/authenticatee.hpp 3088a77 
   src/tests/cram_md5_authentication_tests.cpp 74ea2ad 
   src/tests/module.hpp 7970c28 
   src/tests/module.cpp 482ed22 
 
 Diff: https://reviews.apache.org/r/27651/diff/
 
 
 Testing
 ---
 
 make check
 
 
 Thanks,
 
 Till Toenshoff
 




Re: Review Request 27651: Updated CRAMMD5Authentication tests to additionally run against the test authenticatee module.

2014-11-06 Thread Till Toenshoff

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27651/
---

(Updated Nov. 6, 2014, 1:46 p.m.)


Review request for mesos, Adam B and Vinod Kone.


Changes
---

Removed some useless includes and addressed comments.


Bugs: MESOS-2001
https://issues.apache.org/jira/browse/MESOS-2001


Repository: mesos-git


Description (updated)
---

CRAMMD5Authentication tests now cover all default and modularized 
authenticator/authenticatee combinations.
Also adds a test for missing credential for the authenticatee.


Diffs (updated)
-

  src/authentication/cram_md5/authenticatee.hpp 3088a77 
  src/tests/cram_md5_authentication_tests.cpp 74ea2ad 
  src/tests/module.hpp 7970c28 
  src/tests/module.cpp 482ed22 

Diff: https://reviews.apache.org/r/27651/diff/


Testing
---

make check


Thanks,

Till Toenshoff



Re: Review Request 26476: Remove dynamic allocation from Option.

2014-11-06 Thread Till Toenshoff

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26476/#review60162
---


Seems this one got stuck for a while now. The reasons appear to be founded in 
the doubts expressed in the above reviews. I would highly recommend creating a 
JIRA (if there is none yet), find a shepherd and document the above concerns 
and your argumentation in such JIRA. These points would most likely also form a 
great base for further optimizing in similar cases.

- Till Toenshoff


On Oct. 22, 2014, 7:34 p.m., Joris Van Remoortere wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/26476/
 ---
 
 (Updated Oct. 22, 2014, 7:34 p.m.)
 
 
 Review request for mesos, Benjamin Hindman and Niklas Nielsen.
 
 
 Repository: mesos-git
 
 
 Description
 ---
 
 Remove dynamic allocations from Option class.
 
 
 Diffs
 -
 
   3rdparty/libprocess/3rdparty/stout/include/stout/option.hpp 47fe92c 
 
 Diff: https://reviews.apache.org/r/26476/diff/
 
 
 Testing
 ---
 
 make check
 support/mesos-style.py
 valgrind (reduced allocation count)
 
 
 Thanks,
 
 Joris Van Remoortere
 




Re: Review Request 27483: Fetcher uses hadoop to fetch URIs regardless of the url scheme.

2014-11-06 Thread Mesos ReviewBot

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27483/#review60163
---


Patch looks great!

Reviews applied: [27483]

All tests passed.

- Mesos ReviewBot


On Nov. 6, 2014, 4:12 a.m., Ankur Chauhan wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27483/
 ---
 
 (Updated Nov. 6, 2014, 4:12 a.m.)
 
 
 Review request for mesos, Timothy Chen, Timothy St. Clair, and Vinod Kone.
 
 
 Bugs: MESOS-1711
 https://issues.apache.org/jira/browse/MESOS-1711
 
 
 Repository: mesos-git
 
 
 Description
 ---
 
 Previously, the fetcher used a hardcoded list of schemes to determine what 
 URIs could be fetched by hadoop (if available). This is now changed such that 
 we first check if hadoop can fetch them for us and then we fallback to the 
 os::net and then a local copy method (same as it used to be). This allows 
 users to fetch artifacts from arbitrary filesystems as long as hadoop is 
 correctly configured (in core-site.xml).
 
 
 Diffs
 -
 
   src/launcher/fetcher.cpp bd95928bc3191970330e839bcf41e343d5142c54 
 
 Diff: https://reviews.apache.org/r/27483/diff/
 
 
 Testing
 ---
 
 make check
 sudo bin/mesos-tests.sh --verbose
 support/mesos-style.py
 
 
 Thanks,
 
 Ankur Chauhan
 




Re: Review Request 27651: Updated CRAMMD5Authentication tests to additionally run against the test authenticatee module.

2014-11-06 Thread Mesos ReviewBot

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27651/#review60164
---


Bad patch!

Reviews applied: [27493, 27494, 27495, 27619, 27651]

Failed command: ./support/apply-review.sh -n -r 27651

Error:
 2014-11-06 14:25:41 URL:https://reviews.apache.org/r/27651/diff/raw/ 
[11051/11051] - 27651.patch [1]
error: patch failed: src/tests/module.cpp:70
error: src/tests/module.cpp: patch does not apply
Failed to apply patch

- Mesos ReviewBot


On Nov. 6, 2014, 1:46 p.m., Till Toenshoff wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27651/
 ---
 
 (Updated Nov. 6, 2014, 1:46 p.m.)
 
 
 Review request for mesos, Adam B and Vinod Kone.
 
 
 Bugs: MESOS-2001
 https://issues.apache.org/jira/browse/MESOS-2001
 
 
 Repository: mesos-git
 
 
 Description
 ---
 
 CRAMMD5Authentication tests now cover all default and modularized 
 authenticator/authenticatee combinations.
 Also adds a test for missing credential for the authenticatee.
 
 
 Diffs
 -
 
   src/authentication/cram_md5/authenticatee.hpp 3088a77 
   src/tests/cram_md5_authentication_tests.cpp 74ea2ad 
   src/tests/module.hpp 7970c28 
   src/tests/module.cpp 482ed22 
 
 Diff: https://reviews.apache.org/r/27651/diff/
 
 
 Testing
 ---
 
 make check
 
 
 Thanks,
 
 Till Toenshoff
 




Re: Unable to install Mesos on Ubuntu 14.04. Error during 'make'

2014-11-06 Thread Sweta Rani
Alex,

Done with master but unable to run slave using command ./bin/mesos-slave.sh
--master=127.0.0.1:5050
Also the port 5051 is allocated with random work '-'. See this:
proxima@proxima:~/mesos/build$ netstat -ltnp | grep ':5051'
(Not all processes could be identified, non-owned process info
 will not be shown, you would have to be root to see it all.)
tcp0  0 0.0.0.0:50510.0.0.0:*   LISTEN
 -
proxima@proxima:~/mesos/build$ sudo kill -9 -
kill: failed to parse argument: '-'
proxima@proxima:~/mesos/build$ netstat -ltnp | grep ':5051'
(Not all processes could be identified, non-owned process info
 will not be shown, you would have to be root to see it all.)
tcp0  0 0.0.0.0:50510.0.0.0:*   LISTEN
 -
proxima@:~/mesos/build$ ./bin/mesos-slave.sh --master=127.0.0.1:5050
WARNING: Logging before InitGoogleLogging() is written to STDERR
F1106 19:51:22.737992  6367 process.cpp:1532] Failed to initialize, bind
0.0.0.0:5051: Address already in use [98]
*** Check failure stack trace: ***
Aborted (core dumped)


*Sweta Rani*

On Thu, Nov 6, 2014 at 7:07 PM, Alex Rukletsov a...@mesosphere.io wrote:

 Sweta,

 have you tried another port?

 On Thu, Nov 6, 2014 at 2:24 PM, Sweta Rani swetarani3...@gmail.com
 wrote:

  Joris
 
  Done with above commands, but now it gives following error:
  WARNING: Logging before InitGoogleLogging() is written to STDERR
  F1106 18:50:48.117947  9706 process.cpp:1532] Failed to initialize, bind
  127.0.0.1:5050: Address already in use [98]
  *** Check failure stack trace: ***
 
  I tried to solve it by
 
 
 http://stackoverflow.com/questions/10745878/ubuntu-error-with-apache-98address-already-in-use
   but it didnt work for me.
 
 
  *Sweta Rani*
 
  On Wed, Nov 5, 2014 at 3:50 AM, Joris Van Remoortere 
 jo...@mesosphere.io
  wrote:
 
   You need to make the directory and set the permissions to the user you
  are
   running the master as.
  
   For example:
  
   sudo mkdir /var/lib/mesos
   sudo chown jmlvanre /var/lib/mesos
  
   [jmlvanre] run mesos master...
  
   On Tue, Nov 4, 2014 at 2:07 PM, Sweta Rani swetarani3...@gmail.com
   wrote:
  
Thanks for your help. I have installed mesos but  unable to start
 mesos
master by the command
./bin/mesos-master.sh --ip=127.0.0.1 --work_dir=/var/lib/mesos.
   
The file exists as mesos-master.sh.in in the bin folder.
   
It reports:
Failed to create work directory '/var/lib/mesos': Permission denied
   
*Sweta Rani*
   
On Tue, Nov 4, 2014 at 2:50 PM, Adam Bordelon a...@mesosphere.io
   wrote:
   
 As an alternate workaround, if you don't need the Java support
  (you're
not
 running a JVM framework like Spark/Hadoop/Marathon), then you could
  use
 `configure --disable-java` to skip all the maven/pom build steps.

 By disabling Java, you could at least check that the rest of Mesos
   builds
 and runs fine, which would still work with C++, Python, or Go
   frameworks,
 or even any Java frameworks using the pure Java bindings (jesos).

 On Mon, Nov 3, 2014 at 2:01 PM, Joris Van Remoortere 
jo...@mesosphere.io
 wrote:

  Did you verify that the settings.xml file was parsed using mvn
 -X?
 
  Specifically after the first few lines you will see something
 along
   the
  lines of:
  [DEBUG] Reading global settings from
   /usr/share/maven/conf/settings.xml
  [DEBUG] Reading user settings from /home/abc/.m2/settings.xml
  [DEBUG] Using local repository at /home/abc/.m2/repository
 
  Make sure your settings file is in the right location.
 
  On Mon, Nov 3, 2014 at 11:51 AM, Sweta Rani 
  swetarani3...@gmail.com
   
  wrote:
 
   I use campus proxy configuration for internet. But I am unable
 to
   run
  make
   in installing Mesos on my Ubuntu -14.04, as it ends up with
   ERROR(1).
  Upon
   searching, I created settings.xml file inside .m2 with the
  required
   configuration but failed again. Please help me out, as it is
   greatly
   hindering me from my work.
  
   On Sat Nov 01 2014 at 3:38:57 AM Joris Van Remoortere 
  jo...@mesosphere.io
   
   wrote:
  
I think this suggests you're fetching through a proxy:
*Proxy request sent, awaiting response... 200 OK*
   
When I wget this is my output:
 wget
  http://repo.maven.apache.org/maven2/org/apache/apache/11/
apache-11.pom
--2014-10-31

  
 

   
  
 
 http://repo.maven.apache.org/maven2/org/apache/apache/11/apache-11.pom--2014-10-31
   
15:05:32--
   

  http://repo.maven.apache.org/maven2/org/apache/apache/11/apache-11.pom
Resolving repo.maven.apache.org (repo.maven.apache.org)...
  23.235.47.215
Connecting to repo.maven.apache.org
(repo.maven.apache.org)|23.235.47.215|:80...
connected.
*HTTP 

Re: Unable to install Mesos on Ubuntu 14.04. Error during 'make'

2014-11-06 Thread Sweta Rani
How can I change port for mesos-slave from default 5051 to 5052?

*Sweta Rani*

On Thu, Nov 6, 2014 at 7:58 PM, Sweta Rani swetarani3...@gmail.com wrote:

 Alex,

 Done with master but unable to run slave using command ./bin/mesos-slave.sh
 --master=127.0.0.1:5050
 Also the port 5051 is allocated with random work '-'. See this:
 proxima@proxima:~/mesos/build$ netstat -ltnp | grep ':5051'
 (Not all processes could be identified, non-owned process info
  will not be shown, you would have to be root to see it all.)
 tcp0  0 0.0.0.0:50510.0.0.0:*
 LISTEN  -
 proxima@proxima:~/mesos/build$ sudo kill -9 -
 kill: failed to parse argument: '-'
 proxima@proxima:~/mesos/build$ netstat -ltnp | grep ':5051'
 (Not all processes could be identified, non-owned process info
  will not be shown, you would have to be root to see it all.)
 tcp0  0 0.0.0.0:50510.0.0.0:*
 LISTEN  -
 proxima@:~/mesos/build$ ./bin/mesos-slave.sh --master=127.0.0.1:5050
 WARNING: Logging before InitGoogleLogging() is written to STDERR
 F1106 19:51:22.737992  6367 process.cpp:1532] Failed to initialize, bind
 0.0.0.0:5051: Address already in use [98]
 *** Check failure stack trace: ***
 Aborted (core dumped)


 *Sweta Rani*

 On Thu, Nov 6, 2014 at 7:07 PM, Alex Rukletsov a...@mesosphere.io wrote:

 Sweta,

 have you tried another port?

 On Thu, Nov 6, 2014 at 2:24 PM, Sweta Rani swetarani3...@gmail.com
 wrote:

  Joris
 
  Done with above commands, but now it gives following error:
  WARNING: Logging before InitGoogleLogging() is written to STDERR
  F1106 18:50:48.117947  9706 process.cpp:1532] Failed to initialize, bind
  127.0.0.1:5050: Address already in use [98]
  *** Check failure stack trace: ***
 
  I tried to solve it by
 
 
 http://stackoverflow.com/questions/10745878/ubuntu-error-with-apache-98address-already-in-use
   but it didnt work for me.
 
 
  *Sweta Rani*
 
  On Wed, Nov 5, 2014 at 3:50 AM, Joris Van Remoortere 
 jo...@mesosphere.io
  wrote:
 
   You need to make the directory and set the permissions to the user you
  are
   running the master as.
  
   For example:
  
   sudo mkdir /var/lib/mesos
   sudo chown jmlvanre /var/lib/mesos
  
   [jmlvanre] run mesos master...
  
   On Tue, Nov 4, 2014 at 2:07 PM, Sweta Rani swetarani3...@gmail.com
   wrote:
  
Thanks for your help. I have installed mesos but  unable to start
 mesos
master by the command
./bin/mesos-master.sh --ip=127.0.0.1 --work_dir=/var/lib/mesos.
   
The file exists as mesos-master.sh.in in the bin folder.
   
It reports:
Failed to create work directory '/var/lib/mesos': Permission denied
   
*Sweta Rani*
   
On Tue, Nov 4, 2014 at 2:50 PM, Adam Bordelon a...@mesosphere.io
   wrote:
   
 As an alternate workaround, if you don't need the Java support
  (you're
not
 running a JVM framework like Spark/Hadoop/Marathon), then you
 could
  use
 `configure --disable-java` to skip all the maven/pom build steps.

 By disabling Java, you could at least check that the rest of Mesos
   builds
 and runs fine, which would still work with C++, Python, or Go
   frameworks,
 or even any Java frameworks using the pure Java bindings (jesos).

 On Mon, Nov 3, 2014 at 2:01 PM, Joris Van Remoortere 
jo...@mesosphere.io
 wrote:

  Did you verify that the settings.xml file was parsed using mvn
 -X?
 
  Specifically after the first few lines you will see something
 along
   the
  lines of:
  [DEBUG] Reading global settings from
   /usr/share/maven/conf/settings.xml
  [DEBUG] Reading user settings from /home/abc/.m2/settings.xml
  [DEBUG] Using local repository at /home/abc/.m2/repository
 
  Make sure your settings file is in the right location.
 
  On Mon, Nov 3, 2014 at 11:51 AM, Sweta Rani 
  swetarani3...@gmail.com
   
  wrote:
 
   I use campus proxy configuration for internet. But I am
 unable to
   run
  make
   in installing Mesos on my Ubuntu -14.04, as it ends up with
   ERROR(1).
  Upon
   searching, I created settings.xml file inside .m2 with the
  required
   configuration but failed again. Please help me out, as it is
   greatly
   hindering me from my work.
  
   On Sat Nov 01 2014 at 3:38:57 AM Joris Van Remoortere 
  jo...@mesosphere.io
   
   wrote:
  
I think this suggests you're fetching through a proxy:
*Proxy request sent, awaiting response... 200 OK*
   
When I wget this is my output:
 wget
  http://repo.maven.apache.org/maven2/org/apache/apache/11/
apache-11.pom
--2014-10-31

  
 

   
  
 
 http://repo.maven.apache.org/maven2/org/apache/apache/11/apache-11.pom--2014-10-31
   
15:05:32--
   

  http://repo.maven.apache.org/maven2/org/apache/apache/11/apache-11.pom
Resolving repo.maven.apache.org (repo.maven.apache.org)...

Re: Review Request 27516: Rebased and re-edited patch for MESOS-1316: Abstracted out invoking 'mesos-fetcher'.

2014-11-06 Thread Bernd Mathiske

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27516/
---

(Updated Nov. 6, 2014, 6:55 a.m.)


Review request for mesos and Benjamin Hindman.


Changes
---

Addressed review issues.


Bugs: MESOS-1316
https://issues.apache.org/jira/browse/MESOS-1316


Repository: mesos-git


Description
---

Manually rebasing and re-editing https://reviews.apache.org/r/21233/, which is 
supposed to be replaced now by this patch. 

Original description: To test the mesos-fetcher (and the setting of the 
environment) more cleanly I did some refactoring into a 'fetcher' namespace.

Also moved fetcher environment tests to fetcher test file. Added two fetcher 
tests.


Diffs (updated)
-

  src/Makefile.am 9ab3b9c05d435d18ed1c2966f695857fa205e9fd 
  src/launcher/fetcher.cpp bd95928bc3191970330e839bcf41e343d5142c54 
  src/slave/containerizer/containerizer.hpp 
8a6641269b8e010a708aa8b05bb65e2e72501625 
  src/slave/containerizer/containerizer.cpp 
0254679508167a390fd6fed855f19794354ac081 
  src/slave/containerizer/docker.cpp a6689203adbdcb0ad12583389eaeb83329e4ef6b 
  src/slave/containerizer/mesos/containerizer.hpp 
ab3bb6ffed28e76820020d8fc457f70d948e6657 
  src/slave/containerizer/mesos/containerizer.cpp 
4bd266552fcd28b0dcbc323bfd6a4043bbd835a9 
  src/tests/containerizer_tests.cpp 2c90d2fc18a3268c55b6dfe98699bfb36d093983 
  src/tests/fetcher_tests.cpp e026e875c7197cd214ef0432d7c40207553c8671 

Diff: https://reviews.apache.org/r/27516/diff/


Testing
---

make check on Mac OS 10.10 and Ubuntu 14.4.

In total, 3 tests fail: ExamplesTest.NoExecutorFramework, 
ExamplesTest.JavaFramework
, ExamplesTest.PythonFramework. It is strongly suspected that those are 
unrelated to this code change and just generally flaky.


Thanks,

Bernd Mathiske



Re: Review Request 27516: Rebased and re-edited patch for MESOS-1316: Abstracted out invoking 'mesos-fetcher'.

2014-11-06 Thread Bernd Mathiske


 On Nov. 5, 2014, 8:44 p.m., Benjamin Hindman wrote:
  src/slave/containerizer/fetcher.hpp, line 64
  https://reviews.apache.org/r/27516/diff/3/?file=747421#file747421line64
 
  Mind replacing ' ' with '' everywhere in your patch please? Thanks!

So we are now only using C++ compilers that do not have this bug any more? 
Nice! (Only about 12000 other places in Mesos to fix then :-)


 On Nov. 5, 2014, 8:44 p.m., Benjamin Hindman wrote:
  src/slave/containerizer/fetcher.cpp, line 19
  https://reviews.apache.org/r/27516/diff/3/?file=747422#file747422line19
 
  Please put  includes after  includes.

Actually, there should only be  includes here, according to this: 
http://mesos.apache.org/documentation/latest/mesos-c++-style-guide/, which 
references this: 
http://google-styleguide.googlecode.com/svn/trunk/cppguide.html#Names_and_Order_of_Includes

And yes, the order needs to be swapped then, too. First the header that matches 
the cpp file, then the rest.  

20 minutes later:

Just noticed that in all the places I looked we are NOT following the stated 
style guide in this regard. Hm. Switching back to my original order then.


 On Nov. 5, 2014, 8:44 p.m., Benjamin Hindman wrote:
  src/slave/containerizer/fetcher.cpp, line 112
  https://reviews.apache.org/r/27516/diff/3/?file=747422#file747422line112
 
  Even after making the change above, we shouldn't close the passed in 
  file descriptors. The comment for this function also says that we dup the 
  file descriptors, which implies that we won't be closing them because we're 
  creating our own copy so we can control our copy's lifetime.
  
  In general, the semantics that I've seen compose well are always 
  dup'ing the descriptors ourselves (if we want them to outlive the function 
  call) and then letting the caller close any file descriptors they passed in 
  on their own accord. It's much easier to reason about file descriptor 
  lifetimes with these semantics.
  
  Even better would be to have an FD abstraction that does reference 
  counting, but unfortunately we don't have one of those yet.

Agree. I was a little surprised myself to see this done otherwise, but did not 
want to argue.


 On Nov. 5, 2014, 8:44 p.m., Benjamin Hindman wrote:
  src/slave/containerizer/mesos/containerizer.cpp, line 524
  https://reviews.apache.org/r/27516/diff/3/?file=747423#file747423line524
 
  Why does this need to be a Future? It looks like you're chaining this 
  below via a 'then', which means you can just use Optionint since a 'then' 
  only invokes this if the Future is ready. The fact that you're always doing 
  a 'get()' on the future too implies this precondition.

If we made it a plain Optionint then that value would have to be present at 
the time fetcher::run() returns. This would mean we have to block in 
fetcher::run() to wait for the fetcher.


- Bernd


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27516/#review60098
---


On Nov. 3, 2014, 8:36 a.m., Bernd Mathiske wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27516/
 ---
 
 (Updated Nov. 3, 2014, 8:36 a.m.)
 
 
 Review request for mesos and Benjamin Hindman.
 
 
 Bugs: MESOS-1316
 https://issues.apache.org/jira/browse/MESOS-1316
 
 
 Repository: mesos-git
 
 
 Description
 ---
 
 Manually rebasing and re-editing https://reviews.apache.org/r/21233/, which 
 is supposed to be replaced now by this patch. 
 
 Original description: To test the mesos-fetcher (and the setting of the 
 environment) more cleanly I did some refactoring into a 'fetcher' namespace.
 
 Also moved fetcher environment tests to fetcher test file. Added two fetcher 
 tests.
 
 
 Diffs
 -
 
   src/Makefile.am e6a07150c10b9fa040143e394b2f913a18eeebc1 
   src/launcher/fetcher.cpp 9323c28237010fa065ef34d74435c151ded530a8 
   src/slave/containerizer/fetcher.hpp PRE-CREATION 
   src/slave/containerizer/fetcher.cpp PRE-CREATION 
   src/slave/containerizer/mesos/containerizer.cpp 
 d4b08f54d6feb453f3a9d27ca54c867176e62102 
   src/tests/containerizer_tests.cpp 2c90d2fc18a3268c55b6dfe98699bfb36d093983 
   src/tests/fetcher_tests.cpp d7754009a59fedb43e3422c56b3a786ce80164aa 
 
 Diff: https://reviews.apache.org/r/27516/diff/
 
 
 Testing
 ---
 
 make check on Mac OS 10.10 and Ubuntu 14.4.
 
 In total, 3 tests fail: ExamplesTest.NoExecutorFramework, 
 ExamplesTest.JavaFramework
 , ExamplesTest.PythonFramework. It is strongly suspected that those are 
 unrelated to this code change and just generally flaky.
 
 
 Thanks,
 
 Bernd Mathiske
 




Re: Review Request 27516: Rebased and re-edited patch for MESOS-1316: Abstracted out invoking 'mesos-fetcher'.

2014-11-06 Thread Bernd Mathiske

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27516/
---

(Updated Nov. 6, 2014, 6:59 a.m.)


Review request for mesos and Benjamin Hindman.


Changes
---

Added missing files.


Bugs: MESOS-1316
https://issues.apache.org/jira/browse/MESOS-1316


Repository: mesos-git


Description
---

Manually rebasing and re-editing https://reviews.apache.org/r/21233/, which is 
supposed to be replaced now by this patch. 

Original description: To test the mesos-fetcher (and the setting of the 
environment) more cleanly I did some refactoring into a 'fetcher' namespace.

Also moved fetcher environment tests to fetcher test file. Added two fetcher 
tests.


Diffs (updated)
-

  src/Makefile.am 9ab3b9c05d435d18ed1c2966f695857fa205e9fd 
  src/launcher/fetcher.cpp bd95928bc3191970330e839bcf41e343d5142c54 
  src/slave/containerizer/containerizer.hpp 
8a6641269b8e010a708aa8b05bb65e2e72501625 
  src/slave/containerizer/containerizer.cpp 
0254679508167a390fd6fed855f19794354ac081 
  src/slave/containerizer/docker.cpp a6689203adbdcb0ad12583389eaeb83329e4ef6b 
  src/slave/containerizer/fetcher.hpp PRE-CREATION 
  src/slave/containerizer/fetcher.cpp PRE-CREATION 
  src/slave/containerizer/mesos/containerizer.hpp 
ab3bb6ffed28e76820020d8fc457f70d948e6657 
  src/slave/containerizer/mesos/containerizer.cpp 
4bd266552fcd28b0dcbc323bfd6a4043bbd835a9 
  src/tests/containerizer_tests.cpp 2c90d2fc18a3268c55b6dfe98699bfb36d093983 
  src/tests/fetcher_tests.cpp e026e875c7197cd214ef0432d7c40207553c8671 

Diff: https://reviews.apache.org/r/27516/diff/


Testing
---

make check on Mac OS 10.10 and Ubuntu 14.4.

In total, 3 tests fail: ExamplesTest.NoExecutorFramework, 
ExamplesTest.JavaFramework
, ExamplesTest.PythonFramework. It is strongly suspected that those are 
unrelated to this code change and just generally flaky.


Thanks,

Bernd Mathiske



Re: Review Request 27483: Fetcher uses hadoop to fetch URIs regardless of the url scheme.

2014-11-06 Thread Mesos ReviewBot

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27483/#review60173
---


Patch looks great!

Reviews applied: [27483]

All tests passed.

- Mesos ReviewBot


On Nov. 6, 2014, 4:12 a.m., Ankur Chauhan wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27483/
 ---
 
 (Updated Nov. 6, 2014, 4:12 a.m.)
 
 
 Review request for mesos, Timothy Chen, Timothy St. Clair, and Vinod Kone.
 
 
 Bugs: MESOS-1711
 https://issues.apache.org/jira/browse/MESOS-1711
 
 
 Repository: mesos-git
 
 
 Description
 ---
 
 Previously, the fetcher used a hardcoded list of schemes to determine what 
 URIs could be fetched by hadoop (if available). This is now changed such that 
 we first check if hadoop can fetch them for us and then we fallback to the 
 os::net and then a local copy method (same as it used to be). This allows 
 users to fetch artifacts from arbitrary filesystems as long as hadoop is 
 correctly configured (in core-site.xml).
 
 
 Diffs
 -
 
   src/launcher/fetcher.cpp bd95928bc3191970330e839bcf41e343d5142c54 
 
 Diff: https://reviews.apache.org/r/27483/diff/
 
 
 Testing
 ---
 
 make check
 sudo bin/mesos-tests.sh --verbose
 support/mesos-style.py
 
 
 Thanks,
 
 Ankur Chauhan
 




Re: [VOTE] Release Apache Mesos 0.21.0 (rc1)

2014-11-06 Thread Tim St Clair
$ git tag -l | grep 21

$ git branch -r
  origin/0.21.0-rc1

It looks like you created a branch vs. tag ...? 

Cheers,
Tim

- Original Message -
 From: Ian Downes ian.dow...@gmail.com
 To: dev@mesos.apache.org, u...@mesos.apache.org
 Sent: Wednesday, November 5, 2014 5:12:52 PM
 Subject: [VOTE] Release Apache Mesos 0.21.0 (rc1)
 
 Hi all,
 
 Please vote on releasing the following candidate as Apache Mesos 0.21.0.
 
 
 0.21.0 includes the following:
 
 State reconciliation for frameworks
 Support for Mesos modules
 Task status now includes source and reason
 A shared filesystem isolator
 A pid namespace isolator
 
 The CHANGELOG for the release is available at:
 https://git-wip-us.apache.org/repos/asf?p=mesos.git;a=blob_plain;f=CHANGELOG;hb=0.21.0-rc1
 
 
 The candidate for Mesos 0.21.0 release is available at:
 https://dist.apache.org/repos/dist/dev/mesos/0.21.0-rc1/mesos-0.21.0.tar.gz
 
 The tag to be voted on is 0.21.0-rc1:
 https://git-wip-us.apache.org/repos/asf?p=mesos.git;a=commit;h=0.21.0-rc1
 
 The MD5 checksum of the tarball can be found at:
 https://dist.apache.org/repos/dist/dev/mesos/0.21.0-rc1/mesos-0.21.0.tar.gz.md5
 
 The signature of the tarball can be found at:
 https://dist.apache.org/repos/dist/dev/mesos/0.21.0-rc1/mesos-0.21.0.tar.gz.asc
 
 The PGP key used to sign the release is here:
 https://dist.apache.org/repos/dist/release/mesos/KEYS
 
 The JAR is up in Maven in a staging repository here:
 https://repository.apache.org/content/repositories/orgapachemesos-1038
 
 Please vote on releasing this package as Apache Mesos 0.21.0!
 
 The vote is open until Sat Nov  8 15:09:48 PST 2014 and passes if a
 majority of at least 3 +1 PMC votes are cast.
 
 [ ] +1 Release this package as Apache Mesos 0.21.0
 [ ] -1 Do not release this package because ...
 
 Thanks,
 
 Ian Downes
 

-- 
Cheers,
Timothy St. Clair
Red Hat Inc.


Re: Review Request 27660: Made ProcessBase::route hard-fail rather than soft fail

2014-11-06 Thread Cody Maloney

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27660/
---

(Updated Nov. 6, 2014, 4:44 p.m.)


Review request for mesos and Adam B.


Changes
---

Remove extra return...


Repository: mesos-git


Description
---

It is an API expectation that the path start with '/', and if it doesn't it 
should be a programmer error.

No code actually checked the return code. If a developer did get this wrong 
previously it would silently do nothing and in debugging their endpoint just 
wouldn't appear. This makes it simple/easy for them to find.


Diffs (updated)
-

  3rdparty/libprocess/include/process/process.hpp 
81a1f7a3fece78bb7df1e86850572922c8704ea4 
  3rdparty/libprocess/src/process.cpp 85fb9958342f0bcdde322d9c55333126e6f86668 

Diff: https://reviews.apache.org/r/27660/diff/


Testing
---

make distcheck on ubuntu 14.04


Thanks,

Cody Maloney



Re: Review Request 27483: Fetcher uses hadoop to fetch URIs regardless of the url scheme.

2014-11-06 Thread Mesos ReviewBot

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27483/#review60182
---


Patch looks great!

Reviews applied: [27483]

All tests passed.

- Mesos ReviewBot


On Nov. 6, 2014, 4:12 a.m., Ankur Chauhan wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27483/
 ---
 
 (Updated Nov. 6, 2014, 4:12 a.m.)
 
 
 Review request for mesos, Timothy Chen, Timothy St. Clair, and Vinod Kone.
 
 
 Bugs: MESOS-1711
 https://issues.apache.org/jira/browse/MESOS-1711
 
 
 Repository: mesos-git
 
 
 Description
 ---
 
 Previously, the fetcher used a hardcoded list of schemes to determine what 
 URIs could be fetched by hadoop (if available). This is now changed such that 
 we first check if hadoop can fetch them for us and then we fallback to the 
 os::net and then a local copy method (same as it used to be). This allows 
 users to fetch artifacts from arbitrary filesystems as long as hadoop is 
 correctly configured (in core-site.xml).
 
 
 Diffs
 -
 
   src/launcher/fetcher.cpp bd95928bc3191970330e839bcf41e343d5142c54 
 
 Diff: https://reviews.apache.org/r/27483/diff/
 
 
 Testing
 ---
 
 make check
 sudo bin/mesos-tests.sh --verbose
 support/mesos-style.py
 
 
 Thanks,
 
 Ankur Chauhan
 




Re: Unable to install Mesos on Ubuntu 14.04. Error during 'make'

2014-11-06 Thread Joris Van Remoortere
If you run: mesos-slave.sh --help
Then you will see all the options you can pass to the slave. One of them
is --port=VALUE

Joris

On Thu, Nov 6, 2014 at 6:39 AM, Sweta Rani swetarani3...@gmail.com wrote:

 How can I change port for mesos-slave from default 5051 to 5052?

 *Sweta Rani*

 On Thu, Nov 6, 2014 at 7:58 PM, Sweta Rani swetarani3...@gmail.com
 wrote:

  Alex,
 
  Done with master but unable to run slave using command
 ./bin/mesos-slave.sh
  --master=127.0.0.1:5050
  Also the port 5051 is allocated with random work '-'. See this:
  proxima@proxima:~/mesos/build$ netstat -ltnp | grep ':5051'
  (Not all processes could be identified, non-owned process info
   will not be shown, you would have to be root to see it all.)
  tcp0  0 0.0.0.0:50510.0.0.0:*
  LISTEN  -
  proxima@proxima:~/mesos/build$ sudo kill -9 -
  kill: failed to parse argument: '-'
  proxima@proxima:~/mesos/build$ netstat -ltnp | grep ':5051'
  (Not all processes could be identified, non-owned process info
   will not be shown, you would have to be root to see it all.)
  tcp0  0 0.0.0.0:50510.0.0.0:*
  LISTEN  -
  proxima@:~/mesos/build$ ./bin/mesos-slave.sh --master=127.0.0.1:5050
  WARNING: Logging before InitGoogleLogging() is written to STDERR
  F1106 19:51:22.737992 6367 process.cpp:1532] Failed to initialize, bind
  0.0.0.0:5051: Address already in use [98]
  *** Check failure stack trace: ***
  Aborted (core dumped)
 
 
  *Sweta Rani*
 
  On Thu, Nov 6, 2014 at 7:07 PM, Alex Rukletsov a...@mesosphere.io
 wrote:
 
  Sweta,
 
  have you tried another port?
 
  On Thu, Nov 6, 2014 at 2:24 PM, Sweta Rani swetarani3...@gmail.com
  wrote:
 
   Joris
  
   Done with above commands, but now it gives following error:
   WARNING: Logging before InitGoogleLogging() is written to STDERR
   F1106 18:50:48.117947  9706 process.cpp:1532] Failed to initialize,
 bind
   127.0.0.1:5050: Address already in use [98]
   *** Check failure stack trace: ***
  
   I tried to solve it by
  
  
 
 http://stackoverflow.com/questions/10745878/ubuntu-error-with-apache-98address-already-in-use
but it didnt work for me.
  
  
   *Sweta Rani*
  
   On Wed, Nov 5, 2014 at 3:50 AM, Joris Van Remoortere 
  jo...@mesosphere.io
   wrote:
  
You need to make the directory and set the permissions to the user
 you
   are
running the master as.
   
For example:
   
sudo mkdir /var/lib/mesos
sudo chown jmlvanre /var/lib/mesos
   
[jmlvanre] run mesos master...
   
On Tue, Nov 4, 2014 at 2:07 PM, Sweta Rani swetarani3...@gmail.com
 
wrote:
   
 Thanks for your help. I have installed mesos but  unable to start
  mesos
 master by the command
 ./bin/mesos-master.sh --ip=127.0.0.1 --work_dir=/var/lib/mesos.

 The file exists as mesos-master.sh.in in the bin folder.

 It reports:
 Failed to create work directory '/var/lib/mesos': Permission
 denied

 *Sweta Rani*

 On Tue, Nov 4, 2014 at 2:50 PM, Adam Bordelon a...@mesosphere.io
 
wrote:

  As an alternate workaround, if you don't need the Java support
   (you're
 not
  running a JVM framework like Spark/Hadoop/Marathon), then you
  could
   use
  `configure --disable-java` to skip all the maven/pom build
 steps.
 
  By disabling Java, you could at least check that the rest of
 Mesos
builds
  and runs fine, which would still work with C++, Python, or Go
frameworks,
  or even any Java frameworks using the pure Java bindings
 (jesos).
 
  On Mon, Nov 3, 2014 at 2:01 PM, Joris Van Remoortere 
 jo...@mesosphere.io
  wrote:
 
   Did you verify that the settings.xml file was parsed using mvn
  -X?
  
   Specifically after the first few lines you will see something
  along
the
   lines of:
   [DEBUG] Reading global settings from
/usr/share/maven/conf/settings.xml
   [DEBUG] Reading user settings from /home/abc/.m2/settings.xml
   [DEBUG] Using local repository at /home/abc/.m2/repository
  
   Make sure your settings file is in the right location.
  
   On Mon, Nov 3, 2014 at 11:51 AM, Sweta Rani 
   swetarani3...@gmail.com

   wrote:
  
I use campus proxy configuration for internet. But I am
  unable to
run
   make
in installing Mesos on my Ubuntu -14.04, as it ends up with
ERROR(1).
   Upon
searching, I created settings.xml file inside .m2 with the
   required
configuration but failed again. Please help me out, as it is
greatly
hindering me from my work.
   
On Sat Nov 01 2014 at 3:38:57 AM Joris Van Remoortere 
   jo...@mesosphere.io

wrote:
   
 I think this suggests you're fetching through a proxy:
 *Proxy request sent, awaiting response... 200 OK*

 When I wget this is my output:
  wget
   

Review Request 27685: Introduced TASK_ERROR for unrecoverable errors

2014-11-06 Thread Dominic Hamon

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27685/
---

Review request for mesos and Vinod Kone.


Bugs: MESOS-1143
https://issues.apache.org/jira/browse/MESOS-1143


Repository: mesos-git


Description
---

see summary


Diffs
-

  include/mesos/mesos.proto 6c846f2a688ae721d2895ce51d5d1e68faef28ca 
  src/master/master.cpp a860496d9d75ae4e007d35627d778caf10d08428 
  src/tests/master_authorization_tests.cpp 
5ae855e59036c6cbcec15db5449620a8e5b2aa44 

Diff: https://reviews.apache.org/r/27685/diff/


Testing
---

make check


Thanks,

Dominic Hamon



Re: Review Request 27483: Fetcher uses hadoop to fetch URIs regardless of the url scheme.

2014-11-06 Thread Mesos ReviewBot

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27483/#review60192
---


Patch looks great!

Reviews applied: [27483]

All tests passed.

- Mesos ReviewBot


On Nov. 6, 2014, 4:12 a.m., Ankur Chauhan wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27483/
 ---
 
 (Updated Nov. 6, 2014, 4:12 a.m.)
 
 
 Review request for mesos, Timothy Chen, Timothy St. Clair, and Vinod Kone.
 
 
 Bugs: MESOS-1711
 https://issues.apache.org/jira/browse/MESOS-1711
 
 
 Repository: mesos-git
 
 
 Description
 ---
 
 Previously, the fetcher used a hardcoded list of schemes to determine what 
 URIs could be fetched by hadoop (if available). This is now changed such that 
 we first check if hadoop can fetch them for us and then we fallback to the 
 os::net and then a local copy method (same as it used to be). This allows 
 users to fetch artifacts from arbitrary filesystems as long as hadoop is 
 correctly configured (in core-site.xml).
 
 
 Diffs
 -
 
   src/launcher/fetcher.cpp bd95928bc3191970330e839bcf41e343d5142c54 
 
 Diff: https://reviews.apache.org/r/27483/diff/
 
 
 Testing
 ---
 
 make check
 sudo bin/mesos-tests.sh --verbose
 support/mesos-style.py
 
 
 Thanks,
 
 Ankur Chauhan
 




Re: Review Request 27447: mesos: Replaced the ip and port pairs from UPID class and process namespace with Node class

2014-11-06 Thread Dominic Hamon

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27447/#review60193
---



src/master/master.cpp
https://reviews.apache.org/r/27447/#comment101551

possibility for a future patch: replace info_ ip and port with a protobuf 
'node'?



src/master/master.cpp
https://reviews.apache.org/r/27447/#comment101553

there really should be a utility method on node for this. especially as the 
check will need to be extended to ::1.

Maybe 'isLoopback()'?



src/slave/http.cpp
https://reviews.apache.org/r/27447/#comment101556

clumsy wrapping:

Trystring masterHostname = net::getHostname(
slave-master.get().node.ip);



src/slave/slave.cpp
https://reviews.apache.org/r/27447/#comment101557

hm. so there is a getHostname but we don't use it in other places (see 
previous review where i recommended adding a host() method to Node).


- Dominic Hamon


On Nov. 5, 2014, 6:49 p.m., Evelina Dumitrescu wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27447/
 ---
 
 (Updated Nov. 5, 2014, 6:49 p.m.)
 
 
 Review request for mesos, Dominic Hamon, Joris Van Remoortere, and switched 
 to 'mcypark'.
 
 
 Bugs: MESOS-2019
 https://issues.apache.org/jira/browse/MESOS-2019
 
 
 Repository: mesos-git
 
 
 Description
 ---
 
 At the moment, the Node class is used to keep a mapping from a socket to the 
 ip  port pair in the process namespace.
 I want to propose to extend its use by replacing the ip  port fields from 
 the UPID class and process namespace with this type.
 
 
 Diffs
 -
 
   src/common/protobuf_utils.cpp 33ce782960e88e07b910f844fbdcae4c0f19d9a7 
   src/master/master.cpp a860496d9d75ae4e007d35627d778caf10d08428 
   src/sched/sched.cpp e5f828d0bf9dd03a01920634abfae685a7861b44 
   src/scheduler/scheduler.cpp c74187cfeb57a748fd57198d3badd4b8773490eb 
   src/slave/http.cpp 965d78c3e3ea2e3c922f62aabf6fc87c5cf2ab8d 
   src/slave/slave.cpp dbfd1a8101d78dee8ea3ac19d990a6a7892e59be 
   src/tests/fetcher_tests.cpp e026e875c7197cd214ef0432d7c40207553c8671 
   src/tests/files_tests.cpp a696aa22d56b37ee70c6e64c81a849da6d436451 
   src/tests/gc_tests.cpp f7747e2999cbd64ab0cd0321cca0021489dc3346 
   src/tests/logging_tests.cpp 58e9b3399c3de28b1c5b3a8ec8fdf85b7a31c6d5 
   src/tests/master_contender_detector_tests.cpp 
 8f7fb18f0ca985364565fc4cd50f113b1461fbe7 
   src/tests/master_tests.cpp a6d1a4a5b7d50a88e5b101c43f62d21300cf 
   src/tests/monitor_tests.cpp 4b950e14bd94cdfa21212268b56bebdc1200078d 
 
 Diff: https://reviews.apache.org/r/27447/diff/
 
 
 Testing
 ---
 
 make check
 
 
 Thanks,
 
 Evelina Dumitrescu
 




Re: Review Request 27447: mesos: Replaced the ip and port pairs from UPID class and process namespace with Node class

2014-11-06 Thread Evelina Dumitrescu


 On Nov. 6, 2014, 5:49 p.m., Dominic Hamon wrote:
  src/master/master.cpp, line 321
  https://reviews.apache.org/r/27447/diff/4/?file=751194#file751194line321
 
  there really should be a utility method on node for this. especially as 
  the check will need to be extended to ::1.
  
  Maybe 'isLoopback()'?

I intend to add for the IPAddress struct some methods to check for inaddr_any 
and loopback IPs:

 bool is_loopback() const
  {
if(family == AF_INET)
{
  return ipv4_addr.s_addr == INADDR_LOOPBACK;
}
else if(family == AF_INET6)
{
  IPAddress ipv6_loopback(in6addr_loopback);
  return ipv6_loopback == ipv6_addr;
}
return false;
  }

  bool is_inaddrany() const
  {
if(family == AF_INET)
{
  return ipv4_addr.s_addr == INADDR_ANY;
}
else if(family == AF_INET6)
{
  IPAddress ipv6_inaddr_any(in6addr_any);
  return ipv6_inaddr_any == ipv6_addr;
}
return false;
  }


- Evelina


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27447/#review60193
---


On Nov. 6, 2014, 2:49 a.m., Evelina Dumitrescu wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27447/
 ---
 
 (Updated Nov. 6, 2014, 2:49 a.m.)
 
 
 Review request for mesos, Dominic Hamon, Joris Van Remoortere, and switched 
 to 'mcypark'.
 
 
 Bugs: MESOS-2019
 https://issues.apache.org/jira/browse/MESOS-2019
 
 
 Repository: mesos-git
 
 
 Description
 ---
 
 At the moment, the Node class is used to keep a mapping from a socket to the 
 ip  port pair in the process namespace.
 I want to propose to extend its use by replacing the ip  port fields from 
 the UPID class and process namespace with this type.
 
 
 Diffs
 -
 
   src/common/protobuf_utils.cpp 33ce782960e88e07b910f844fbdcae4c0f19d9a7 
   src/master/master.cpp a860496d9d75ae4e007d35627d778caf10d08428 
   src/sched/sched.cpp e5f828d0bf9dd03a01920634abfae685a7861b44 
   src/scheduler/scheduler.cpp c74187cfeb57a748fd57198d3badd4b8773490eb 
   src/slave/http.cpp 965d78c3e3ea2e3c922f62aabf6fc87c5cf2ab8d 
   src/slave/slave.cpp dbfd1a8101d78dee8ea3ac19d990a6a7892e59be 
   src/tests/fetcher_tests.cpp e026e875c7197cd214ef0432d7c40207553c8671 
   src/tests/files_tests.cpp a696aa22d56b37ee70c6e64c81a849da6d436451 
   src/tests/gc_tests.cpp f7747e2999cbd64ab0cd0321cca0021489dc3346 
   src/tests/logging_tests.cpp 58e9b3399c3de28b1c5b3a8ec8fdf85b7a31c6d5 
   src/tests/master_contender_detector_tests.cpp 
 8f7fb18f0ca985364565fc4cd50f113b1461fbe7 
   src/tests/master_tests.cpp a6d1a4a5b7d50a88e5b101c43f62d21300cf 
   src/tests/monitor_tests.cpp 4b950e14bd94cdfa21212268b56bebdc1200078d 
 
 Diff: https://reviews.apache.org/r/27447/diff/
 
 
 Testing
 ---
 
 make check
 
 
 Thanks,
 
 Evelina Dumitrescu
 




Re: Review Request 27446: libprocess: Replaced the ip and port pairs from UPID class and process namespace with Node class

2014-11-06 Thread Evelina Dumitrescu


 On Nov. 6, 2014, 5:44 p.m., Dominic Hamon wrote:
  3rdparty/libprocess/src/http.cpp, line 90
  https://reviews.apache.org/r/27446/diff/3/?file=751387#file751387line90
 
  actually, a utility method on Node to return the string host might be 
  useful. Or a Trystring host, actually.

I think we can just add stringify(node) since the Node class overloads the  
operator.


 On Nov. 6, 2014, 5:44 p.m., Dominic Hamon wrote:
  3rdparty/libprocess/src/http.cpp, line 84
  https://reviews.apache.org/r/27446/diff/3/?file=751387#file751387line84
 
  worth adding an accessor to Node to return the in_addr*?

I want to replace the uint32_t type with IPAddress for the ip field and all the 
inet_ntop/inet_pton calls will be handled in the IPAddress struct. For example, 
we can replace this call with stringify method for IPAddress and overload the 
 operator:

inline
  std::ostream 
operator (std::ostream  stream, const IPAddress  ipaddr)
{
  if (ipaddr.family == AF_INET)
{
  char str[INET_ADDRSTRLEN];
  struct in_addr addr;
  inet_ntop (AF_INET, addr, str, INET_ADDRSTRLEN);
  stream  str;
}
  else
{
  char str[INET6_ADDRSTRLEN];
  struct in6_addr addr;
  inet_ntop (AF_INET6, addr, str, INET6_ADDRSTRLEN);
  stream  str;
}
  return stream;
}


- Evelina


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27446/#review60189
---


On Nov. 6, 2014, 3:07 a.m., Evelina Dumitrescu wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27446/
 ---
 
 (Updated Nov. 6, 2014, 3:07 a.m.)
 
 
 Review request for mesos, Dominic Hamon, Joris Van Remoortere, and switched 
 to 'mcypark'.
 
 
 Bugs: MESOS-2019
 https://issues.apache.org/jira/browse/MESOS-2019
 
 
 Repository: mesos-git
 
 
 Description
 ---
 
 At the moment, the Node class is used to keep a mapping from a socket to the 
 ip  port pair in the process namespace.
 I want to propose to extend its use by replacing the ip  port fields from 
 the UPID class and process namespace with this type.
 
 
 Diffs
 -
 
   3rdparty/libprocess/include/process/node.hpp 
 7a96894456683be60595a8f96f85df37947ff5b6 
   3rdparty/libprocess/include/process/pid.hpp 
 23453229c2a868a62b143416de33cf630268a64e 
   3rdparty/libprocess/include/process/process.hpp 
 81a1f7a3fece78bb7df1e86850572922c8704ea4 
   3rdparty/libprocess/src/http.cpp 4ef00d11de4ff4e7eb4e44aac204147799f85ac2 
   3rdparty/libprocess/src/pid.cpp 20ff25c0959c247687955535f02e0397653af52a 
   3rdparty/libprocess/src/process.cpp 
 85fb9958342f0bcdde322d9c55333126e6f86668 
   3rdparty/libprocess/src/tests/benchmarks.cpp 
 3177a8ecac6adf8b52c3160587dc3a4fe2707d26 
   3rdparty/libprocess/src/tests/http_tests.cpp 
 a1c3685c88d268c7ecdc2dad22608198dc0b295d 
   3rdparty/libprocess/src/tests/metrics_tests.cpp 
 33539e45bab8f79afa773a325465016f01fd3b75 
   3rdparty/libprocess/src/tests/process_tests.cpp 
 b985fb77ea05fae5c0b144ea48814acc7bb5135b 
 
 Diff: https://reviews.apache.org/r/27446/diff/
 
 
 Testing
 ---
 
 make check
 
 
 Thanks,
 
 Evelina Dumitrescu
 




Re: Review Request 27446: libprocess: Replaced the ip and port pairs from UPID class and process namespace with Node class

2014-11-06 Thread Evelina Dumitrescu


 On Nov. 6, 2014, 5:44 p.m., Dominic Hamon wrote:
  3rdparty/libprocess/include/process/pid.hpp, line 59
  https://reviews.apache.org/r/27446/diff/3/?file=751385#file751385line59
 
  this can be simplified by adding an operator  to Node.

I didn't noticed that. Actually there is a  operator defined for the Node 
class.


- Evelina


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27446/#review60189
---


On Nov. 6, 2014, 3:07 a.m., Evelina Dumitrescu wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27446/
 ---
 
 (Updated Nov. 6, 2014, 3:07 a.m.)
 
 
 Review request for mesos, Dominic Hamon, Joris Van Remoortere, and switched 
 to 'mcypark'.
 
 
 Bugs: MESOS-2019
 https://issues.apache.org/jira/browse/MESOS-2019
 
 
 Repository: mesos-git
 
 
 Description
 ---
 
 At the moment, the Node class is used to keep a mapping from a socket to the 
 ip  port pair in the process namespace.
 I want to propose to extend its use by replacing the ip  port fields from 
 the UPID class and process namespace with this type.
 
 
 Diffs
 -
 
   3rdparty/libprocess/include/process/node.hpp 
 7a96894456683be60595a8f96f85df37947ff5b6 
   3rdparty/libprocess/include/process/pid.hpp 
 23453229c2a868a62b143416de33cf630268a64e 
   3rdparty/libprocess/include/process/process.hpp 
 81a1f7a3fece78bb7df1e86850572922c8704ea4 
   3rdparty/libprocess/src/http.cpp 4ef00d11de4ff4e7eb4e44aac204147799f85ac2 
   3rdparty/libprocess/src/pid.cpp 20ff25c0959c247687955535f02e0397653af52a 
   3rdparty/libprocess/src/process.cpp 
 85fb9958342f0bcdde322d9c55333126e6f86668 
   3rdparty/libprocess/src/tests/benchmarks.cpp 
 3177a8ecac6adf8b52c3160587dc3a4fe2707d26 
   3rdparty/libprocess/src/tests/http_tests.cpp 
 a1c3685c88d268c7ecdc2dad22608198dc0b295d 
   3rdparty/libprocess/src/tests/metrics_tests.cpp 
 33539e45bab8f79afa773a325465016f01fd3b75 
   3rdparty/libprocess/src/tests/process_tests.cpp 
 b985fb77ea05fae5c0b144ea48814acc7bb5135b 
 
 Diff: https://reviews.apache.org/r/27446/diff/
 
 
 Testing
 ---
 
 make check
 
 
 Thanks,
 
 Evelina Dumitrescu
 




Re: Review Request 27675: Updated upgrades.md to reflect changes on AuthenticationStartMessage.

2014-11-06 Thread Kapil Arya

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27675/#review60206
---

Ship it!


Ship It!

- Kapil Arya


On Nov. 6, 2014, 5:51 a.m., Till Toenshoff wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27675/
 ---
 
 (Updated Nov. 6, 2014, 5:51 a.m.)
 
 
 Review request for mesos, Adam B and Vinod Kone.
 
 
 Bugs: MESOS-2001
 https://issues.apache.org/jira/browse/MESOS-2001
 
 
 Repository: mesos-git
 
 
 Description
 ---
 
 see summary.
 
 
 Diffs
 -
 
   docs/upgrades.md 3aff335 
 
 Diff: https://reviews.apache.org/r/27675/diff/
 
 
 Testing
 ---
 
 
 Thanks,
 
 Till Toenshoff
 




Re: Review Request 27447: mesos: Replaced the ip and port pairs from UPID class and process namespace with Node class

2014-11-06 Thread Evelina Dumitrescu


 On Nov. 6, 2014, 5:49 p.m., Dominic Hamon wrote:
  src/slave/slave.cpp, line 300
  https://reviews.apache.org/r/27447/diff/4/?file=751198#file751198line300
 
  hm. so there is a getHostname but we don't use it in other places (see 
  previous review where i recommended adding a host() method to Node).

Here is returned the name of the host. In the other case, is is used the string 
representation of the numerical value of the IP.


- Evelina


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27447/#review60193
---


On Nov. 6, 2014, 2:49 a.m., Evelina Dumitrescu wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27447/
 ---
 
 (Updated Nov. 6, 2014, 2:49 a.m.)
 
 
 Review request for mesos, Dominic Hamon, Joris Van Remoortere, and switched 
 to 'mcypark'.
 
 
 Bugs: MESOS-2019
 https://issues.apache.org/jira/browse/MESOS-2019
 
 
 Repository: mesos-git
 
 
 Description
 ---
 
 At the moment, the Node class is used to keep a mapping from a socket to the 
 ip  port pair in the process namespace.
 I want to propose to extend its use by replacing the ip  port fields from 
 the UPID class and process namespace with this type.
 
 
 Diffs
 -
 
   src/common/protobuf_utils.cpp 33ce782960e88e07b910f844fbdcae4c0f19d9a7 
   src/master/master.cpp a860496d9d75ae4e007d35627d778caf10d08428 
   src/sched/sched.cpp e5f828d0bf9dd03a01920634abfae685a7861b44 
   src/scheduler/scheduler.cpp c74187cfeb57a748fd57198d3badd4b8773490eb 
   src/slave/http.cpp 965d78c3e3ea2e3c922f62aabf6fc87c5cf2ab8d 
   src/slave/slave.cpp dbfd1a8101d78dee8ea3ac19d990a6a7892e59be 
   src/tests/fetcher_tests.cpp e026e875c7197cd214ef0432d7c40207553c8671 
   src/tests/files_tests.cpp a696aa22d56b37ee70c6e64c81a849da6d436451 
   src/tests/gc_tests.cpp f7747e2999cbd64ab0cd0321cca0021489dc3346 
   src/tests/logging_tests.cpp 58e9b3399c3de28b1c5b3a8ec8fdf85b7a31c6d5 
   src/tests/master_contender_detector_tests.cpp 
 8f7fb18f0ca985364565fc4cd50f113b1461fbe7 
   src/tests/master_tests.cpp a6d1a4a5b7d50a88e5b101c43f62d21300cf 
   src/tests/monitor_tests.cpp 4b950e14bd94cdfa21212268b56bebdc1200078d 
 
 Diff: https://reviews.apache.org/r/27447/diff/
 
 
 Testing
 ---
 
 make check
 
 
 Thanks,
 
 Evelina Dumitrescu
 




Re: Review Request 27447: mesos: Replaced the ip and port pairs from UPID class and process namespace with Node class

2014-11-06 Thread Evelina Dumitrescu


 On Nov. 6, 2014, 5:49 p.m., Dominic Hamon wrote:
  src/master/master.cpp, line 290
  https://reviews.apache.org/r/27447/diff/4/?file=751194#file751194line290
 
  possibility for a future patch: replace info_ ip and port with a 
  protobuf 'node'?

I will consider this.


- Evelina


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27447/#review60193
---


On Nov. 6, 2014, 2:49 a.m., Evelina Dumitrescu wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27447/
 ---
 
 (Updated Nov. 6, 2014, 2:49 a.m.)
 
 
 Review request for mesos, Dominic Hamon, Joris Van Remoortere, and switched 
 to 'mcypark'.
 
 
 Bugs: MESOS-2019
 https://issues.apache.org/jira/browse/MESOS-2019
 
 
 Repository: mesos-git
 
 
 Description
 ---
 
 At the moment, the Node class is used to keep a mapping from a socket to the 
 ip  port pair in the process namespace.
 I want to propose to extend its use by replacing the ip  port fields from 
 the UPID class and process namespace with this type.
 
 
 Diffs
 -
 
   src/common/protobuf_utils.cpp 33ce782960e88e07b910f844fbdcae4c0f19d9a7 
   src/master/master.cpp a860496d9d75ae4e007d35627d778caf10d08428 
   src/sched/sched.cpp e5f828d0bf9dd03a01920634abfae685a7861b44 
   src/scheduler/scheduler.cpp c74187cfeb57a748fd57198d3badd4b8773490eb 
   src/slave/http.cpp 965d78c3e3ea2e3c922f62aabf6fc87c5cf2ab8d 
   src/slave/slave.cpp dbfd1a8101d78dee8ea3ac19d990a6a7892e59be 
   src/tests/fetcher_tests.cpp e026e875c7197cd214ef0432d7c40207553c8671 
   src/tests/files_tests.cpp a696aa22d56b37ee70c6e64c81a849da6d436451 
   src/tests/gc_tests.cpp f7747e2999cbd64ab0cd0321cca0021489dc3346 
   src/tests/logging_tests.cpp 58e9b3399c3de28b1c5b3a8ec8fdf85b7a31c6d5 
   src/tests/master_contender_detector_tests.cpp 
 8f7fb18f0ca985364565fc4cd50f113b1461fbe7 
   src/tests/master_tests.cpp a6d1a4a5b7d50a88e5b101c43f62d21300cf 
   src/tests/monitor_tests.cpp 4b950e14bd94cdfa21212268b56bebdc1200078d 
 
 Diff: https://reviews.apache.org/r/27447/diff/
 
 
 Testing
 ---
 
 make check
 
 
 Thanks,
 
 Evelina Dumitrescu
 




Re: Review Request 27670: Bug fix: Check for non-zero status code and hadoop client not found

2014-11-06 Thread Ankur Chauhan

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27670/
---

(Updated Nov. 6, 2014, 7:03 p.m.)


Review request for mesos, Benjamin Hindman and Vinod Kone.


Changes
---

Added MESOS-1711 as the 'bugs' field.


Bugs: MESOS-1711
https://issues.apache.org/jira/browse/MESOS-1711


Repository: mesos-git


Description
---

@benh noticed that the fetcher would detect the condition where hadoop returns 
a success in case of a success non zero value


Diffs
-

  src/launcher/fetcher.cpp bd95928bc3191970330e839bcf41e343d5142c54 

Diff: https://reviews.apache.org/r/27670/diff/


Testing
---

make check
sudo bin/mesos-tests.sh --verbose
support/mesos-style.py


Thanks,

Ankur Chauhan



Re: Review Request 26476: Remove dynamic allocation from Option.

2014-11-06 Thread Joris Van Remoortere

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26476/
---

(Updated Nov. 6, 2014, 7:13 p.m.)


Review request for mesos, Benjamin Hindman and Niklas Nielsen.


Changes
---

Add the JIRA issue for this.


Bugs: MESOS-1991
https://issues.apache.org/jira/browse/MESOS-1991


Repository: mesos-git


Description
---

Remove dynamic allocations from Option class.


Diffs
-

  3rdparty/libprocess/3rdparty/stout/include/stout/option.hpp 47fe92c 

Diff: https://reviews.apache.org/r/26476/diff/


Testing
---

make check
support/mesos-style.py
valgrind (reduced allocation count)


Thanks,

Joris Van Remoortere



Re: Unable to install Mesos on Ubuntu 14.04. Error during 'make'

2014-11-06 Thread Sweta Rani
Cloning from svn repo gave following error:

proxima@proxima-HP-Pavilion-m6-Notebook-PC:~/mesodoc$ rake
Cloning Apache Mesos codebase
Cloning into '/home/proxima/mesodoc/tmp/mesos'...
remote: Counting objects: 1075, done.
remote: Compressing objects: 100% (935/935), done.
remote: Total 1075 (delta 221), reused 537 (delta 83)
Receiving objects: 100% (1075/1075), 26.77 MiB | 171.00 KiB/s, done.
Resolving deltas: 100% (221/221), done.
Checking connectivity... done.
Updating latest documentation from the Apache Mesos codebase
Updating docs to the latest version
rake aborted!
Errno::ENOENT: No such file or directory -
/home/proxima/mesodoc/source/documentation/latest/
running-torque-or-mpi-on-mesos.md
/home/proxima/mesodoc/Rakefile:43:in `block in top (required)'
Tasks: TOP = default = update_docs

Neither I could get assets folder in this. How will I make changes?

On Thu Nov 06 2014 at 10:55:38 PM Joris Van Remoortere jo...@mesosphere.io
wrote:

 If you run: mesos-slave.sh --help
 Then you will see all the options you can pass to the slave. One of them
 is --port=VALUE

 Joris

 On Thu, Nov 6, 2014 at 6:39 AM, Sweta Rani swetarani3...@gmail.com
 wrote:

  How can I change port for mesos-slave from default 5051 to 5052?
 
  *Sweta Rani*
 
  On Thu, Nov 6, 2014 at 7:58 PM, Sweta Rani swetarani3...@gmail.com
  wrote:
 
   Alex,
  
   Done with master but unable to run slave using command
  ./bin/mesos-slave.sh
   --master=127.0.0.1:5050
   Also the port 5051 is allocated with random work '-'. See this:
   proxima@proxima:~/mesos/build$ netstat -ltnp | grep ':5051'
   (Not all processes could be identified, non-owned process info
will not be shown, you would have to be root to see it all.)
   tcp0  0 0.0.0.0:50510.0.0.0:*
   LISTEN  -
   proxima@proxima:~/mesos/build$ sudo kill -9 -
   kill: failed to parse argument: '-'
   proxima@proxima:~/mesos/build$ netstat -ltnp | grep ':5051'
   (Not all processes could be identified, non-owned process info
will not be shown, you would have to be root to see it all.)
   tcp0  0 0.0.0.0:50510.0.0.0:*
   LISTEN  -
   proxima@:~/mesos/build$ ./bin/mesos-slave.sh --master=127.0.0.1:5050
   WARNING: Logging before InitGoogleLogging() is written to STDERR
   F1106 19:51:22.737992 6367 process.cpp:1532] Failed to initialize, bind
   0.0.0.0:5051: Address already in use [98]
   *** Check failure stack trace: ***
   Aborted (core dumped)
  
  
   *Sweta Rani*
  
   On Thu, Nov 6, 2014 at 7:07 PM, Alex Rukletsov a...@mesosphere.io
  wrote:
  
   Sweta,
  
   have you tried another port?
  
   On Thu, Nov 6, 2014 at 2:24 PM, Sweta Rani swetarani3...@gmail.com
   wrote:
  
Joris
   
Done with above commands, but now it gives following error:
WARNING: Logging before InitGoogleLogging() is written to STDERR
F1106 18:50:48.117947  9706 process.cpp:1532] Failed to initialize,
  bind
127.0.0.1:5050: Address already in use [98]
*** Check failure stack trace: ***
   
I tried to solve it by
   
   
  
  http://stackoverflow.com/questions/10745878/ubuntu-
 error-with-apache-98address-already-in-use
 but it didnt work for me.
   
   
*Sweta Rani*
   
On Wed, Nov 5, 2014 at 3:50 AM, Joris Van Remoortere 
   jo...@mesosphere.io
wrote:
   
 You need to make the directory and set the permissions to the user
  you
are
 running the master as.

 For example:

 sudo mkdir /var/lib/mesos
 sudo chown jmlvanre /var/lib/mesos

 [jmlvanre] run mesos master...

 On Tue, Nov 4, 2014 at 2:07 PM, Sweta Rani 
 swetarani3...@gmail.com
  
 wrote:

  Thanks for your help. I have installed mesos but  unable to
 start
   mesos
  master by the command
  ./bin/mesos-master.sh --ip=127.0.0.1
 --work_dir=/var/lib/mesos.
 
  The file exists as mesos-master.sh.in in the bin folder.
 
  It reports:
  Failed to create work directory '/var/lib/mesos': Permission
  denied
 
  *Sweta Rani*
 
  On Tue, Nov 4, 2014 at 2:50 PM, Adam Bordelon 
 a...@mesosphere.io
  
 wrote:
 
   As an alternate workaround, if you don't need the Java support
(you're
  not
   running a JVM framework like Spark/Hadoop/Marathon), then you
   could
use
   `configure --disable-java` to skip all the maven/pom build
  steps.
  
   By disabling Java, you could at least check that the rest of
  Mesos
 builds
   and runs fine, which would still work with C++, Python, or Go
 frameworks,
   or even any Java frameworks using the pure Java bindings
  (jesos).
  
   On Mon, Nov 3, 2014 at 2:01 PM, Joris Van Remoortere 
  jo...@mesosphere.io
   wrote:
  
Did you verify that the settings.xml file was parsed using
 mvn
   -X?
   
Specifically after the first few lines you will see
 something
   along
 the
lines of:
[DEBUG] Reading global 

Re: Review Request 27556: Updated docs/configuration.md to reflect current state.

2014-11-06 Thread Ian Downes

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27556/#review60229
---

Ship it!


Ship It!

- Ian Downes


On Nov. 4, 2014, 7:37 p.m., Kapil Arya wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27556/
 ---
 
 (Updated Nov. 4, 2014, 7:37 p.m.)
 
 
 Review request for mesos, Ian Downes and Till Toenshoff.
 
 
 Bugs: MESOS-2037
 https://issues.apache.org/jira/browse/MESOS-2037
 
 
 Repository: mesos-git
 
 
 Description
 ---
 
 Copied the output of --help messages from configure, master, and slave.
 
 The option tables are now in html format.
 
 Also removed the explanation of the default configure flags.
 
 
 Diffs
 -
 
   docs/configuration.md 5845ae324181d01cb65990fbf8dd38a621e1c351 
 
 Diff: https://reviews.apache.org/r/27556/diff/
 
 
 Testing
 ---
 
 Pushed and tested output on github at:
 https://github.com/karya0/mesos/blob/modules/docs/configuration.md
 
 
 Thanks,
 
 Kapil Arya
 




Re: [VOTE] Release Apache Mesos 0.21.0 (rc1)

2014-11-06 Thread Ian Downes
Apologies: I used support/tag.sh but had a local branch *and* local tag and
it pushed the branch only.

$ git ls-remote --tags origin-wip | grep 0.21.0
a7733493dc9e6f2447f825671d8a745602c9bf7a refs/tags/0.21.0-rc1

On Thu, Nov 6, 2014 at 8:11 AM, Tim St Clair tstcl...@redhat.com wrote:

 $ git tag -l | grep 21

 $ git branch -r
   origin/0.21.0-rc1

 It looks like you created a branch vs. tag ...?

 Cheers,
 Tim

 - Original Message -
  From: Ian Downes ian.dow...@gmail.com
  To: dev@mesos.apache.org, u...@mesos.apache.org
  Sent: Wednesday, November 5, 2014 5:12:52 PM
  Subject: [VOTE] Release Apache Mesos 0.21.0 (rc1)
 
  Hi all,
 
  Please vote on releasing the following candidate as Apache Mesos 0.21.0.
 
 
  0.21.0 includes the following:
 
 
  State reconciliation for frameworks
  Support for Mesos modules
  Task status now includes source and reason
  A shared filesystem isolator
  A pid namespace isolator
 
  The CHANGELOG for the release is available at:
 
 https://git-wip-us.apache.org/repos/asf?p=mesos.git;a=blob_plain;f=CHANGELOG;hb=0.21.0-rc1
 
 
 
  The candidate for Mesos 0.21.0 release is available at:
 
 https://dist.apache.org/repos/dist/dev/mesos/0.21.0-rc1/mesos-0.21.0.tar.gz
 
  The tag to be voted on is 0.21.0-rc1:
 
 https://git-wip-us.apache.org/repos/asf?p=mesos.git;a=commit;h=0.21.0-rc1
 
  The MD5 checksum of the tarball can be found at:
 
 https://dist.apache.org/repos/dist/dev/mesos/0.21.0-rc1/mesos-0.21.0.tar.gz.md5
 
  The signature of the tarball can be found at:
 
 https://dist.apache.org/repos/dist/dev/mesos/0.21.0-rc1/mesos-0.21.0.tar.gz.asc
 
  The PGP key used to sign the release is here:
  https://dist.apache.org/repos/dist/release/mesos/KEYS
 
  The JAR is up in Maven in a staging repository here:
  https://repository.apache.org/content/repositories/orgapachemesos-1038
 
  Please vote on releasing this package as Apache Mesos 0.21.0!
 
  The vote is open until Sat Nov  8 15:09:48 PST 2014 and passes if a
  majority of at least 3 +1 PMC votes are cast.
 
  [ ] +1 Release this package as Apache Mesos 0.21.0
  [ ] -1 Do not release this package because ...
 
  Thanks,
 
  Ian Downes
 

 --
 Cheers,
 Timothy St. Clair
 Red Hat Inc.



Re: [VOTE] Release Apache Mesos 0.21.0 (rc1)

2014-11-06 Thread Tom Arnfeld
+1




`make check` passed on Ubuntu 12.04 LTS (kernel 3.2.0-67)


--


Tom Arnfeld

Developer // DueDil





(+44) 7525940046

25 Christopher Street, London, EC2A 2BS

On Thu, Nov 6, 2014 at 8:43 PM, Ian Downes idow...@twitter.com.invalid
wrote:

 Apologies: I used support/tag.sh but had a local branch *and* local tag and
 it pushed the branch only.
 $ git ls-remote --tags origin-wip | grep 0.21.0
 a7733493dc9e6f2447f825671d8a745602c9bf7a refs/tags/0.21.0-rc1
 On Thu, Nov 6, 2014 at 8:11 AM, Tim St Clair tstcl...@redhat.com wrote:
 $ git tag -l | grep 21

 $ git branch -r
   origin/0.21.0-rc1

 It looks like you created a branch vs. tag ...?

 Cheers,
 Tim

 - Original Message -
  From: Ian Downes ian.dow...@gmail.com
  To: dev@mesos.apache.org, u...@mesos.apache.org
  Sent: Wednesday, November 5, 2014 5:12:52 PM
  Subject: [VOTE] Release Apache Mesos 0.21.0 (rc1)
 
  Hi all,
 
  Please vote on releasing the following candidate as Apache Mesos 0.21.0.
 
 
  0.21.0 includes the following:
 
 
  State reconciliation for frameworks
  Support for Mesos modules
  Task status now includes source and reason
  A shared filesystem isolator
  A pid namespace isolator
 
  The CHANGELOG for the release is available at:
 
 https://git-wip-us.apache.org/repos/asf?p=mesos.git;a=blob_plain;f=CHANGELOG;hb=0.21.0-rc1
 
 
 
  The candidate for Mesos 0.21.0 release is available at:
 
 https://dist.apache.org/repos/dist/dev/mesos/0.21.0-rc1/mesos-0.21.0.tar.gz
 
  The tag to be voted on is 0.21.0-rc1:
 
 https://git-wip-us.apache.org/repos/asf?p=mesos.git;a=commit;h=0.21.0-rc1
 
  The MD5 checksum of the tarball can be found at:
 
 https://dist.apache.org/repos/dist/dev/mesos/0.21.0-rc1/mesos-0.21.0.tar.gz.md5
 
  The signature of the tarball can be found at:
 
 https://dist.apache.org/repos/dist/dev/mesos/0.21.0-rc1/mesos-0.21.0.tar.gz.asc
 
  The PGP key used to sign the release is here:
  https://dist.apache.org/repos/dist/release/mesos/KEYS
 
  The JAR is up in Maven in a staging repository here:
  https://repository.apache.org/content/repositories/orgapachemesos-1038
 
  Please vote on releasing this package as Apache Mesos 0.21.0!
 
  The vote is open until Sat Nov  8 15:09:48 PST 2014 and passes if a
  majority of at least 3 +1 PMC votes are cast.
 
  [ ] +1 Release this package as Apache Mesos 0.21.0
  [ ] -1 Do not release this package because ...
 
  Thanks,
 
  Ian Downes
 

 --
 Cheers,
 Timothy St. Clair
 Red Hat Inc.


Re: Review Request 27651: Updated CRAMMD5Authentication tests to additionally run against the test authenticatee module.

2014-11-06 Thread Till Toenshoff

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27651/
---

(Updated Nov. 6, 2014, 9:05 p.m.)


Review request for mesos, Adam B and Vinod Kone.


Changes
---

updated jira id to tie into specific instead of epic


Bugs: MESOS-2041
https://issues.apache.org/jira/browse/MESOS-2041


Repository: mesos-git


Description
---

CRAMMD5Authentication tests now cover all default and modularized 
authenticator/authenticatee combinations.
Also adds a test for missing credential for the authenticatee.


Diffs
-

  src/authentication/cram_md5/authenticatee.hpp 3088a77 
  src/tests/cram_md5_authentication_tests.cpp 74ea2ad 
  src/tests/module.hpp 7970c28 
  src/tests/module.cpp 482ed22 

Diff: https://reviews.apache.org/r/27651/diff/


Testing
---

make check


Thanks,

Till Toenshoff



Re: Review Request 27651: Updated CRAMMD5Authentication tests to additionally run against the test authenticatee module.

2014-11-06 Thread Kapil Arya

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27651/#review60242
---

Ship it!


Ship It!

- Kapil Arya


On Nov. 6, 2014, 4:05 p.m., Till Toenshoff wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27651/
 ---
 
 (Updated Nov. 6, 2014, 4:05 p.m.)
 
 
 Review request for mesos, Adam B and Vinod Kone.
 
 
 Bugs: MESOS-2041
 https://issues.apache.org/jira/browse/MESOS-2041
 
 
 Repository: mesos-git
 
 
 Description
 ---
 
 CRAMMD5Authentication tests now cover all default and modularized 
 authenticator/authenticatee combinations.
 Also adds a test for missing credential for the authenticatee.
 
 
 Diffs
 -
 
   src/authentication/cram_md5/authenticatee.hpp 3088a77 
   src/tests/cram_md5_authentication_tests.cpp 74ea2ad 
   src/tests/module.hpp 7970c28 
   src/tests/module.cpp 482ed22 
 
 Diff: https://reviews.apache.org/r/27651/diff/
 
 
 Testing
 ---
 
 make check
 
 
 Thanks,
 
 Till Toenshoff
 




Re: Review Request 27619: Updated CRAMMD5Authentication tests to additionally run against the test authenticator module.

2014-11-06 Thread Kapil Arya

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27619/#review60243
---

Ship it!


Ship It!

- Kapil Arya


On Nov. 5, 2014, 7:05 p.m., Till Toenshoff wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27619/
 ---
 
 (Updated Nov. 5, 2014, 7:05 p.m.)
 
 
 Review request for mesos, Adam B, Kapil Arya, and Vinod Kone.
 
 
 Bugs: MESOS-1894
 https://issues.apache.org/jira/browse/MESOS-1894
 
 
 Repository: mesos-git
 
 
 Description
 ---
 
 see summary.
 
 
 Diffs
 -
 
   src/authentication/cram_md5/authenticator.hpp 601248d 
   src/tests/cram_md5_authentication_tests.cpp 74ea2ad 
   src/tests/module.hpp 7970c28 
   src/tests/module.cpp 482ed22 
 
 Diff: https://reviews.apache.org/r/27619/diff/
 
 
 Testing
 ---
 
 make check
 
 
 Thanks,
 
 Till Toenshoff
 




Re: Review Request 27493: Added CRAM-MD5 Test Authenticatee Module.

2014-11-06 Thread Kapil Arya

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27493/#review60244
---

Ship it!



src/authentication/authenticatee.hpp
https://reviews.apache.org/r/27493/#comment101596

Don't we want to put the directory name in it as well?



src/authentication/authenticatee.hpp
https://reviews.apache.org/r/27493/#comment101597

Do we need a newline here?


- Kapil Arya


On Nov. 6, 2014, 5:33 a.m., Till Toenshoff wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27493/
 ---
 
 (Updated Nov. 6, 2014, 5:33 a.m.)
 
 
 Review request for mesos, Adam B, Kapil Arya, and Vinod Kone.
 
 
 Bugs: MESOS-2001
 https://issues.apache.org/jira/browse/MESOS-2001
 
 
 Repository: mesos-git
 
 
 Description
 ---
 
 see summary.
 
 
 Diffs
 -
 
   src/Makefile.am 9ab3b9c 
   src/authentication/authenticatee.hpp PRE-CREATION 
   src/authentication/cram_md5/authenticatee.hpp 3088a77 
   src/examples/test_authentication_modules.cpp PRE-CREATION 
   src/examples/test_authenticator_module.cpp 4398d71 
   src/module/authenticatee.hpp PRE-CREATION 
   src/module/manager.cpp 0d71e33 
 
 Diff: https://reviews.apache.org/r/27493/diff/
 
 
 Testing
 ---
 
 make check
 
 Note: All three CRAM-MD5 Authenticatee patches need to get applied before 
 running make check! 
 
 
 Thanks,
 
 Till Toenshoff
 




Review Request 27701: Added check for apr and svn headers besides libraries

2014-11-06 Thread Timothy Chen

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27701/
---

Review request for mesos and Benjamin Hindman.


Repository: mesos-git


Description
---

Added check for apr and svn headers besides libraries


Diffs
-

  3rdparty/libprocess/configure.ac 18eb681a5779b15cc5c7b7187ba5da8b4059c530 
  configure.ac 4504006b25e2b3e45fefd36c9ed3a46e6c1d8430 

Diff: https://reviews.apache.org/r/27701/diff/


Testing
---

./bootstrap  configure  make


Thanks,

Timothy Chen



Re: Review Request 27494: Integrated CRAM-MD5 Test Authenticatee module into slave.

2014-11-06 Thread Kapil Arya

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27494/#review60251
---



src/slave/flags.hpp
https://reviews.apache.org/r/27494/#comment101599

Remove space at the end of the line (before newline char)?


- Kapil Arya


On Nov. 6, 2014, 6:21 a.m., Till Toenshoff wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27494/
 ---
 
 (Updated Nov. 6, 2014, 6:21 a.m.)
 
 
 Review request for mesos, Adam B and Vinod Kone.
 
 
 Bugs: MESOS-2001
 https://issues.apache.org/jira/browse/MESOS-2001
 
 
 Repository: mesos-git
 
 
 Description
 ---
 
 Also fixes messages.proto to use a raw bytestream instead of a string for 
 AuthenticationStartMessage as non CRAM-MD5 authentication may transmit binary 
 data.
 Note that the change of AuthenticationStartMessage does basically have no 
 impact on C++ based proto code other than the prevention of a warning due to 
 non-UTF8 characters being encoded. That does in fact occur when using non 
 CRAM-MD5 based SASL authentication mechanisms.
 
 Note that this patch covers modularized slave authentication only. Framework 
 authentication currently is currently covered by the default (built-in) 
 implementation. There will be a subsequent patch for modularized framework 
 authentication. 
 
 
 Diffs
 -
 
   src/messages/messages.proto de0e2a2 
   src/sched/sched.cpp e5f828d 
   src/scheduler/scheduler.cpp c74187c 
   src/slave/constants.hpp 701dd89 
   src/slave/constants.cpp d6ad78c 
   src/slave/flags.hpp efbd35d 
   src/slave/slave.hpp 5b082fc 
   src/slave/slave.cpp dbfd1a8 
 
 Diff: https://reviews.apache.org/r/27494/diff/
 
 
 Testing
 ---
 
 make check
 
 NOTE all three CRAM-MD5 authenticatee module related RRs need to get applied 
 before running make check.
 
 
 Thanks,
 
 Till Toenshoff
 




Build failed in Jenkins: Mesos-Trunk-Ubuntu-Build-Out-Of-Src-Disable-Java-Disable-Python-Disable-Webui #2533

2014-11-06 Thread Apache Jenkins Server
 
mechanisms: CRAM-MD5
I1106 21:50:13.831666 10762 authenticatee.hpp:250] Attempting to authenticate 
with mechanism 'CRAM-MD5'
I1106 21:50:13.831845 10767 authenticator.hpp:275] Received SASL authentication 
start
I1106 21:50:13.831892 10767 authenticator.hpp:397] Authentication requires more 
steps
I1106 21:50:13.831956 10767 authenticatee.hpp:270] Received SASL authentication 
step
I1106 21:50:13.832058 10767 authenticator.hpp:303] Received SASL authentication 
step
I1106 21:50:13.832082 10767 auxprop.cpp:99] Request to lookup properties for 
user: 'test-principal' realm: 'pomona.apache.org' server FQDN: 
'pomona.apache.org' SASL_AUXPROP_VERIFY_AGAINST_HASH: false 
SASL_AUXPROP_OVERRIDE: false SASL_AUXPROP_AUTHZID: false 
I1106 21:50:13.832095 10767 auxprop.cpp:171] Looking up auxiliary property 
'*userPassword'
I1106 21:50:13.832171 10767 auxprop.cpp:171] Looking up auxiliary property 
'*cmusaslsecretCRAM-MD5'
I1106 21:50:13.832197 10767 auxprop.cpp:99] Request to lookup properties for 
user: 'test-principal' realm: 'pomona.apache.org' server FQDN: 
'pomona.apache.org' SASL_AUXPROP_VERIFY_AGAINST_HASH: false 
SASL_AUXPROP_OVERRIDE: false SASL_AUXPROP_AUTHZID: true 
I1106 21:50:13.832206 10767 auxprop.cpp:121] Skipping auxiliary property 
'*userPassword' since SASL_AUXPROP_AUTHZID == true
I1106 21:50:13.832211 10767 auxprop.cpp:121] Skipping auxiliary property 
'*cmusaslsecretCRAM-MD5' since SASL_AUXPROP_AUTHZID == true
I1106 21:50:13.832224 10767 authenticator.hpp:389] Authentication success
I1106 21:50:13.832356 10755 authenticatee.hpp:310] Authentication success
I1106 21:50:13.832412 10756 master.cpp:3957] Successfully authenticated 
principal 'test-principal' at slave(51)@67.195.81.187:42265
I1106 21:50:13.832648 10755 slave.cpp:722] Successfully authenticated with 
master master@67.195.81.187:42265
I1106 21:50:13.832798 10755 slave.cpp:1052] Will retry registration in 
4.334745ms if necessary
I1106 21:50:13.833009 10756 master.cpp:3063] Registering slave at 
slave(51)@67.195.81.187:42265 (pomona.apache.org) with id 
20141106-215013-3142697795-42265-10739-S0
I1106 21:50:13.833498 10754 registrar.cpp:445] Applied 1 operations in 72886ns; 
attempting to update the 'registry'
I1106 21:50:13.836539 10763 log.cpp:680] Attempting to append 316 bytes to the 
log
I1106 21:50:13.836688 10759 coordinator.cpp:340] Coordinator attempting to 
write APPEND action at position 3
I1106 21:50:13.837621 10765 replica.cpp:508] Replica received write request for 
position 3
I1106 21:50:13.838235 10765 leveldb.cpp:343] Persisting action (335 bytes) to 
leveldb took 580209ns
I1106 21:50:13.838243 10753 slave.cpp:1052] Will retry registration in 
34.28488ms if necessary
I1106 21:50:13.838259 10765 replica.cpp:676] Persisted action at 3
I1106 21:50:13.838379 10766 master.cpp:3051] Ignoring register slave message 
from slave(51)@67.195.81.187:42265 (pomona.apache.org) as admission is already 
in progress
I1106 21:50:13.839057 10755 replica.cpp:655] Replica received learned notice 
for position 3
I1106 21:50:13.839463 10755 leveldb.cpp:343] Persisting action (337 bytes) to 
leveldb took 359634ns
I1106 21:50:13.839483 10755 replica.cpp:676] Persisted action at 3
I1106 21:50:13.839498 10755 replica.cpp:661] Replica learned APPEND action at 
position 3
I1106 21:50:13.840714 10765 registrar.cpp:490] Successfully updated the 
'registry' in 7.154176ms
I1106 21:50:13.841016 10768 log.cpp:699] Attempting to truncate the log to 3
I1106 21:50:13.841130 10763 coordinator.cpp:340] Coordinator attempting to 
write TRUNCATE action at position 4
I1106 21:50:13.841276 10766 master.cpp:3117] Registered slave 
20141106-215013-3142697795-42265-10739-S0 at slave(51)@67.195.81.187:42265 
(pomona.apache.org) with cpus(*):2; mem(*):1024; disk(*):1024; 
ports(*):[31000-32000]
I1106 21:50:13.841367 10759 slave.cpp:2538] Received ping from 
slave-observer(52)@67.195.81.187:42265
I1106 21:50:13.841577 10755 hierarchical_allocator_process.hpp:442] Added slave 
20141106-215013-3142697795-42265-10739-S0 (pomona.apache.org) with cpus(*):2; 
mem(*):1024; disk(*):1024; ports(*):[31000-32000] (and cpus(*):2; mem(*):1024; 
disk(*):1024; ports(*):[31000-32000] available)
I1106 21:50:13.841961 10764 replica.cpp:508] Replica received write request for 
position 4
I1106 21:50:13.900509 10755 hierarchical_allocator_process.hpp:679] Performed 
allocation for slave 20141106-215013-3142697795-42265-10739-S0 in 104213ns
I1106 21:50:13.900746 10759 slave.cpp:756] Registered with master 
master@67.195.81.187:42265; given slave ID 
20141106-215013-3142697795-42265-10739-S0
I1106 21:50:13.900835 10754 status_update_manager.cpp:178] Resuming sending 
status updates
I1106 21:50:13.901160 10764 leveldb.cpp:343] Persisting action (16 bytes) to 
leveldb took 742687ns
I1106 21:50:13.901190 10764 replica.cpp:676] Persisted action at 4
I1106 21:50:13.901860 10753 replica.cpp:655] Replica received learned notice 
for position 4
I1106 21:50:13.902582 10753 leveldb.cpp:343] Persisting action

Re: Review Request 27701: Added check for apr and svn headers besides libraries

2014-11-06 Thread Mesos ReviewBot

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27701/#review60254
---


Bad patch!

Reviews applied: [27701]

Failed command: ./support/apply-review.sh -n -r 27701

Error:
 2014-11-06 21:52:25 URL:https://reviews.apache.org/r/27701/diff/raw/ 
[5155/5155] - 27701.patch [1]
Successfully applied: Added check for apr and svn headers besides libraries

Added check for apr and svn headers besides libraries


Review: https://reviews.apache.org/r/27701
No files to lint

ERROR: Commit spanning multiple projects.

Please use separate commits for mesos, libprocess and stout.

Paths grouped by project:
mesos:
  configure.ac
libprocess:
  3rdparty/libprocess/configure.ac
Failed to commit patch

- Mesos ReviewBot


On Nov. 6, 2014, 9:37 p.m., Timothy Chen wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27701/
 ---
 
 (Updated Nov. 6, 2014, 9:37 p.m.)
 
 
 Review request for mesos and Benjamin Hindman.
 
 
 Repository: mesos-git
 
 
 Description
 ---
 
 Added check for apr and svn headers besides libraries
 
 
 Diffs
 -
 
   3rdparty/libprocess/configure.ac 18eb681a5779b15cc5c7b7187ba5da8b4059c530 
   configure.ac 4504006b25e2b3e45fefd36c9ed3a46e6c1d8430 
 
 Diff: https://reviews.apache.org/r/27701/diff/
 
 
 Testing
 ---
 
 ./bootstrap  configure  make
 
 
 Thanks,
 
 Timothy Chen
 




Build failed in Jenkins: Mesos-Trunk-Ubuntu-Build-In-Src-Set-JAVA_HOME #2255

2014-11-06 Thread Apache Jenkins Server
See 
https://builds.apache.org/job/Mesos-Trunk-Ubuntu-Build-In-Src-Set-JAVA_HOME/2255/changes

Changes:

[idownes] Updated docs/configuration.md.

--
[...truncated 65481 lines...]
I1106 21:52:48.399585 26726 master.cpp:3907] Using default CRAM-MD5 
authenticator
I1106 21:52:48.399814 26735 authenticator.hpp:169] Creating new server SASL 
connection
I1106 21:52:48.399960 26730 authenticatee.hpp:224] Received SASL authentication 
mechanisms: CRAM-MD5
I1106 21:52:48.399986 26730 authenticatee.hpp:250] Attempting to authenticate 
with mechanism 'CRAM-MD5'
I1106 21:52:48.400079 26730 authenticator.hpp:275] Received SASL authentication 
start
I1106 21:52:48.400126 26730 authenticator.hpp:397] Authentication requires more 
steps
I1106 21:52:48.400244 26729 authenticatee.hpp:270] Received SASL authentication 
step
I1106 21:52:48.400329 26729 authenticator.hpp:303] Received SASL authentication 
step
I1106 21:52:48.400352 26729 auxprop.cpp:99] Request to lookup properties for 
user: 'test-principal' realm: 'proserpina.apache.org' server FQDN: 
'proserpina.apache.org' SASL_AUXPROP_VERIFY_AGAINST_HASH: false 
SASL_AUXPROP_OVERRIDE: false SASL_AUXPROP_AUTHZID: false 
I1106 21:52:48.400362 26729 auxprop.cpp:171] Looking up auxiliary property 
'*userPassword'
I1106 21:52:48.400395 26729 auxprop.cpp:171] Looking up auxiliary property 
'*cmusaslsecretCRAM-MD5'
I1106 21:52:48.400413 26729 auxprop.cpp:99] Request to lookup properties for 
user: 'test-principal' realm: 'proserpina.apache.org' server FQDN: 
'proserpina.apache.org' SASL_AUXPROP_VERIFY_AGAINST_HASH: false 
SASL_AUXPROP_OVERRIDE: false SASL_AUXPROP_AUTHZID: true 
I1106 21:52:48.400420 26729 auxprop.cpp:121] Skipping auxiliary property 
'*userPassword' since SASL_AUXPROP_AUTHZID == true
I1106 21:52:48.400426 26729 auxprop.cpp:121] Skipping auxiliary property 
'*cmusaslsecretCRAM-MD5' since SASL_AUXPROP_AUTHZID == true
I1106 21:52:48.400437 26729 authenticator.hpp:389] Authentication success
I1106 21:52:48.400516 26734 authenticatee.hpp:310] Authentication success
I1106 21:52:48.400591 26736 master.cpp:3957] Successfully authenticated 
principal 'test-principal' at 
scheduler-cb0abfc3-17a9-4471-b54a-8528a79e5279@67.195.81.189:56794
I1106 21:52:48.400902 26728 sched.cpp:357] Successfully authenticated with 
master master@67.195.81.189:56794
I1106 21:52:48.400949 26728 sched.cpp:476] Sending registration request to 
master@67.195.81.189:56794
I1106 21:52:48.401149 26736 master.cpp:1378] Received registration request for 
framework 'default' at 
scheduler-cb0abfc3-17a9-4471-b54a-8528a79e5279@67.195.81.189:56794
I1106 21:52:48.401211 26736 master.cpp:1337] Authorizing framework principal 
'test-principal' to receive offers for role '*'
I1106 21:52:48.401655 26737 master.cpp:1442] Registering framework 
20141106-215248-3176252227-56794-26710- (default) at 
scheduler-cb0abfc3-17a9-4471-b54a-8528a79e5279@67.195.81.189:56794
I1106 21:52:48.401954 26727 hierarchical_allocator_process.hpp:329] Added 
framework 20141106-215248-3176252227-56794-26710-
I1106 21:52:48.401974 26727 hierarchical_allocator_process.hpp:697] No 
resources available to allocate!
I1106 21:52:48.401985 26727 hierarchical_allocator_process.hpp:659] Performed 
allocation for 0 slaves in 13303ns
I1106 21:52:48.402000 26737 sched.cpp:407] Framework registered with 
20141106-215248-3176252227-56794-26710-
I1106 21:52:48.402042 26737 sched.cpp:421] Scheduler::registered took 16407ns
I1106 21:52:48.402606 26730 master.cpp:784] Framework 
20141106-215248-3176252227-56794-26710- (default) at 
scheduler-cb0abfc3-17a9-4471-b54a-8528a79e5279@67.195.81.189:56794 disconnected
I1106 21:52:48.402632 26730 master.cpp:1747] Disconnecting framework 
20141106-215248-3176252227-56794-26710- (default) at 
scheduler-cb0abfc3-17a9-4471-b54a-8528a79e5279@67.195.81.189:56794
I1106 21:52:48.402660 26730 master.cpp:1763] Deactivating framework 
20141106-215248-3176252227-56794-26710- (default) at 
scheduler-cb0abfc3-17a9-4471-b54a-8528a79e5279@67.195.81.189:56794
I1106 21:52:48.402724 26730 master.cpp:806] Giving framework 
20141106-215248-3176252227-56794-26710- (default) at 
scheduler-cb0abfc3-17a9-4471-b54a-8528a79e5279@67.195.81.189:56794 0ns to 
failover
I1106 21:52:48.403165 26739 hierarchical_allocator_process.hpp:405] Deactivated 
framework 20141106-215248-3176252227-56794-26710-
I1106 21:52:48.403228 26727 master.cpp:3708] Framework failover timeout, 
removing framework 20141106-215248-3176252227-56794-26710- (default) at 
scheduler-cb0abfc3-17a9-4471-b54a-8528a79e5279@67.195.81.189:56794
I1106 21:52:48.403256 26727 master.cpp:4269] Removing framework 
20141106-215248-3176252227-56794-26710- (default) at 
scheduler-cb0abfc3-17a9-4471-b54a-8528a79e5279@67.195.81.189:56794
I1106 21:52:48.403633 26725 hierarchical_allocator_process.hpp:360] Removed 
framework 20141106-215248-3176252227-56794-26710-
2014-11-06 
21:52:48,421:26710

Re: Review Request 26736: Ensure docker calls stop if a docker_stop_timeout is provided Added the flag docker_stop_timeout that defaults to 0, if the timeout is 0 the a docker kill will be run, otherw

2014-11-06 Thread Ryan Thomas

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26736/
---

(Updated Nov. 6, 2014, 9:55 p.m.)


Review request for mesos, Benjamin Hindman and Timothy Chen.


Changes
---

Rebase from master


Bugs: MESOS-1925
https://issues.apache.org/jira/browse/MESOS-1925


Repository: mesos-git


Description
---

Ensure docker calls stop if a docker_stop_timeout is provided Added the flag 
docker_stop_timeout that defaults to 0, if the timeout is 0 the a docker kill 
will be run, otherwise a docker stop with the timeout in seconds is used. A 
negative value will result in a Failure.


Diffs (updated)
-

  src/docker/docker.hpp 2dc692c51d6f9d45d8d7c59f8678991fd80c6e89 
  src/docker/docker.cpp 0c0a1bfd3c1260ebdf342d0a6e0e04d7b6a111a4 
  src/slave/containerizer/docker.cpp a6689203adbdcb0ad12583389eaeb83329e4ef6b 
  src/slave/flags.hpp efbd35dd744d8c8a53c36dd5cdb8d1c50fcfec3a 
  src/tests/docker_containerizer_tests.cpp 
9d4ccc57f58d61c62aab5cdc79a129e987920bf6 
  src/tests/docker_tests.cpp ff06a01d7f20656c05bcf5afd3bc10059dc5378d 

Diff: https://reviews.apache.org/r/26736/diff/


Testing
---

make check
sudo bin/mesos-tests.sh --gtest_filter=DOCKER --verbose 
support/mesos-style.py


Thanks,

Ryan Thomas



Re: Review Request 27701: Added check for apr and svn headers besides libraries

2014-11-06 Thread Timothy Chen

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27701/
---

(Updated Nov. 6, 2014, 10:02 p.m.)


Review request for mesos and Benjamin Hindman.


Repository: mesos-git


Description (updated)
---

Review: https://reviews.apache.org/r/27701


Diffs (updated)
-

  configure.ac 4504006b25e2b3e45fefd36c9ed3a46e6c1d8430 

Diff: https://reviews.apache.org/r/27701/diff/


Testing
---

./bootstrap  configure  make


Thanks,

Timothy Chen



Review Request 27704: Added check for apr and svn headers besides libraries in 3rdparty

2014-11-06 Thread Timothy Chen

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27704/
---

Review request for mesos and Benjamin Hindman.


Repository: mesos-git


Description
---

Added check for apr and svn headers besides libraries in 3rdparty


Diffs
-

  3rdparty/libprocess/configure.ac 18eb681a5779b15cc5c7b7187ba5da8b4059c530 

Diff: https://reviews.apache.org/r/27704/diff/


Testing
---

bootstrap  configure  make


Thanks,

Timothy Chen



Re: Review Request 27704: Added check for apr and svn headers besides libraries in 3rdparty

2014-11-06 Thread Timothy Chen

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27704/
---

(Updated Nov. 6, 2014, 10:03 p.m.)


Review request for mesos and Benjamin Hindman.


Repository: mesos-git


Description
---

Added check for apr and svn headers besides libraries in 3rdparty


Diffs
-

  3rdparty/libprocess/configure.ac 18eb681a5779b15cc5c7b7187ba5da8b4059c530 

Diff: https://reviews.apache.org/r/27704/diff/


Testing
---

bootstrap  configure  make


Thanks,

Timothy Chen



Re: Review Request 27315: Updated scheduler driver to exponentially backoff during registration retries.

2014-11-06 Thread Vinod Kone

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27315/
---

(Updated Nov. 6, 2014, 10:20 p.m.)


Review request for mesos and Ben Mahler.


Changes
---

benm's comments. NNFR.


Bugs: MESOS-1903
https://issues.apache.org/jira/browse/MESOS-1903


Repository: mesos-git


Description
---

Uses the same backoff (except no initial backoff) strategy used by the slave 
during registration.


Diffs (updated)
-

  src/Makefile.am 9ab3b9c05d435d18ed1c2966f695857fa205e9fd 
  src/sched/constants.hpp PRE-CREATION 
  src/sched/constants.cpp PRE-CREATION 
  src/sched/flags.hpp PRE-CREATION 
  src/sched/sched.cpp e5f828d0bf9dd03a01920634abfae685a7861b44 
  src/slave/slave.hpp 5b082fc6956238d27f5d3352a3b40ce778a1a3a4 
  src/slave/slave.cpp dbfd1a8101d78dee8ea3ac19d990a6a7892e59be 
  src/tests/fault_tolerance_tests.cpp 372c4fdec1ef70991d4d550d8d73ac363436fdb9 

Diff: https://reviews.apache.org/r/27315/diff/


Testing
---

make check


Thanks,

Vinod Kone



Re: [VOTE] Release Apache Mesos 0.21.0 (rc1)

2014-11-06 Thread Adam Bordelon
Also note, in addition to 'Support for Mesos modules' we also now allow
pluggable isolator modules.

On Thu, Nov 6, 2014 at 2:28 PM, Adam Bordelon a...@mesosphere.io wrote:

 make check passed on LinuxMint16 (Ubuntu 13.10)

 Would love to see the following two cherry-picked, especially the
 configuration.md change before you update the website for the release
 blog post.

 d09478d73cb3ddd76c7b886bef691c8034fa6908 Updated docs/configuration.md.
 c7227471f98c0dc62c8700d41534be142a3fcfad Fetcher uses Hadoop to fetch URIs
 with unknown schemes.


 On Thu, Nov 6, 2014 at 1:00 PM, Tom Arnfeld t...@duedil.com wrote:

 +1




 `make check` passed on Ubuntu 12.04 LTS (kernel 3.2.0-67)


 --


 Tom Arnfeld

 Developer // DueDil





 (+44) 7525940046

 25 Christopher Street, London, EC2A 2BS

 On Thu, Nov 6, 2014 at 8:43 PM, Ian Downes idow...@twitter.com.invalid
 wrote:

  Apologies: I used support/tag.sh but had a local branch *and* local tag
 and
  it pushed the branch only.
  $ git ls-remote --tags origin-wip | grep 0.21.0
  a7733493dc9e6f2447f825671d8a745602c9bf7a refs/tags/0.21.0-rc1
  On Thu, Nov 6, 2014 at 8:11 AM, Tim St Clair tstcl...@redhat.com
 wrote:
  $ git tag -l | grep 21
 
  $ git branch -r
origin/0.21.0-rc1
 
  It looks like you created a branch vs. tag ...?
 
  Cheers,
  Tim
 
  - Original Message -
   From: Ian Downes ian.dow...@gmail.com
   To: dev@mesos.apache.org, u...@mesos.apache.org
   Sent: Wednesday, November 5, 2014 5:12:52 PM
   Subject: [VOTE] Release Apache Mesos 0.21.0 (rc1)
  
   Hi all,
  
   Please vote on releasing the following candidate as Apache Mesos
 0.21.0.
  
  
   0.21.0 includes the following:
  
 
 
   State reconciliation for frameworks
   Support for Mesos modules
   Task status now includes source and reason
   A shared filesystem isolator
   A pid namespace isolator
  
   The CHANGELOG for the release is available at:
  
 
 https://git-wip-us.apache.org/repos/asf?p=mesos.git;a=blob_plain;f=CHANGELOG;hb=0.21.0-rc1
  
 
 
  
   The candidate for Mesos 0.21.0 release is available at:
  
 
 https://dist.apache.org/repos/dist/dev/mesos/0.21.0-rc1/mesos-0.21.0.tar.gz
  
   The tag to be voted on is 0.21.0-rc1:
  
 
 https://git-wip-us.apache.org/repos/asf?p=mesos.git;a=commit;h=0.21.0-rc1
  
   The MD5 checksum of the tarball can be found at:
  
 
 https://dist.apache.org/repos/dist/dev/mesos/0.21.0-rc1/mesos-0.21.0.tar.gz.md5
  
   The signature of the tarball can be found at:
  
 
 https://dist.apache.org/repos/dist/dev/mesos/0.21.0-rc1/mesos-0.21.0.tar.gz.asc
  
   The PGP key used to sign the release is here:
   https://dist.apache.org/repos/dist/release/mesos/KEYS
  
   The JAR is up in Maven in a staging repository here:
  
 https://repository.apache.org/content/repositories/orgapachemesos-1038
  
   Please vote on releasing this package as Apache Mesos 0.21.0!
  
   The vote is open until Sat Nov  8 15:09:48 PST 2014 and passes if a
   majority of at least 3 +1 PMC votes are cast.
  
   [ ] +1 Release this package as Apache Mesos 0.21.0
   [ ] -1 Do not release this package because ...
  
   Thanks,
  
   Ian Downes
  
 
  --
  Cheers,
  Timothy St. Clair
  Red Hat Inc.
 





Re: [VOTE] Release Apache Mesos 0.21.0 (rc1)

2014-11-06 Thread Adam Bordelon
make check passed on LinuxMint16 (Ubuntu 13.10)

Would love to see the following two cherry-picked, especially the
configuration.md change before you update the website for the release blog
post.

d09478d73cb3ddd76c7b886bef691c8034fa6908 Updated docs/configuration.md.
c7227471f98c0dc62c8700d41534be142a3fcfad Fetcher uses Hadoop to fetch URIs
with unknown schemes.

On Thu, Nov 6, 2014 at 1:00 PM, Tom Arnfeld t...@duedil.com wrote:

 +1




 `make check` passed on Ubuntu 12.04 LTS (kernel 3.2.0-67)


 --


 Tom Arnfeld

 Developer // DueDil





 (+44) 7525940046

 25 Christopher Street, London, EC2A 2BS

 On Thu, Nov 6, 2014 at 8:43 PM, Ian Downes idow...@twitter.com.invalid
 wrote:

  Apologies: I used support/tag.sh but had a local branch *and* local tag
 and
  it pushed the branch only.
  $ git ls-remote --tags origin-wip | grep 0.21.0
  a7733493dc9e6f2447f825671d8a745602c9bf7a refs/tags/0.21.0-rc1
  On Thu, Nov 6, 2014 at 8:11 AM, Tim St Clair tstcl...@redhat.com
 wrote:
  $ git tag -l | grep 21
 
  $ git branch -r
origin/0.21.0-rc1
 
  It looks like you created a branch vs. tag ...?
 
  Cheers,
  Tim
 
  - Original Message -
   From: Ian Downes ian.dow...@gmail.com
   To: dev@mesos.apache.org, u...@mesos.apache.org
   Sent: Wednesday, November 5, 2014 5:12:52 PM
   Subject: [VOTE] Release Apache Mesos 0.21.0 (rc1)
  
   Hi all,
  
   Please vote on releasing the following candidate as Apache Mesos
 0.21.0.
  
  
   0.21.0 includes the following:
  
 
 
   State reconciliation for frameworks
   Support for Mesos modules
   Task status now includes source and reason
   A shared filesystem isolator
   A pid namespace isolator
  
   The CHANGELOG for the release is available at:
  
 
 https://git-wip-us.apache.org/repos/asf?p=mesos.git;a=blob_plain;f=CHANGELOG;hb=0.21.0-rc1
  
 
 
  
   The candidate for Mesos 0.21.0 release is available at:
  
 
 https://dist.apache.org/repos/dist/dev/mesos/0.21.0-rc1/mesos-0.21.0.tar.gz
  
   The tag to be voted on is 0.21.0-rc1:
  
 
 https://git-wip-us.apache.org/repos/asf?p=mesos.git;a=commit;h=0.21.0-rc1
  
   The MD5 checksum of the tarball can be found at:
  
 
 https://dist.apache.org/repos/dist/dev/mesos/0.21.0-rc1/mesos-0.21.0.tar.gz.md5
  
   The signature of the tarball can be found at:
  
 
 https://dist.apache.org/repos/dist/dev/mesos/0.21.0-rc1/mesos-0.21.0.tar.gz.asc
  
   The PGP key used to sign the release is here:
   https://dist.apache.org/repos/dist/release/mesos/KEYS
  
   The JAR is up in Maven in a staging repository here:
  
 https://repository.apache.org/content/repositories/orgapachemesos-1038
  
   Please vote on releasing this package as Apache Mesos 0.21.0!
  
   The vote is open until Sat Nov  8 15:09:48 PST 2014 and passes if a
   majority of at least 3 +1 PMC votes are cast.
  
   [ ] +1 Release this package as Apache Mesos 0.21.0
   [ ] -1 Do not release this package because ...
  
   Thanks,
  
   Ian Downes
  
 
  --
  Cheers,
  Timothy St. Clair
  Red Hat Inc.
 



Build failed in Jenkins: Mesos-Trunk-Ubuntu-Build-Out-Of-Src-Disable-Java-Disable-Python-Disable-Webui #2534

2014-11-06 Thread Apache Jenkins Server
 if necessary
I1106 23:02:54.200470 28523 master.cpp:3063] Registering slave at 
slave(220)@67.195.81.186:57799 (penates.apache.org) with id 
20141106-230254-3125920579-57799-28500-S0
I1106 23:02:54.200935 28521 registrar.cpp:445] Applied 1 operations in 54768ns; 
attempting to update the 'registry'
I1106 23:02:54.203893 28517 log.cpp:680] Attempting to append 318 bytes to the 
log
I1106 23:02:54.204052 28522 coordinator.cpp:340] Coordinator attempting to 
write APPEND action at position 3
I1106 23:02:54.205118 28526 replica.cpp:508] Replica received write request for 
position 3
I1106 23:02:54.205765 28526 leveldb.cpp:343] Persisting action (337 bytes) to 
leveldb took 613666ns
I1106 23:02:54.205795 28526 replica.cpp:676] Persisted action at 3
I1106 23:02:54.206488 28517 replica.cpp:655] Replica received learned notice 
for position 3
I1106 23:02:54.207089 28517 leveldb.cpp:343] Persisting action (339 bytes) to 
leveldb took 538746ns
I1106 23:02:54.207120 28517 replica.cpp:676] Persisted action at 3
I1106 23:02:54.207144 28517 replica.cpp:661] Replica learned APPEND action at 
position 3
I1106 23:02:54.208432 28525 registrar.cpp:490] Successfully updated the 
'registry' in 7.442176ms
I1106 23:02:54.208752 28515 log.cpp:699] Attempting to truncate the log to 3
I1106 23:02:54.208971 28514 coordinator.cpp:340] Coordinator attempting to 
write TRUNCATE action at position 4
I1106 23:02:54.209048 28516 slave.cpp:2538] Received ping from 
slave-observer(198)@67.195.81.186:57799
I1106 23:02:54.209051 28524 master.cpp:3117] Registered slave 
20141106-230254-3125920579-57799-28500-S0 at slave(220)@67.195.81.186:57799 
(penates.apache.org) with cpus(*):2; mem(*):1024; disk(*):1024; 
ports(*):[31000-32000]
I1106 23:02:54.209252 28522 hierarchical_allocator_process.hpp:442] Added slave 
20141106-230254-3125920579-57799-28500-S0 (penates.apache.org) with cpus(*):2; 
mem(*):1024; disk(*):1024; ports(*):[31000-32000] (and cpus(*):2; mem(*):1024; 
disk(*):1024; ports(*):[31000-32000] available)
I1106 23:02:54.209369 28516 slave.cpp:756] Registered with master 
master@67.195.81.186:57799; given slave ID 
20141106-230254-3125920579-57799-28500-S0
I1106 23:02:54.209388 28522 hierarchical_allocator_process.hpp:679] Performed 
allocation for slave 20141106-230254-3125920579-57799-28500-S0 in 65073ns
I1106 23:02:54.209499 28519 status_update_manager.cpp:178] Resuming sending 
status updates
I1106 23:02:54.209813 28521 master.cpp:693] Master terminating
I1106 23:02:54.210151 28522 replica.cpp:508] Replica received write request for 
position 4
I1106 23:02:54.210420 28524 slave.cpp:2623] master@67.195.81.186:57799 exited
W1106 23:02:54.210445 28524 slave.cpp:2626] Master disconnected! Waiting for a 
new master to be elected
I1106 23:02:54.210741 28522 leveldb.cpp:343] Persisting action (16 bytes) to 
leveldb took 557583ns
I1106 23:02:54.210770 28522 replica.cpp:676] Persisted action at 4
I1106 23:02:54.211560 28522 replica.cpp:655] Replica received learned notice 
for position 4
I1106 23:02:54.211920 28522 leveldb.cpp:343] Persisting action (18 bytes) to 
leveldb took 333280ns
I1106 23:02:54.211990 28522 leveldb.cpp:401] Deleting ~2 keys from leveldb took 
41644ns
I1106 23:02:54.212013 28522 replica.cpp:676] Persisted action at 4
I1106 23:02:54.212035 28522 replica.cpp:661] Replica learned TRUNCATE action at 
position 4
I1106 23:02:54.216079 28500 slave.cpp:484] Slave terminating
[   OK ] CredentialsTest.authenticatedSlaveText (151 ms)
[ RUN  ] CredentialsTest.authenticatedSlave
Using temporary directory '/tmp/CredentialsTest_authenticatedSlave_9KPvWC'
I1106 23:02:54.295954 28500 leveldb.cpp:176] Opened db in 2.619329ms
I1106 23:02:54.296875 28500 leveldb.cpp:183] Compacted db in 887542ns
I1106 23:02:54.296923 28500 leveldb.cpp:198] Created db iterator in 16639ns
I1106 23:02:54.296942 28500 leveldb.cpp:204] Seeked to beginning of db in 1843ns
I1106 23:02:54.296952 28500 leveldb.cpp:273] Iterated through 0 keys in the db 
in 422ns
I1106 23:02:54.296986 28500 replica.cpp:741] Replica recovered with log 
positions 0 - 0 with 1 holes and 0 unlearned
I1106 23:02:54.297473 28515 recover.cpp:437] Starting replica recovery
I1106 23:02:54.297916 28515 recover.cpp:463] Replica is in EMPTY status
I1106 23:02:54.299103 28527 replica.cpp:638] Replica in EMPTY status received a 
broadcasted recover request
I1106 23:02:54.299681 28521 recover.cpp:188] Received a recover response from a 
replica in EMPTY status
I1106 23:02:54.300174 28515 recover.cpp:554] Updating replica status to STARTING
I1106 23:02:54.300956 28526 leveldb.cpp:306] Persisting metadata (8 bytes) to 
leveldb took 568815ns
I1106 23:02:54.300987 28526 replica.cpp:320] Persisted replica status to 
STARTING
I1106 23:02:54.301194 28526 recover.cpp:463] Replica is in STARTING status
I1106 23:02:54.302402 28524 master.cpp:318] Master 
20141106-230254-3125920579-57799-28500 (penates.apache.org) started on 
67.195.81.186:57799
I1106 23:02:54.302441 28524 master.cpp:364] Master only

Re: Review Request 26736: Ensure docker calls stop if a docker_stop_timeout is provided Added the flag docker_stop_timeout that defaults to 0, if the timeout is 0 the a docker kill will be run, otherw

2014-11-06 Thread Mesos ReviewBot

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26736/#review60266
---


Patch looks great!

Reviews applied: [26736]

All tests passed.

- Mesos ReviewBot


On Nov. 6, 2014, 9:55 p.m., Ryan Thomas wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/26736/
 ---
 
 (Updated Nov. 6, 2014, 9:55 p.m.)
 
 
 Review request for mesos, Benjamin Hindman and Timothy Chen.
 
 
 Bugs: MESOS-1925
 https://issues.apache.org/jira/browse/MESOS-1925
 
 
 Repository: mesos-git
 
 
 Description
 ---
 
 Ensure docker calls stop if a docker_stop_timeout is provided Added the flag 
 docker_stop_timeout that defaults to 0, if the timeout is 0 the a docker kill 
 will be run, otherwise a docker stop with the timeout in seconds is used. A 
 negative value will result in a Failure.
 
 
 Diffs
 -
 
   src/docker/docker.hpp 2dc692c51d6f9d45d8d7c59f8678991fd80c6e89 
   src/docker/docker.cpp 0c0a1bfd3c1260ebdf342d0a6e0e04d7b6a111a4 
   src/slave/containerizer/docker.cpp a6689203adbdcb0ad12583389eaeb83329e4ef6b 
   src/slave/flags.hpp efbd35dd744d8c8a53c36dd5cdb8d1c50fcfec3a 
   src/tests/docker_containerizer_tests.cpp 
 9d4ccc57f58d61c62aab5cdc79a129e987920bf6 
   src/tests/docker_tests.cpp ff06a01d7f20656c05bcf5afd3bc10059dc5378d 
 
 Diff: https://reviews.apache.org/r/26736/diff/
 
 
 Testing
 ---
 
 make check
 sudo bin/mesos-tests.sh --gtest_filter=DOCKER --verbose 
 support/mesos-style.py
 
 
 Thanks,
 
 Ryan Thomas
 




Build failed in Jenkins: Mesos-Trunk-Ubuntu-Build-In-Src-Set-JAVA_HOME #2256

2014-11-06 Thread Apache Jenkins Server
See 
https://builds.apache.org/job/Mesos-Trunk-Ubuntu-Build-In-Src-Set-JAVA_HOME/2256/changes

Changes:

[idownes] Only push the tag when tagging a release.

--
[...truncated 70292 lines...]
I1106 23:13:54.828126  3456 authenticatee.hpp:224] Received SASL authentication 
mechanisms: CRAM-MD5
I1106 23:13:54.828155  3456 authenticatee.hpp:250] Attempting to authenticate 
with mechanism 'CRAM-MD5'
I1106 23:13:54.828255  3453 authenticator.hpp:275] Received SASL authentication 
start
I1106 23:13:54.828373  3453 authenticator.hpp:397] Authentication requires more 
steps
I1106 23:13:54.828474  3443 authenticatee.hpp:270] Received SASL authentication 
step
I1106 23:13:54.828627  3443 authenticator.hpp:303] Received SASL authentication 
step
I1106 23:13:54.828660  3443 auxprop.cpp:99] Request to lookup properties for 
user: 'test-principal' realm: 'pomona.apache.org' server FQDN: 
'pomona.apache.org' SASL_AUXPROP_VERIFY_AGAINST_HASH: false 
SASL_AUXPROP_OVERRIDE: false SASL_AUXPROP_AUTHZID: false 
I1106 23:13:54.828677  3443 auxprop.cpp:171] Looking up auxiliary property 
'*userPassword'
I1106 23:13:54.828727  3443 auxprop.cpp:171] Looking up auxiliary property 
'*cmusaslsecretCRAM-MD5'
I1106 23:13:54.828768  3443 auxprop.cpp:99] Request to lookup properties for 
user: 'test-principal' realm: 'pomona.apache.org' server FQDN: 
'pomona.apache.org' SASL_AUXPROP_VERIFY_AGAINST_HASH: false 
SASL_AUXPROP_OVERRIDE: false SASL_AUXPROP_AUTHZID: true 
I1106 23:13:54.828788  3443 auxprop.cpp:121] Skipping auxiliary property 
'*userPassword' since SASL_AUXPROP_AUTHZID == true
I1106 23:13:54.828800  3443 auxprop.cpp:121] Skipping auxiliary property 
'*cmusaslsecretCRAM-MD5' since SASL_AUXPROP_AUTHZID == true
I1106 23:13:54.828822  3443 authenticator.hpp:389] Authentication success
I1106 23:13:54.828901  3453 authenticatee.hpp:310] Authentication success
I1106 23:13:54.828928  3456 master.cpp:3957] Successfully authenticated 
principal 'test-principal' at 
scheduler-d1c8083e-fd2b-4ecf-aa3c-132b24dbbd89@67.195.81.187:50469
I1106 23:13:54.829361  3453 sched.cpp:357] Successfully authenticated with 
master master@67.195.81.187:50469
I1106 23:13:54.829390  3453 sched.cpp:476] Sending registration request to 
master@67.195.81.187:50469
I1106 23:13:54.829540  3454 master.cpp:1378] Received registration request for 
framework 'framework4' at 
scheduler-d1c8083e-fd2b-4ecf-aa3c-132b24dbbd89@67.195.81.187:50469
I1106 23:13:54.829622  3454 master.cpp:1337] Authorizing framework principal 
'test-principal' to receive offers for role 'role1'
I1106 23:13:54.830108  3454 master.cpp:1442] Registering framework 
20141106-231354-3142697795-50469-3427-0003 (framework4) at 
scheduler-d1c8083e-fd2b-4ecf-aa3c-132b24dbbd89@67.195.81.187:50469
I1106 23:13:54.830492  3453 sched.cpp:407] Framework registered with 
20141106-231354-3142697795-50469-3427-0003
I1106 23:13:54.830544  3453 sched.cpp:421] Scheduler::registered took 28088ns
I1106 23:13:54.830608  3447 hierarchical_allocator_process.hpp:329] Added 
framework 20141106-231354-3142697795-50469-3427-0003
I1106 23:13:54.831528  3447 hierarchical_allocator_process.hpp:659] Performed 
allocation for 4 slaves in 834096ns
I1106 23:13:54.832275  3427 containerizer.cpp:100] Using isolation: 
posix/cpu,posix/mem
I1106 23:13:54.835315  3445 slave.cpp:169] Slave started on 
223)@67.195.81.187:50469
I1106 23:13:54.835350  3445 credentials.hpp:84] Loading credential for 
authentication from 
'/tmp/DRFAllocatorTest_DRFAllocatorProcess_gbSZvx/credential'
I1106 23:13:54.835489  3445 slave.cpp:276] Slave using credential for: 
test-principal
I1106 23:13:54.835692  3445 slave.cpp:289] Slave resources: cpus(*):1; 
mem(*):512; disk(*):0; ports(*):[31000-32000]
I1106 23:13:54.835767  3445 slave.cpp:318] Slave hostname: pomona.apache.org
I1106 23:13:54.835783  3445 slave.cpp:319] Slave checkpoint: false
W1106 23:13:54.835789  3445 slave.cpp:321] Disabling checkpointing is 
deprecated and the --checkpoint flag will be removed in a future release. 
Please avoid using this flag
I1106 23:13:54.836673  3441 state.cpp:33] Recovering state from 
'/tmp/DRFAllocatorTest_DRFAllocatorProcess_gbSZvx/meta'
I1106 23:13:54.836874  3453 status_update_manager.cpp:197] Recovering status 
update manager
I1106 23:13:54.837056  3450 containerizer.cpp:281] Recovering containerizer
I1106 23:13:54.837754  3442 slave.cpp:3466] Finished recovery
I1106 23:13:54.838178  3443 status_update_manager.cpp:171] Pausing sending 
status updates
I1106 23:13:54.838196  3445 slave.cpp:602] New master detected at 
master@67.195.81.187:50469
I1106 23:13:54.838232  3445 slave.cpp:665] Authenticating with master 
master@67.195.81.187:50469
I1106 23:13:54.838366  3445 slave.cpp:638] Detecting new master
I1106 23:13:54.838379  3449 authenticatee.hpp:133] Creating new client SASL 
connection
I1106 23:13:54.838553  3454 master.cpp:3896] Authenticating 
slave(223)@67.195.81.187:50469
I1106 23:13:54.838587  3454 master.cpp:3907

Re: [VOTE] Release Apache Mesos 0.21.0 (rc1)

2014-11-06 Thread Timothy Chen
The Fetcher patch currently contains a bug that is going to be patched
up by a new reviewboard, so I suggest we leave it for the next
version.

Tim

On Thu, Nov 6, 2014 at 2:29 PM, Adam Bordelon a...@mesosphere.io wrote:
 Also note, in addition to 'Support for Mesos modules' we also now allow
 pluggable isolator modules.

 On Thu, Nov 6, 2014 at 2:28 PM, Adam Bordelon a...@mesosphere.io wrote:

 make check passed on LinuxMint16 (Ubuntu 13.10)

 Would love to see the following two cherry-picked, especially the
 configuration.md change before you update the website for the release
 blog post.

 d09478d73cb3ddd76c7b886bef691c8034fa6908 Updated docs/configuration.md.
 c7227471f98c0dc62c8700d41534be142a3fcfad Fetcher uses Hadoop to fetch URIs
 with unknown schemes.


 On Thu, Nov 6, 2014 at 1:00 PM, Tom Arnfeld t...@duedil.com wrote:

 +1




 `make check` passed on Ubuntu 12.04 LTS (kernel 3.2.0-67)


 --


 Tom Arnfeld

 Developer // DueDil





 (+44) 7525940046

 25 Christopher Street, London, EC2A 2BS

 On Thu, Nov 6, 2014 at 8:43 PM, Ian Downes idow...@twitter.com.invalid
 wrote:

  Apologies: I used support/tag.sh but had a local branch *and* local tag
 and
  it pushed the branch only.
  $ git ls-remote --tags origin-wip | grep 0.21.0
  a7733493dc9e6f2447f825671d8a745602c9bf7a refs/tags/0.21.0-rc1
  On Thu, Nov 6, 2014 at 8:11 AM, Tim St Clair tstcl...@redhat.com
 wrote:
  $ git tag -l | grep 21
 
  $ git branch -r
origin/0.21.0-rc1
 
  It looks like you created a branch vs. tag ...?
 
  Cheers,
  Tim
 
  - Original Message -
   From: Ian Downes ian.dow...@gmail.com
   To: dev@mesos.apache.org, u...@mesos.apache.org
   Sent: Wednesday, November 5, 2014 5:12:52 PM
   Subject: [VOTE] Release Apache Mesos 0.21.0 (rc1)
  
   Hi all,
  
   Please vote on releasing the following candidate as Apache Mesos
 0.21.0.
  
  
   0.21.0 includes the following:
  
 
 
   State reconciliation for frameworks
   Support for Mesos modules
   Task status now includes source and reason
   A shared filesystem isolator
   A pid namespace isolator
  
   The CHANGELOG for the release is available at:
  
 
 https://git-wip-us.apache.org/repos/asf?p=mesos.git;a=blob_plain;f=CHANGELOG;hb=0.21.0-rc1
  
 
 
  
   The candidate for Mesos 0.21.0 release is available at:
  
 
 https://dist.apache.org/repos/dist/dev/mesos/0.21.0-rc1/mesos-0.21.0.tar.gz
  
   The tag to be voted on is 0.21.0-rc1:
  
 
 https://git-wip-us.apache.org/repos/asf?p=mesos.git;a=commit;h=0.21.0-rc1
  
   The MD5 checksum of the tarball can be found at:
  
 
 https://dist.apache.org/repos/dist/dev/mesos/0.21.0-rc1/mesos-0.21.0.tar.gz.md5
  
   The signature of the tarball can be found at:
  
 
 https://dist.apache.org/repos/dist/dev/mesos/0.21.0-rc1/mesos-0.21.0.tar.gz.asc
  
   The PGP key used to sign the release is here:
   https://dist.apache.org/repos/dist/release/mesos/KEYS
  
   The JAR is up in Maven in a staging repository here:
  
 https://repository.apache.org/content/repositories/orgapachemesos-1038
  
   Please vote on releasing this package as Apache Mesos 0.21.0!
  
   The vote is open until Sat Nov  8 15:09:48 PST 2014 and passes if a
   majority of at least 3 +1 PMC votes are cast.
  
   [ ] +1 Release this package as Apache Mesos 0.21.0
   [ ] -1 Do not release this package because ...
  
   Thanks,
  
   Ian Downes
  
 
  --
  Cheers,
  Timothy St. Clair
  Red Hat Inc.
 





Re: [VOTE] Release Apache Mesos 0.21.0 (rc1)

2014-11-06 Thread Ankur Chauhan
Hi Tim,

FYI: I posted a fix for the bug pointed out by @benh at 
https://reviews.apache.org/r/27670/ https://reviews.apache.org/r/27670/

-- Ankur
 On 6 Nov 2014, at 15:31, Timothy Chen tnac...@gmail.com wrote:
 
 The Fetcher patch currently contains a bug that is going to be patched
 up by a new reviewboard, so I suggest we leave it for the next
 version.
 
 Tim
 
 On Thu, Nov 6, 2014 at 2:29 PM, Adam Bordelon a...@mesosphere.io wrote:
 Also note, in addition to 'Support for Mesos modules' we also now allow
 pluggable isolator modules.
 
 On Thu, Nov 6, 2014 at 2:28 PM, Adam Bordelon a...@mesosphere.io wrote:
 
 make check passed on LinuxMint16 (Ubuntu 13.10)
 
 Would love to see the following two cherry-picked, especially the
 configuration.md change before you update the website for the release
 blog post.
 
 d09478d73cb3ddd76c7b886bef691c8034fa6908 Updated docs/configuration.md.
 c7227471f98c0dc62c8700d41534be142a3fcfad Fetcher uses Hadoop to fetch URIs
 with unknown schemes.
 
 
 On Thu, Nov 6, 2014 at 1:00 PM, Tom Arnfeld t...@duedil.com wrote:
 
 +1
 
 
 
 
 `make check` passed on Ubuntu 12.04 LTS (kernel 3.2.0-67)
 
 
 --
 
 
 Tom Arnfeld
 
 Developer // DueDil
 
 
 
 
 
 (+44) 7525940046
 
 25 Christopher Street, London, EC2A 2BS
 
 On Thu, Nov 6, 2014 at 8:43 PM, Ian Downes idow...@twitter.com.invalid
 wrote:
 
 Apologies: I used support/tag.sh but had a local branch *and* local tag
 and
 it pushed the branch only.
 $ git ls-remote --tags origin-wip | grep 0.21.0
 a7733493dc9e6f2447f825671d8a745602c9bf7a refs/tags/0.21.0-rc1
 On Thu, Nov 6, 2014 at 8:11 AM, Tim St Clair tstcl...@redhat.com
 wrote:
 $ git tag -l | grep 21
 
 $ git branch -r
  origin/0.21.0-rc1
 
 It looks like you created a branch vs. tag ...?
 
 Cheers,
 Tim
 
 - Original Message -
 From: Ian Downes ian.dow...@gmail.com
 To: dev@mesos.apache.org, u...@mesos.apache.org
 Sent: Wednesday, November 5, 2014 5:12:52 PM
 Subject: [VOTE] Release Apache Mesos 0.21.0 (rc1)
 
 Hi all,
 
 Please vote on releasing the following candidate as Apache Mesos
 0.21.0.
 
 
 0.21.0 includes the following:
 
 
 
 State reconciliation for frameworks
 Support for Mesos modules
 Task status now includes source and reason
 A shared filesystem isolator
 A pid namespace isolator
 
 The CHANGELOG for the release is available at:
 
 
 https://git-wip-us.apache.org/repos/asf?p=mesos.git;a=blob_plain;f=CHANGELOG;hb=0.21.0-rc1
 
 
 
 
 The candidate for Mesos 0.21.0 release is available at:
 
 
 https://dist.apache.org/repos/dist/dev/mesos/0.21.0-rc1/mesos-0.21.0.tar.gz
 
 The tag to be voted on is 0.21.0-rc1:
 
 
 https://git-wip-us.apache.org/repos/asf?p=mesos.git;a=commit;h=0.21.0-rc1
 
 The MD5 checksum of the tarball can be found at:
 
 
 https://dist.apache.org/repos/dist/dev/mesos/0.21.0-rc1/mesos-0.21.0.tar.gz.md5
 
 The signature of the tarball can be found at:
 
 
 https://dist.apache.org/repos/dist/dev/mesos/0.21.0-rc1/mesos-0.21.0.tar.gz.asc
 
 The PGP key used to sign the release is here:
 https://dist.apache.org/repos/dist/release/mesos/KEYS
 
 The JAR is up in Maven in a staging repository here:
 
 https://repository.apache.org/content/repositories/orgapachemesos-1038
 
 Please vote on releasing this package as Apache Mesos 0.21.0!
 
 The vote is open until Sat Nov  8 15:09:48 PST 2014 and passes if a
 majority of at least 3 +1 PMC votes are cast.
 
 [ ] +1 Release this package as Apache Mesos 0.21.0
 [ ] -1 Do not release this package because ...
 
 Thanks,
 
 Ian Downes
 
 
 --
 Cheers,
 Timothy St. Clair
 Red Hat Inc.
 
 
 
 



Re: [VOTE] Release Apache Mesos 0.21.0 (rc1)

2014-11-06 Thread Niklas Nielsen
+1 for Ubuntu 14.04.1 LTS and Mac OS X 10.9.5.

Niklas

On 6 November 2014 13:00, Tom Arnfeld t...@duedil.com wrote:

 +1

 `make check` passed on Ubuntu 12.04 LTS (kernel 3.2.0-67)

 --

 Tom Arnfeld
 Developer // DueDil

 (+44) 7525940046
 25 Christopher Street, London, EC2A 2BS


 On Thu, Nov 6, 2014 at 8:43 PM, Ian Downes idow...@twitter.com.invalid
 wrote:

 Apologies: I used support/tag.sh but had a local branch *and* local tag
 and
 it pushed the branch only.

 $ git ls-remote --tags origin-wip | grep 0.21.0
 a7733493dc9e6f2447f825671d8a745602c9bf7a refs/tags/0.21.0-rc1

 On Thu, Nov 6, 2014 at 8:11 AM, Tim St Clair tstcl...@redhat.com
 wrote:

  $ git tag -l | grep 21
 
  $ git branch -r
  origin/0.21.0-rc1
 
  It looks like you created a branch vs. tag ...?
 
  Cheers,
  Tim
 
  - Original Message -
   From: Ian Downes ian.dow...@gmail.com
   To: dev@mesos.apache.org, u...@mesos.apache.org
   Sent: Wednesday, November 5, 2014 5:12:52 PM
   Subject: [VOTE] Release Apache Mesos 0.21.0 (rc1)
  
   Hi all,
  
   Please vote on releasing the following candidate as Apache Mesos
 0.21.0.
  
  
   0.21.0 includes the following:
  
 
 

   State reconciliation for frameworks
   Support for Mesos modules
   Task status now includes source and reason
   A shared filesystem isolator
   A pid namespace isolator
  
   The CHANGELOG for the release is available at:
  
 
 https://git-wip-us.apache.org/repos/asf?p=mesos.git;a=blob_plain;f=CHANGELOG;hb=0.21.0-rc1
  
 
 

  
   The candidate for Mesos 0.21.0 release is available at:
  
 
 https://dist.apache.org/repos/dist/dev/mesos/0.21.0-rc1/mesos-0.21.0.tar.gz
  
   The tag to be voted on is 0.21.0-rc1:
  
 
 https://git-wip-us.apache.org/repos/asf?p=mesos.git;a=commit;h=0.21.0-rc1
  
   The MD5 checksum of the tarball can be found at:
  
 
 https://dist.apache.org/repos/dist/dev/mesos/0.21.0-rc1/mesos-0.21.0.tar.gz.md5
  
   The signature of the tarball can be found at:
  
 
 https://dist.apache.org/repos/dist/dev/mesos/0.21.0-rc1/mesos-0.21.0.tar.gz.asc
  
   The PGP key used to sign the release is here:
   https://dist.apache.org/repos/dist/release/mesos/KEYS
  
   The JAR is up in Maven in a staging repository here:
  
 https://repository.apache.org/content/repositories/orgapachemesos-1038
  
   Please vote on releasing this package as Apache Mesos 0.21.0!
  
   The vote is open until Sat Nov 8 15:09:48 PST 2014 and passes if a
   majority of at least 3 +1 PMC votes are cast.
  
   [ ] +1 Release this package as Apache Mesos 0.21.0
   [ ] -1 Do not release this package because ...
  
   Thanks,
  
   Ian Downes
  
 
  --
  Cheers,
  Timothy St. Clair
  Red Hat Inc.
 





Re: Review Request 27315: Updated scheduler driver to exponentially backoff during registration retries.

2014-11-06 Thread Mesos ReviewBot

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27315/#review60284
---


Patch looks great!

Reviews applied: [27315]

All tests passed.

- Mesos ReviewBot


On Nov. 6, 2014, 10:20 p.m., Vinod Kone wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27315/
 ---
 
 (Updated Nov. 6, 2014, 10:20 p.m.)
 
 
 Review request for mesos and Ben Mahler.
 
 
 Bugs: MESOS-1903
 https://issues.apache.org/jira/browse/MESOS-1903
 
 
 Repository: mesos-git
 
 
 Description
 ---
 
 Uses the same backoff (except no initial backoff) strategy used by the slave 
 during registration.
 
 
 Diffs
 -
 
   src/Makefile.am 9ab3b9c05d435d18ed1c2966f695857fa205e9fd 
   src/sched/constants.hpp PRE-CREATION 
   src/sched/constants.cpp PRE-CREATION 
   src/sched/flags.hpp PRE-CREATION 
   src/sched/sched.cpp e5f828d0bf9dd03a01920634abfae685a7861b44 
   src/slave/slave.hpp 5b082fc6956238d27f5d3352a3b40ce778a1a3a4 
   src/slave/slave.cpp dbfd1a8101d78dee8ea3ac19d990a6a7892e59be 
   src/tests/fault_tolerance_tests.cpp 
 372c4fdec1ef70991d4d550d8d73ac363436fdb9 
 
 Diff: https://reviews.apache.org/r/27315/diff/
 
 
 Testing
 ---
 
 make check
 
 
 Thanks,
 
 Vinod Kone
 




Re: Review Request 27707: Document authentication prerequisites in docs/release-guide.md.

2014-11-06 Thread Mesos ReviewBot

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27707/#review60286
---


Patch looks great!

Reviews applied: [27707]

All tests passed.

- Mesos ReviewBot


On Nov. 6, 2014, 10:43 p.m., Ian Downes wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27707/
 ---
 
 (Updated Nov. 6, 2014, 10:43 p.m.)
 
 
 Review request for mesos and Vinod Kone.
 
 
 Repository: mesos-git
 
 
 Description
 ---
 
 Document authentication prerequisites in docs/release-guide.md.
 
 
 Diffs
 -
 
   docs/release-guide.md 3aa6262894a4ec0e9197b770d47531b84258b801 
 
 Diff: https://reviews.apache.org/r/27707/diff/
 
 
 Testing
 ---
 
 
 Thanks,
 
 Ian Downes
 




Re: Review Request 27707: Document authentication prerequisites in docs/release-guide.md.

2014-11-06 Thread Vinod Kone

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27707/#review60290
---

Ship it!



docs/release-guide.md
https://reviews.apache.org/r/27707/#comment101655

Can you add a step here to update configure.ac to next version and do a 
commit?


- Vinod Kone


On Nov. 6, 2014, 10:43 p.m., Ian Downes wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27707/
 ---
 
 (Updated Nov. 6, 2014, 10:43 p.m.)
 
 
 Review request for mesos and Vinod Kone.
 
 
 Repository: mesos-git
 
 
 Description
 ---
 
 Document authentication prerequisites in docs/release-guide.md.
 
 
 Diffs
 -
 
   docs/release-guide.md 3aa6262894a4ec0e9197b770d47531b84258b801 
 
 Diff: https://reviews.apache.org/r/27707/diff/
 
 
 Testing
 ---
 
 
 Thanks,
 
 Ian Downes
 




Re: [VOTE] Release Apache Mesos 0.21.0 (rc1)

2014-11-06 Thread Vinod Kone
-1

There is a SEGFAULT issue in Authenticator.

https://issues.apache.org/jira/browse/MESOS-2050

On Thu, Nov 6, 2014 at 3:59 PM, Niklas Nielsen nik...@mesosphere.io wrote:

 +1 for Ubuntu 14.04.1 LTS and Mac OS X 10.9.5.

 Niklas

 On 6 November 2014 13:00, Tom Arnfeld t...@duedil.com wrote:

  +1
 
  `make check` passed on Ubuntu 12.04 LTS (kernel 3.2.0-67)
 
  --
 
  Tom Arnfeld
  Developer // DueDil
 
  (+44) 7525940046
  25 Christopher Street, London, EC2A 2BS
 
 
  On Thu, Nov 6, 2014 at 8:43 PM, Ian Downes idow...@twitter.com.invalid
  wrote:
 
  Apologies: I used support/tag.sh but had a local branch *and* local tag
  and
  it pushed the branch only.
 
  $ git ls-remote --tags origin-wip | grep 0.21.0
  a7733493dc9e6f2447f825671d8a745602c9bf7a refs/tags/0.21.0-rc1
 
  On Thu, Nov 6, 2014 at 8:11 AM, Tim St Clair tstcl...@redhat.com
  wrote:
 
   $ git tag -l | grep 21
  
   $ git branch -r
   origin/0.21.0-rc1
  
   It looks like you created a branch vs. tag ...?
  
   Cheers,
   Tim
  
   - Original Message -
From: Ian Downes ian.dow...@gmail.com
To: dev@mesos.apache.org, u...@mesos.apache.org
Sent: Wednesday, November 5, 2014 5:12:52 PM
Subject: [VOTE] Release Apache Mesos 0.21.0 (rc1)
   
Hi all,
   
Please vote on releasing the following candidate as Apache Mesos
  0.21.0.
   
   
0.21.0 includes the following:
   
  
 
 
 
State reconciliation for frameworks
Support for Mesos modules
Task status now includes source and reason
A shared filesystem isolator
A pid namespace isolator
   
The CHANGELOG for the release is available at:
   
  
 
 https://git-wip-us.apache.org/repos/asf?p=mesos.git;a=blob_plain;f=CHANGELOG;hb=0.21.0-rc1
   
  
 
 
 
   
The candidate for Mesos 0.21.0 release is available at:
   
  
 
 https://dist.apache.org/repos/dist/dev/mesos/0.21.0-rc1/mesos-0.21.0.tar.gz
   
The tag to be voted on is 0.21.0-rc1:
   
  
 
 https://git-wip-us.apache.org/repos/asf?p=mesos.git;a=commit;h=0.21.0-rc1
   
The MD5 checksum of the tarball can be found at:
   
  
 
 https://dist.apache.org/repos/dist/dev/mesos/0.21.0-rc1/mesos-0.21.0.tar.gz.md5
   
The signature of the tarball can be found at:
   
  
 
 https://dist.apache.org/repos/dist/dev/mesos/0.21.0-rc1/mesos-0.21.0.tar.gz.asc
   
The PGP key used to sign the release is here:
https://dist.apache.org/repos/dist/release/mesos/KEYS
   
The JAR is up in Maven in a staging repository here:
   
  https://repository.apache.org/content/repositories/orgapachemesos-1038
   
Please vote on releasing this package as Apache Mesos 0.21.0!
   
The vote is open until Sat Nov 8 15:09:48 PST 2014 and passes if a
majority of at least 3 +1 PMC votes are cast.
   
[ ] +1 Release this package as Apache Mesos 0.21.0
[ ] -1 Do not release this package because ...
   
Thanks,
   
Ian Downes
   
  
   --
   Cheers,
   Timothy St. Clair
   Red Hat Inc.
  
 
 
 



Re: A problem with resource offers

2014-11-06 Thread Benjamin Mahler
Which version of the master are you using and do you have the logs? The
fact that no offers were coming back sounds like a bug!

As for using O1 after a disconnection, all offers are invalid once a
disconnection occurs. The scheduler driver does not automatically rescind
offers upon disconnection, so I'd recommend clearing all cached offers when
your scheduler gets disconnected, to avoid the unnecessary TASK_LOST
updates.

On Thu, Nov 6, 2014 at 6:25 PM, Sharma Podila spod...@netflix.com wrote:

 We had an interesting problem with resource offers today and I would like
 to confirm this problem and request an enhancement. Here's the summary in
 the right sequence of events:

 1. resource offer O1 for slave A arrives
 2. mesos disconnects
 3. mesos reregisters
 4. mesos offer O2 for slave A arrives
 (our framework keeps offers for sometime if unused, therefore, we now
 have both O1 and O2, incorrectly)
 5. launch task T1 using offers O1 and O2
 6. framework thinks it has no offers with it now for slave A, will wait
 for new offer after mesos consumes resources for task T1
 7. mesos sends TASK_LOST for T1 saying it was using an invalid offer
 (even though only O1 was invalid, O2 is gone missing silently)
 8. no more offers come for slave A
 9. basically we have an offer leak problem.

 To work around this, I am changing my framework so that when it receives
 mesos reregistration callback (step 3 above), it removes all existing
 offers. This should fix the problem.

 However, I am wondering if #7 can be improved in Mesos. When a task is (or
 set of tasks are) launched using multiple offers, if at least one of the
 offers is invalid, then Mesos should treat all offers as given up by the
 framework. This will send TASK_LOST to the framework, but, also make the
 valid offers available again through new offers.

 I am thinking this will be critical to do when Mesos starts rescinding
 offers. Because in that case the frameworks cannot rely on the strategy
 like the one I am using with reregistration.

 Sharma




Re: Review Request 27670: Bug fix: Check for non-zero status code and hadoop client not found

2014-11-06 Thread Adam B

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27670/#review60295
---

Ship it!


Looks good.


src/launcher/fetcher.cpp
https://reviews.apache.org/r/27670/#comment101662

Since Try is either Error or Some, you could just write:
if (available.isError() || !available.get()) {


- Adam B


On Nov. 6, 2014, 11:03 a.m., Ankur Chauhan wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27670/
 ---
 
 (Updated Nov. 6, 2014, 11:03 a.m.)
 
 
 Review request for mesos, Benjamin Hindman and Vinod Kone.
 
 
 Bugs: MESOS-1711
 https://issues.apache.org/jira/browse/MESOS-1711
 
 
 Repository: mesos-git
 
 
 Description
 ---
 
 @benh noticed that the fetcher would detect the condition where hadoop 
 returns a success in case of a success non zero value
 
 
 Diffs
 -
 
   src/launcher/fetcher.cpp bd95928bc3191970330e839bcf41e343d5142c54 
 
 Diff: https://reviews.apache.org/r/27670/diff/
 
 
 Testing
 ---
 
 make check
 sudo bin/mesos-tests.sh --verbose
 support/mesos-style.py
 
 
 Thanks,
 
 Ankur Chauhan
 




Re: Review Request 27446: libprocess: Replaced the ip and port pairs from UPID class and process namespace with Node class.

2014-11-06 Thread Evelina Dumitrescu

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27446/
---

(Updated Nov. 7, 2014, 3:23 a.m.)


Review request for mesos, Dominic Hamon, Joris Van Remoortere, and switched to 
'mcypark'.


Summary (updated)
-

libprocess:  Replaced the ip and port pairs from UPID class and process 
namespace with Node class.


Bugs: MESOS-2019
https://issues.apache.org/jira/browse/MESOS-2019


Repository: mesos-git


Description
---

At the moment, the Node class is used to keep a mapping from a socket to the ip 
 port pair in the process namespace.
I want to propose to extend its use by replacing the ip  port fields from the 
UPID class and process namespace with this type.


Diffs (updated)
-

  3rdparty/libprocess/include/process/node.hpp 
7a96894456683be60595a8f96f85df37947ff5b6 
  3rdparty/libprocess/include/process/pid.hpp 
23453229c2a868a62b143416de33cf630268a64e 
  3rdparty/libprocess/include/process/process.hpp 
81a1f7a3fece78bb7df1e86850572922c8704ea4 
  3rdparty/libprocess/src/http.cpp 4ef00d11de4ff4e7eb4e44aac204147799f85ac2 
  3rdparty/libprocess/src/pid.cpp 20ff25c0959c247687955535f02e0397653af52a 
  3rdparty/libprocess/src/process.cpp 85fb9958342f0bcdde322d9c55333126e6f86668 
  3rdparty/libprocess/src/tests/benchmarks.cpp 
3177a8ecac6adf8b52c3160587dc3a4fe2707d26 
  3rdparty/libprocess/src/tests/http_tests.cpp 
a1c3685c88d268c7ecdc2dad22608198dc0b295d 
  3rdparty/libprocess/src/tests/metrics_tests.cpp 
33539e45bab8f79afa773a325465016f01fd3b75 
  3rdparty/libprocess/src/tests/process_tests.cpp 
b985fb77ea05fae5c0b144ea48814acc7bb5135b 

Diff: https://reviews.apache.org/r/27446/diff/


Testing
---

make check


Thanks,

Evelina Dumitrescu



Re: Review Request 27447: mesos: Replaced the ip and port pairs from UPID class and process namespace with Node class.

2014-11-06 Thread Evelina Dumitrescu

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27447/
---

(Updated Nov. 7, 2014, 3:23 a.m.)


Review request for mesos, Dominic Hamon, Joris Van Remoortere, and switched to 
'mcypark'.


Summary (updated)
-

mesos:  Replaced the ip and port pairs from UPID class and process namespace 
with Node class.


Bugs: MESOS-2019
https://issues.apache.org/jira/browse/MESOS-2019


Repository: mesos-git


Description
---

At the moment, the Node class is used to keep a mapping from a socket to the ip 
 port pair in the process namespace.
I want to propose to extend its use by replacing the ip  port fields from the 
UPID class and process namespace with this type.


Diffs (updated)
-

  src/common/protobuf_utils.cpp 33ce782960e88e07b910f844fbdcae4c0f19d9a7 
  src/master/master.cpp a860496d9d75ae4e007d35627d778caf10d08428 
  src/sched/sched.cpp 8ca052635bf8772a6fa95972931b6254c6e452ef 
  src/scheduler/scheduler.cpp c74187cfeb57a748fd57198d3badd4b8773490eb 
  src/slave/http.cpp 965d78c3e3ea2e3c922f62aabf6fc87c5cf2ab8d 
  src/slave/slave.cpp 81e0c4ba702d492976f9c771b8d779853c423187 
  src/tests/fetcher_tests.cpp e026e875c7197cd214ef0432d7c40207553c8671 
  src/tests/files_tests.cpp a696aa22d56b37ee70c6e64c81a849da6d436451 
  src/tests/gc_tests.cpp f7747e2999cbd64ab0cd0321cca0021489dc3346 
  src/tests/logging_tests.cpp 58e9b3399c3de28b1c5b3a8ec8fdf85b7a31c6d5 
  src/tests/master_contender_detector_tests.cpp 
8f7fb18f0ca985364565fc4cd50f113b1461fbe7 
  src/tests/master_tests.cpp a6d1a4a5b7d50a88e5b101c43f62d21300cf 
  src/tests/monitor_tests.cpp 4b950e14bd94cdfa21212268b56bebdc1200078d 

Diff: https://reviews.apache.org/r/27447/diff/


Testing
---

make check


Thanks,

Evelina Dumitrescu



Build failed in Jenkins: Mesos-Trunk-Ubuntu-Build-Out-Of-Src-Disable-Java-Disable-Python-Disable-Webui #2535

2014-11-06 Thread Apache Jenkins Server
See 
https://builds.apache.org/job/Mesos-Trunk-Ubuntu-Build-Out-Of-Src-Disable-Java-Disable-Python-Disable-Webui/2535/changes

Changes:

[vinodkone] Updated scheduler driver to exponentially backoff during 
registration

--
[...truncated 65144 lines...]
I1107 03:34:44.615044 30026 slave.cpp:1798] Status update manager successfully 
handled status update acknowledgement (UUID: 
beefce9a-5f3c-4fc1-bcd5-5954aaf1b6f3) for task 1 of framework 
20141107-033439-3193029443-48289-29993-
I1107 03:34:44.615097 30026 slave.cpp:4258] Completing task 1
I1107 03:34:44.615115 30030 status_update_manager.cpp:371] Forwarding update 
TASK_FINISHED (UUID: f0bad1d8-9aa6-4126-a8b8-2bbc4f08a0aa) for task 3 of 
framework 20141107-033439-3193029443-48289-29993- to the slave
I1107 03:34:44.615272 30028 slave.cpp:2456] Forwarding the update TASK_FINISHED 
(UUID: f0bad1d8-9aa6-4126-a8b8-2bbc4f08a0aa) for task 3 of framework 
20141107-033439-3193029443-48289-29993- to master@67.195.81.190:48289
I1107 03:34:44.615298 30017 slave.cpp:1798] Status update manager successfully 
handled status update acknowledgement (UUID: 
17a5f53a-0c16-4bab-b246-aa2d59e3772e) for task 4 of framework 
20141107-033439-3193029443-48289-29993-
I1107 03:34:44.615352 30017 slave.cpp:4258] Completing task 4
I1107 03:34:44.615381 30028 slave.cpp:2383] Status update manager successfully 
handled status update TASK_FINISHED (UUID: 
f0bad1d8-9aa6-4126-a8b8-2bbc4f08a0aa) for task 3 of framework 
20141107-033439-3193029443-48289-29993-
I1107 03:34:44.615404 30028 slave.cpp:2389] Sending acknowledgement for status 
update TASK_FINISHED (UUID: f0bad1d8-9aa6-4126-a8b8-2bbc4f08a0aa) for task 3 of 
framework 20141107-033439-3193029443-48289-29993- to 
executor(1)@67.195.81.190:48983
I1107 03:34:44.615466 30023 master.cpp:3441] Forwarding status update 
TASK_FINISHED (UUID: f0bad1d8-9aa6-4126-a8b8-2bbc4f08a0aa) for task 3 of 
framework 20141107-033439-3193029443-48289-29993-
I1107 03:34:44.615562 30023 master.cpp:3413] Status update TASK_FINISHED (UUID: 
f0bad1d8-9aa6-4126-a8b8-2bbc4f08a0aa) for task 3 of framework 
20141107-033439-3193029443-48289-29993- from slave 
20141107-033439-3193029443-48289-29993-S0 at slave(2)@67.195.81.190:48289 
(pietas.apache.org)
I1107 03:34:44.615609 30023 master.cpp:4691] Updating the latest state of task 
3 of framework 20141107-033439-3193029443-48289-29993- to TASK_FINISHED
Task 3 is in state TASK_FINISHED
I1107 03:34:44.615715 30030 sched.cpp:669] Scheduler::statusUpdate took 40909ns
I1107 03:34:44.615792 30030 sched.cpp:779] Stopping framework 
'20141107-033439-3193029443-48289-29993-'
I1107 03:34:44.615818 30368 exec.cpp:339] Executor received status update 
acknowledgement f0bad1d8-9aa6-4126-a8b8-2bbc4f08a0aa for task 3 of framework 
20141107-033439-3193029443-48289-29993-
I1107 03:34:44.615849 30019 hierarchical_allocator_process.hpp:563] Recovered 
cpus(*):1; mem(*):128 (total allocatable: mem(*):10240; disk(*):3.70122e+06; 
ports(*):[31000-32000]; cpus(*):2) on slave 
20141107-033439-3193029443-48289-29993-S0 from framework 
20141107-033439-3193029443-48289-29993-
I1107 03:34:44.615875 30023 master.cpp:4750] Removing task 3 with resources 
cpus(*):1; mem(*):128 of framework 20141107-033439-3193029443-48289-29993- 
on slave 20141107-033439-3193029443-48289-29993-S0 at 
slave(2)@67.195.81.190:48289 (pietas.apache.org)
I1107 03:34:44.615988 30023 master.cpp:2913] Forwarding status update 
acknowledgement f0bad1d8-9aa6-4126-a8b8-2bbc4f08a0aa for task 3 of framework 
20141107-033439-3193029443-48289-29993- (No Executor Framework (C++)) at 
scheduler-4b61160a-cbba-45a8-a7d1-3d4971a8574f@67.195.81.190:48289 to slave 
20141107-033439-3193029443-48289-29993-S0 at slave(2)@67.195.81.190:48289 
(pietas.apache.org)
I1107 03:34:44.616142 30023 master.cpp:1702] Asked to unregister framework 
20141107-033439-3193029443-48289-29993-
I1107 03:34:44.616168 30023 master.cpp:4269] Removing framework 
20141107-033439-3193029443-48289-29993- (No Executor Framework (C++)) at 
scheduler-4b61160a-cbba-45a8-a7d1-3d4971a8574f@67.195.81.190:48289
I1107 03:34:44.616230 30027 status_update_manager.cpp:389] Received status 
update acknowledgement (UUID: f0bad1d8-9aa6-4126-a8b8-2bbc4f08a0aa) for task 3 
of framework 20141107-033439-3193029443-48289-29993-
I1107 03:34:44.616335 30024 hierarchical_allocator_process.hpp:405] Deactivated 
framework 20141107-033439-3193029443-48289-29993-
I1107 03:34:44.616341 30027 status_update_manager.cpp:525] Cleaning up status 
update stream for task 3 of framework 
20141107-033439-3193029443-48289-29993-
I1107 03:34:44.616360 30026 slave.cpp:1531] Asked to shut down framework 
20141107-033439-3193029443-48289-29993- by master@67.195.81.190:48289
I1107 03:34:44.616392 30022 slave.cpp:1531] Asked to shut down framework 
20141107-033439-3193029443-48289-29993- by master@67.195.81.190:48289
I1107 

Build failed in Jenkins: Mesos-Trunk-Ubuntu-Build-In-Src-Set-JAVA_HOME #2257

2014-11-06 Thread Apache Jenkins Server
See 
https://builds.apache.org/job/Mesos-Trunk-Ubuntu-Build-In-Src-Set-JAVA_HOME/2257/changes

Changes:

[vinodkone] Updated scheduler driver to exponentially backoff during 
registration

--
[...truncated 70446 lines...]
2014-11-07 03:34:49,335:19187(0x2ae0adb1e700):ZOO_INFO@zookeeper_init@786: 
Initiating client connection, host=127.0.0.1:48480 sessionTimeout=1 
watcher=0x2ae0a923a2ca sessionId=0 sessionPasswd=null context=0x2ae0f40a0100 
flags=0
2014-11-07 03:34:49,335:19187(0x2ae468a04700):ZOO_INFO@check_events@1703: 
initiated connection to server [127.0.0.1:48480]
2014-11-07 03:34:49,338:19187(0x2ae468a04700):ZOO_INFO@check_events@1750: 
session establishment complete on server [127.0.0.1:48480], 
sessionId=0x149885198b9, negotiated timeout=1
2014-11-07 03:34:49,353:19187(0x2ae0ae925700):ZOO_INFO@zookeeper_close@2505: 
Closing zookeeper sessionId=0x149885198b9 to [127.0.0.1:48480]

I1107 03:34:49.354049 19187 zookeeper_test_server.cpp:118] Shutting down 
ZooKeeperTestServer on port 48480
[   OK ] ZooKeeperStateTest.FetchAndStoreAndStoreFailAndFetch (84 ms)
[ RUN  ] ZooKeeperStateTest.Names
I1107 03:34:49.359802 19187 zookeeper_test_server.cpp:158] Started 
ZooKeeperTestServer on port 45825
2014-11-07 03:34:49,360:19187(0x2ae0af52b700):ZOO_INFO@log_env@712: Client 
environment:zookeeper.version=zookeeper C client 3.4.5
2014-11-07 03:34:49,360:19187(0x2ae0af52b700):ZOO_INFO@log_env@716: Client 
environment:host.name=pomona.apache.org
2014-11-07 03:34:49,360:19187(0x2ae0af52b700):ZOO_INFO@log_env@723: Client 
environment:os.name=Linux
2014-11-07 03:34:49,360:19187(0x2ae0af52b700):ZOO_INFO@log_env@724: Client 
environment:os.arch=3.13.0-36-lowlatency
2014-11-07 03:34:49,360:19187(0x2ae0af52b700):ZOO_INFO@log_env@725: Client 
environment:os.version=#63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014
2014-11-07 03:34:49,360:19187(0x2ae0af52b700):ZOO_INFO@log_env@733: Client 
environment:user.name=jenkins
2014-11-07 03:34:49,360:19187(0x2ae0af52b700):ZOO_INFO@log_env@741: Client 
environment:user.home=/home/jenkins
2014-11-07 03:34:49,360:19187(0x2ae0af52b700):ZOO_INFO@log_env@753: Client 
environment:user.dir=https://builds.apache.org/job/Mesos-Trunk-Ubuntu-Build-In-Src-Set-JAVA_HOME/ws/src
2014-11-07 03:34:49,360:19187(0x2ae0af52b700):ZOO_INFO@zookeeper_init@786: 
Initiating client connection, host=127.0.0.1:45825 sessionTimeout=1 
watcher=0x2ae0a923a2ca sessionId=0 sessionPasswd=null context=0x2ae0e808c7d0 
flags=0
2014-11-07 03:34:49,360:19187(0x2ae468a04700):ZOO_INFO@check_events@1703: 
initiated connection to server [127.0.0.1:45825]
2014-11-07 03:34:49,363:19187(0x2ae468a04700):ZOO_INFO@check_events@1750: 
session establishment complete on server [127.0.0.1:45825], 
sessionId=0x1498851990e, negotiated timeout=1
2014-11-07 03:34:49,374:19187(0x2ae0adb1e700):ZOO_INFO@zookeeper_close@2505: 
Closing zookeeper sessionId=0x1498851990e to [127.0.0.1:45825]

I1107 03:34:49.374914 19187 zookeeper_test_server.cpp:118] Shutting down 
ZooKeeperTestServer on port 45825
[   OK ] ZooKeeperStateTest.Names (21 ms)
[ RUN  ] ZooKeeperStateTest.FetchAndStoreAndStoreAndFetch
I1107 03:34:49.380270 19187 zookeeper_test_server.cpp:158] Started 
ZooKeeperTestServer on port 34105
2014-11-07 03:34:49,380:19187(0x2ae0ae724700):ZOO_INFO@log_env@712: Client 
environment:zookeeper.version=zookeeper C client 3.4.5
2014-11-07 03:34:49,380:19187(0x2ae0ae724700):ZOO_INFO@log_env@716: Client 
environment:host.name=pomona.apache.org
2014-11-07 03:34:49,380:19187(0x2ae0ae724700):ZOO_INFO@log_env@723: Client 
environment:os.name=Linux
2014-11-07 03:34:49,380:19187(0x2ae0ae724700):ZOO_INFO@log_env@724: Client 
environment:os.arch=3.13.0-36-lowlatency
2014-11-07 03:34:49,380:19187(0x2ae0ae724700):ZOO_INFO@log_env@725: Client 
environment:os.version=#63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014
2014-11-07 03:34:49,380:19187(0x2ae0ae724700):ZOO_INFO@log_env@733: Client 
environment:user.name=jenkins
2014-11-07 03:34:49,380:19187(0x2ae0ae724700):ZOO_INFO@log_env@741: Client 
environment:user.home=/home/jenkins
2014-11-07 03:34:49,380:19187(0x2ae0ae724700):ZOO_INFO@log_env@753: Client 
environment:user.dir=https://builds.apache.org/job/Mesos-Trunk-Ubuntu-Build-In-Src-Set-JAVA_HOME/ws/src
2014-11-07 03:34:49,380:19187(0x2ae0ae724700):ZOO_INFO@zookeeper_init@786: 
Initiating client connection, host=127.0.0.1:34105 sessionTimeout=1 
watcher=0x2ae0a923a2ca sessionId=0 sessionPasswd=null context=0x2ae0e808c7d0 
flags=0
2014-11-07 03:34:49,381:19187(0x2ae468a04700):ZOO_INFO@check_events@1703: 
initiated connection to server [127.0.0.1:34105]
2014-11-07 03:34:49,383:19187(0x2ae468a04700):ZOO_INFO@check_events@1750: 
session establishment complete on server [127.0.0.1:34105], 
sessionId=0x14988519923, negotiated timeout=1
2014-11-07 03:34:49,403:19187(0x2ae0af32a700):ZOO_INFO@zookeeper_close@2505: 
Closing zookeeper sessionId=0x14988519923 to 

Re: Review Request 27447: mesos: Replaced the ip and port pairs from UPID class and process namespace with Node class.

2014-11-06 Thread Mesos ReviewBot

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27447/#review60301
---


Patch looks great!

Reviews applied: [27446, 27447]

All tests passed.

- Mesos ReviewBot


On Nov. 7, 2014, 3:23 a.m., Evelina Dumitrescu wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27447/
 ---
 
 (Updated Nov. 7, 2014, 3:23 a.m.)
 
 
 Review request for mesos, Dominic Hamon, Joris Van Remoortere, and switched 
 to 'mcypark'.
 
 
 Bugs: MESOS-2019
 https://issues.apache.org/jira/browse/MESOS-2019
 
 
 Repository: mesos-git
 
 
 Description
 ---
 
 At the moment, the Node class is used to keep a mapping from a socket to the 
 ip  port pair in the process namespace.
 I want to propose to extend its use by replacing the ip  port fields from 
 the UPID class and process namespace with this type.
 
 
 Diffs
 -
 
   src/common/protobuf_utils.cpp 33ce782960e88e07b910f844fbdcae4c0f19d9a7 
   src/master/master.cpp a860496d9d75ae4e007d35627d778caf10d08428 
   src/sched/sched.cpp 8ca052635bf8772a6fa95972931b6254c6e452ef 
   src/scheduler/scheduler.cpp c74187cfeb57a748fd57198d3badd4b8773490eb 
   src/slave/http.cpp 965d78c3e3ea2e3c922f62aabf6fc87c5cf2ab8d 
   src/slave/slave.cpp 81e0c4ba702d492976f9c771b8d779853c423187 
   src/tests/fetcher_tests.cpp e026e875c7197cd214ef0432d7c40207553c8671 
   src/tests/files_tests.cpp a696aa22d56b37ee70c6e64c81a849da6d436451 
   src/tests/gc_tests.cpp f7747e2999cbd64ab0cd0321cca0021489dc3346 
   src/tests/logging_tests.cpp 58e9b3399c3de28b1c5b3a8ec8fdf85b7a31c6d5 
   src/tests/master_contender_detector_tests.cpp 
 8f7fb18f0ca985364565fc4cd50f113b1461fbe7 
   src/tests/master_tests.cpp a6d1a4a5b7d50a88e5b101c43f62d21300cf 
   src/tests/monitor_tests.cpp 4b950e14bd94cdfa21212268b56bebdc1200078d 
 
 Diff: https://reviews.apache.org/r/27447/diff/
 
 
 Testing
 ---
 
 make check
 
 
 Thanks,
 
 Evelina Dumitrescu
 




Re: Review Request 27675: Updated upgrades.md to reflect changes on AuthenticationStartMessage.

2014-11-06 Thread Adam B

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27675/#review60304
---



docs/upgrades.md
https://reviews.apache.org/r/27675/#comment101671

My suggested rewording:
The Authentication API has changed slightly in this release to support 
additional authentication mechanisms. The change from 'string' to 'bytes' for 
AuthenticationStartMessage.data has no impact on C++ or the over-the-wire 
representation, so it only impacts pure language bindings for languages like 
Java and Python that use different types for UTF-8 strings vs. byte arrays.


- Adam B


On Nov. 6, 2014, 2:51 a.m., Till Toenshoff wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27675/
 ---
 
 (Updated Nov. 6, 2014, 2:51 a.m.)
 
 
 Review request for mesos, Adam B and Vinod Kone.
 
 
 Bugs: MESOS-2001
 https://issues.apache.org/jira/browse/MESOS-2001
 
 
 Repository: mesos-git
 
 
 Description
 ---
 
 see summary.
 
 
 Diffs
 -
 
   docs/upgrades.md 3aff335 
 
 Diff: https://reviews.apache.org/r/27675/diff/
 
 
 Testing
 ---
 
 
 Thanks,
 
 Till Toenshoff
 




Re: Review Request 27493: Added CRAM-MD5 Test Authenticatee Module.

2014-11-06 Thread Adam B


 On Nov. 6, 2014, 1:33 p.m., Kapil Arya wrote:
  src/authentication/authenticatee.hpp, lines 19-20
  https://reviews.apache.org/r/27493/diff/2/?file=751526#file751526line19
 
  Don't we want to put the directory name in it as well?

We decided against it for authentication/authenticator.hpp, since it's pretty 
redundant, like master/master.hpp or files/files.hpp; and it's not likely that 
we'll have another Authenticatee at the root scope.
If you feel at all strongly about it, I wouldn't fight it, but I would ask that 
we make the same change to authenticator.hpp.


- Adam


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27493/#review60244
---


On Nov. 6, 2014, 2:33 a.m., Till Toenshoff wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/27493/
 ---
 
 (Updated Nov. 6, 2014, 2:33 a.m.)
 
 
 Review request for mesos, Adam B, Kapil Arya, and Vinod Kone.
 
 
 Bugs: MESOS-2001
 https://issues.apache.org/jira/browse/MESOS-2001
 
 
 Repository: mesos-git
 
 
 Description
 ---
 
 see summary.
 
 
 Diffs
 -
 
   src/Makefile.am 9ab3b9c 
   src/authentication/authenticatee.hpp PRE-CREATION 
   src/authentication/cram_md5/authenticatee.hpp 3088a77 
   src/examples/test_authentication_modules.cpp PRE-CREATION 
   src/examples/test_authenticator_module.cpp 4398d71 
   src/module/authenticatee.hpp PRE-CREATION 
   src/module/manager.cpp 0d71e33 
 
 Diff: https://reviews.apache.org/r/27493/diff/
 
 
 Testing
 ---
 
 make check
 
 Note: All three CRAM-MD5 Authenticatee patches need to get applied before 
 running make check! 
 
 
 Thanks,
 
 Till Toenshoff