diff options
author | Chen Rong <rong.a.chen@intel.com> | 2019-07-30 15:12:38 +0800 |
---|---|---|
committer | Philip Li <philip.li@intel.com> | 2019-07-30 16:35:36 +0800 |
commit | 02ce5342f7e97bf167e1f55610564743b3165fad (patch) | |
tree | 863f84a0d2af36a0f09de3a535063c5784c29f11 | |
parent | 6f0cb3662ddf97f6110050af93fb31762d727809 (diff) | |
download | lkp-tests-02ce5342f7e97bf167e1f55610564743b3165fad.tar.gz |
sbin/compare: don't check commit id for local run
The commit name is kernel version in local run.
--------------
before:
$ lkp compare -f "will-it-scale.per_process_ops" /lkp/result/will-it-scale/process-100%-brk1/shao2-debian/debian/x86_64-rhel-7.6/gcc-7/4.19.0-4-amd64/0/ /lkp/result/will-it-scale/process-100%-brk1/shao2-debian/debian/x86_64-rhel-7.6/gcc-7/4.19.0-5-amd64/0/
after:
$ lkp compare -f "will-it-scale.per_process_ops" /lkp/result/will-it-scale/process-100%-brk1/shao2-debian/debian/x86_64-rhel-7.6/gcc-7/4.19.0-4-amd64/0/ /lkp/result/will-it-scale/process-100%-brk1/shao2-debian/debian/x86_64-rhel-7.6/gcc-7/4.19.0-5-amd64/0/
4.19.0-4-amd64 4.19.0-5-amd64 testcase/testparams/testbox
---------------- -------------------------- ---------------------------
%stddev change %stddev
\ | \
1678120 1691865 will-it-scale/process-100%-brk1/shao2-debian
1678120 1691865 GEO-MEAN will-it-scale.per_process_ops
--------------
Signed-off-by: Chen Rong <rong.a.chen@intel.com>
Signed-off-by: Philip Li <philip.li@intel.com>
-rwxr-xr-x | lib/result.rb | 7 | ||||
-rwxr-xr-x | sbin/compare | 2 |
2 files changed, 6 insertions, 3 deletions
diff --git a/lib/result.rb b/lib/result.rb index 8c1c46dd..440a2dfa 100755 --- a/lib/result.rb +++ b/lib/result.rb @@ -57,7 +57,7 @@ class ResultPath < Hash end end - def parse_result_root(rt) + def parse_result_root(rt, is_local_run = false) dirs = rt.sub(/#{RESULT_MNT}\d*/, '').split('/') dirs.shift if dirs[0] == '' @@ -79,7 +79,10 @@ class ResultPath < Hash end if ps.include?('commit') - each_commit { |_type, commit| return false unless self[commit] && sha1_40?(self[commit]) } + each_commit do |_type, commit| + return false unless self[commit] + return false if !is_local_run && !sha1_40?(self[commit]) + end end # for rt and _rt diff --git a/sbin/compare b/sbin/compare index 952b53d8..b6f2e6c8 100755 --- a/sbin/compare +++ b/sbin/compare @@ -266,7 +266,7 @@ def add_stats(path) end result_path = ResultPath.new - return unless result_path.parse_result_root(File.realpath(path)) + return unless result_path.parse_result_root(File.realpath(path), local_run?) if $dim_not_a_param dim = result_path[$opt_dimension] |