Actions
action #128588
closedInvestigation Tab broken for some jobs size:M
Description
Observation¶
In https://openqa.opensuse.org/tests/3260493#investigation and https://openqa.opensuse.org/tests/3260745#investigation the tab is loading forever. The console points to a Javascript error:
test_result.js:105 Uncaught TypeError: Cannot read properties of null (reading '1')
at githashToLink (test_result.js:105:218)
at test_result.js:127:1680
at Array.forEach (<anonymous>)
at Object.renderInvestigationTab [as renderContents] (test_result.js:127:747)
at Object.success (test_result.js:85:187)
at fire (bootstrap.js:245:231)
at Object.fireWith [as resolveWith] (bootstrap.js:251:181)
at done (bootstrap.js:680:95)
at XMLHttpRequest.<anonymous> (bootstrap.js:693:314)
function githashToLink(value, repo) {
const logItems = value.split(/(?=^[0-9a-f])/gm);
const commits = [];
for (let i = 0; i < logItems.length; i++) {
const item = logItems[i];
const match = item.match(/^([0-9a-f]{9}) (.*)/);
const sha = match[1]; // <--------
const msg = match[2];
commits.push({
link: sha.link(repo + sha),
msg: msg,
stat: item.match(/^ .*/gm)
});
}
return commits;
}
Steps to reproduce¶
- Click on https://openqa.opensuse.org/tests/3260493#investigation and observe the error from developer console
Acceptance criteria¶
- AC1: No more JavaScript errors on the investigation tab (done)
- AC2: A ui test exists
Suggestions¶
- See https://openqa.opensuse.org/tests/3260493/investigation_ajax for the returned json
- Narrow down what cases exhibit this issue (it did not fail tests and doesn't always break)
- Always check for a null result on regex matches
Updated by okurz over 1 year ago
- Subject changed from Investigation Tab broken for some jobs to Investigation Tab broken for some jobs size:M
- Description updated (diff)
- Status changed from New to Workable
Updated by tinita over 1 year ago
- Status changed from Workable to In Progress
- Assignee set to tinita
Updated by tinita over 1 year ago
- Status changed from In Progress to Feedback
https://github.com/os-autoinst/openQA/pull/5126 Improve non-matching log output in investigation tab
Updated by tinita over 1 year ago
- Status changed from Feedback to In Progress
https://github.com/os-autoinst/openQA/pull/5126 merged, now trying to add a test
Updated by tinita over 1 year ago
- Description updated (diff)
- Status changed from In Progress to Workable
- Assignee deleted (
tinita)
Actually there are more urgent tasks, so I put this back to workable for now
Updated by mkittler over 1 year ago
- Status changed from Workable to In Progress
- Assignee set to mkittler
Updated by mkittler over 1 year ago
- Status changed from In Progress to Feedback
Updated by okurz over 1 year ago
- Status changed from Feedback to Resolved
https://github.com/os-autoinst/openQA/pull/5182 merged, AC2 covered now as well
Actions