diff options
author | Markus Lehtonen <markus.lehtonen@linux.intel.com> | 2016-06-23 18:38:35 +0300 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2016-07-01 16:22:47 +0100 |
commit | 8c596369ddcd1d079bec5c163132676f3940767b (patch) | |
tree | 5f9240954b014e21936ca414a81fdac4dd9167f3 | |
parent | 964fffa514fd60bd06eedfdda358133659b30ea2 (diff) | |
download | poky-8c596369ddcd1d079bec5c163132676f3940767b.tar.gz |
oeqa.buildperf: add git revision and branch to result data
BuildPerfTestRunner determines these from the Git repository under which
it is being run (i.e. where the build directory exists). The branch and
revision may be defined/overridden with OE_BUILDPERFTEST_GIT_BRANCH
and OE_BUILDPERFTEST_GIT_BRANCH environment variables, if needed. This
makes it possible to run the build performance test script even if the
top directory is not a git repository clone, for example.
(From OE-Core rev: e6004582454d8c6a18f617c12e6e408ded5be8df)
Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>
Signed-off-by: Ross Burton <ross.burton@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
-rw-r--r-- | meta/lib/oeqa/buildperf/base.py | 32 |
1 files changed, 32 insertions, 0 deletions
diff --git a/meta/lib/oeqa/buildperf/base.py b/meta/lib/oeqa/buildperf/base.py index c0318a18e0..e10cbf4572 100644 --- a/meta/lib/oeqa/buildperf/base.py +++ b/meta/lib/oeqa/buildperf/base.py | |||
@@ -22,6 +22,7 @@ import traceback | |||
22 | from datetime import datetime, timedelta | 22 | from datetime import datetime, timedelta |
23 | 23 | ||
24 | from oeqa.utils.commands import runCmd, get_bb_vars | 24 | from oeqa.utils.commands import runCmd, get_bb_vars |
25 | from oeqa.utils.git import GitError, GitRepo | ||
25 | 26 | ||
26 | # Get logger for this module | 27 | # Get logger for this module |
27 | log = logging.getLogger('build-perf') | 28 | log = logging.getLogger('build-perf') |
@@ -85,10 +86,41 @@ class BuildPerfTestRunner(object): | |||
85 | if not os.path.exists(self.out_dir): | 86 | if not os.path.exists(self.out_dir): |
86 | os.makedirs(self.out_dir) | 87 | os.makedirs(self.out_dir) |
87 | 88 | ||
89 | # Get Git parameters | ||
90 | try: | ||
91 | self.repo = GitRepo('.') | ||
92 | except GitError: | ||
93 | self.repo = None | ||
94 | self.git_rev, self.git_branch = self.get_git_revision() | ||
95 | log.info("Using Git branch:revision %s:%s", self.git_branch, | ||
96 | self.git_rev) | ||
97 | |||
98 | def get_git_revision(self): | ||
99 | """Get git branch and revision under testing""" | ||
100 | rev = os.getenv('OE_BUILDPERFTEST_GIT_REVISION') | ||
101 | branch = os.getenv('OE_BUILDPERFTEST_GIT_BRANCH') | ||
102 | if not self.repo and (not rev or not branch): | ||
103 | log.info("The current working directory doesn't seem to be a Git " | ||
104 | "repository clone. You can specify branch and revision " | ||
105 | "used in test results with OE_BUILDPERFTEST_GIT_REVISION " | ||
106 | "and OE_BUILDPERFTEST_GIT_BRANCH environment variables") | ||
107 | else: | ||
108 | if not rev: | ||
109 | rev = self.repo.run_cmd(['rev-parse', 'HEAD']) | ||
110 | if not branch: | ||
111 | try: | ||
112 | # Strip 11 chars, i.e. 'refs/heads' from the beginning | ||
113 | branch = self.repo.run_cmd(['symbolic-ref', 'HEAD'])[11:] | ||
114 | except GitError: | ||
115 | log.debug('Currently on detached HEAD') | ||
116 | branch = None | ||
117 | return str(rev), str(branch) | ||
88 | 118 | ||
89 | def run_tests(self): | 119 | def run_tests(self): |
90 | """Method that actually runs the tests""" | 120 | """Method that actually runs the tests""" |
91 | self.results['schema_version'] = 1 | 121 | self.results['schema_version'] = 1 |
122 | self.results['git_revision'] = self.git_rev | ||
123 | self.results['git_branch'] = self.git_branch | ||
92 | self.results['tester_host'] = socket.gethostname() | 124 | self.results['tester_host'] = socket.gethostname() |
93 | start_time = datetime.utcnow() | 125 | start_time = datetime.utcnow() |
94 | self.results['start_time'] = start_time | 126 | self.results['start_time'] = start_time |