glusterfs.git/xlators/features/quota/src/quota.c, branch v3.5.4beta1

features/quota: Send the immediate parent with limit in quota statfs

2015-02-23T12:56:47+00:00

adjustment

This is a backport of http://review.gluster.org/#/c/7330

> Problem:
> Assume the directory structure /quota_limit_dir/subdir and
> quota_limit_dir is
> set with some limit. When quota-deem-statfs is enabled the output of
> 'df /quota_limit_dir' would display quota modified values wrt to
> quota_limit_dir where as 'df /quota_limit_subdir/subdir' would display
> the quota modified values wrt volume root (/).
>
> This behaviour is not expected since, when mounted with subdirectory
> admin doesn't want users to know information above the subdirectory mounted.
>
> Solution:
> Any subdirectory within a quota_limit_dir would show the modified values
> as in the /quota_limit_dir. It searches for the nearest parent that has quota
> limit set and modifies the statvfs wrt that.
>
> Change-Id: Ie10fae8999bddbb766b1dbeb881723ed80dce688
> BUG: 1080296
> Signed-off-by: Varun Shastry 
> Reviewed-on: http://review.gluster.org/7330
> Reviewed-by: Raghavendra G 
> Tested-by: Raghavendra G 

Change-Id: I5232a82da20d3dfd48303ba2748b86cfc96ff864
BUG: 1115197
Signed-off-by: vmallika 
Reviewed-on: http://review.gluster.org/9510
Tested-by: Gluster Build System 
Reviewed-by: Niels de Vos

features/quota: Fix dict leak

2014-06-23T09:53:55+00:00

Change-Id: Id4542d1629175cce5fec5ab8f9a5899eec48e2eb
BUG: 1110777
Signed-off-by: Varun Shastry 
Reviewed-on: http://review.gluster.org/8132
Tested-by: Gluster Build System 
Reviewed-by: Niels de Vos

features/quota: fix the dict leak when quota is off

2014-03-15T12:46:50+00:00

Change-Id: I15036da90e96b69858eac5a19bd438df3bd8cc53
BUG: 1075506
Signed-off-by: Varun Shastry 
Reviewed-on: http://review.gluster.org/7233
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

features/quota: fix crash in error handling after building ancestry.

2014-02-08T16:22:22+00:00

Change-Id: Ifbebf1aa496d49a6c4bb30258b83aaf9792828e5
BUG: 1059833
Signed-off-by: Raghavendra G 
Reviewed-on: http://review.gluster.org/6924
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

quota: get directory size before enforcing quota on rename

2014-02-06T07:05:59+00:00

        Backport of http://review.gluster.org/6155

BUG: 1023974
Change-Id: Icb4bca8c4a5fa9b14855478e69b1315f2e9a3c3d
Signed-off-by: Krishnan Parthasarathi 
Reviewed-on: http://review.gluster.org/6887
Reviewed-by: Raghavendra G 
Tested-by: Gluster Build System

quota: fix recording of last alert log message

2014-02-06T07:05:51+00:00

        Backport of http://review.gluster.org/6532

PROBLEM:

Alert log messages, corresponding to disk usage crossing soft-limit on a
directory, weren't being logged as often as expected.

CAUSE:

The mechanism in place to log alert messages, once every alert-time
seconds, set the previous logged time incorrectly.

FIX:

Update previous logged time only if we logged an alert message, ie. when
the "time was right" to alert.

BUG: 969461
Change-Id: Ie019c180c97d2b049fa664b14ce087b88e3b578f
Signed-off-by: Krishnan Parthasarathi 
Reviewed-on: http://review.gluster.org/6886
Reviewed-by: Raghavendra G 
Tested-by: Gluster Build System

mgmt/glusterd: make sure quota enforcer has established connection with quotad before marking quota as enabled.

2014-01-29T06:46:35+00:00

without this patch there is a window of time when quota is marked as
enabled in quota-enforcer, but connection to quotad wouldn't have been
established. Any checklimit done during this period can result in a
failed fop because of unavailability of quotad.

Change-Id: I0d509fabc434dd55ce9ec59157123524197fcc80
Signed-off-by: Raghavendra G 
BUG: 969461
Signed-off-by: Raghavendra G 
Reviewed-on: http://review.gluster.org/6572
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur 
Reviewed-on: http://review.gluster.org/6820

features/quota: Handle the corner case in statfs call

2014-01-29T06:41:59+00:00

Problem: Even though limit is not set quota used to send 'quota-deem-statfs'
key in the dictionary resulting in incorrect calculations.

Change-Id: I643cb35cca6648e40f1c745135280876958ddaa9
BUG: 1046894
Signed-off-by: Varun Shastry 
Reviewed-on: http://review.gluster.org/6607
Tested-by: Gluster Build System 
Reviewed-by: Raghavendra G 
Reviewed-on: http://review.gluster.org/6840
Reviewed-by: Vijay Bellur

features/quota: maintain correct link_count in ancestry building

2014-01-28T16:49:30+00:00

codepath.

Ancestry building codepath can be executed simultaneously when a file
has hardlinks. So, following fixes are needed:

* modify local->link_count under locks in ancestry building codepath.
* before invoking check_limit on newly constructed parents,
  link_count should not be set, but instead incremented by as many
  number of new invocations of check_limit.
* decrementing link_count and the check whether the count has hit zero
  to resume the waiting call_stub should be atomic.

Change-Id: I6f52e50547362a5ded6bd9f085570223aea372d1
BUG: 969461
Signed-off-by: Raghavendra G 
Reviewed-on: http://review.gluster.org/6491
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur 
Reviewed-on: http://review.gluster.org/6819

features/quota: remove in-memory accounting of files in enforcer

2014-01-28T16:47:26+00:00

Accounting was done in enforcer (though marker is the ultimate source
of truth) to offset cached directory size becoming stale. However,
with enforcer being moved to brick we can no longer maintain correct
cluster wide size for a directory. Hence removing accounting code from
enforcer.

Change-Id: I5ea94234da4da85ed5f5ced1354d8de3454b3fcb
BUG: 969461
Signed-off-by: Raghavendra G 
Reviewed-on: http://review.gluster.org/6434
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur 
Reviewed-on: http://review.gluster.org/6818