----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58366/#review171658 -----------------------------------------------------------
Ship it! Master (cc2aa46) is green with this patch. ./build-support/jenkins/build.sh I will refresh this build result if you post a review containing "@ReviewBot retry" - Aurora ReviewBot On April 11, 2017, 11:47 p.m., David Robinson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/58366/ > ----------------------------------------------------------- > > (Updated April 11, 2017, 11:47 p.m.) > > > Review request for Aurora. > > > Bugs: AURORA-1918 > https://issues.apache.org/jira/browse/AURORA-1918 > > > Repository: aurora > > > Description > ------- > > Allow resource monitors to be disabled. > > > Diffs > ----- > > src/main/python/apache/aurora/executor/bin/thermos_executor_main.py > c6c08983fb2d204afedd419798135d667169bce4 > src/main/python/apache/thermos/monitoring/resource.py > 434666696e600a0e6c19edd986c86575539976f2 > > src/test/python/apache/aurora/executor/bin/test_thermos_executor_entry_point.py > 38deae69dce7d88dab87f70e0128e0b86091fdef > > > Diff: https://reviews.apache.org/r/58366/diff/3/ > > > Testing > ------- > > Tested -thermos_executor_flags="--task_disk_collection_interval_secs=0 > --task_process_collection_interval_secs=0" (and different combinations: disk > enabled and process disabled, disk disabled and process enabled, both > enabled, both disabled). Launched various jobs and ensured the specific > resource monitors were enabled or disabled as per the executor flags. > > > Thanks, > > David Robinson > >
