-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/42197/
-----------------------------------------------------------

(Updated Jan. 14, 2016, 2:07 p.m.)


Review request for mesos, Benjamin Bannier and Till Toenshoff.


Changes
-------

Added an error message for non-Linux machines.


Bugs: MESOS-3082
    https://issues.apache.org/jira/browse/MESOS-3082


Repository: mesos


Description
-------

Virtual machines may not always support 'CPU performance counters'. Tests
trying to collect to 'cycle' value of 'perf' will fail under these
circumstances. This will be avoided by disabling these tests if 'perf' hardware
events are not available.


Diffs (updated)
-----

  src/tests/environment.cpp 20218a086baefcefb310eb45ed9024e5425ce787 

Diff: https://reviews.apache.org/r/42197/diff/


Testing (updated)
-------

Running `sudo ./bin/mesos-tests.sh 
--gtest_filter="*ROOT_CGROUPS_Perf:*ROOT_CGROUPS_Sample:*ROOT_CGROUPS_UserCgroup:*CGROUPS_ROOT_PerfRollForward:*ROOT_Sample"`

1) In a Linux VM without 'CPU performance counters': This doesn't run the tests 
and shows an error message that 'cycles' is not available, as expected
2) On OSX: This doesn't run the tests and shows an error message that 'perf' is 
not available, as expected
3) NOT TESTED YET!!! On native Linux (i.e. with 'CPU performance counters'): 
This runs the tests.


Thanks,

Jan Schlicht

Reply via email to