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
|
#!/usr/bin/env python3
"""Runs clang-tidy in parallel and combines the the results for easier viewing."""
import argparse
import datetime
import json
import os
import subprocess
import sys
import locale
import time
from typing import Any, Dict, List, Optional, Tuple
import multiprocessing
from pathlib import Path
from concurrent import futures
from simple_report import Result, Report, put_report, try_combine_reports, make_report
import yaml
def _clang_tidy_executor(clang_tidy_filename: str, clang_tidy_binary: str,
clang_tidy_cfg: Dict[str, Any], output_dir: str, show_stdout: bool,
mongo_check_module: str = '') -> Tuple[str, Optional[str]]:
clang_tidy_parent_dir = output_dir / clang_tidy_filename.parent
os.makedirs(clang_tidy_parent_dir, exist_ok=True)
output_filename_base = clang_tidy_parent_dir / clang_tidy_filename.name
output_filename_fixes = output_filename_base.with_suffix(".yml")
if mongo_check_module:
load_module_option = ['-load', mongo_check_module]
else:
load_module_option = []
clang_tidy_command = [
clang_tidy_binary, *load_module_option, clang_tidy_filename,
f"-export-fixes={output_filename_fixes}", f"-config={json.dumps(clang_tidy_cfg)}"
]
proc = subprocess.run(clang_tidy_command, capture_output=True, check=False)
files_to_parse = None
if proc.returncode != 0:
output_filename_out = output_filename_base.with_suffix(".fail")
files_to_parse = output_filename_fixes
if not show_stdout:
print(
f"Running clang-tidy on {clang_tidy_filename} had errors see {output_filename_out}")
else:
print(f"Running clang-tidy on {clang_tidy_filename}")
print(f"{proc.stderr.decode(locale.getpreferredencoding())}")
print(f"{proc.stdout.decode(locale.getpreferredencoding())}")
else:
output_filename_out = output_filename_base.with_suffix(".pass")
if not show_stdout:
print(f"Running clang-tidy on {clang_tidy_filename} had no errors")
with open(output_filename_out, 'wb') as output:
output.write(proc.stderr)
output.write(proc.stdout)
return proc.stdout.decode(locale.getpreferredencoding()), files_to_parse
def _combine_errors(fixes_filename: str, files_to_parse: List[str]) -> int:
failed_files = 0
all_fixes = {}
#loop files_to_parse and count the number of failed_files
for item in files_to_parse:
if item is None:
continue
failed_files += 1
# Read the yaml fixes for the file to combine them with the other suggested fixes
with open(item) as input_yml:
fixes = yaml.safe_load(input_yml)
for fix in fixes['Diagnostics']:
fix_data = all_fixes.setdefault(fix["DiagnosticName"], {}).setdefault(
fix["FilePath"], {}).setdefault(
fix["FileOffset"], {
"replacements": fix["Replacements"], "message": fix["Message"], "count": 0,
"source_files": []
})
fix_data["count"] += 1
fix_data["source_files"].append(fixes['MainSourceFile'])
with open(fixes_filename, "w") as files_file:
json.dump(all_fixes, files_file, indent=4, sort_keys=True)
return failed_files
def __dedup_errors(clang_tidy_errors_threads: List[str]) -> str:
#use dict as an 'ordered set'(in python 3.6+), set value to dummy value(true here)
error_to_dummy_value = dict()
for errs in clang_tidy_errors_threads:
if errs:
for val in errs.splitlines():
error_to_dummy_value[val] = True
return os.linesep.join(error_to_dummy_value.keys())
def main():
"""Execute Main entry point."""
parser = argparse.ArgumentParser(description='Run multithreaded clang-tidy')
parser.add_argument('-t', "--threads", type=int, default=multiprocessing.cpu_count(),
help="Run with a specific number of threads")
parser.add_argument("-d", "--output-dir", type=str, default="clang_tidy_fixes",
help="Directory to write all clang-tidy output to")
parser.add_argument("-o", "--fixes-file", type=str, default="clang_tidy_fixes.json",
help="Report json file to write combined fixes to")
parser.add_argument("-c", "--compile-commands", type=str, default="compile_commands.json",
help="compile_commands.json file to use to find the files to tidy")
parser.add_argument("-q", "--show-stdout", type=bool, default=True,
help="Log errors to console")
parser.add_argument("-l", "--log-file", type=str, default="clang_tidy",
help="clang tidy log from evergreen")
parser.add_argument("-m", "--check-module", type=str,
default="build/install/lib/libmongo_tidy_checks.so",
help="Path to load the custom mongo checks module.")
# TODO: Is there someway to get this without hardcoding this much
parser.add_argument("-y", "--clang-tidy-toolchain", type=str, default="v4")
parser.add_argument("-f", "--clang-tidy-cfg", type=str, default=".clang-tidy")
args = parser.parse_args()
clang_tidy_binary = f'/opt/mongodbtoolchain/{args.clang_tidy_toolchain}/bin/clang-tidy'
if os.path.exists(args.check_module):
mongo_tidy_check_module = args.check_module
else:
mongo_tidy_check_module = ''
with open(args.compile_commands) as compile_commands:
compile_commands = json.load(compile_commands)
with open(args.clang_tidy_cfg) as clang_tidy_cfg:
clang_tidy_cfg = yaml.safe_load(clang_tidy_cfg)
files_to_tidy = list()
files_to_parse = list()
for file_doc in compile_commands:
# A few special cases of files to ignore
if not "src/mongo" in file_doc["file"]:
continue
# TODO SERVER-49884 Remove this when we no longer check in generated Bison.
if "parser_gen.cpp" in file_doc["file"]:
continue
files_to_tidy.append(Path(file_doc["file"]))
total_jobs = len(files_to_tidy)
workers = args.threads
clang_tidy_errors_futures: List[str] = []
clang_tidy_executor_futures: List[futures.ThreadPoolExecutor.submit] = []
# total completed tasks
tasks_completed = 0
with futures.ThreadPoolExecutor(max_workers=workers) as executor:
start_time = time.time()
# submit all futures
for clang_tidy_filename in files_to_tidy:
clang_tidy_executor_futures.append(
executor.submit(_clang_tidy_executor, clang_tidy_filename, clang_tidy_binary,
clang_tidy_cfg, args.output_dir, args.show_stdout,
mongo_tidy_check_module))
for future in futures.as_completed(clang_tidy_executor_futures):
clang_tidy_errors_futures.append(future.result()[0])
files_to_parse.append(future.result()[1])
tasks_completed += 1
pretty_time_duration = str(datetime.timedelta(seconds=time.time() - start_time))
print(
f" The number of jobs completed is {tasks_completed}/{total_jobs}. Duration {pretty_time_duration}"
)
failed_files = _combine_errors(Path(args.output_dir, args.fixes_file), files_to_parse)
# Zip up all the files for upload
subprocess.run(["tar", "-czvf", args.output_dir + ".tgz", args.output_dir], check=False)
# create report and dump to report.json
error_file_contents = __dedup_errors(clang_tidy_errors_futures)
report = make_report(args.log_file, error_file_contents, 1 if failed_files > 0 else 0)
try_combine_reports(report)
put_report(report)
return failed_files
if __name__ == "__main__":
sys.exit(main())
|