Bug 1473399 - [RFE]: rgw: nfs-ganesha store v4 clientids in RADOS
Summary: [RFE]: rgw: nfs-ganesha store v4 clientids in RADOS
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Ceph Storage
Classification: Red Hat Storage
Component: RGW
Version: 3.0
Hardware: All
OS: All
low
urgent
Target Milestone: rc
: 3.0
Assignee: Matt Benjamin (redhat)
QA Contact: ceph-qe-bugs
Bara Ancincova
URL:
Whiteboard:
Depends On:
Blocks: 1494421
TreeView+ depends on / blocked
 
Reported: 2017-07-20 17:25 UTC by Matt Benjamin (redhat)
Modified: 2017-12-05 23:36 UTC (History)
11 users (show)

Fixed In Version: RHEL: nfs-ganesha-2.5.2-2.el7cp Ubuntu: nfs-ganesha_2.5.2-3redhat1xenial
Doc Type: Enhancement
Doc Text:
.NFSv4 recovery state data can be stored in Ceph RADOS NFS version 4 (NFSv4) recovery state data such as, `clientids`, can now be stored in Ceph RADOS objects. This change increases the resilience of clustered NFS servers exposing Ceph storage resources.
Clone Of:
Environment:
Last Closed: 2017-12-05 23:36:49 UTC
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2017:3387 0 normal SHIPPED_LIVE Red Hat Ceph Storage 3.0 bug fix and enhancement update 2017-12-06 03:03:45 UTC

Description Matt Benjamin (redhat) 2017-07-20 17:25:35 UTC
This bug contains the following actions

1. absorb the following 2 upstream nfs-ganesha commits from Gui Hecheng <guihecheng.com>

commit 7a97788340772013dceb603dfee5e6f2a353bcb2
Author: Gui Hecheng <guihecheng.com>
Date:   Wed May 3 09:58:34 2017 +0800

    cmake: make modulized recovery backends compile as modules
    
    - add USE_RADOS_RECOV option for new rados kv backend
    - keep original fs backend as default
    
    Change-Id: I26c2c4f9a433e6cd70f113fa05194d6817b9377a
    Signed-off-by: Gui Hecheng <guihecheng.com>

commit c3ff11cca87c3963ea44beff9f5753fbb2150b7a
Author: Gui Hecheng <guihecheng.com>
Date:   Thu Mar 30 10:50:18 2017 +0800

    SAL: introduce new recovery backend based on rados kv store
    
    Use rados OMAP API to implement a kv store for client tracking data
    
    Change-Id: I1aec1e110a2fba87ae39a1439818a363b6cfc822
    Signed-off-by: Gui Hecheng <guihecheng.com>

Comment 2 Harish NV Rao 2017-07-25 14:13:51 UTC
@Matt, could you please provide more information on this bz and the steps to verify the fix?

Comment 4 Harish NV Rao 2017-07-28 10:57:00 UTC
(In reply to Harish NV Rao from comment #2)
> @Matt, could you please provide more information on this bz and the steps to
> verify the fix?

@Matt, A gentle reminder

Comment 5 Harish NV Rao 2017-08-02 08:57:45 UTC
(In reply to Harish NV Rao from comment #4)
> (In reply to Harish NV Rao from comment #2)
> > @Matt, could you please provide more information on this bz and the steps to
> > verify the fix?
> 
> @Matt, A gentle reminder

@Matt, A gentle reminder

Comment 6 Drew Harris 2017-08-30 15:22:40 UTC
This is required OSP 13 manilla support. Does not need to be tested for 3.0 GA.

Comment 10 Daniel Gryniewicz 2017-09-07 14:04:39 UTC
There are actually 4 commits in this sequence, not 2.  Here they are:

commit 227e63a3f75ac8cec73533079c2396dc8b3e108f
Author:     Gui Hecheng <guihecheng.com>
AuthorDate: Thu May 4 22:43:17 2017 +0800
Commit:     Daniel Gryniewicz <dang>
CommitDate: Thu Sep 7 08:22:57 2017 -0400

    config: add new config options for rados_kv recovery backend
    
    - new config block: RADOS_KV
    - new option: ceph_conf, userid, pool
    
    Change-Id: Id44afa70e8b5adb2cb2b9d48a807b0046f604f30
    Signed-off-by: Gui Hecheng <guihecheng.com>

commit eaaaa8dc4038f1653ec16964c5a247c8be9fc1cf
Author:     Gui Hecheng <guihecheng.com>
AuthorDate: Wed May 3 09:58:34 2017 +0800
Commit:     Daniel Gryniewicz <dang>
CommitDate: Thu Sep 7 08:21:56 2017 -0400

    cmake: make modulized recovery backends compile as modules
    
    - add USE_RADOS_RECOV option for new rados kv backend
    - keep original fs backend as default
    
    Change-Id: I26c2c4f9a433e6cd70f113fa05194d6817b9377a
    Signed-off-by: Gui Hecheng <guihecheng.com>

commit 1a6e79dc748ecf172b7eb77a89f7ab093f482dfc
Author:     Gui Hecheng <guihecheng.com>
AuthorDate: Thu Mar 30 10:50:18 2017 +0800
Commit:     Daniel Gryniewicz <dang>
CommitDate: Thu Sep 7 08:20:50 2017 -0400

    SAL: introduce new recovery backend based on rados kv store
    
    Use rados OMAP API to implement a kv store for client tracking data
    
    Change-Id: I1aec1e110a2fba87ae39a1439818a363b6cfc822
    Signed-off-by: Gui Hecheng <guihecheng.com>

commit 75e568201a1b7d5b487fe529db62dd672f41695a
Author:     Gui Hecheng <guihecheng.com>
AuthorDate: Thu Mar 30 10:44:25 2017 +0800
Commit:     Daniel Gryniewicz <dang>
CommitDate: Thu Sep 7 08:20:34 2017 -0400

    SAL: extract fs logic from nfs4_recovery
    
    This is a prepare patch for modulized recovery backends.
    - define recovery apis: struct nfs_recovery_backend
    - define hooks for recovery_fs module
    
    Change-Id: I45523ef9a0e6f9a801fc733b095ba2965dd8751b
    Signed-off-by: Gui Hecheng <guihecheng.com>


There's only one slight conflict, in the spec file.

Comment 22 errata-xmlrpc 2017-12-05 23:36:49 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2017:3387


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