> On April 20, 2017, 9:40 p.m., Greg Mann wrote: > > support/test-upgrade.py > > Lines 298-302 (patched) > > <https://reviews.apache.org/r/58357/diff/3/?file=1694387#file1694387line326> > > > > Would it be possible to allow the user to exclude more than one at a > > time? I would also be find with the user specifying explicitly a list of > > the frameworks they _do_ want to run. > > Zhitao Li wrote: > Yes I can make this a comma separated list. > > Do you think we need both of `excluding tests` and `only run tests`?
I think just one or the other is fine. - Greg ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58357/#review172551 ----------------------------------------------------------- On April 19, 2017, 4:28 p.m., Zhitao Li wrote: > > ----------------------------------------------------------- > 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, Xiaojian Huang, and Greg Mann. > > > Bugs: MESOS-7228 > https://issues.apache.org/jira/browse/MESOS-7228 > > > Repository: mesos > > > Description > ------- > > This patch added support to java and python based test framework in > `test-upgrade.py` script. > > > Diffs > ----- > > support/test-upgrade.py 84df21dd6e63653e1a18e700ef904aa3a04b2b45 > > > Diff: https://reviews.apache.org/r/58357/diff/3/ > > > Testing > ------- > > Ran this on all three languages options for cpp, java and python. > > > Thanks, > > Zhitao Li > >
