Bug 1436411 - Unrecognized filesystems (i.e. btrfs, zfs) log many errors about "getinode size"
Summary: Unrecognized filesystems (i.e. btrfs, zfs) log many errors about "getinode size"
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: glusterd
Version: 3.10
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Niels de Vos
QA Contact:
URL: http://lists.gluster.org/pipermail/gl...
Whiteboard:
Depends On: 1433425
Blocks:
TreeView+ depends on / blocked
 
Reported: 2017-03-27 21:07 UTC by Niels de Vos
Modified: 2017-04-05 00:02 UTC (History)
1 user (show)

Fixed In Version: glusterfs-3.10.1
Doc Type: If docs needed, set a value
Doc Text:
Clone Of: 1433425
Environment:
Last Closed: 2017-04-05 00:02:39 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Description Niels de Vos 2017-03-27 21:07:02 UTC
+++ This bug was initially created as a clone of Bug #1433425 +++

Description of problem:
hi, I have replicated glusterfs on 3 nodes with zfs, the logs are flooded
with inodes error

[2017-03-06 16:24:15.019386] E [MSGID: 106419]
[glusterd-utils.c:5458:glusterd_add_inode_size_to_dict] 0-management: could
not find (null) to getinode size for zclei21/01 (zfs): (null) package
missing?

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

How reproducible:
100%

Steps to Reproduce:
1. configure a volume with bricks on btrfs or zfs
2. run 'gluster volume status <VOLNAME> detail'

Actual results:
Upon each execution of 'gluster volume status <VOLNAME> detail' log entries are added.

Expected results:
No logs, or maybe once.

Additional info:
Reported on http://lists.gluster.org/pipermail/gluster-users/2017-March/030189.html

--- Additional comment from Worker Ant on 2017-03-17 16:53:58 CET ---

REVIEW: https://review.gluster.org/16867 (glusterd: support filesystems with dynamic inode sizes) posted (#2) for review on master by Niels de Vos (ndevos)

--- Additional comment from Worker Ant on 2017-03-27 18:57:34 CEST ---

COMMIT: https://review.gluster.org/16867 committed in master by Atin Mukherjee (amukherj) 
------
commit 12921693b572f642156d3167d1c92d3449dfc8ec
Author: Niels de Vos <ndevos>
Date:   Tue Mar 7 18:01:25 2017 +0100

    glusterd: support filesystems with dynamic inode sizes
    
    btrfs and zfs are two filesystems that do not have fixed sizes for
    inodes. Instead of logging an error, skip checking and mark the size as
    "N/A" like other properties that can not be reported.
    
    The error message that was reported by users on the mailinglist shows up
    like:
    
      [glusterd-utils.c:5458:glusterd_add_inode_size_to_dict] 0-management: could not find (null) to getinode size for /dev/vdb (btrfs): (null) package missing?
    
    Change-Id: Ib10b7a3669f2f4221075715d9fd44ce1ffc35324
    Reported-by: Arman Khalatyan <arm2arm>
    URL: http://lists.gluster.org/pipermail/gluster-users/2017-March/030189.html
    BUG: 1433425
    Signed-off-by: Niels de Vos <ndevos>
    Reviewed-on: https://review.gluster.org/16867
    Smoke: Gluster Build System <jenkins.org>
    Reviewed-by: Atin Mukherjee <amukherj>
    Reviewed-by: Prashanth Pai <ppai>
    NetBSD-regression: NetBSD Build System <jenkins.org>
    CentOS-regression: Gluster Build System <jenkins.org>

Comment 1 Worker Ant 2017-03-27 21:08:59 UTC
REVIEW: https://review.gluster.org/16959 (glusterd: support filesystems with dynamic inode sizes) posted (#1) for review on release-3.10 by Niels de Vos (ndevos)

Comment 2 Worker Ant 2017-03-28 12:04:21 UTC
COMMIT: https://review.gluster.org/16959 committed in release-3.10 by Shyamsundar Ranganathan (srangana) 
------
commit 1d38219967ed8a5cb5f47c20e228c2a9c3484d83
Author: Niels de Vos <ndevos>
Date:   Tue Mar 7 18:01:25 2017 +0100

    glusterd: support filesystems with dynamic inode sizes
    
    btrfs and zfs are two filesystems that do not have fixed sizes for
    inodes. Instead of logging an error, skip checking and mark the size as
    "N/A" like other properties that can not be reported.
    
    The error message that was reported by users on the mailinglist shows up
    like:
    
      [glusterd-utils.c:5458:glusterd_add_inode_size_to_dict] 0-management: could not find (null) to getinode size for /dev/vdb (btrfs): (null) package missing?
    
    Cherry picked from commit 12921693b572f642156d3167d1c92d3449dfc8ec:
    > Change-Id: Ib10b7a3669f2f4221075715d9fd44ce1ffc35324
    > Reported-by: Arman Khalatyan <arm2arm>
    > URL: http://lists.gluster.org/pipermail/gluster-users/2017-March/030189.html
    > BUG: 1433425
    > Signed-off-by: Niels de Vos <ndevos>
    > Reviewed-on: https://review.gluster.org/16867
    > Smoke: Gluster Build System <jenkins.org>
    > Reviewed-by: Atin Mukherjee <amukherj>
    > Reviewed-by: Prashanth Pai <ppai>
    > NetBSD-regression: NetBSD Build System <jenkins.org>
    > CentOS-regression: Gluster Build System <jenkins.org>
    
    Change-Id: Ib10b7a3669f2f4221075715d9fd44ce1ffc35324
    Reported-by: Arman Khalatyan <arm2arm>
    BUG: 1436411
    Signed-off-by: Niels de Vos <ndevos>
    Reviewed-on: https://review.gluster.org/16959
    Smoke: Gluster Build System <jenkins.org>
    NetBSD-regression: NetBSD Build System <jenkins.org>
    CentOS-regression: Gluster Build System <jenkins.org>
    Reviewed-by: Shyamsundar Ranganathan <srangana>

Comment 3 Shyamsundar 2017-04-05 00:02:39 UTC
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-3.10.1, please open a new bug report.

glusterfs-3.10.1 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/gluster-users/2017-April/030494.html
[2] https://www.gluster.org/pipermail/gluster-users/


Note You need to log in before you can comment on or make changes to this bug.