Skip to content

Commit ae81173

Browse files
committed
[UpstreamMergeAutomation] Better logging
1 parent 1aa95de commit ae81173

2 files changed

Lines changed: 19 additions & 8 deletions

File tree

.vscode/settings.json

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
{
2+
"python.autoComplete.extraPaths": [
3+
"${workspaceFolder}/sources/poky/bitbake/lib",
4+
"${workspaceFolder}/sources/poky/meta/lib"
5+
],
6+
"python.analysis.extraPaths": [
7+
"${workspaceFolder}/sources/poky/bitbake/lib",
8+
"${workspaceFolder}/sources/poky/meta/lib"
9+
],
10+
"files.associations": {
11+
"*.conf": "bitbake",
12+
"*.inc": "bitbake"
13+
}
14+
}

scripts/dev/upstream_merge/log_and_email_utils.py

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -50,17 +50,14 @@ def format_status(status, message):
5050
:param message: Associated message or details.
5151
:return: Tuple of formatted status strings for summary and detailed logs.
5252
"""
53-
error_msg = f" ... ERRORS\n {message or ''}\n\n\n"
54-
ok_no_changes = " ... OK (no changes)"
55-
ok_msg = f" ... OK\n {message}\n\n\n"
5653

5754
if status != 0:
58-
return (" ... ERRORS", error_msg)
55+
return (" Merge Conflict\n", f" Merge Conflict\n {message or ''}\n\n\n")
5956

6057
if message is None:
61-
return (ok_no_changes, f"{ok_no_changes}\n\n\n")
58+
return (" Up to date (no changes)\n", " Up to date (no changes)\n\n\n")
6259

63-
return (" ... OK", ok_msg)
60+
return (" Upstream Merge Successful\n", f"{message}\n\n\n")
6461

6562

6663
def format_merge_report(merge_report, email_log_level, skip_push_and_pr=False):
@@ -102,7 +99,7 @@ def format_merge_report(merge_report, email_log_level, skip_push_and_pr=False):
10299
for git_obj, (status, message) in merge_report.items():
103100
min_line, additional_line = format_status(status, message)
104101

105-
layer_name = git_obj.local_repo.split("/")[-1]
102+
layer_name = git_obj.local_repo.split("/")[-1].upper()
106103
min_detail += f"{layer_name}\n{min_line}\n"
107104

108105
if status != 0:
@@ -151,7 +148,7 @@ def format_merge_report(merge_report, email_log_level, skip_push_and_pr=False):
151148

152149
if email_log_level == 0:
153150
return min_detail + "\n\n" + error_detail
154-
return min_detail + "\n\n" + error_detail + "\n\n" + diff_detail
151+
return min_detail + "\n\n\t\t\t\t\t\t\t\t\tERROR DETAILS\n\n" + error_detail + "\n\n\t\t\t\t\t\t\t\t\tSUMMARY\n\n" + diff_detail
155152

156153

157154
def write_log(email_file_name, contents):

0 commit comments

Comments
 (0)