Skip to content

Update ci-prune-buildcache to also prune the stack-specific buildcaches#1294

Open
zackgalbreath wants to merge 4 commits intomainfrom
prune_stacks
Open

Update ci-prune-buildcache to also prune the stack-specific buildcaches#1294
zackgalbreath wants to merge 4 commits intomainfrom
prune_stacks

Conversation

@zackgalbreath
Copy link
Copy Markdown
Collaborator

No description provided.

@zackgalbreath
Copy link
Copy Markdown
Collaborator Author

zackgalbreath commented Jan 8, 2026

Note that the relevant cron job is still intentionally disabled. My plan is to monitor a manual run of this job after this PR and spack/spack#51818 are merged to verify that it is behaving as expected. If everything looks good then we'll re-enable automatic scheduled pruning.

Comment thread images/ci-prune-buildcache/entrypoint.sh Outdated
Comment thread images/ci-prune-buildcache/entrypoint.sh

# Update the mirror index
echo "Updating mirror index for $stack..."
spack buildcache update-index "${stack}"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Doesn't have to be here, but a while back we had briefly discussed doing this as part of the pruning command optionally so that the index would be update before the pruning to exclude the pruned specs. Looking at the pruning function in spack it seems that is still not there, just something to remember for later.

namespace: custom
spec:
suspend: true
schedule: "0 0 * * 6"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This could maybe be more frequent, mostly for testing initially.

Comment thread images/ci-prune-buildcache/entrypoint.sh Outdated
@zackgalbreath zackgalbreath force-pushed the prune_stacks branch 3 times, most recently from c020d44 to f4e8b44 Compare April 29, 2026 19:01
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