Allen Li | 24bf818 | 2017-03-02 16:41:20 -0800 | [diff] [blame] | 1 | # Copyright 2017 The Chromium OS Authors. All rights reserved. |
| 2 | # Use of this source code is governed by a BSD-style license that can be |
| 3 | # found in the LICENSE file. |
| 4 | |
| 5 | """Git repo metrics.""" |
| 6 | |
| 7 | from __future__ import absolute_import |
| 8 | from __future__ import print_function |
Allen Li | 24bf818 | 2017-03-02 16:41:20 -0800 | [diff] [blame] | 9 | |
| 10 | import os |
| 11 | import subprocess |
| 12 | |
Allen Li | 24bf818 | 2017-03-02 16:41:20 -0800 | [diff] [blame] | 13 | from chromite.lib import cros_logging as logging |
Allen Li | a9c6e80 | 2017-07-11 15:42:47 -0700 | [diff] [blame^] | 14 | from chromite.lib import metrics |
Allen Li | 24bf818 | 2017-03-02 16:41:20 -0800 | [diff] [blame] | 15 | |
| 16 | logger = logging.getLogger(__name__) |
| 17 | |
| 18 | |
| 19 | class _GitRepo(object): |
Paul Hobbs | e46a42b | 2017-03-21 14:04:13 -0700 | [diff] [blame] | 20 | """Helper class for running git commands.""" |
Allen Li | 24bf818 | 2017-03-02 16:41:20 -0800 | [diff] [blame] | 21 | |
| 22 | def __init__(self, gitdir): |
| 23 | self._gitdir = gitdir |
| 24 | |
| 25 | def _get_git_command(self): |
| 26 | return ['git', '--git-dir', self._gitdir] |
| 27 | |
| 28 | def _check_output(self, args, **kwargs): |
| 29 | return subprocess.check_output( |
| 30 | self._get_git_command() + list(args), **kwargs) |
| 31 | |
| 32 | def get_commit_hash(self): |
Allen Li | d233398 | 2017-04-04 17:03:32 -0700 | [diff] [blame] | 33 | """Return commit hash string.""" |
Allen Li | 24bf818 | 2017-03-02 16:41:20 -0800 | [diff] [blame] | 34 | return self._check_output(['rev-parse', 'HEAD']).strip() |
| 35 | |
| 36 | def get_commit_time(self): |
Allen Li | d233398 | 2017-04-04 17:03:32 -0700 | [diff] [blame] | 37 | """Return commit time as UNIX timestamp int.""" |
Allen Li | 24bf818 | 2017-03-02 16:41:20 -0800 | [diff] [blame] | 38 | return int(self._check_output(['show', '-s', '--format=%ct', 'HEAD']) |
| 39 | .strip()) |
| 40 | |
Allen Li | 8934e04 | 2017-06-21 15:54:42 -0700 | [diff] [blame] | 41 | def get_unstaged_changes(self): |
| 42 | """Return number of unstaged changes as (added, deleted).""" |
Allen Li | 5956da6 | 2017-06-27 12:38:49 -0700 | [diff] [blame] | 43 | added_total, deleted_total = 0, 0 |
| 44 | # output looks like: |
| 45 | # '1\t2\tfoo\n3\t4\tbar\n' |
| 46 | # '-\t-\tbinary_file\n' |
Allen Li | 8934e04 | 2017-06-21 15:54:42 -0700 | [diff] [blame] | 47 | output = self._check_output(['diff-index', '--numstat', 'HEAD']) |
| 48 | stats_strings = (line.split() for line in output.splitlines()) |
Allen Li | 5956da6 | 2017-06-27 12:38:49 -0700 | [diff] [blame] | 49 | for added, deleted, _path in stats_strings: |
| 50 | if added != '-': |
| 51 | added_total += int(added) |
| 52 | if deleted != '-': |
| 53 | deleted_total += int(deleted) |
| 54 | return added_total, deleted_total |
Allen Li | 8934e04 | 2017-06-21 15:54:42 -0700 | [diff] [blame] | 55 | |
Allen Li | 24bf818 | 2017-03-02 16:41:20 -0800 | [diff] [blame] | 56 | |
| 57 | class _GitMetricCollector(object): |
Allen Li | bdb9f04 | 2017-04-10 13:25:47 -0700 | [diff] [blame] | 58 | """Class for collecting metrics about a git repository. |
| 59 | |
| 60 | The constructor takes the arguments: `gitdir`, `metric_path`. |
| 61 | `gitdir` is the path to the Git directory to collect metrics for and |
| 62 | may start with a tilde (expanded to a user's home directory). |
| 63 | `metric_path` is the Monarch metric path to report to. |
| 64 | """ |
Allen Li | 24bf818 | 2017-03-02 16:41:20 -0800 | [diff] [blame] | 65 | |
Allen Li | a9c6e80 | 2017-07-11 15:42:47 -0700 | [diff] [blame^] | 66 | _commit_hash_metric = metrics.StringMetric( |
Allen Li | 24bf818 | 2017-03-02 16:41:20 -0800 | [diff] [blame] | 67 | 'git/hash', |
| 68 | description='Current Git commit hash.') |
| 69 | |
Allen Li | a9c6e80 | 2017-07-11 15:42:47 -0700 | [diff] [blame^] | 70 | _timestamp_metric = metrics.GaugeMetric( |
Allen Li | d523d96 | 2017-04-04 16:48:36 -0700 | [diff] [blame] | 71 | 'git/timestamp', |
Allen Li | 24bf818 | 2017-03-02 16:41:20 -0800 | [diff] [blame] | 72 | description='Current Git commit time as seconds since Unix Epoch.') |
| 73 | |
Allen Li | a9c6e80 | 2017-07-11 15:42:47 -0700 | [diff] [blame^] | 74 | _unstaged_changes_metric = metrics.GaugeMetric( |
Allen Li | 8934e04 | 2017-06-21 15:54:42 -0700 | [diff] [blame] | 75 | 'git/unstaged_changes', |
| 76 | description='Unstaged Git changes.') |
| 77 | |
Allen Li | 24bf818 | 2017-03-02 16:41:20 -0800 | [diff] [blame] | 78 | def __init__(self, gitdir, metric_path): |
| 79 | self._gitdir = gitdir |
Allen Li | bdb9f04 | 2017-04-10 13:25:47 -0700 | [diff] [blame] | 80 | self._gitrepo = _GitRepo(os.path.expanduser(gitdir)) |
Allen Li | 24bf818 | 2017-03-02 16:41:20 -0800 | [diff] [blame] | 81 | self._fields = {'repo': gitdir} |
| 82 | self._metric_path = metric_path |
| 83 | |
| 84 | def collect(self): |
| 85 | """Collect metrics.""" |
| 86 | try: |
| 87 | self._collect_commit_hash_metric() |
Allen Li | d523d96 | 2017-04-04 16:48:36 -0700 | [diff] [blame] | 88 | self._collect_timestamp_metric() |
Allen Li | 8934e04 | 2017-06-21 15:54:42 -0700 | [diff] [blame] | 89 | self._collect_unstaged_changes_metric() |
Allen Li | 24bf818 | 2017-03-02 16:41:20 -0800 | [diff] [blame] | 90 | except subprocess.CalledProcessError as e: |
Allen Li | 867d458 | 2017-05-24 18:00:43 -0700 | [diff] [blame] | 91 | logger.warning(u'Error collecting git metrics for %s: %s', |
Allen Li | 24bf818 | 2017-03-02 16:41:20 -0800 | [diff] [blame] | 92 | self._gitdir, e) |
| 93 | |
| 94 | def _collect_commit_hash_metric(self): |
| 95 | commit_hash = self._gitrepo.get_commit_hash() |
Allen Li | 867d458 | 2017-05-24 18:00:43 -0700 | [diff] [blame] | 96 | logger.debug(u'Collecting Git hash %r for %r', commit_hash, self._gitdir) |
Allen Li | 24bf818 | 2017-03-02 16:41:20 -0800 | [diff] [blame] | 97 | self._commit_hash_metric.set(commit_hash, self._fields) |
| 98 | |
Allen Li | d523d96 | 2017-04-04 16:48:36 -0700 | [diff] [blame] | 99 | def _collect_timestamp_metric(self): |
Allen Li | 24bf818 | 2017-03-02 16:41:20 -0800 | [diff] [blame] | 100 | commit_time = self._gitrepo.get_commit_time() |
Allen Li | 867d458 | 2017-05-24 18:00:43 -0700 | [diff] [blame] | 101 | logger.debug(u'Collecting Git timestamp %r for %r', |
Allen Li | 24bf818 | 2017-03-02 16:41:20 -0800 | [diff] [blame] | 102 | commit_time, self._gitdir) |
Allen Li | d523d96 | 2017-04-04 16:48:36 -0700 | [diff] [blame] | 103 | self._timestamp_metric.set(commit_time, self._fields) |
Allen Li | 24bf818 | 2017-03-02 16:41:20 -0800 | [diff] [blame] | 104 | |
Allen Li | 8934e04 | 2017-06-21 15:54:42 -0700 | [diff] [blame] | 105 | def _collect_unstaged_changes_metric(self): |
Allen Li | c8acdec | 2017-06-26 15:09:41 -0700 | [diff] [blame] | 106 | added, deleted = self._gitrepo.get_unstaged_changes() |
Allen Li | 36b8a8b | 2017-06-28 14:47:16 -0700 | [diff] [blame] | 107 | self._unstaged_changes_metric.set( |
Allen Li | 8934e04 | 2017-06-21 15:54:42 -0700 | [diff] [blame] | 108 | added, fields=dict(change_type='added', **self._fields)) |
Allen Li | 36b8a8b | 2017-06-28 14:47:16 -0700 | [diff] [blame] | 109 | self._unstaged_changes_metric.set( |
Allen Li | 8934e04 | 2017-06-21 15:54:42 -0700 | [diff] [blame] | 110 | deleted, fields=dict(change_type='deleted', **self._fields)) |
| 111 | |
Allen Li | 24bf818 | 2017-03-02 16:41:20 -0800 | [diff] [blame] | 112 | |
Allen Li | bdb9f04 | 2017-04-10 13:25:47 -0700 | [diff] [blame] | 113 | _CHROMIUMOS_DIR = '~chromeos-test/chromiumos/' |
Allen Li | 24bf818 | 2017-03-02 16:41:20 -0800 | [diff] [blame] | 114 | |
| 115 | _repo_collectors = ( |
Allen Li | a02d34a | 2017-04-04 17:13:46 -0700 | [diff] [blame] | 116 | # TODO(ayatane): We cannot access chromeos-admin because we are |
| 117 | # running as non-root. |
| 118 | _GitMetricCollector(gitdir='/root/chromeos-admin/.git', |
| 119 | metric_path='chromeos-admin'), |
| 120 | _GitMetricCollector(gitdir=_CHROMIUMOS_DIR + 'chromite/.git', |
| 121 | metric_path='chromite'), |
| 122 | _GitMetricCollector(gitdir='/usr/local/autotest/.git', |
| 123 | metric_path='installed_autotest'), |
Allen Li | 24bf818 | 2017-03-02 16:41:20 -0800 | [diff] [blame] | 124 | ) |
| 125 | |
| 126 | |
| 127 | def collect_git_metrics(): |
| 128 | """Collect metrics for Git repository state.""" |
| 129 | for collector in _repo_collectors: |
| 130 | collector.collect() |