----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49334/#review139857 -----------------------------------------------------------
config/legacy_untested_classes.txt (line 44) <https://reviews.apache.org/r/49334/#comment205208> Isn't this a fairly easy class to test? - David McLaughlin On June 28, 2016, 8:42 p.m., Mehrdad Nurolahzade wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/49334/ > ----------------------------------------------------------- > > (Updated June 28, 2016, 8:42 p.m.) > > > Review request for Aurora, David McLaughlin and Maxim Khutornenko. > > > Repository: aurora > > > Description > ------- > > AURORA-1725 Expose tier configurations as a debug page > > > Diffs > ----- > > config/legacy_untested_classes.txt 1ea2183ab20cc5c6bca147bcea4e5c708d576b62 > src/main/java/org/apache/aurora/scheduler/http/JettyServerModule.java > 496cd5590d6042d30e7bd0410a6c40d804c15023 > src/main/java/org/apache/aurora/scheduler/http/Tiers.java PRE-CREATION > src/main/resources/scheduler/assets/index.html > eca27e081950a66b743f79ae7ae3d08e4625d2c8 > src/test/java/org/apache/aurora/scheduler/http/AbstractJettyTest.java > 561b134152113a50ff495cef28d2396b56d02f1c > > Diff: https://reviews.apache.org/r/49334/diff/ > > > Testing > ------- > > Comments: > - Logged the work done so far (new ```/tiers``` debug page) to this ticket > and kept AURORA-1710 for modifications to scheduler UI. > - Added comments to AURORA-1710 to clarify the suggested approach for > implementation. > > Testing: > ``` > ./build-support/jenkins/build.sh > ... > > *** OK (All tests passed) *** > > ./src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh > ... > mesos-master start/running, process 18776 > + RETCODE=0 > + restore_netrc > + mv /home/vagrant/.netrc.bak /home/vagrant/.netrc > + true > Connection to 127.0.0.1 closed. > > real 15m5.795s > user 0m1.318s > sys 0m0.533s > ``` > > > Thanks, > > Mehrdad Nurolahzade > >
