Skip to content

Remove completed queue-record migration - #465

Merged
trask merged 1 commit into
mainfrom
trask-remove-completed-queue-migration
Oct 14, 2026
Merged

trask merged 1 commit into
mainfrom
trask-remove-completed-queue-migration

Conversation

@trask

@trask trask commented Oct 13, 2026 •

Copy link
Copy Markdown
Member

Remove the one-time queue-record migration now that the production cleanup is complete.

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

Copilot-Session: 3a86c6ad-177f-4cc3-a518-62e99a74dedb
@opentelemetry-pr-dashboard

opentelemetry-pr-dashboard Bot commented Oct 13, 2026 •

Copy link
Copy Markdown

Pull request dashboard status

Merged · refreshed 2026-10-14 14:56 UTC

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

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.

🟢 Approval recommended

The obsolete migration is removed consistently while ongoing filtering, reporting, and runtime behavior remain intact.

0 open findings

What changed in this PR

Removes the completed one-time queue-record migration while preserving ongoing filtering and incremental reporting.

Changes:

  • Deletes the migration script and dedicated tests.
  • Removes migration workflow logic, restoring the standard 40-minute collection budget.
  • Documents the retained migration receipt.
File Description
github-actions-queue/​README.md Documents the completed cleanup receipt.
.github/​workflows/​github-actions-queue-collector.yml Removes migration execution and conditional report rebuilding.
.github/​scripts/​github-actions-queue/​test_migrate_queue_records.py Deletes obsolete migration tests.
.github/​scripts/​github-actions-queue/​migrate_queue_records.py Deletes the completed migration utility.

🧠 Review effort: Balanced


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

@trask
trask marked this pull request as ready for review October 13, 2026 20:57
@trask
trask requested a review from a team as a code owner October 13, 2026 20:57
@trask
trask requested a review from adrielp October 13, 2026 20:57
@trask
trask merged commit 1e7a880 into main Oct 14, 2026
8 checks passed
@trask
trask deleted the trask-remove-completed-queue-migration branch October 14, 2026 14:55
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