Skip to content

Commit

Permalink
Fix Perf top Page
Browse files Browse the repository at this point in the history
Signed-off-by: Lan Xia <[email protected]>
  • Loading branch information
llxia committed Jan 31, 2025
1 parent a7a6b4d commit 78af9b3
Show file tree
Hide file tree
Showing 4 changed files with 11 additions and 8 deletions.
7 changes: 5 additions & 2 deletions test-result-summary-client/src/Build/ReleaseSummary.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ const ReleaseSummary = () => {
const rerunBuildsRes = fetchData(
`/api/getAllChildBuilds${params({
parentId,
buildNameRegex: 'Test_openjdk.*_rerun',
buildNameRegex: '(Test|Perf)_openjdk.*_rerun',
})}`
);

Expand Down Expand Up @@ -83,7 +83,10 @@ const ReleaseSummary = () => {
buildResultStr = ` ⚠️ ${buildResult} ⚠️${nl}`;
}

if (buildName.startsWith('Test_openjdk')) {
if (
buildName.startsWith('Test_openjdk') ||
buildName.startsWith('Perf_openjdk')
) {
let rerunLinkInfo = '';
if (rerunFailedLink) {
rerunLinkInfo = `Rerun [failed](${rerunFailedLink})${nl}`;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ export default function ResultSummary() {
})}`
);
// get all child builds info based on buildNameRegex
const buildNameRegex = '^Test_openjdk.*';
const buildNameRegex = '^(Test|Perf)_openjdk.*';
const testSummaryResult = undefined;
const buildResult = undefined;

Expand Down
8 changes: 4 additions & 4 deletions test-result-summary-client/src/Build/TopLevelBuilds.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -14,13 +14,13 @@ function TopLevelBuilds() {
const updateData = async (type) => {
if (!type) type = 'Test';
let results = '';
if (type === 'Test') {
if (type === 'AQAvitCert') {
results = await fetchData(
`/api/getTopLevelBuildNames?type=${type}`
`/api/getTopLevelBuildNames?type=Test&AQAvitCert=true`
);
} else if (type === 'AQAvitCert') {
} else {
results = await fetchData(
`/api/getTopLevelBuildNames?type=Test&AQAvitCert=true`
`/api/getTopLevelBuildNames?type=${type}`
);
}
const builds = {};
Expand Down
2 changes: 1 addition & 1 deletion test-result-summary-client/src/utils/Utils.js
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ export const order = (a, b) => {
};
export const getInfoFromBuildName = (buildName) => {
const regex =
/^Test_openjdk(\w+)_(\w+)_(\w+).(.+?)_(.+?_.+?(_xl|_fips140_2|_fips140_3_openjceplusfips|_fips140_3_openjceplusfips.fips140-3|_criu)?)(_.+)?$/i;
/^(Test|Perf)_openjdk(\w+)_(\w+)_(\w+).(.+?)_(.+?_.+?(_xl|_fips140_2|_fips140_3_openjceplusfips|_fips140_3_openjceplusfips.fips140-3|_criu)?)(_.+)?$/i;
const tokens = buildName.match(regex);
if (Array.isArray(tokens) && tokens.length > 5) {
const [_, jdkVersion, jdkImpl, level, group, platform] = tokens;
Expand Down

0 comments on commit 78af9b3

Please sign in to comment.