Patrick Williams | c0f7c04 | 2017-02-23 20:41:17 -0600 | [diff] [blame] | 1 | #!/usr/bin/python3 |
| 2 | # |
| 3 | # Script for comparing buildstats from two different builds |
| 4 | # |
| 5 | # Copyright (c) 2016, Intel Corporation. |
| 6 | # |
| 7 | # This program is free software; you can redistribute it and/or modify it |
| 8 | # under the terms and conditions of the GNU General Public License, |
| 9 | # version 2, as published by the Free Software Foundation. |
| 10 | # |
| 11 | # This program is distributed in the hope it will be useful, but WITHOUT |
| 12 | # ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
| 13 | # FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for |
| 14 | # more details. |
| 15 | # |
| 16 | import argparse |
| 17 | import glob |
| 18 | import json |
| 19 | import logging |
| 20 | import math |
| 21 | import os |
| 22 | import re |
| 23 | import sys |
| 24 | from collections import namedtuple |
| 25 | from datetime import datetime, timedelta, tzinfo |
| 26 | from operator import attrgetter |
| 27 | |
| 28 | # Setup logging |
| 29 | logging.basicConfig(level=logging.INFO, format="%(levelname)s: %(message)s") |
| 30 | log = logging.getLogger() |
| 31 | |
| 32 | |
| 33 | class ScriptError(Exception): |
| 34 | """Exception for internal error handling of this script""" |
| 35 | pass |
| 36 | |
| 37 | |
| 38 | class TimeZone(tzinfo): |
| 39 | """Simple fixed-offset tzinfo""" |
| 40 | def __init__(self, seconds, name): |
| 41 | self._offset = timedelta(seconds=seconds) |
| 42 | self._name = name |
| 43 | |
| 44 | def utcoffset(self, dt): |
| 45 | return self._offset |
| 46 | |
| 47 | def tzname(self, dt): |
| 48 | return self._name |
| 49 | |
| 50 | def dst(self, dt): |
| 51 | return None |
| 52 | |
| 53 | TIMEZONES = {'UTC': TimeZone(0, 'UTC'), |
| 54 | 'EET': TimeZone(7200, 'EET'), |
| 55 | 'EEST': TimeZone(10800, 'EEST')} |
| 56 | |
| 57 | taskdiff_fields = ('pkg', 'pkg_op', 'task', 'task_op', 'value1', 'value2', |
| 58 | 'absdiff', 'reldiff') |
| 59 | TaskDiff = namedtuple('TaskDiff', ' '.join(taskdiff_fields)) |
| 60 | |
| 61 | |
| 62 | def to_datetime_obj(obj): |
| 63 | """Helper for getting timestamps in datetime format""" |
| 64 | if isinstance(obj, datetime): |
| 65 | return obj |
| 66 | else: |
| 67 | return datetime.utcfromtimestamp(obj).replace(tzinfo=TIMEZONES['UTC']) |
| 68 | |
| 69 | |
| 70 | class BSTask(dict): |
| 71 | def __init__(self, *args, **kwargs): |
| 72 | self['start_time'] = None |
| 73 | self['elapsed_time'] = None |
| 74 | self['status'] = None |
| 75 | self['iostat'] = {} |
| 76 | self['rusage'] = {} |
| 77 | self['child_rusage'] = {} |
| 78 | super(BSTask, self).__init__(*args, **kwargs) |
| 79 | |
| 80 | @property |
| 81 | def cputime(self): |
| 82 | """Sum of user and system time taken by the task""" |
| 83 | return self['rusage']['ru_stime'] + self['rusage']['ru_utime'] + \ |
| 84 | self['child_rusage']['ru_stime'] + self['child_rusage']['ru_utime'] |
| 85 | |
| 86 | @property |
| 87 | def walltime(self): |
| 88 | """Elapsed wall clock time""" |
| 89 | return self['elapsed_time'].total_seconds() |
| 90 | |
| 91 | @property |
| 92 | def read_bytes(self): |
| 93 | """Bytes read from the block layer""" |
| 94 | return self['iostat']['read_bytes'] |
| 95 | |
| 96 | @property |
| 97 | def write_bytes(self): |
| 98 | """Bytes written to the block layer""" |
| 99 | return self['iostat']['write_bytes'] |
| 100 | |
| 101 | @property |
| 102 | def read_ops(self): |
| 103 | """Number of read operations on the block layer""" |
| 104 | return self['rusage']['ru_inblock'] + self['child_rusage']['ru_inblock'] |
| 105 | |
| 106 | @property |
| 107 | def write_ops(self): |
| 108 | """Number of write operations on the block layer""" |
| 109 | return self['rusage']['ru_oublock'] + self['child_rusage']['ru_oublock'] |
| 110 | |
| 111 | |
| 112 | def read_buildstats_file(buildstat_file): |
| 113 | """Convert buildstat text file into dict/json""" |
| 114 | bs_task = BSTask() |
| 115 | log.debug("Reading task buildstats from %s", buildstat_file) |
| 116 | with open(buildstat_file) as fobj: |
| 117 | for line in fobj.readlines(): |
| 118 | key, val = line.split(':', 1) |
| 119 | val = val.strip() |
| 120 | if key == 'Started': |
| 121 | start_time = to_datetime_obj(float(val)) |
| 122 | bs_task['start_time'] = start_time |
| 123 | elif key == 'Ended': |
| 124 | end_time = to_datetime_obj(float(val)) |
| 125 | elif key.startswith('IO '): |
| 126 | split = key.split() |
| 127 | bs_task['iostat'][split[1]] = int(val) |
| 128 | elif key.find('rusage') >= 0: |
| 129 | split = key.split() |
| 130 | ru_key = split[-1] |
| 131 | if ru_key in ('ru_stime', 'ru_utime'): |
| 132 | val = float(val) |
| 133 | else: |
| 134 | val = int(val) |
| 135 | ru_type = 'rusage' if split[0] == 'rusage' else \ |
| 136 | 'child_rusage' |
| 137 | bs_task[ru_type][ru_key] = val |
| 138 | elif key == 'Status': |
| 139 | bs_task['status'] = val |
| 140 | bs_task['elapsed_time'] = end_time - start_time |
| 141 | return bs_task |
| 142 | |
| 143 | |
| 144 | def read_buildstats_dir(bs_dir): |
| 145 | """Read buildstats directory""" |
| 146 | def split_nevr(nevr): |
| 147 | """Split name and version information from recipe "nevr" string""" |
| 148 | n_e_v, revision = nevr.rsplit('-', 1) |
| 149 | match = re.match(r'^(?P<name>\S+)-((?P<epoch>[0-9]{1,5})_)?(?P<version>[0-9]\S*)$', |
| 150 | n_e_v) |
| 151 | if not match: |
| 152 | # If we're not able to parse a version starting with a number, just |
| 153 | # take the part after last dash |
| 154 | match = re.match(r'^(?P<name>\S+)-((?P<epoch>[0-9]{1,5})_)?(?P<version>[^-]+)$', |
| 155 | n_e_v) |
| 156 | name = match.group('name') |
| 157 | version = match.group('version') |
| 158 | epoch = match.group('epoch') |
| 159 | return name, epoch, version, revision |
| 160 | |
| 161 | if not os.path.isfile(os.path.join(bs_dir, 'build_stats')): |
| 162 | raise ScriptError("{} does not look like a buildstats directory".format(bs_dir)) |
| 163 | |
| 164 | log.debug("Reading buildstats directory %s", bs_dir) |
| 165 | |
| 166 | buildstats = {} |
| 167 | subdirs = os.listdir(bs_dir) |
| 168 | for dirname in subdirs: |
| 169 | recipe_dir = os.path.join(bs_dir, dirname) |
| 170 | if not os.path.isdir(recipe_dir): |
| 171 | continue |
| 172 | name, epoch, version, revision = split_nevr(dirname) |
| 173 | recipe_bs = {'nevr': dirname, |
| 174 | 'name': name, |
| 175 | 'epoch': epoch, |
| 176 | 'version': version, |
| 177 | 'revision': revision, |
| 178 | 'tasks': {}} |
| 179 | for task in os.listdir(recipe_dir): |
| 180 | recipe_bs['tasks'][task] = [read_buildstats_file( |
| 181 | os.path.join(recipe_dir, task))] |
| 182 | if name in buildstats: |
| 183 | raise ScriptError("Cannot handle multiple versions of the same " |
| 184 | "package ({})".format(name)) |
| 185 | buildstats[name] = recipe_bs |
| 186 | |
| 187 | return buildstats |
| 188 | |
| 189 | |
| 190 | def bs_append(dst, src): |
| 191 | """Append data from another buildstats""" |
| 192 | if set(dst.keys()) != set(src.keys()): |
| 193 | raise ScriptError("Refusing to join buildstats, set of packages is " |
| 194 | "different") |
| 195 | for pkg, data in dst.items(): |
| 196 | if data['nevr'] != src[pkg]['nevr']: |
| 197 | raise ScriptError("Refusing to join buildstats, package version " |
| 198 | "differs: {} vs. {}".format(data['nevr'], src[pkg]['nevr'])) |
| 199 | if set(data['tasks'].keys()) != set(src[pkg]['tasks'].keys()): |
| 200 | raise ScriptError("Refusing to join buildstats, set of tasks " |
| 201 | "in {} differ".format(pkg)) |
| 202 | for taskname, taskdata in data['tasks'].items(): |
| 203 | taskdata.extend(src[pkg]['tasks'][taskname]) |
| 204 | |
| 205 | |
| 206 | def read_buildstats_json(path): |
| 207 | """Read buildstats from JSON file""" |
| 208 | buildstats = {} |
| 209 | with open(path) as fobj: |
| 210 | bs_json = json.load(fobj) |
| 211 | for recipe_bs in bs_json: |
| 212 | if recipe_bs['name'] in buildstats: |
| 213 | raise ScriptError("Cannot handle multiple versions of the same " |
| 214 | "package ({})".format(recipe_bs['name'])) |
| 215 | |
| 216 | if recipe_bs['epoch'] is None: |
| 217 | recipe_bs['nevr'] = "{}-{}-{}".format(recipe_bs['name'], recipe_bs['version'], recipe_bs['revision']) |
| 218 | else: |
| 219 | recipe_bs['nevr'] = "{}-{}_{}-{}".format(recipe_bs['name'], recipe_bs['epoch'], recipe_bs['version'], recipe_bs['revision']) |
| 220 | |
| 221 | for task, data in recipe_bs['tasks'].copy().items(): |
| 222 | recipe_bs['tasks'][task] = [BSTask(data)] |
| 223 | |
| 224 | buildstats[recipe_bs['name']] = recipe_bs |
| 225 | |
| 226 | return buildstats |
| 227 | |
| 228 | |
| 229 | def read_buildstats(path, multi): |
| 230 | """Read buildstats""" |
| 231 | if not os.path.exists(path): |
| 232 | raise ScriptError("No such file or directory: {}".format(path)) |
| 233 | |
| 234 | if os.path.isfile(path): |
| 235 | return read_buildstats_json(path) |
| 236 | |
| 237 | if os.path.isfile(os.path.join(path, 'build_stats')): |
| 238 | return read_buildstats_dir(path) |
| 239 | |
| 240 | # Handle a non-buildstat directory |
| 241 | subpaths = sorted(glob.glob(path + '/*')) |
| 242 | if len(subpaths) > 1: |
| 243 | if multi: |
| 244 | log.info("Averaging over {} buildstats from {}".format( |
| 245 | len(subpaths), path)) |
| 246 | else: |
| 247 | raise ScriptError("Multiple buildstats found in '{}'. Please give " |
| 248 | "a single buildstat directory of use the --multi " |
| 249 | "option".format(path)) |
| 250 | bs = None |
| 251 | for subpath in subpaths: |
| 252 | if os.path.isfile(subpath): |
| 253 | tmpbs = read_buildstats_json(subpath) |
| 254 | else: |
| 255 | tmpbs = read_buildstats_dir(subpath) |
| 256 | if not bs: |
| 257 | bs = tmpbs |
| 258 | else: |
| 259 | log.debug("Joining buildstats") |
| 260 | bs_append(bs, tmpbs) |
| 261 | |
| 262 | if not bs: |
| 263 | raise ScriptError("No buildstats found under {}".format(path)) |
| 264 | return bs |
| 265 | |
| 266 | |
| 267 | def print_ver_diff(bs1, bs2): |
| 268 | """Print package version differences""" |
| 269 | pkgs1 = set(bs1.keys()) |
| 270 | pkgs2 = set(bs2.keys()) |
| 271 | new_pkgs = pkgs2 - pkgs1 |
| 272 | deleted_pkgs = pkgs1 - pkgs2 |
| 273 | |
| 274 | echanged = [] |
| 275 | vchanged = [] |
| 276 | rchanged = [] |
| 277 | unchanged = [] |
| 278 | common_pkgs = pkgs2.intersection(pkgs1) |
| 279 | if common_pkgs: |
| 280 | for pkg in common_pkgs: |
| 281 | if bs1[pkg]['epoch'] != bs2[pkg]['epoch']: |
| 282 | echanged.append(pkg) |
| 283 | elif bs1[pkg]['version'] != bs2[pkg]['version']: |
| 284 | vchanged.append(pkg) |
| 285 | elif bs1[pkg]['revision'] != bs2[pkg]['revision']: |
| 286 | rchanged.append(pkg) |
| 287 | else: |
| 288 | unchanged.append(pkg) |
| 289 | |
| 290 | maxlen = max([len(pkg) for pkg in pkgs1.union(pkgs2)]) |
| 291 | fmt_str = " {:{maxlen}} ({})" |
| 292 | # if unchanged: |
| 293 | # print("\nUNCHANGED PACKAGES:") |
| 294 | # print("-------------------") |
| 295 | # maxlen = max([len(pkg) for pkg in unchanged]) |
| 296 | # for pkg in sorted(unchanged): |
| 297 | # print(fmt_str.format(pkg, bs2[pkg]['nevr'], maxlen=maxlen)) |
| 298 | |
| 299 | if new_pkgs: |
| 300 | print("\nNEW PACKAGES:") |
| 301 | print("-------------") |
| 302 | for pkg in sorted(new_pkgs): |
| 303 | print(fmt_str.format(pkg, bs2[pkg]['nevr'], maxlen=maxlen)) |
| 304 | |
| 305 | if deleted_pkgs: |
| 306 | print("\nDELETED PACKAGES:") |
| 307 | print("-----------------") |
| 308 | for pkg in sorted(deleted_pkgs): |
| 309 | print(fmt_str.format(pkg, bs1[pkg]['nevr'], maxlen=maxlen)) |
| 310 | |
| 311 | fmt_str = " {0:{maxlen}} {1:<20} ({2})" |
| 312 | if rchanged: |
| 313 | print("\nREVISION CHANGED:") |
| 314 | print("-----------------") |
| 315 | for pkg in sorted(rchanged): |
| 316 | field1 = "{} -> {}".format(pkg, bs1[pkg]['revision'], bs2[pkg]['revision']) |
| 317 | field2 = "{} -> {}".format(bs1[pkg]['nevr'], bs2[pkg]['nevr']) |
| 318 | print(fmt_str.format(pkg, field1, field2, maxlen=maxlen)) |
| 319 | |
| 320 | if vchanged: |
| 321 | print("\nVERSION CHANGED:") |
| 322 | print("----------------") |
| 323 | for pkg in sorted(vchanged): |
| 324 | field1 = "{} -> {}".format(bs1[pkg]['version'], bs2[pkg]['version']) |
| 325 | field2 = "{} -> {}".format(bs1[pkg]['nevr'], bs2[pkg]['nevr']) |
| 326 | print(fmt_str.format(pkg, field1, field2, maxlen=maxlen)) |
| 327 | |
| 328 | if echanged: |
| 329 | print("\nEPOCH CHANGED:") |
| 330 | print("--------------") |
| 331 | for pkg in sorted(echanged): |
| 332 | field1 = "{} -> {}".format(pkg, bs1[pkg]['epoch'], bs2[pkg]['epoch']) |
| 333 | field2 = "{} -> {}".format(bs1[pkg]['nevr'], bs2[pkg]['nevr']) |
| 334 | print(fmt_str.format(pkg, field1, field2, maxlen=maxlen)) |
| 335 | |
| 336 | |
| 337 | def print_task_diff(bs1, bs2, val_type, min_val=0, min_absdiff=0, sort_by=('absdiff',)): |
| 338 | """Diff task execution times""" |
| 339 | def val_to_str(val, human_readable=False): |
| 340 | """Convert raw value to printable string""" |
| 341 | def hms_time(secs): |
| 342 | """Get time in human-readable HH:MM:SS format""" |
| 343 | h = int(secs / 3600) |
| 344 | m = int((secs % 3600) / 60) |
| 345 | s = secs % 60 |
| 346 | if h == 0: |
| 347 | return "{:02d}:{:04.1f}".format(m, s) |
| 348 | else: |
| 349 | return "{:d}:{:02d}:{:04.1f}".format(h, m, s) |
| 350 | |
| 351 | if 'time' in val_type: |
| 352 | if human_readable: |
| 353 | return hms_time(val) |
| 354 | else: |
| 355 | return "{:.1f}s".format(val) |
| 356 | elif 'bytes' in val_type and human_readable: |
| 357 | prefix = ['', 'Ki', 'Mi', 'Gi', 'Ti', 'Pi'] |
| 358 | dec = int(math.log(val, 2) / 10) |
| 359 | prec = 1 if dec > 0 else 0 |
| 360 | return "{:.{prec}f}{}B".format(val / (2 ** (10 * dec)), |
| 361 | prefix[dec], prec=prec) |
| 362 | elif 'ops' in val_type and human_readable: |
| 363 | prefix = ['', 'k', 'M', 'G', 'T', 'P'] |
| 364 | dec = int(math.log(val, 1000)) |
| 365 | prec = 1 if dec > 0 else 0 |
| 366 | return "{:.{prec}f}{}ops".format(val / (1000 ** dec), |
| 367 | prefix[dec], prec=prec) |
| 368 | return str(int(val)) |
| 369 | |
| 370 | def sum_vals(buildstats): |
| 371 | """Get cumulative sum of all tasks""" |
| 372 | total = 0.0 |
| 373 | for recipe_data in buildstats.values(): |
| 374 | for bs_task in recipe_data['tasks'].values(): |
| 375 | total += sum([getattr(b, val_type) for b in bs_task]) / len(bs_task) |
| 376 | return total |
| 377 | |
| 378 | tasks_diff = [] |
| 379 | |
| 380 | if min_val: |
| 381 | print("Ignoring tasks less than {} ({})".format( |
| 382 | val_to_str(min_val, True), val_to_str(min_val))) |
| 383 | if min_absdiff: |
| 384 | print("Ignoring differences less than {} ({})".format( |
| 385 | val_to_str(min_absdiff, True), val_to_str(min_absdiff))) |
| 386 | |
| 387 | # Prepare the data |
| 388 | pkgs = set(bs1.keys()).union(set(bs2.keys())) |
| 389 | for pkg in pkgs: |
| 390 | tasks1 = bs1[pkg]['tasks'] if pkg in bs1 else {} |
| 391 | tasks2 = bs2[pkg]['tasks'] if pkg in bs2 else {} |
| 392 | if not tasks1: |
| 393 | pkg_op = '+ ' |
| 394 | elif not tasks2: |
| 395 | pkg_op = '- ' |
| 396 | else: |
| 397 | pkg_op = ' ' |
| 398 | |
| 399 | for task in set(tasks1.keys()).union(set(tasks2.keys())): |
| 400 | task_op = ' ' |
| 401 | if task in tasks1: |
| 402 | # Average over all values |
| 403 | val1 = [getattr(b, val_type) for b in bs1[pkg]['tasks'][task]] |
| 404 | val1 = sum(val1) / len(val1) |
| 405 | else: |
| 406 | task_op = '+ ' |
| 407 | val1 = 0 |
| 408 | if task in tasks2: |
| 409 | # Average over all values |
| 410 | val2 = [getattr(b, val_type) for b in bs2[pkg]['tasks'][task]] |
| 411 | val2 = sum(val2) / len(val2) |
| 412 | else: |
| 413 | val2 = 0 |
| 414 | task_op = '- ' |
| 415 | |
| 416 | if val1 == 0: |
| 417 | reldiff = float('inf') |
| 418 | else: |
| 419 | reldiff = 100 * (val2 - val1) / val1 |
| 420 | |
| 421 | if max(val1, val2) < min_val: |
| 422 | log.debug("Filtering out %s:%s (%s)", pkg, task, |
| 423 | val_to_str(max(val1, val2))) |
| 424 | continue |
| 425 | if abs(val2 - val1) < min_absdiff: |
| 426 | log.debug("Filtering out %s:%s (difference of %s)", pkg, task, |
| 427 | val_to_str(val2-val1)) |
| 428 | continue |
| 429 | tasks_diff.append(TaskDiff(pkg, pkg_op, task, task_op, val1, val2, |
| 430 | val2-val1, reldiff)) |
| 431 | |
| 432 | # Sort our list |
| 433 | for field in reversed(sort_by): |
| 434 | if field.startswith('-'): |
| 435 | field = field[1:] |
| 436 | reverse = True |
| 437 | else: |
| 438 | reverse = False |
| 439 | tasks_diff = sorted(tasks_diff, key=attrgetter(field), reverse=reverse) |
| 440 | |
| 441 | linedata = [(' ', 'PKG', ' ', 'TASK', 'ABSDIFF', 'RELDIFF', |
| 442 | val_type.upper() + '1', val_type.upper() + '2')] |
| 443 | field_lens = dict([('len_{}'.format(i), len(f)) for i, f in enumerate(linedata[0])]) |
| 444 | |
| 445 | # Prepare fields in string format and measure field lengths |
| 446 | for diff in tasks_diff: |
| 447 | task_prefix = diff.task_op if diff.pkg_op == ' ' else ' ' |
| 448 | linedata.append((diff.pkg_op, diff.pkg, task_prefix, diff.task, |
| 449 | val_to_str(diff.absdiff), |
| 450 | '{:+.1f}%'.format(diff.reldiff), |
| 451 | val_to_str(diff.value1), |
| 452 | val_to_str(diff.value2))) |
| 453 | for i, field in enumerate(linedata[-1]): |
| 454 | key = 'len_{}'.format(i) |
| 455 | if len(field) > field_lens[key]: |
| 456 | field_lens[key] = len(field) |
| 457 | |
| 458 | # Print data |
| 459 | print() |
| 460 | for fields in linedata: |
| 461 | print("{:{len_0}}{:{len_1}} {:{len_2}}{:{len_3}} {:>{len_4}} {:>{len_5}} {:>{len_6}} -> {:{len_7}}".format( |
| 462 | *fields, **field_lens)) |
| 463 | |
| 464 | # Print summary of the diffs |
| 465 | total1 = sum_vals(bs1) |
| 466 | total2 = sum_vals(bs2) |
| 467 | print("\nCumulative {}:".format(val_type)) |
| 468 | print (" {} {:+.1f}% {} ({}) -> {} ({})".format( |
| 469 | val_to_str(total2 - total1), 100 * (total2-total1) / total1, |
| 470 | val_to_str(total1, True), val_to_str(total1), |
| 471 | val_to_str(total2, True), val_to_str(total2))) |
| 472 | |
| 473 | |
| 474 | def parse_args(argv): |
| 475 | """Parse cmdline arguments""" |
| 476 | description=""" |
| 477 | Script for comparing buildstats of two separate builds.""" |
| 478 | parser = argparse.ArgumentParser( |
| 479 | formatter_class=argparse.ArgumentDefaultsHelpFormatter, |
| 480 | description=description) |
| 481 | |
| 482 | min_val_defaults = {'cputime': 3.0, |
| 483 | 'read_bytes': 524288, |
| 484 | 'write_bytes': 524288, |
| 485 | 'read_ops': 500, |
| 486 | 'write_ops': 500, |
| 487 | 'walltime': 5} |
| 488 | min_absdiff_defaults = {'cputime': 1.0, |
| 489 | 'read_bytes': 131072, |
| 490 | 'write_bytes': 131072, |
| 491 | 'read_ops': 50, |
| 492 | 'write_ops': 50, |
| 493 | 'walltime': 2} |
| 494 | |
| 495 | parser.add_argument('--debug', '-d', action='store_true', |
| 496 | help="Verbose logging") |
| 497 | parser.add_argument('--ver-diff', action='store_true', |
| 498 | help="Show package version differences and exit") |
| 499 | parser.add_argument('--diff-attr', default='cputime', |
| 500 | choices=min_val_defaults.keys(), |
| 501 | help="Buildstat attribute which to compare") |
| 502 | parser.add_argument('--min-val', default=min_val_defaults, type=float, |
| 503 | help="Filter out tasks less than MIN_VAL. " |
| 504 | "Default depends on --diff-attr.") |
| 505 | parser.add_argument('--min-absdiff', default=min_absdiff_defaults, type=float, |
| 506 | help="Filter out tasks whose difference is less than " |
| 507 | "MIN_ABSDIFF, Default depends on --diff-attr.") |
| 508 | parser.add_argument('--sort-by', default='absdiff', |
| 509 | help="Comma-separated list of field sort order. " |
| 510 | "Prepend the field name with '-' for reversed sort. " |
| 511 | "Available fields are: {}".format(', '.join(taskdiff_fields))) |
| 512 | parser.add_argument('--multi', action='store_true', |
| 513 | help="Read all buildstats from the given paths and " |
| 514 | "average over them") |
| 515 | parser.add_argument('buildstats1', metavar='BUILDSTATS1', help="'Left' buildstat") |
| 516 | parser.add_argument('buildstats2', metavar='BUILDSTATS2', help="'Right' buildstat") |
| 517 | |
| 518 | args = parser.parse_args(argv) |
| 519 | |
| 520 | # We do not nedd/want to read all buildstats if we just want to look at the |
| 521 | # package versions |
| 522 | if args.ver_diff: |
| 523 | args.multi = False |
| 524 | |
| 525 | # Handle defaults for the filter arguments |
| 526 | if args.min_val is min_val_defaults: |
| 527 | args.min_val = min_val_defaults[args.diff_attr] |
| 528 | if args.min_absdiff is min_absdiff_defaults: |
| 529 | args.min_absdiff = min_absdiff_defaults[args.diff_attr] |
| 530 | |
| 531 | return args |
| 532 | |
| 533 | |
| 534 | def main(argv=None): |
| 535 | """Script entry point""" |
| 536 | args = parse_args(argv) |
| 537 | if args.debug: |
| 538 | log.setLevel(logging.DEBUG) |
| 539 | |
| 540 | # Validate sort fields |
| 541 | sort_by = [] |
| 542 | for field in args.sort_by.split(','): |
| 543 | if field.lstrip('-') not in taskdiff_fields: |
| 544 | log.error("Invalid sort field '%s' (must be one of: %s)" % |
| 545 | (field, ', '.join(taskdiff_fields))) |
| 546 | sys.exit(1) |
| 547 | sort_by.append(field) |
| 548 | |
| 549 | try: |
| 550 | bs1 = read_buildstats(args.buildstats1, args.multi) |
| 551 | bs2 = read_buildstats(args.buildstats2, args.multi) |
| 552 | |
| 553 | if args.ver_diff: |
| 554 | print_ver_diff(bs1, bs2) |
| 555 | else: |
| 556 | print_task_diff(bs1, bs2, args.diff_attr, args.min_val, |
| 557 | args.min_absdiff, sort_by) |
| 558 | except ScriptError as err: |
| 559 | log.error(str(err)) |
| 560 | return 1 |
| 561 | return 0 |
| 562 | |
| 563 | if __name__ == "__main__": |
| 564 | sys.exit(main()) |