> On Feb. 27, 2013, 1:32 a.m., Benjamin Hindman wrote: > > src/tests/monitor_tests.cpp, line 93 > > <https://reviews.apache.org/r/9636/diff/2/?file=262939#file262939line93> > > > > I'm fine to have you expose the pid (we've done this in a few other > > places too) but you also can construct this PID very easily (since you know > > it's ID and can get the IP/port as well).
Ah yes I wanted to do that, but missed the process::ip and process::port calls! I will change http::get to take a UPID instead as well, but I thought I recalled an aversion to that on your part? > On Feb. 27, 2013, 1:32 a.m., Benjamin Hindman wrote: > > src/tests/monitor_tests.cpp, line 78 > > <https://reviews.apache.org/r/9636/diff/2/?file=262939#file262939line78> > > > > Don't even need the temporary if you don't want it. Couldn't get it to compile used inline without the temporary. - Ben ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9636/#review17125 ----------------------------------------------------------- On Feb. 27, 2013, 1:08 a.m., Ben Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/9636/ > ----------------------------------------------------------- > > (Updated Feb. 27, 2013, 1:08 a.m.) > > > Review request for mesos, Benjamin Hindman and Vinod Kone. > > > Description > ------- > > Ultimately I'd like to be testing even more functionality of the > ResourceMonitor: > 1. Check that the data has been published to statistics. > 2. Check that metering is occurring on subsequent resource data (cpu_usage). > 3. Add tests for process based isolation usage. > 4. Add tests for cgroups based isolation usage. > > 1 and 2 are possible within this change. > 3 and 4 are trickier given our current testing setup. > > For now, I've done testing locally and a test cluster to ensure the above > were working as intended. > > > Diffs > ----- > > src/Makefile.am 8c74525ff6381b6d8624622a758b792fd631cd8c > src/slave/monitor.hpp e2ef3bdf6cd5b2718db01184d98d4d29cd5b00c9 > src/slave/monitor.cpp 16a555c15014bd964003be0666908ae1720bf1fc > src/tests/monitor_tests.cpp PRE-CREATION > > Diff: https://reviews.apache.org/r/9636/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Ben Mahler > >
