[Bugs] [Bug 1597154] New: When storage reserve limit is reached, appending data to an existing file throws EROFS error

[bugzilla at redhat.com]

https://bugzilla.redhat.com/show_bug.cgi?id=1597154

            Bug ID: 1597154
           Summary: When storage reserve limit is reached, appending data
                    to an existing file throws EROFS error
           Product: GlusterFS
           Version: 3.12
         Component: replicate
          Keywords: Triaged
          Severity: medium
          Assignee: bugs at gluster.org
          Reporter: ravishankar at redhat.com
                CC: bugs at gluster.org
        Depends On: 1554291, 1561129



+++ This bug was initially created as a clone of Bug #1561129 +++

Description of problem:
=======================
When storage reserve limit is reached, appending data to an existing file
throws EROFS error.

Version-Release number of selected component (if applicable):
3.12.2-5.el7rhgs.x86_64

How reproducible:
always

Steps to Reproduce:
===================
1) create a x3 volume and start it.
2) Set storage.reserve volume set option to 90,
gluster volume set distrepx3 storage.reserve 90
3) Create data using dd until the backend bricks reaches reserve limit.
4) Now, try appending data to the existing file.

Actual results:
===============
appending data to an existing file throws EROFS error
cat /etc/redhat-release  > 25MB_997
-bash: 25MB_997: Read-only file system

Expected results:
=================
It should throw appropriate error, ENOSPC.

-

--- Additional comment from Worker Ant on 2018-03-27 20:13:23 EDT ---

REVIEW: https://review.gluster.org/19781 (afr: add quorum checks in pre-op)
posted (#1) for review on master by Ravishankar N

--- Additional comment from Worker Ant on 2018-04-05 08:22:41 EDT ---

COMMIT: https://review.gluster.org/19781 committed in master by "Pranith Kumar
Karampuri" <pkarampu at redhat.com> with a commit message- afr: add quorum checks
in pre-op

Problem:
We seem to be winding the FOP if pre-op did not succeed on quorum bricks
and then failing the FOP with EROFS since the fop did not meet quorum.
This essentially masks the actual error due to which pre-op failed. (See
BZ).

Fix:
Skip FOP phase if pre-op quorum is not met and go to post-op.

Fixes: 1561129

Change-Id: Ie58a41e8fa1ad79aa06093706e96db8eef61b6d9
fixes: bz#1561129
Signed-off-by: Ravishankar N <ravishankar at redhat.com>

--- Additional comment from Ravishankar N on 2018-04-16 06:28:26 EDT ---

Moving it back to POST since one more patch for eager lock fixes is needed to
verify the steps in the bug description without causing the mount to crash.

--- Additional comment from Worker Ant on 2018-04-16 06:29:43 EDT ---

REVIEW: https://review.gluster.org/19879 (afr: fixes to afr-eager locking)
posted (#1) for review on master by Ravishankar N

--- Additional comment from Worker Ant on 2018-04-18 03:49:38 EDT ---

COMMIT: https://review.gluster.org/19879 committed in master by "Pranith Kumar
Karampuri" <pkarampu at redhat.com> with a commit message- afr: fixes to afr-eager
locking

1. If pre-op fails on all bricks,set lock->release to true in
afr_handle_lock_acquire_failure so that the GF_ASSERT in afr_unlock() does not
crash.

2. Added a missing 'return' after handling pre-op failure in
afr_transaction_perform_fop(), fixing a use-after-free issue.

Change-Id: If0627a9124cb5d6405037cab3f17f8325eed2d83
fixes: bz#1561129
Signed-off-by: Ravishankar N <ravishankar at redhat.com>

--- Additional comment from Shyamsundar on 2018-06-20 14:03:13 EDT ---

This bug is getting closed because a release has been made available that
should address the reported issue. In case the problem is still not fixed with
glusterfs-v4.1.0, please open a new bug report.

glusterfs-v4.1.0 has been announced on the Gluster mailinglists [1], packages
for several distributions should become available in the near future. Keep an
eye on the Gluster Users mailinglist [2] and the update infrastructure for your
distribution.

[1] http://lists.gluster.org/pipermail/announce/2018-June/000102.html
[2] https://www.gluster.org/pipermail/gluster-users/


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1554291
[Bug 1554291] When storage reserve limit is reached, appending data to an
existing file throws EROFS error
https://bugzilla.redhat.com/show_bug.cgi?id=1561129
[Bug 1561129] When storage reserve limit is reached, appending data to an
existing file throws EROFS error
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.