power_LoadTest: skip web_page domains for keyvals
After introducing checkpoint logic for each website, the keyval file is
blowing up with detailed stats about each website. The keyval file is
not the place for that. This change skips all checkpoint data for
domains that contain [web_page_] in their name as its the prefix used
for specific web page data. The raw data is still preserved & stored.
If and when there are more usecases to skip domains besides web-pages,
then a more generalizable domain-skipping system (either in PLT, or in
the calc() function) might be appropiate, but that's not the case yet.
BUG=b:117680074
TEST=test_that ... power_LoadTest.fast
Change-Id: Ib49433809960736015ce67b21190bedc89933ac2
Signed-off-by: Ruben Rodriguez Buchillon <coconutruben@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/1280482
Reviewed-by: Puthikorn Voravootivat <puthik@chromium.org>
diff --git a/client/site_tests/power_LoadTest/power_LoadTest.py b/client/site_tests/power_LoadTest/power_LoadTest.py
index d6bd8fc..e770f23 100755
--- a/client/site_tests/power_LoadTest/power_LoadTest.py
+++ b/client/site_tests/power_LoadTest/power_LoadTest.py
@@ -502,11 +502,14 @@
minutes_battery_life_tested = keyvals['minutes_battery_life_tested']
+ # Avoid polluting the keyvals with non-core domains.
+ core_keyvals = {k: v for k, v in keyvals.iteritems()
+ if not k.startswith('_')}
if not self._gaia_login:
- keyvals = dict(map(lambda (key, value):
- ('INVALID_' + str(key), value), keyvals.items()))
+ core_keyvals = {'INVALID_%s' % str(k): v for k, v in
+ core_keyvals.iteritems()}
else:
- for key, value in keyvals.iteritems():
+ for key, value in core_keyvals.iteritems():
if key.startswith('percent_cpuidle') and \
key.endswith('C0_time'):
self.output_perf_value(description=key,
@@ -514,7 +517,7 @@
units='percent',
higher_is_better=False)
- self.write_perf_keyval(keyvals)
+ self.write_perf_keyval(core_keyvals)
for log in self._meas_logs:
log.save_results(self.resultsdir)
self._checkpoint_logger.save_checkpoint_data(self.resultsdir)