adoptium / adoptium/aqa-test-tools

Fix TWAS benchmark parsing gaps in BenchmarkMetric.js

Open Beginner friendly
#1,266 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Jupyter Notebook
Stars
33
Forks
97
Avg merge
7h 9m
Merged PRs (30d)
5

Description

Fix TWAS benchmark parsing gaps in BenchmarkMetric.js (SOABench key, DayTrader30/WebSphereFootprint memory regex, CPU Util pct whitespace)

## Problem
After standing up Perf_Pipeline_TWAS(https://hyc-runtimes-jenkins.swg-devops.com/job/Perf_Pipeline_TWAS/1/) , the Traffic Light table only showed 2 of 6 expected benchmark rows (WebSphereStartup + DayTrader30 Throughput). The other 4 were silently dropped -- not a missing-data issue, a parsing issue in `TestResultSummaryService/parsers/BenchmarkMetric.js`.

## Fix
Updated `BenchmarkMetric.js`:
- Added a `SOABench` key (in addition to existing `SOABENCH`) matching the exact benchmark name casing.
- Loosened CPU Util pct regex whitespace matching to non-greedy.
- Updated Adjusted Single Server Memory regex + units field from KB to MB for `DayTrader30` and `WebSphereFootprint`.

Contributor guide

Open the contributing guide

Research direction

Start in TestResultSummaryService/parsers/BenchmarkMetric.js and inspect the existing benchmark keys and metric regular expressions. Verify the SOABench casing, CPU Util pct whitespace, and adjusted single-server memory handling for DayTrader30 and WebSphereFootprint. Done means the TWAS results produce all six expected benchmark rows, including the previously dropped entries.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
backend
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Quiet
Clarity
Clearly specified
Newbie friendliness
82/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.