----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67639/#review204974 -----------------------------------------------------------
Master (9c9b592) is green with this patch. ./build-support/jenkins/build.sh However, it appears that it might lack test coverage. I will refresh this build result if you post a review containing "@ReviewBot retry" - Aurora ReviewBot On June 19, 2018, 1:21 a.m., Santhosh Kumar Shanmugham wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/67639/ > ----------------------------------------------------------- > > (Updated June 19, 2018, 1:21 a.m.) > > > Review request for Aurora, Franck Cuny and Jordan Ly. > > > Repository: aurora > > > Description > ------- > > Since the scheduler enforces a maximum timeout on each > maintenance request and we now allow CoordinatorSlaPolicy > to block maintenance, we need to know which tasks are > running into the force maintenance timeout. Export maintenace > count down time as a metric brokwen down by task keys. > > > Diffs > ----- > > src/main/java/org/apache/aurora/scheduler/base/InstanceKeys.java > b12ac83168401c15fb1d30179ea8e4816f09cd3d > > src/main/java/org/apache/aurora/scheduler/maintenance/MaintenanceController.java > 7fc5990dfb04c5528a44142c3efdd6d60d08188d > > > Diff: https://reviews.apache.org/r/67639/diff/1/ > > > Testing > ------- > > ./gradlew test > > **Tested in Vagrant** > sshanmugham::tw-mbp-sshanmugham {~}$ curl http://192.168.33.7:8081/vars | > grep maintenance_countdown > ######################################################################## > 100.0% > maintenance_countdown_ms_vagrant/test/coordinator/0 264523 > maintenance_countdown_ms_vagrant/test/coordinator/1 24476 > sshanmugham::tw-mbp-sshanmugham {~}$ curl http://192.168.33.7:8081/vars | > grep maintenance_countdown > ######################################################################## > 100.0% > maintenance_countdown_ms_vagrant/test/coordinator/0 264523 > maintenance_countdown_ms_vagrant/test/coordinator/1 24476 > sshanmugham::tw-mbp-sshanmugham {~}$ curl http://192.168.33.7:8081/vars | > grep maintenance_countdown > ######################################################################## > 100.0% > maintenance_countdown_ms_vagrant/test/coordinator/0 264523 > maintenance_countdown_ms_vagrant/test/coordinator/1 0 > > > Thanks, > > Santhosh Kumar Shanmugham > >