Skip to content

[10.2.x] cache: apply per-volume settings on first start after clear (#13252)#13340

Open
masaori335 wants to merge 1 commit into
apache:10.2.xfrom
masaori335:asf-10.2.x-13252
Open

[10.2.x] cache: apply per-volume settings on first start after clear (#13252)#13340
masaori335 wants to merge 1 commit into
apache:10.2.xfrom
masaori335:asf-10.2.x-13252

Conversation

@masaori335

Copy link
Copy Markdown
Contributor

Per-volume tuning (ram_cache, ram_cache_size, ram_cache_cutoff, avg_obj_size, fragment_size) was only copied onto the CacheVol when matching an existing on-disk volume, so volumes created fresh (e.g. the first start after a cache clear) ignored the config until the next restart. Apply the settings in one pass after all CacheVols exist.

Extend cache_volume_features to verify the settings take effect via per-volume metrics.

(cherry picked from commit ec72cf4)

Conflicts:
tests/gold_tests/cache/cache_volume_features.replay.yaml

@masaori335 masaori335 requested a review from cmcfarlen June 26, 2026 04:56
@masaori335 masaori335 self-assigned this Jun 26, 2026
@masaori335 masaori335 added the Backport Marked for backport for an LTS patch release label Jun 26, 2026
@masaori335 masaori335 added this to the 10.2.0 milestone Jun 26, 2026
…3252)

Per-volume tuning (ram_cache, ram_cache_size, ram_cache_cutoff,
avg_obj_size, fragment_size) was only copied onto the CacheVol when
matching an existing on-disk volume, so volumes created fresh (e.g.
the first start after a cache clear) ignored the config until the
next restart. Apply the settings in one pass after all CacheVols
exist.

Extend cache_volume_features to verify the settings take effect via
per-volume metrics.

(cherry picked from commit ec72cf4)

Conflicts:
  tests/gold_tests/cache/cache_volume_features.replay.yaml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Backport Marked for backport for an LTS patch release Cache

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant