Re: Review Request 58357: Support more test frameworks in test-upgrade script.

2017-04-29 Thread Mesos Reviewbot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58357/#review173424 --- Patch looks great! Reviews applied: [58357] Passed command:

Re: Review Request 58357: Support more test frameworks in test-upgrade script.

2017-04-28 Thread Zhitao Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58357/ --- (Updated April 29, 2017, 12:17 a.m.) Review request for mesos, Anand Mazumdar,

Re: Review Request 58357: Support more test frameworks in test-upgrade script.

2017-04-28 Thread Mesos Reviewbot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58357/#review173303 --- Patch looks great! Reviews applied: [58357] Passed command:

Re: Review Request 58357: Support more test frameworks in test-upgrade script.

2017-04-27 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58357/#review173277 --- Fix it, then Ship it! support/test-upgrade.py Lines 342-343

Re: Review Request 58357: Support more test frameworks in test-upgrade script.

2017-04-27 Thread Zhitao Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58357/ --- (Updated April 27, 2017, 5:58 p.m.) Review request for mesos, Anand Mazumdar,

Re: Review Request 58357: Support more test frameworks in test-upgrade script.

2017-04-24 Thread Greg Mann
> On April 20, 2017, 9:40 p.m., Greg Mann wrote: > > support/test-upgrade.py > > Lines 298-302 (patched) > > > > > > Would it be possible to allow the user to exclude more than one at a > > time? I would also be

Re: Review Request 58357: Support more test frameworks in test-upgrade script.

2017-04-24 Thread Zhitao Li
> On April 20, 2017, 9:40 p.m., Greg Mann wrote: > > support/test-upgrade.py > > Lines 298-302 (patched) > > > > > > Would it be possible to allow the user to exclude more than one at a > > time? I would also be

Re: Review Request 58357: Support more test frameworks in test-upgrade script.

2017-04-20 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58357/#review172551 --- Tested this out; works great!! Just a couple small things below.

Re: Review Request 58357: Support more test frameworks in test-upgrade script.

2017-04-19 Thread Mesos Reviewbot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58357/#review172403 --- Patch looks great! Reviews applied: [58357] Passed command:

Re: Review Request 58357: Support more test frameworks in test-upgrade script.

2017-04-19 Thread Zhitao Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58357/ --- (Updated April 19, 2017, 4:28 p.m.) Review request for mesos, Anand Mazumdar,