[MG_Test, CI] (P6): the last two readers the role split left behind

build-linux-verify could not compile PipeStatsTest: RoleDerivedJsonDumpPathForTesting is DISAGGREGATED-only and the verify flavour is deliberately monolith. The case name stays registered and GTEST_SKIPs there, matching the PIPE_PUSH pattern, so ctest -N still matches between flavours.

runtime_mode_proof.py logs opened MOBILEGL_LOG_FILE_PATH as a file. After the split that name is a base and the sink writes <base>.client.log / <base>.server.log, so the inproc proof died on FileNotFoundError after a green lane. It now reads through split_log_paths' role helpers and names the paths it looked for when none exist.
This commit is contained in:
2026-09-22 04:40:03 -04:00
parent 2ec8e7f4a1
commit e6c51d0711
2 changed files with 21 additions and 1 deletions
+14 -1
View File
@@ -70,8 +70,21 @@ def logs_proof(discovery, junit):
logs = helper.marker_log_paths(lane)
proof = {}
for name in sorted(RSP_CASES):
# P6 splits the library log per role: MOBILEGL_LOG_FILE_PATH is a BASE NAME and the
# files that exist are <base>.client.log / <base>.server.log. Reading the base directly
# is FileNotFoundError after the split - the same class of reader the role-split
# landing already fixed everywhere else.
path = Path(logs[name])
proof[name] = {'private_log': str(path), **inproc_log_proof(path.read_text(encoding='utf-8', errors='replace'))}
if not helper.any_role_file(path):
raise RuntimeError(
f'{name}: no role log under {path} '
f'(looked for {helper.role_paths(path)}); the library never opened the sink')
text = helper.read_role_logs(path)
proof[name] = {
'private_log': str(path),
'role_logs': helper.role_paths(path),
**inproc_log_proof(text),
}
return proof