Skip to content
This repository was archived by the owner on Jun 8, 2026. It is now read-only.

dashboard: Add single PR view#31

Open
afinn12 wants to merge 3 commits into
kata-containers:mainfrom
a1icja:single-pr-view
Open

dashboard: Add single PR view#31
afinn12 wants to merge 3 commits into
kata-containers:mainfrom
a1icja:single-pr-view

Conversation

@afinn12

@afinn12 afinn12 commented Nov 18, 2024

Copy link
Copy Markdown
Contributor

Needed

Description

image

After selecting a PR:

image

  • The display (and PR number) is appended to the URL, so specific views can be shared/reloaded
  • The result is the latest result if the check was rerun
  • Total runs is for the given PR number
    • The rowExpansionTemplate still shows the results across all 10 PRs, could change to add different information here.

Testing

  • Tested in dev
  • Tested in prod
    • Used data from our forked repo instead since data hasn't been updated in this repo yet

@afinn12 afinn12 marked this pull request as ready for review November 18, 2024 22:21
Comment on lines -22 to -24
# Use bash to fail fast:
# https://docs.github.com/en/actions/writing-workflows/workflow-syntax-for-github-actions#jobsjob_idstepsshell
shell: bash

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.

We should definitely keep this to prevent false workflow passes

Added a script that fetches PR data and created a separate view on the dashboard.
Tweaked dotenv require.

Fixes kata-containers#1

Signed-off-by: Anna Finn <afinn12@bu.edu>
Added code to get rerun information to both fetch scripts.
Display the reruns as a superscript in the rowExpansionTemplate with the result/URL.

Fixes: kata-containers#8

Signed-off-by: Anna Finn <afinn12@bu.edu>
Added a separate view to display all tests for a given PR.
Added the display to the URL, rowExpansionTemplate is unchanged.

Fixes: kata-containers#12

Signed-off-by: Anna Finn <afinn12@bu.edu>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants