Bug 764706 (GLUSTER-2974) - volume set for 'cache-min-file-size' succeeds even if 'min-file size' is greater than 'max-file-size'
Summary: volume set for 'cache-min-file-size' succeeds even if 'min-file size' is grea...
Keywords:
Status: CLOSED DUPLICATE of bug 782095
Alias: GLUSTER-2974
Product: GlusterFS
Classification: Community
Component: cli
Version: mainline
Hardware: x86_64
OS: Linux
low
medium
Target Milestone: ---
Assignee: Avra Sengupta
QA Contact:
URL:
Whiteboard:
Depends On: 764708
Blocks: 849287
TreeView+ depends on / blocked
 
Reported: 2011-06-02 07:29 UTC by M S Vishwanath Bhat
Modified: 2016-06-01 01:55 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
: 849287 (view as bug list)
Environment:
Last Closed: 2013-02-21 08:28:46 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:

Attachments (Terms of Use)

Description Amar Tumballi 2011-06-02 05:18:00 UTC 
*** Bug 2976 has been marked as a duplicate of this bug. ***
Comment 1 M S Vishwanath Bhat 2011-06-02 07:29:09 UTC 
I did the following operation 

root@tortuga:/tmp# gluster volume set hosdu performance.cache-min-file-size 5MB
Set volume successful
root@tortuga:/tmp# gluster volume set hosdu performance.cache-max-file-size 1MB
Set volume successful

min-file-size for caching exceeds max-file-size for caching. There should be a validation to verify it.

root@tortuga:/tmp# gluster volume info
 
Volume Name: hosdu
Type: Distribute
Status: Started
Number of Bricks: 3
Transport-type: tcp
Bricks:
Brick1: tortuga:/tmp/brick1
Brick2: tortuga:/tmp/brick2
Brick3: tortuga:/tmp/brick3
Options Reconfigured:
performance.cache-max-file-size: 1MB
performance.cache-min-file-size: 5MB
Comment 2 Anand Avati 2011-06-08 13:57:39 UTC 
PATCH: http://patches.gluster.com/patch/7362 in master (performance/io-cache: fix the check of min and max file size.)
Comment 3 Anand Avati 2011-06-08 13:58:02 UTC 
PATCH: http://patches.gluster.com/patch/7364 in release-3.1 (performance/io-cache: fix the check of min and max file size.)
Comment 4 Anand Avati 2011-06-08 13:58:26 UTC 
PATCH: http://patches.gluster.com/patch/7363 in release-3.2 (performance/io-cache: fix the check of min and max file size.)
Comment 5 Amar Tumballi 2011-06-08 14:55:28 UTC 
The patches committed fixes the bug 764708. But this bug will remain open.
Comment 6 M S Vishwanath Bhat 2011-06-10 03:54:16 UTC 
The issue still persists in 3.2.1qa4... Is this supposed to be fixed in 3.2.1?
Comment 7 Amar Tumballi 2011-06-10 04:15:58 UTC 
Bug fix happened is for bug 764708, please verify that.
Comment 8 Amar Tumballi 2012-07-11 05:28:15 UTC 
Currently known issue. We don't do the check during the 'volume set' but the check happens when the process is started (volume start/mounting).

I guess, best way to fix it is by fixing bug 762935, so the proper error code can be sent to the user too.


Currently, lets just document this behavior, so that user is aware of such behavior.
Comment 9 Avra Sengupta 2013-02-21 08:28:46 UTC 
Resolved in http://review.gluster.org/4541.

*** This bug has been marked as a duplicate of bug 782095 ***
Note You need to log in before you can comment on or make changes to this bug.