[Bugs] [Bug 1396332] New: Refresh config fails while exporting subdirectories within a volume

[bugzilla at redhat.com]

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

            Bug ID: 1396332
           Summary: Refresh config fails while exporting subdirectories
                    within a volume
           Product: GlusterFS
           Version: 3.9
         Component: common-ha
          Keywords: Triaged
          Severity: high
          Assignee: jthottan at redhat.com
          Reporter: jthottan at redhat.com
                CC: aloganat at redhat.com, bugs at gluster.org,
                    jthottan at redhat.com, kkeithle at redhat.com,
                    mzywusko at redhat.com, ndevos at redhat.com,
                    rhs-bugs at redhat.com, skoduri at redhat.com,
                    storage-qa-internal at redhat.com
        Depends On: 1389661, 1389746



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

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

Description of problem:
Refresh config failed while exporting subdirectories within a volume.

Version-Release number of selected component (if applicable):
nfs-ganesha-2.4.0-2.el7rhgs.x86_64
glusterfs-server-3.8.4-3.el7rhgs.x86_64

How reproducible:
Always

Steps to Reproduce:
1. Create a ganesha cluster, create a volume and enable nfs-ganesha on that
volume.
2. Mount the volume and create subdirectories.
3. Edit the export.conf file to export subdirectories within the volume.
    conf file path:
/var/run/gluster/shared_storage/nfs-ganesha/exports/export.testvol1.conf
4. Run refresh config.
    /usr/libexec/ganesha/ganesha-ha.sh --refresh-config
/var/run/gluster/shared_storage/nfs-ganesha/ testvol1

Actual results:

Refresh config failed.

Expected results:
Refresh config should succeed and subdirectory should be exported.

Additional info:

Edited export.conf file:
-------------------------

EXPORT{
      Export_Id = 2;
      Path = "/testvol1/user2/dir0";
      FSAL {
           name = GLUSTER;
           hostname="localhost";
          volume="testvol1";
          volpath="/user2/dir0";
           }
      Access_type = RW;
      Disable_ACL = true;
      Squash="No_root_squash";
      Pseudo="/testvol1_ps";
      Protocols = "3", "4" ;
      Transports = "UDP","TCP";
      SecType = "sys";
     }


Error message in command prompt:
--------------------------------

[root at dhcp47-155 ~]# /usr/libexec/ganesha/ganesha-ha.sh --refresh-config
/var/run/gluster/shared_storage/nfs-ganesha/ testvol1
Error: refresh-config failed on dhcp46-101.
[root at dhcp47-155 ~]# 


ganesha.log log snippet:
-------------------------

28/10/2016 11:36:09 : epoch 40270000 : dhcp46-101.lab.eng.blr.redhat.com :
ganesha.nfsd-22434[dbus_heartbeat] dbus_message_entrypoint :DBUS :MAJ :Method
(RemoveExport) on (org.ganesha.nfsd.exportmgr) failed: name =
(org.freedesktop.DBus.Error.InvalidArgs), message = (lookup_export failed with
Export id not found)

--- Additional comment from Red Hat Bugzilla Rules Engine on 2016-10-28
03:00:08 EDT ---

This bug is automatically being proposed for the current release of Red Hat
Gluster Storage 3 under active development, by setting the release flag
'rhgs‑3.2.0' to '?'. 

If this bug should be proposed for a different release, please manually change
the proposed release flag.

--- Additional comment from Arthy Loganathan on 2016-10-28 03:00 EDT ---

--- Additional comment from Worker Ant on 2016-10-28 08:15:48 EDT ---

REVIEW: http://review.gluster.org/15751 (ganesha/scripts : use export id for
dbus signals) posted (#1) for review on master by jiffin tony Thottan
(jthottan at redhat.com)

--- Additional comment from Worker Ant on 2016-11-17 00:14:06 EST ---

REVIEW: http://review.gluster.org/15751 (ganesha/scripts : use export id for
dbus signals) posted (#2) for review on master by jiffin tony Thottan
(jthottan at redhat.com)

--- Additional comment from Worker Ant on 2016-11-17 16:17:34 EST ---

COMMIT: http://review.gluster.org/15751 committed in master by Kaleb KEITHLEY
(kkeithle at redhat.com) 
------
commit 2f70f47979d204d5ee29811f5f5fa61d0a52c3b1
Author: Jiffin Tony Thottan <jthottan at redhat.com>
Date:   Fri Oct 28 17:37:39 2016 +0530

    ganesha/scripts : use export id for dbus signals

    Currently for add export and update export parameter passed for executing
those signal
    is "PATH". This is based on assumption that volume name and PATH will
always be same.
    But it is wrong for subdir exports. The only reliable parameter in export
configuration
    file is "Export_Id".

    Change-Id: Ic63ff44ac7736e14502034b74beaae27292eddf9
    BUG: 1389746
    Signed-off-by: Jiffin Tony Thottan <jthottan at redhat.com>
    Reviewed-on: http://review.gluster.org/15751
    Smoke: Gluster Build System <jenkins at build.gluster.org>
    NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>
    CentOS-regression: Gluster Build System <jenkins at build.gluster.org>
    Reviewed-by: soumya k <skoduri at redhat.com>
    Reviewed-by: Kaleb KEITHLEY <kkeithle at redhat.com>


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1389661
[Bug 1389661] Refresh config fails while exporting subdirectories within a
volume
https://bugzilla.redhat.com/show_bug.cgi?id=1389746
[Bug 1389746] Refresh config fails while exporting subdirectories within a
volume
-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=1DowuHkVXB&a=cc_unsubscribe