Re: Review Request 52855: Re-enabled the change user test in slave tests.

2017-03-01 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52855/#review167395 --- Closing this review due to inactivity. Please see our

Re: Review Request 52855: Re-enabled the change user test in slave tests.

2016-10-14 Thread Gilbert Song
> On Oct. 14, 2016, 11:56 a.m., Gilbert Song wrote: > > src/tests/slave_tests.cpp, lines 1163-1165 > > > > > > Use `createDockerImage()`. Please discard this comment. - Gilbert

Re: Review Request 52855: Re-enabled the change user test in slave tests.

2016-10-14 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52855/#review152709 --- Fix it, then Ship it! src/tests/slave_tests.cpp (line 1045)

Re: Review Request 52855: Re-enabled the change user test in slave tests.

2016-10-14 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52855/#review152661 --- Patch looks great! Reviews applied: [52854, 52855] Passed

Review Request 52855: Re-enabled the change user test in slave tests.

2016-10-13 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52855/ --- Review request for mesos and Gilbert Song. Bugs: MESOS-6391