Skip to content

Restore dashboard workflow Git context - #287

Merged
maryliag merged 2 commits into
mainfrom
trask-fix-dashboard-git-config
Aug 21, 2026
Merged

Restore dashboard workflow Git context#287
maryliag merged 2 commits into
mainfrom
trask-fix-dashboard-git-config

Conversation

@trask

@trask trask commented Aug 21, 2026

Copy link
Copy Markdown
Member

Fixes the hourly dashboard canary failures introduced by #285. The reusable workflow still loads scripts from its own commit, but now checks out the triggering commit at the workspace root so state-branch operations have the Git metadata and origin they require.

Failure observed in https://github.com/open-telemetry/shared-workflows/actions/runs/32490943133.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: a3f8d097-6dfe-467e-9e20-9be7e3fab4a6
@trask
trask requested a balanced review from Copilot August 21, 2026 14:20

Copilot AI 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.

Pull request overview

Restores the Git repository context required by dashboard state operations while retaining immutable workflow scripts.

Changes:

  • Checks out the triggering commit at the workspace root.
  • Reads live dashboard configuration from that checkout.
  • Updates rollout assertions for the restored layout.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.github/workflows/pull-request-dashboard-repo.yml Restores root Git checkout for both dashboard jobs.
.github/scripts/pull-request-dashboard/test_rollout.py Verifies checkout and configuration behavior.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: a3f8d097-6dfe-467e-9e20-9be7e3fab4a6
@trask
trask marked this pull request as ready for review August 21, 2026 14:33
@trask
trask requested a review from a team as a code owner August 21, 2026 14:34
@trask
trask requested a review from maryliag August 21, 2026 14:34
@opentelemetry-pr-dashboard

opentelemetry-pr-dashboard Bot commented Aug 21, 2026

Copy link
Copy Markdown

Pull request dashboard status

Merged · refreshed 2026-08-21 15:52 UTC

Status above doesn't look right?
  • Anything look wrong? Report it with what you expected; it helps us improve the dashboard.

@maryliag
maryliag merged commit 7fd7c3a into main Aug 21, 2026
6 checks passed
@maryliag
maryliag deleted the trask-fix-dashboard-git-config branch August 21, 2026 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants