1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
|
#!/usr/bin/env python3
"""Helpers for unittests. Note that when you import this we
automatically wrap unittest.main() with a call to
bootstrap.initialize so that we getLogger config, commandline args,
logging control, etc... this works fine but it's a little hacky so
caveat emptor.
"""
from abc import ABC, abstractmethod
import contextlib
import functools
import inspect
import logging
import os
import pickle
import random
import statistics
import time
import tempfile
from typing import Callable, Dict, List
import unittest
import warnings
import bootstrap
import config
import function_utils
import scott_secrets
import sqlalchemy as sa
logger = logging.getLogger(__name__)
cfg = config.add_commandline_args(
f'Logging ({__file__})', 'Args related to function decorators'
)
cfg.add_argument(
'--unittests_ignore_perf',
action='store_true',
default=False,
help='Ignore unittest perf regression in @check_method_for_perf_regressions',
)
cfg.add_argument(
'--unittests_num_perf_samples',
type=int,
default=50,
help='The count of perf timing samples we need to see before blocking slow runs on perf grounds',
)
cfg.add_argument(
'--unittests_drop_perf_traces',
type=str,
nargs=1,
default=None,
help='The identifier (i.e. file!test_fixture) for which we should drop all perf data',
)
cfg.add_argument(
'--unittests_persistance_strategy',
choices=['FILE', 'DATABASE'],
default='DATABASE',
help='Should we persist perf data in a file or db?',
)
cfg.add_argument(
'--unittests_perfdb_filename',
type=str,
metavar='FILENAME',
default=f'{os.environ["HOME"]}/.python_unittest_performance_db',
help='File in which to store perf data (iff --unittests_persistance_strategy is FILE)',
)
cfg.add_argument(
'--unittests_perfdb_spec',
type=str,
metavar='DBSPEC',
default='mariadb+pymysql://python_unittest:<PASSWORD>@db.house:3306/python_unittest_performance',
help='Db connection spec for perf data (iff --unittest_persistance_strategy is DATABASE)',
)
# >>> This is the hacky business, FYI. <<<
unittest.main = bootstrap.initialize(unittest.main)
class PerfRegressionDataPersister(ABC):
def __init__(self):
pass
@abstractmethod
def load_performance_data(self) -> Dict[str, List[float]]:
pass
@abstractmethod
def save_performance_data(self, method_id: str, data: Dict[str, List[float]]):
pass
@abstractmethod
def delete_performance_data(self, method_id: str):
pass
class FileBasedPerfRegressionDataPersister(PerfRegressionDataPersister):
def __init__(self, filename: str):
self.filename = filename
self.traces_to_delete = []
def load_performance_data(self, method_id: str) -> Dict[str, List[float]]:
with open(self.filename, 'rb') as f:
return pickle.load(f)
def save_performance_data(self, method_id: str, data: Dict[str, List[float]]):
for trace in self.traces_to_delete:
if trace in data:
data[trace] = []
with open(self.filename, 'wb') as f:
pickle.dump(data, f, pickle.HIGHEST_PROTOCOL)
def delete_performance_data(self, method_id: str):
self.traces_to_delete.append(method_id)
class DatabasePerfRegressionDataPersister(PerfRegressionDataPersister):
def __init__(self, dbspec: str):
self.dbspec = dbspec
self.engine = sa.create_engine(self.dbspec)
self.conn = self.engine.connect()
def load_performance_data(self, method_id: str) -> Dict[str, List[float]]:
results = self.conn.execute(
sa.text(
f'SELECT * FROM runtimes_by_function WHERE function = "{method_id}";'
)
)
ret = {method_id: []}
for result in results.all():
ret[method_id].append(result['runtime'])
results.close()
return ret
def save_performance_data(self, method_id: str, data: Dict[str, List[float]]):
self.delete_performance_data(method_id)
for (method_id, perf_data) in data.items():
sql = 'INSERT INTO runtimes_by_function (function, runtime) VALUES '
for perf in perf_data:
self.conn.execute(sql + f'("{method_id}", {perf});')
def delete_performance_data(self, method_id: str):
sql = f'DELETE FROM runtimes_by_function WHERE function = "{method_id}"'
self.conn.execute(sql)
def check_method_for_perf_regressions(func: Callable) -> Callable:
"""
This is meant to be used on a method in a class that subclasses
unittest.TestCase. When thus decorated it will time the execution
of the code in the method, compare it with a database of
historical perfmance, and fail the test with a perf-related
message if it has become too slow.
"""
@functools.wraps(func)
def wrapper_perf_monitor(*args, **kwargs):
if config.config['unittests_persistance_strategy'] == 'FILE':
filename = config.config['unittests_perfdb_filename']
helper = FileBasedPerfRegressionDataPersister(filename)
elif config.config['unittests_persistance_strategy'] == 'DATABASE':
dbspec = config.config['unittests_perfdb_spec']
dbspec = dbspec.replace(
'<PASSWORD>', scott_secrets.MARIADB_UNITTEST_PERF_PASSWORD
)
helper = DatabasePerfRegressionDataPersister(dbspec)
else:
raise Exception('Unknown/unexpected --unittests_persistance_strategy value')
func_id = function_utils.function_identifier(func)
func_name = func.__name__
logger.debug(f'Watching {func_name}\'s performance...')
logger.debug(f'Canonical function identifier = {func_id}')
try:
perfdb = helper.load_performance_data(func_id)
except Exception as e:
logger.exception(e)
msg = 'Unable to load perfdb; skipping it...'
logger.warning(msg)
warnings.warn(msg)
perfdb = {}
# cmdline arg to forget perf traces for function
drop_id = config.config['unittests_drop_perf_traces']
if drop_id is not None:
helper.delete_performance_data(drop_id)
# Run the wrapped test paying attention to latency.
start_time = time.perf_counter()
value = func(*args, **kwargs)
end_time = time.perf_counter()
run_time = end_time - start_time
# See if it was unexpectedly slow.
hist = perfdb.get(func_id, [])
if len(hist) < config.config['unittests_num_perf_samples']:
hist.append(run_time)
logger.debug(f'Still establishing a perf baseline for {func_name}')
else:
stdev = statistics.stdev(hist)
logger.debug(f'For {func_name}, performance stdev={stdev}')
slowest = hist[-1]
logger.debug(f'For {func_name}, slowest perf on record is {slowest:f}s')
limit = slowest + stdev * 4
logger.debug(f'For {func_name}, max acceptable runtime is {limit:f}s')
logger.debug(f'For {func_name}, actual observed runtime was {run_time:f}s')
if run_time > limit and not config.config['unittests_ignore_perf']:
msg = f'''{func_id} performance has regressed unacceptably.
{slowest:f}s is the slowest runtime on record in {len(hist)} perf samples.
It just ran in {run_time:f}s which is 4+ stdevs slower than the slowest.
Here is the current, full db perf timing distribution:
'''
for x in hist:
msg += f'{x:f}\n'
logger.error(msg)
slf = args[0] # Peek at the wrapped function's self ref.
slf.fail(msg) # ...to fail the testcase.
else:
hist.append(run_time)
# Don't spam the database with samples; just pick a random
# sample from what we have and store that back.
n = min(config.config['unittests_num_perf_samples'], len(hist))
hist = random.sample(hist, n)
hist.sort()
perfdb[func_id] = hist
helper.save_performance_data(func_id, perfdb)
return value
return wrapper_perf_monitor
def check_all_methods_for_perf_regressions(prefix='test_'):
"""Decorate unittests with this to pay attention to the perf of the
testcode and flag perf regressions. e.g.
import unittest_utils as uu
@uu.check_all_methods_for_perf_regressions()
class TestMyClass(unittest.TestCase):
def test_some_part_of_my_class(self):
...
"""
def decorate_the_testcase(cls):
if issubclass(cls, unittest.TestCase):
for name, m in inspect.getmembers(cls, inspect.isfunction):
if name.startswith(prefix):
setattr(cls, name, check_method_for_perf_regressions(m))
logger.debug(f'Wrapping {cls.__name__}:{name}.')
return cls
return decorate_the_testcase
def breakpoint():
"""Hard code a breakpoint somewhere; drop into pdb."""
import pdb
pdb.set_trace()
class RecordStdout(object):
"""
Record what is emitted to stdout.
>>> with RecordStdout() as record:
... print("This is a test!")
>>> print({record().readline()})
{'This is a test!\\n'}
>>> record().close()
"""
def __init__(self) -> None:
self.destination = tempfile.SpooledTemporaryFile(mode='r+')
self.recorder = None
def __enter__(self) -> Callable[[], tempfile.SpooledTemporaryFile]:
self.recorder = contextlib.redirect_stdout(self.destination)
self.recorder.__enter__()
return lambda: self.destination
def __exit__(self, *args) -> bool:
self.recorder.__exit__(*args)
self.destination.seek(0)
return None
class RecordStderr(object):
"""
Record what is emitted to stderr.
>>> import sys
>>> with RecordStderr() as record:
... print("This is a test!", file=sys.stderr)
>>> print({record().readline()})
{'This is a test!\\n'}
>>> record().close()
"""
def __init__(self) -> None:
self.destination = tempfile.SpooledTemporaryFile(mode='r+')
self.recorder = None
def __enter__(self) -> Callable[[], tempfile.SpooledTemporaryFile]:
self.recorder = contextlib.redirect_stderr(self.destination)
self.recorder.__enter__()
return lambda: self.destination
def __exit__(self, *args) -> bool:
self.recorder.__exit__(*args)
self.destination.seek(0)
return None
class RecordMultipleStreams(object):
"""
Record the output to more than one stream.
"""
def __init__(self, *files) -> None:
self.files = [*files]
self.destination = tempfile.SpooledTemporaryFile(mode='r+')
self.saved_writes = []
def __enter__(self) -> Callable[[], tempfile.SpooledTemporaryFile]:
for f in self.files:
self.saved_writes.append(f.write)
f.write = self.destination.write
return lambda: self.destination
def __exit__(self, *args) -> bool:
for f in self.files:
f.write = self.saved_writes.pop()
self.destination.seek(0)
if __name__ == '__main__':
import doctest
doctest.testmod()
|