Skip to content

Conversation

@jmcarp
Copy link
Contributor

@jmcarp jmcarp commented Oct 8, 2025

  • Vendor the latest omicron config to fix tests.
  • Speed up the cleanup step by a few minutes.

@jmcarp jmcarp force-pushed the jmcarp/debug-gha-cleanup branch 3 times, most recently from 4863567 to 5a58607 Compare October 8, 2025 18:42
* Vendor the latest omicron config to fix tests.
* Speed up the cleanup step by a few minutes.
@jmcarp jmcarp force-pushed the jmcarp/debug-gha-cleanup branch from 5a58607 to 34b34f5 Compare October 9, 2025 14:28
@jmcarp jmcarp marked this pull request as ready for review October 9, 2025 14:28
@jmcarp jmcarp requested a review from a team as a code owner October 9, 2025 14:28
Comment on lines +40 to +43
# Note: deleting these directories can be very slow, so we only remove
# enough to allow the build to complete. We'll drop this workaround and
# get better build times once we switch to self-hosted runners with
# persistent storage and more disk.
Copy link
Member

Choose a reason for hiding this comment

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

Do we need the extra space just to build the image, or is the image itself so big that we can't even run it? If this only needed for build, then we can probably live with slow rm once we publish the image to a registry.

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