Skip to content

Associate remote JCK statuses with their targets - #1306

Open
andrew-m-leonard with Copilot wants to merge 3 commits into
masterfrom
copilot/fix-remote-trigger-display-in-grid
Open

Associate remote JCK statuses with their targets#1306
andrew-m-leonard with Copilot wants to merge 3 commits into
masterfrom
copilot/fix-remote-trigger-display-in-grid

Conversation

Copilot AI commented Oct 5, 2026 •
edited by andrew-m-leonard
Loading

Copy link
Copy Markdown
Contributor

Parallel remote JCK jobs can finish in a different order from their triggers, causing dev and special results to appear in each other’s Grid rows.

  • Status mapping: Match each status display name to its target, platform, and JDK; retain trigger-order fallback when the display name does not identify a target.
  • Regression coverage: Add a case where special finishes before dev.
Trigger order: dev, special
Completion order: special SUCCESS, dev FAILURE
Grid results: dev FAILURE, special SUCCESS

Pipeline job with new remote target info for testing this PR: https://ci.adoptium.net/job/build-scripts/job/openjdk21-pipeline/502/

Copilot AI linked an issue Oct 5, 2026 that may be closed by this pull request
Fix remote trigger display in Grid #1268
Open
@netlify

netlify Bot commented Oct 5, 2026 •
edited
Loading

Copy link
Copy Markdown

✅ Deploy Preview for eclipsefdn-adoptium-trss canceled.

Name Link
🔨 Latest commit 635d79a
🔍 Latest deploy log https://app.netlify.com/projects/eclipsefdn-adoptium-trss/deploys/6ac5198ed6447b0008f28b7f

@andrew-m-leonard
Match remote JCK status to target
385ffd3 
Co-authored-by: andrew-m-leonard <31470007+andrew-m-leonard@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix remote trigger display in Grid Associate remote JCK statuses with their targets Oct 5, 2026
Copilot AI requested a review from andrew-m-leonard October 5, 2026 15:56
andrew-m-leonard
andrew-m-leonard reviewed Oct 6, 2026
View reviewed changes
Comment thread TestResultSummaryService/routes/getRemoteJckBuildInfo.js
Copilot AI requested a review from andrew-m-leonard October 6, 2026 09:19
andrew-m-leonard
andrew-m-leonard reviewed Oct 6, 2026
View reviewed changes
Comment thread TestResultSummaryService/routes/getRemoteJckBuildInfo.js
Copilot AI requested a review from andrew-m-leonard October 6, 2026 09:35
@andrew-m-leonard
Parse explicit remote JCK targets
635d79a 
Co-authored-by: andrew-m-leonard <31470007+andrew-m-leonard@users.noreply.github.com>
@andrew-m-leonard
andrew-m-leonard marked this pull request as ready for review October 8, 2026 12:56
andrew-m-leonard
andrew-m-leonard approved these changes Oct 8, 2026
View reviewed changes

@andrew-m-leonard andrew-m-leonard left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@smlambert smlambert Awaiting requested review from smlambert
1 more reviewer
@andrew-m-leonard andrew-m-leonard andrew-m-leonard approved these changes
Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

@andrew-m-leonard andrew-m-leonard

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Fix remote trigger display in Grid

2 participants

@andrew-m-leonard