Commit 11bc878
authored
[CRCR] Add nightly results view to per-repo dashboard (#8441)
## Summary
- Adds `?event=nightly` query parameter support to the per-repo CRCR
dashboard page (`/crcr/{org}/{repo}`)
- When `event=nightly`, the page shows nightly CI results grouped by SHA
with Time/SHA/Commit columns (no PR number or author)
- The CRCR summary page's nightly tab now links repos to
`/crcr/{org}/{repo}?event=nightly` so navigation is context-aware
- Creates a new `crcr_nightly_dashboard` ClickHouse query that filters
by `event_type = 'nightly'` and deduplicates by `max(run_attempt)` per
`run_id + job_name`
### Navigation flow
| Source | Link Target |
|--------|-------------|
| CRCR Summary → Pull Requests tab → click repo | `/crcr/{org}/{repo}`
(default, PR results) |
| CRCR Summary → Nightly tab → click repo |
`/crcr/{org}/{repo}?event=nightly` (nightly results) |
### Mockup
- [Nightly per-repo
mockup](https://subinz1.github.io/CRCR/oot-hud-mockup-crcr-nightly.html)
- [CRCR Summary
mockup](https://subinz1.github.io/CRCR/oot-hud-mockup-crcr-summary.html)
## Test plan
- [ ] Visit `/crcr/{org}/{repo}` — should show PR-based results (default
behavior unchanged)
- [ ] Visit `/crcr/{org}/{repo}?event=nightly` — should show nightly
results grouped by SHA
- [ ] CRCR summary nightly tab repo links navigate to `?event=nightly`
- [ ] Nightly view hides PR/Author columns and stat cards
- [ ] Time range selector works for nightly view
- [ ] Empty state displays correctly when no nightly data exists1 parent a575ab9 commit 11bc878
4 files changed
Lines changed: 450 additions & 28 deletions
File tree
- torchci
- clickhouse_queries/crcr_nightly_dashboard
- pages/crcr
- [org]
Lines changed: 12 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
Lines changed: 42 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
0 commit comments