Skip to content

Fix manual run IDs and speed up arch tag cleanup#345

Open
bdice wants to merge 4 commits into
mainfrom
fix-manual-run-id-workflow-call
Open

Fix manual run IDs and speed up arch tag cleanup#345
bdice wants to merge 4 commits into
mainfrom
fix-manual-run-id-workflow-call

Conversation

@bdice
Copy link
Copy Markdown
Contributor

@bdice bdice commented May 15, 2026

Summary

  • Pass the parent run ID into called test and benchmark workflows so manual build images resolve to the run that produced them.
  • Speed up arch-specific tag cleanup by resolving multiple tags in one lookup path and stopping package-version pagination once all requested tags are found.
  • Use merged multi-arch deps images as build contexts so deps arch-specific intermediates can be cleaned up in the deps merge jobs.

The cleanup step was taking more than 6 minutes to run because it was fetching all pages of the container registry, once for every image being deleted. The updates in ci/delete-arch-tags.sh now fetch only enough pages needed to resolve the IDs for the desired tags, which now only takes a second or two to run.

Since that is so much faster, I also moved the presto-deps image cleanup and refactored the other presto image builds to use the merged manifests rather than the arch-specific intermediates.

Testing

@bdice bdice requested a review from a team as a code owner May 15, 2026 21:35
@bdice bdice requested a review from mattgara May 15, 2026 21:35
@bdice bdice changed the title Fix manual run IDs for called test workflows Fix manual run IDs and speed up arch tag cleanup May 15, 2026
Copy link
Copy Markdown
Contributor

@Avinash-Raj Avinash-Raj left a comment

Choose a reason for hiding this comment

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

LGTM.

@Avinash-Raj
Copy link
Copy Markdown
Contributor

/ok to test d9874e2

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.

2 participants