Skip to content

Fixed problem with all "forced" volumes cache (#7028)#7037

Merged
zwoop merged 1 commit intoapache:8.1.xfrom
gtenev:multiple_all_forced_volumes_8.1.x
Jul 23, 2020
Merged

Fixed problem with all "forced" volumes cache (#7028)#7037
zwoop merged 1 commit intoapache:8.1.xfrom
gtenev:multiple_all_forced_volumes_8.1.x

Conversation

@gtenev
Copy link
Contributor

@gtenev gtenev commented Jul 23, 2020

Fixed a problem with cache volume configurations where all volumes are
forced to a span (disk) in storage.config and percentage size is used
in volume.config.

Since "forced" volumes are now excluded from "total cache space" used
for volume size percentage calculations when all spans (disks) are
"forced" to volumes the "total cache space" becomes 0 and then volume
sizes calculated as percentage become 0. This patch fixes the issue and
also handles the case when a single volume is forced to multiple spans.

(cherry picked from commit 26ddf1e)

Fixed a problem with cache volume configurations where all volumes are
forced to a span (disk) in storage.config and percentage size is used
in volume.config.

Since "forced" volumes are now excluded from "total cache space" used
for volume size percentage calculations when all spans (disks) are
"forced" to volumes the "total cache space" becomes 0 and then volume
sizes calculated as percentage become 0. This patch fixes the issue and
also handles the case when a single volume is forced to multiple spans.

(cherry picked from commit 26ddf1e)
@gtenev gtenev added this to the 8.1.0 milestone Jul 23, 2020
@gtenev gtenev requested review from SolidWallOfCode and scw00 July 23, 2020 22:28
@gtenev gtenev self-assigned this Jul 23, 2020
@zwoop
Copy link
Contributor

zwoop commented Jul 23, 2020

[approve ci freebsd]

@zwoop zwoop merged commit da39ef3 into apache:8.1.x Jul 23, 2020
@zwoop zwoop modified the milestones: 8.1.0, Backported Jul 23, 2020
masaori335 pushed a commit to masaori335/trafficserver that referenced this pull request Mar 31, 2021
* asf/8.1.x:
  Updated ChangeLog
  Fixed problem with all "forced" volumes cache (apache#7028) (apache#7037)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants