Bug 782095

Summary:	[glusterfs-3.2.6qa1]: if io-cache min-file-size is greater than max-file-size, then glusterd shows the wrong values
Product:	[Community] GlusterFS	Reporter:	Raghavendra Bhat <rabhat>
Component:	glusterd	Assignee:	Avra Sengupta <asengupt>
Status:	CLOSED CURRENTRELEASE	QA Contact:
Severity:	medium	Docs Contact:
Priority:	low
Version:	pre-release	CC:	amarts, gluster-bugs, vbhat
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	glusterfs-3.4.0	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2013-07-24 17:20:52 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Bug Depends On:
Bug Blocks:	849287

Description Raghavendra Bhat 2012-01-16 14:00:31 UTC

Description of problem:
If maximum file size to be cached is lesser than the minimum file size to be cached, then io-cache reconfigure will set both min-file-size and max-file-size to the old value (according to the log message).
[2012-01-16 19:23:30.995499] E [io-cache.c:1673:reconfigure] 0-io-cache: minimum size (23068672) of a file that can be cached is greater than maximum size (23068672). Hence Defaulting to old value

But gluster volume info still shows the wrong input value given by the user only.

gluster volume info

Volume Name: vol
Type: Distribute
Status: Started
Number of Bricks: 1
Transport-type: tcp
Bricks:
Brick1: hyperspace:/mnt/sda7/export4
Options Reconfigured:
performance.cache-max-file-size: 2MB
performance.cache-min-file-size: 22MB
performance.flush-behind: on


Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:

Comment 1 Amar Tumballi 2012-12-20 07:58:00 UTC

see if its still valid with master code.

Comment 2 Avra Sengupta 2012-12-20 11:54:11 UTC

Checked in master. It is reproducible in master.

Comment 3 Vijay Bellur 2013-02-20 02:58:44 UTC

CHANGE: http://review.gluster.org/4541 (glusterd: Added validation function for performance cache max and min size.) merged in master by Anand Avati (avati)

Comment 4 Avra Sengupta 2013-02-21 08:28:46 UTC

*** Bug 764706 has been marked as a duplicate of this bug. ***