Edward Lemur | 32e3d1e | 2018-07-12 00:54:05 +0000 | [diff] [blame] | 1 | #!/usr/bin/env python |
| 2 | # Copyright (c) 2018 The Chromium Authors. All rights reserved. |
| 3 | # Use of this source code is governed by a BSD-style license that can be |
| 4 | # found in the LICENSE file. |
| 5 | |
Edward Lemur | dd5051f | 2018-08-08 00:56:41 +0000 | [diff] [blame] | 6 | from __future__ import print_function |
| 7 | |
Edward Lemur | 6f812e1 | 2018-07-31 22:45:57 +0000 | [diff] [blame] | 8 | import contextlib |
Edward Lemur | 32e3d1e | 2018-07-12 00:54:05 +0000 | [diff] [blame] | 9 | import functools |
| 10 | import json |
| 11 | import os |
| 12 | import subprocess |
| 13 | import sys |
| 14 | import tempfile |
| 15 | import threading |
| 16 | import time |
| 17 | import traceback |
| 18 | import urllib2 |
| 19 | |
| 20 | import detect_host_arch |
| 21 | import gclient_utils |
| 22 | import metrics_utils |
| 23 | |
| 24 | |
| 25 | DEPOT_TOOLS = os.path.dirname(os.path.abspath(__file__)) |
| 26 | CONFIG_FILE = os.path.join(DEPOT_TOOLS, 'metrics.cfg') |
| 27 | UPLOAD_SCRIPT = os.path.join(DEPOT_TOOLS, 'upload_metrics.py') |
| 28 | |
Edward Lemur | 32e3d1e | 2018-07-12 00:54:05 +0000 | [diff] [blame] | 29 | DISABLE_METRICS_COLLECTION = os.environ.get('DEPOT_TOOLS_METRICS') == '0' |
| 30 | DEFAULT_COUNTDOWN = 10 |
| 31 | |
| 32 | INVALID_CONFIG_WARNING = ( |
Edward Lemur | dd5051f | 2018-08-08 00:56:41 +0000 | [diff] [blame] | 33 | 'WARNING: Your metrics.cfg file was invalid or nonexistent. A new one will ' |
Tomasz Ĺšniatowski | bc56d8c | 2018-08-28 17:56:25 +0000 | [diff] [blame] | 34 | 'be created.' |
Edward Lemur | dd5051f | 2018-08-08 00:56:41 +0000 | [diff] [blame] | 35 | ) |
| 36 | PERMISSION_DENIED_WARNING = ( |
| 37 | 'Could not write the metrics collection config:\n\t%s\n' |
| 38 | 'Metrics collection will be disabled.' |
Edward Lemur | 32e3d1e | 2018-07-12 00:54:05 +0000 | [diff] [blame] | 39 | ) |
| 40 | |
| 41 | |
| 42 | class _Config(object): |
| 43 | def __init__(self): |
| 44 | self._initialized = False |
| 45 | self._config = {} |
| 46 | |
| 47 | def _ensure_initialized(self): |
| 48 | if self._initialized: |
| 49 | return |
| 50 | |
| 51 | try: |
| 52 | config = json.loads(gclient_utils.FileRead(CONFIG_FILE)) |
| 53 | except (IOError, ValueError): |
| 54 | config = {} |
| 55 | |
| 56 | self._config = config.copy() |
| 57 | |
| 58 | if 'is-googler' not in self._config: |
| 59 | # /should-upload is only accessible from Google IPs, so we only need to |
| 60 | # check if we can reach the page. An external developer would get access |
| 61 | # denied. |
| 62 | try: |
Edward Lemur | 5ba1e9c | 2018-07-23 18:19:02 +0000 | [diff] [blame] | 63 | req = urllib2.urlopen(metrics_utils.APP_URL + '/should-upload') |
Edward Lemur | 32e3d1e | 2018-07-12 00:54:05 +0000 | [diff] [blame] | 64 | self._config['is-googler'] = req.getcode() == 200 |
| 65 | except (urllib2.URLError, urllib2.HTTPError): |
| 66 | self._config['is-googler'] = False |
| 67 | |
| 68 | # Make sure the config variables we need are present, and initialize them to |
| 69 | # safe values otherwise. |
| 70 | self._config.setdefault('countdown', DEFAULT_COUNTDOWN) |
| 71 | self._config.setdefault('opt-in', None) |
Edward Lemur | 4883626 | 2018-10-18 02:08:06 +0000 | [diff] [blame] | 72 | self._config.setdefault('version', metrics_utils.CURRENT_VERSION) |
Edward Lemur | 32e3d1e | 2018-07-12 00:54:05 +0000 | [diff] [blame] | 73 | |
| 74 | if config != self._config: |
Edward Lemur | dd5051f | 2018-08-08 00:56:41 +0000 | [diff] [blame] | 75 | print(INVALID_CONFIG_WARNING, file=sys.stderr) |
Edward Lemur | 32e3d1e | 2018-07-12 00:54:05 +0000 | [diff] [blame] | 76 | self._write_config() |
| 77 | |
| 78 | self._initialized = True |
| 79 | |
| 80 | def _write_config(self): |
Edward Lemur | dd5051f | 2018-08-08 00:56:41 +0000 | [diff] [blame] | 81 | try: |
| 82 | gclient_utils.FileWrite(CONFIG_FILE, json.dumps(self._config)) |
| 83 | except IOError as e: |
| 84 | print(PERMISSION_DENIED_WARNING % e, file=sys.stderr) |
| 85 | self._config['opt-in'] = False |
Edward Lemur | 32e3d1e | 2018-07-12 00:54:05 +0000 | [diff] [blame] | 86 | |
| 87 | @property |
Edward Lemur | 4883626 | 2018-10-18 02:08:06 +0000 | [diff] [blame] | 88 | def version(self): |
| 89 | self._ensure_initialized() |
| 90 | return self._config['version'] |
| 91 | |
| 92 | @property |
Edward Lemur | 32e3d1e | 2018-07-12 00:54:05 +0000 | [diff] [blame] | 93 | def is_googler(self): |
| 94 | self._ensure_initialized() |
| 95 | return self._config['is-googler'] |
| 96 | |
| 97 | @property |
| 98 | def opted_in(self): |
| 99 | self._ensure_initialized() |
| 100 | return self._config['opt-in'] |
| 101 | |
| 102 | @opted_in.setter |
| 103 | def opted_in(self, value): |
| 104 | self._ensure_initialized() |
| 105 | self._config['opt-in'] = value |
Edward Lemur | 4883626 | 2018-10-18 02:08:06 +0000 | [diff] [blame] | 106 | self._config['version'] = metrics_utils.CURRENT_VERSION |
Edward Lemur | 32e3d1e | 2018-07-12 00:54:05 +0000 | [diff] [blame] | 107 | self._write_config() |
| 108 | |
| 109 | @property |
| 110 | def countdown(self): |
| 111 | self._ensure_initialized() |
| 112 | return self._config['countdown'] |
| 113 | |
Edward Lemur | 4883626 | 2018-10-18 02:08:06 +0000 | [diff] [blame] | 114 | @property |
| 115 | def should_collect_metrics(self): |
| 116 | # Don't collect the metrics unless the user is a googler, the user has opted |
| 117 | # in, or the countdown has expired. |
| 118 | if not self.is_googler: |
| 119 | return False |
| 120 | if self.opted_in is False: |
| 121 | return False |
| 122 | if self.opted_in is None and self.countdown > 0: |
| 123 | return False |
| 124 | return True |
| 125 | |
Edward Lemur | 32e3d1e | 2018-07-12 00:54:05 +0000 | [diff] [blame] | 126 | def decrease_countdown(self): |
| 127 | self._ensure_initialized() |
| 128 | if self.countdown == 0: |
| 129 | return |
| 130 | self._config['countdown'] -= 1 |
Edward Lemur | 4883626 | 2018-10-18 02:08:06 +0000 | [diff] [blame] | 131 | if self.countdown == 0: |
| 132 | self._config['version'] = metrics_utils.CURRENT_VERSION |
Edward Lemur | 32e3d1e | 2018-07-12 00:54:05 +0000 | [diff] [blame] | 133 | self._write_config() |
| 134 | |
Edward Lemur | 4883626 | 2018-10-18 02:08:06 +0000 | [diff] [blame] | 135 | def reset_config(self): |
| 136 | # Only reset countdown if we're already collecting metrics. |
| 137 | if self.should_collect_metrics: |
| 138 | self._ensure_initialized() |
| 139 | self._config['countdown'] = DEFAULT_COUNTDOWN |
| 140 | self._config['opt-in'] = None |
| 141 | |
Edward Lemur | 32e3d1e | 2018-07-12 00:54:05 +0000 | [diff] [blame] | 142 | |
| 143 | class MetricsCollector(object): |
| 144 | def __init__(self): |
| 145 | self._metrics_lock = threading.Lock() |
| 146 | self._reported_metrics = {} |
| 147 | self._config = _Config() |
Edward Lemur | 3298e7b | 2018-07-17 18:21:27 +0000 | [diff] [blame] | 148 | self._collecting_metrics = False |
Edward Lemur | 6f812e1 | 2018-07-31 22:45:57 +0000 | [diff] [blame] | 149 | self._collect_custom_metrics = True |
Edward Lemur | 32e3d1e | 2018-07-12 00:54:05 +0000 | [diff] [blame] | 150 | |
| 151 | @property |
| 152 | def config(self): |
| 153 | return self._config |
| 154 | |
Edward Lemur | 3298e7b | 2018-07-17 18:21:27 +0000 | [diff] [blame] | 155 | @property |
| 156 | def collecting_metrics(self): |
| 157 | return self._collecting_metrics |
| 158 | |
Edward Lemur | 32e3d1e | 2018-07-12 00:54:05 +0000 | [diff] [blame] | 159 | def add(self, name, value): |
Edward Lemur | 6f812e1 | 2018-07-31 22:45:57 +0000 | [diff] [blame] | 160 | if self._collect_custom_metrics: |
| 161 | with self._metrics_lock: |
| 162 | self._reported_metrics[name] = value |
| 163 | |
| 164 | @contextlib.contextmanager |
| 165 | def pause_metrics_collection(self): |
| 166 | collect_custom_metrics = self._collect_custom_metrics |
| 167 | self._collect_custom_metrics = False |
| 168 | try: |
| 169 | yield |
| 170 | finally: |
| 171 | self._collect_custom_metrics = collect_custom_metrics |
Edward Lemur | 32e3d1e | 2018-07-12 00:54:05 +0000 | [diff] [blame] | 172 | |
| 173 | def _upload_metrics_data(self): |
| 174 | """Upload the metrics data to the AppEngine app.""" |
| 175 | # We invoke a subprocess, and use stdin.write instead of communicate(), |
| 176 | # so that we are able to return immediately, leaving the upload running in |
| 177 | # the background. |
| 178 | p = subprocess.Popen([sys.executable, UPLOAD_SCRIPT], stdin=subprocess.PIPE) |
| 179 | p.stdin.write(json.dumps(self._reported_metrics)) |
| 180 | |
| 181 | def _collect_metrics(self, func, command_name, *args, **kwargs): |
Edward Lemur | 6f812e1 | 2018-07-31 22:45:57 +0000 | [diff] [blame] | 182 | # If we're already collecting metrics, just execute the function. |
| 183 | # e.g. git-cl split invokes git-cl upload several times to upload each |
| 184 | # splitted CL. |
| 185 | if self.collecting_metrics: |
| 186 | # Don't collect metrics for this function. |
| 187 | # e.g. Don't record the arguments git-cl split passes to git-cl upload. |
| 188 | with self.pause_metrics_collection(): |
| 189 | return func(*args, **kwargs) |
Edward Lemur | 7fa0f19 | 2018-07-17 21:33:37 +0000 | [diff] [blame] | 190 | |
| 191 | self._collecting_metrics = True |
Edward Lemur | 32e3d1e | 2018-07-12 00:54:05 +0000 | [diff] [blame] | 192 | self.add('command', command_name) |
| 193 | try: |
| 194 | start = time.time() |
Edward Lemur | 6f812e1 | 2018-07-31 22:45:57 +0000 | [diff] [blame] | 195 | result = func(*args, **kwargs) |
Edward Lemur | 32e3d1e | 2018-07-12 00:54:05 +0000 | [diff] [blame] | 196 | exception = None |
| 197 | # pylint: disable=bare-except |
| 198 | except: |
| 199 | exception = sys.exc_info() |
| 200 | finally: |
| 201 | self.add('execution_time', time.time() - start) |
| 202 | |
Edward Lemur | 32e3d1e | 2018-07-12 00:54:05 +0000 | [diff] [blame] | 203 | exit_code = metrics_utils.return_code_from_exception(exception) |
| 204 | self.add('exit_code', exit_code) |
| 205 | |
Edward Lemur | 32e3d1e | 2018-07-12 00:54:05 +0000 | [diff] [blame] | 206 | # Add metrics regarding environment information. |
| 207 | self.add('timestamp', metrics_utils.seconds_to_weeks(time.time())) |
| 208 | self.add('python_version', metrics_utils.get_python_version()) |
| 209 | self.add('host_os', gclient_utils.GetMacWinOrLinux()) |
| 210 | self.add('host_arch', detect_host_arch.HostArch()) |
| 211 | self.add('depot_tools_age', metrics_utils.get_repo_timestamp(DEPOT_TOOLS)) |
| 212 | |
| 213 | self._upload_metrics_data() |
Edward Lemur | 6f812e1 | 2018-07-31 22:45:57 +0000 | [diff] [blame] | 214 | if exception: |
| 215 | raise exception[0], exception[1], exception[2] |
| 216 | return result |
Edward Lemur | 32e3d1e | 2018-07-12 00:54:05 +0000 | [diff] [blame] | 217 | |
| 218 | def collect_metrics(self, command_name): |
| 219 | """A decorator used to collect metrics over the life of a function. |
| 220 | |
| 221 | This decorator executes the function and collects metrics about the system |
Edward Lemur | 6f812e1 | 2018-07-31 22:45:57 +0000 | [diff] [blame] | 222 | environment and the function performance. |
Edward Lemur | 32e3d1e | 2018-07-12 00:54:05 +0000 | [diff] [blame] | 223 | """ |
| 224 | def _decorator(func): |
| 225 | # Do this first so we don't have to read, and possibly create a config |
| 226 | # file. |
| 227 | if DISABLE_METRICS_COLLECTION: |
| 228 | return func |
Edward Lemur | 4883626 | 2018-10-18 02:08:06 +0000 | [diff] [blame] | 229 | if not self.config.should_collect_metrics: |
Edward Lemur | 32e3d1e | 2018-07-12 00:54:05 +0000 | [diff] [blame] | 230 | return func |
Edward Lemur | 32e3d1e | 2018-07-12 00:54:05 +0000 | [diff] [blame] | 231 | # Otherwise, collect the metrics. |
| 232 | # Needed to preserve the __name__ and __doc__ attributes of func. |
| 233 | @functools.wraps(func) |
| 234 | def _inner(*args, **kwargs): |
Edward Lemur | 6f812e1 | 2018-07-31 22:45:57 +0000 | [diff] [blame] | 235 | return self._collect_metrics(func, command_name, *args, **kwargs) |
Edward Lemur | 32e3d1e | 2018-07-12 00:54:05 +0000 | [diff] [blame] | 236 | return _inner |
| 237 | return _decorator |
| 238 | |
Edward Lemur | 6f812e1 | 2018-07-31 22:45:57 +0000 | [diff] [blame] | 239 | @contextlib.contextmanager |
| 240 | def print_notice_and_exit(self): |
| 241 | """A context manager used to print the notice and terminate execution. |
| 242 | |
| 243 | This decorator executes the function and prints the monitoring notice if |
| 244 | necessary. If an exception is raised, we will catch it, and print it before |
| 245 | printing the metrics collection notice. |
| 246 | This will call sys.exit() with an appropriate exit code to ensure the notice |
| 247 | is the last thing printed.""" |
| 248 | # Needed to preserve the __name__ and __doc__ attributes of func. |
| 249 | try: |
| 250 | yield |
| 251 | exception = None |
| 252 | # pylint: disable=bare-except |
| 253 | except: |
| 254 | exception = sys.exc_info() |
| 255 | |
| 256 | # Print the exception before the metrics notice, so that the notice is |
| 257 | # clearly visible even if gclient fails. |
| 258 | if exception: |
| 259 | if isinstance(exception[1], KeyboardInterrupt): |
| 260 | sys.stderr.write('Interrupted\n') |
| 261 | elif not isinstance(exception[1], SystemExit): |
| 262 | traceback.print_exception(*exception) |
| 263 | |
Edward Lemur | 4883626 | 2018-10-18 02:08:06 +0000 | [diff] [blame] | 264 | # Check if the version has changed |
| 265 | if (not DISABLE_METRICS_COLLECTION and self.config.is_googler |
| 266 | and self.config.opted_in is not False |
| 267 | and self.config.version != metrics_utils.CURRENT_VERSION): |
| 268 | metrics_utils.print_version_change(self.config.version) |
| 269 | self.config.reset_config() |
| 270 | |
Edward Lemur | 6f812e1 | 2018-07-31 22:45:57 +0000 | [diff] [blame] | 271 | # Print the notice |
| 272 | if (not DISABLE_METRICS_COLLECTION and self.config.is_googler |
| 273 | and self.config.opted_in is None): |
| 274 | metrics_utils.print_notice(self.config.countdown) |
| 275 | self.config.decrease_countdown() |
| 276 | |
| 277 | sys.exit(metrics_utils.return_code_from_exception(exception)) |
| 278 | |
Edward Lemur | 32e3d1e | 2018-07-12 00:54:05 +0000 | [diff] [blame] | 279 | |
| 280 | collector = MetricsCollector() |