Re: Review Request 65959: Added a test to make sure `slave/recovery_secs` is reported in metrics.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65959/#review198852 --- Patch looks great! Reviews applied: [65954, 65959] Passed command: export OS='ubuntu:14.04' BUILDTOOL='autotools' COMPILER='gcc' CONFIGURATION='--verbose --disable-libtool-wrappers' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; ./support/docker-build.sh - Mesos Reviewbot On March 7, 2018, 11:21 p.m., Zhitao Li wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/65959/ > --- > > (Updated March 7, 2018, 11:21 p.m.) > > > Review request for mesos, Gilbert Song, Greg Mann, Jason Lai, and James Peach. > > > Bugs: MESOS-8609 > https://issues.apache.org/jira/browse/MESOS-8609 > > > Repository: mesos > > > Description > --- > > Added a test to make sure `slave/recovery_secs` is reported in metrics. > > > Diffs > - > > src/tests/slave_tests.cpp e253317a67019302f18afe11e2a314e716cec226 > > > Diff: https://reviews.apache.org/r/65959/diff/1/ > > > Testing > --- > > > Thanks, > > Zhitao Li > >
Re: Review Request 65959: Added a test to make sure `slave/recovery_secs` is reported in metrics.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65959/#review198837 --- PASS: Mesos patch 65959 was successfully built and tested. Reviews applied: `['65954', '65959']` All the build artifacts available at: http://dcos-win.westus.cloudapp.azure.com/mesos-build/review/65959 - Mesos Reviewbot Windows On March 7, 2018, 11:21 p.m., Zhitao Li wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/65959/ > --- > > (Updated March 7, 2018, 11:21 p.m.) > > > Review request for mesos, Gilbert Song, Greg Mann, Jason Lai, and James Peach. > > > Bugs: MESOS-8609 > https://issues.apache.org/jira/browse/MESOS-8609 > > > Repository: mesos > > > Description > --- > > Added a test to make sure `slave/recovery_secs` is reported in metrics. > > > Diffs > - > > src/tests/slave_tests.cpp e253317a67019302f18afe11e2a314e716cec226 > > > Diff: https://reviews.apache.org/r/65959/diff/1/ > > > Testing > --- > > > Thanks, > > Zhitao Li > >