----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27531/#review71124 -----------------------------------------------------------
Bad patch! Reviews applied: [27531] Failed command: ./support/apply-review.sh -n -r 27531 Error: 2015-02-05 01:53:52 URL:https://reviews.apache.org/r/27531/diff/raw/ [14793/14793] -> "27531.patch" [1] error: patch failed: src/master/master.cpp:2220 error: src/master/master.cpp: patch does not apply Failed to apply patch - Mesos ReviewBot On Feb. 5, 2015, 12:19 a.m., Dominic Hamon wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/27531/ > ----------------------------------------------------------- > > (Updated Feb. 5, 2015, 12:19 a.m.) > > > Review request for mesos and Vinod Kone. > > > Bugs: MESOS-1830 > https://issues.apache.org/jira/browse/MESOS-1830 > > > Repository: mesos > > > Description > ------- > > Update metrics in Master to match the source and reason split for task > statuses. > > > Diffs > ----- > > src/master/master.cpp f4b6463b4af967afc2ff78862687cddc8617962b > src/master/metrics.hpp 6a43abc914dce24c60b5db57ee01d172c8258e82 > src/master/metrics.cpp 956fe5042d7478d90d4f03059e248694ba2b95e5 > src/slave/slave.hpp 70bd8c1fde4ea09fa54c76aa93424a1adb0309f6 > src/slave/slave.cpp a8b262174ab5c9a524db8318d3d1438cd75a702b > src/tests/master_authorization_tests.cpp > 20adaa954a0ddfe3459d8a3f9696b5ca9ae07f24 > src/tests/master_slave_reconciliation_tests.cpp > 04806ede4e727fa4de1464017a06797d69b54e29 > src/tests/master_tests.cpp 678d27f41a2f246c714c77adb132263c0c2c61ed > src/tests/mesos.hpp 17c2d8f0cb6326b08fc506143e823ee2c3a32e09 > src/tests/mesos.cpp 5ed4df530cf1bf11eec3b29542641822e0f702b2 > src/tests/rate_limiting_tests.cpp 7f5ca251fef5b1852bb273505f5eaed750578b9f > src/tests/slave_tests.cpp e7e2af63da785644f3f7e6e23607c02be962a2c6 > > Diff: https://reviews.apache.org/r/27531/diff/ > > > Testing > ------- > > added metric tests to master tests > make check > > > Thanks, > > Dominic Hamon > >
