Skip to content

Add enable_droplet_caching property to gate rep's droplet cache - #1193

Merged
ebroberson merged 2 commits into
developfrom
feature/TNZ-126780
Aug 12, 2026
Merged

Add enable_droplet_caching property to gate rep's droplet cache#1193
ebroberson merged 2 commits into
developfrom
feature/TNZ-126780

Conversation

@geofffranks

Copy link
Copy Markdown
Contributor

Summary

The cached downloader will not evict cached files marked as "in use". As a result, on densely packed cells, the download cache will grow and grow beyond the max cache directory size, as all the cached droplet files are actually in use.

For these cases, this change lets operators disable rep's local-disk caching of app droplet downloads, avoiding ephemeral-disk-full incidents on densely-packed cells at the cost of re-downloading droplets from the blobstore on every restart/scale. Buildpack/lifecycle bundle caching is unaffected. Defaults to true (current behavior unchanged) via the BOSH job spec.

ai-assisted=yes
TNZ-126780

Backward Compatibility

Breaking Change? yes

Lets operators disable rep's local-disk caching of app droplet
downloads, avoiding ephemeral-disk-full incidents on densely-packed
cells at the cost of re-downloading droplets from the blobstore on
every restart/scale. Buildpack/lifecycle bundle caching is unaffected.
Defaults to true (current behavior unchanged) via the BOSH job spec.

ai-assisted=yes
TNZ-126780
Keeps jobs/rep_windows in sync with jobs/rep: Task 3 added the
diego.executor.enable_droplet_caching property to jobs/rep but missed
the Windows job, which shares the same executor config. tnz-runtime-
containers-release already had rep_windows parity from Task 8b.

ai-assisted=yes
TNZ-126780

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
ebroberson
ebroberson previously approved these changes Aug 12, 2026

@ebroberson ebroberson 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.

LGTM

@ebroberson
ebroberson merged commit ce41050 into develop Aug 12, 2026
10 checks passed
@github-project-automation github-project-automation Bot moved this from Pending Merge | Prioritized to Done in Application Runtime Platform Working Group Aug 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

2 participants