Re: Review Request 35711: Disallow special characters in role name.

2015-06-27 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35711/ --- (Updated June 27, 2015, 8:36 a.m.) Review request for mesos, Adam B and Jie

Re: Review Request 35711: Disallow special characters in role name.

2015-06-27 Thread haosdent huang
On June 24, 2015, 9:30 a.m., Adam B wrote: Looks great! We recently added validation lambdas to flag definitions, so you may be able to take advantage of that. We probably also want to do validation in the slave for the --resources flag, since resources can be declared as reserved for

Re: Review Request 35711: Disallow special characters in role name.

2015-06-27 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35711/ --- (Updated June 27, 2015, 11:32 a.m.) Review request for mesos, Adam B and Jie

Re: Review Request 35711: Disallow special characters in role name.

2015-06-27 Thread haosdent huang
On June 24, 2015, 6:34 p.m., Jie Yu wrote: src/master/master.hpp, lines 630-651 https://reviews.apache.org/r/35711/diff/4/?file=989196#file989196line630 Can you move this function to src/master/validation.hpp|cpp ``` namespace role { OptionError

Re: Review Request 35711: Disallow special characters in role name.

2015-06-27 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35711/ --- (Updated June 27, 2015, 11:35 a.m.) Review request for mesos, Adam B and Jie

Re: Review Request 35728: Fix failing test: SlaveTest.ROOT_RunTaskWithCommandInfoWithUser.

2015-06-27 Thread haosdent huang
On June 26, 2015, 2:30 a.m., Adam B wrote: Nice work! So, running the command as root first will guarantee that lt-mesos-executor exists before trying to run the task as the test-user `nobody`? We might be able to fix this in a cleaner way, but this looks good enough to me. Just fix

Re: Review Request 35711: Disallow special characters in role name.

2015-06-27 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35711/#review89627 --- src/Makefile.am (line 1489)

Re: Review Request 35711: Disallow special characters in role name.

2015-06-27 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35711/#review89626 --- src/Makefile.am (line 362)

Re: Review Request 35728: Fix failing test: SlaveTest.ROOT_RunTaskWithCommandInfoWithUser.

2015-06-27 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35728/ --- (Updated June 27, 2015, 4:37 p.m.) Review request for mesos and Adam B.

Re: Review Request 35728: Fix failing test: SlaveTest.ROOT_RunTaskWithCommandInfoWithUser.

2015-06-27 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35728/ --- (Updated June 27, 2015, 4:37 p.m.) Review request for mesos and Adam B.

Review Request 35974: Fix missing new line in ssl_client.

2015-06-27 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35974/ --- Review request for mesos, Benjamin Hindman and Vinod Kone. Repository: mesos

Re: Review Request 35974: Fix missing new line in ssl_client.

2015-06-27 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35974/#review89633 --- Ship it! - Joerg Schad On June 27, 2015, 7:30 p.m., Joris Van

Re: Review Request 35974: Fix missing new line in ssl_client.

2015-06-27 Thread Kapil Arya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35974/#review89632 --- Ship it! Ship It! - Kapil Arya On June 27, 2015, 3:30 p.m.,

Re: Review Request 32982: Added reservation user guide.

2015-06-27 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32982/ --- (Updated June 28, 2015, 3:33 a.m.) Review request for mesos, Alexander

Review Request 35981: Added persistent volume user guide.

2015-06-27 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35981/ --- Review request for mesos, Adam B, Jie Yu, and Timothy Chen. Bugs: MESOS-2405

Re: Review Request 35911: Moved StatusUpdate.uuid from required to optional.

2015-06-27 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35911/#review89651 --- Ship it! Ship It! - Vinod Kone On June 27, 2015, 12:36 a.m.,

Re: Review Request 32982: Added reservation user guide.

2015-06-27 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32982/ --- (Updated June 28, 2015, 3:23 a.m.) Review request for mesos, Alexander

Re: Review Request 35721: Set the ownership of persistent volume to match the sandbox directory.

2015-06-27 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35721/ --- (Updated June 28, 2015, 5:51 a.m.) Review request for mesos, Adam B, Jie Yu,

Re: Review Request 32982: Added reservation user guide.

2015-06-27 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32982/ --- (Updated June 28, 2015, 3:27 a.m.) Review request for mesos, Alexander

Re: Review Request 32982: Added reservation user guide.

2015-06-27 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32982/ --- (Updated June 28, 2015, 3:28 a.m.) Review request for mesos, Alexander

Re: Review Request 35981: Added persistent volume user guide.

2015-06-27 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35981/#review89655 --- Patch looks great! Reviews applied: [32982, 35981] All tests

Re: Review Request 35721: Set the owner of persistent volumes to frameworkInfo.user .

2015-06-27 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35721/ --- (Updated June 28, 2015, 5:42 a.m.) Review request for mesos, Adam B and Jie

Re: Review Request 35721: Set the owner of persistent volumes to frameworkInfo.user .

2015-06-27 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35721/ --- (Updated June 28, 2015, 5:46 a.m.) Review request for mesos, Adam B and Jie