Bug 1273652

Summary: RHEL6 sssd spec file disables building NFSv4 idmapd plugin
Product: Red Hat Enterprise Linux 6 Reporter: James Ralston <ralston>
Component: sssdAssignee: SSSD Maintainers <sssd-maint>
Status: CLOSED ERRATA QA Contact: Namita Soman <nsoman>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 6.7CC: grajaiya, jgalipea, jhrozek, lslebodn, mkosek, mzidek, pbrezina, preichl, sgoveas
Target Milestone: rc   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: sssd-1.13.2-1.el6 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-05-10 20:21:29 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
changes to spec file need to build idmapd plugin none

Description James Ralston 2015-10-20 21:34:35 UTC
Description of problem:

The %configure macro in the sssd.spec file for RHEL6 deliberately includes the --without-nfsv4-idmapd-plugin flag, which disables building the /usr/lib64/libnfsidmap/sss.so idmapd plugin. This makes it impossible for RHEL6 NFSv4 clients to leverage sssd to perform NFSv4 id mapping.

It contrast, the sssd.spec file for RHEL7 builds and packages the /usr/lib64/libnfsidmap/sss.so idmapd plugin, along with several other plugins that are excluded from the RHEL6 sssd packages.

We still have many RHEL6 clients that we need to configure as NFSv4 clients. We need the /usr/lib64/libnfsidmap/sss.so idmapd plugin.

I don't understand why the idmapd plugin (as well as other plugins) were excluded from the RHEL6 sssd packages. The reason can't be because they are too old to work effectively, because the latest RHEL6 sssd packages (1.12.2) are currently more recent than the latest RHEL7 sssd packages (1.12.4).

For now, we've rebuilt the sssd packages from source, patching the spec file to properly build and package the idmapd plugin. But we are highly disinterested in maintaining our own local fork of sssd for RHEL6. Please update the RHEL6 packages to include the idmapd plugin.

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

0:sssd-common-1.12.4-47.el6.x86_64

Comment 1 James Ralston 2015-10-20 21:38:38 UTC
Created attachment 1084930 [details]
changes to spec file need to build idmapd plugin

Here are the (trivial) changes we made to the spec file in order to build the idmapd plugin.

Comment 3 James Ralston 2015-10-20 22:02:21 UTC
Cross-filed as support case 01527018.

Comment 4 Lukas Slebodnik 2015-10-21 06:08:40 UTC
(In reply to James Ralston from comment #0)
> Description of problem:
> 
> The %configure macro in the sssd.spec file for RHEL6 deliberately includes
> the --without-nfsv4-idmapd-plugin flag, which disables building the
> /usr/lib64/libnfsidmap/sss.so idmapd plugin. This makes it impossible for
> RHEL6 NFSv4 clients to leverage sssd to perform NFSv4 id mapping.
> 
> It contrast, the sssd.spec file for RHEL7 builds and packages the
> /usr/lib64/libnfsidmap/sss.so idmapd plugin, along with several other
> plugins that are excluded from the RHEL6 sssd packages.
> 
> We still have many RHEL6 clients that we need to configure as NFSv4 clients.
> We need the /usr/lib64/libnfsidmap/sss.so idmapd plugin.
> 
> I don't understand why the idmapd plugin (as well as other plugins) were
> excluded from the RHEL6 sssd packages. The reason can't be because they are
> too old to work effectively, because the latest RHEL6 sssd packages (1.12.2)
> are currently more recent than the latest RHEL7 sssd packages (1.12.4).
> 
> For now, we've rebuilt the sssd packages from source, patching the spec file
> to properly build and package the idmapd plugin. But we are highly
> disinterested in maintaining our own local fork of sssd for RHEL6. Please
> update the RHEL6 packages to include the idmapd plugin.
> 
If you do not want to maintain packages yourself than you can use back-ported version from Fedora 21 as a temporary solution.
https://copr.fedoraproject.org/coprs/lslebodn/sssd-1-12/

Comment 5 Jakub Hrozek 2015-10-21 10:50:26 UTC
Dev ack since this is just a specfile change.

Comment 6 Jakub Hrozek 2015-10-21 10:51:42 UTC
I tried to find the reason we explicitly disable the plugin but couldn't...I suspect it's because we were trying to minimize the amount of new features added into 6.7 since it's already pretty late even in the RHEL cycle..

Comment 7 Jakub Hrozek 2015-10-28 19:03:54 UTC
James, thanks for filing this bug. It's tentatively scheduled for 6.8.

Comment 8 James Ralston 2015-10-29 02:24:10 UTC
Jakub: understood; thanks.

We don't have an issue with maintaining a local fork, as long as we know that we can eventually reintegrate with upstream.

Comment 9 Jakub Hrozek 2015-12-11 16:00:50 UTC
FYI, here is a test repo: https://copr.fedoraproject.org/coprs/jhrozek/SSSD-6.8-preview/

Comment 11 Jakub Hrozek 2016-02-18 10:33:18 UTC
To verify the bug, just make sure the sssd-common subpackage contains the sss.so file under libdir/libnfsidmap.

We don't test the functionality itself IIRC and moreover, it was tested by a user/customer who also reported a leak in the plugin. so we already know the plugin works :)

Comment 12 Steeve Goveas 2016-02-18 10:57:23 UTC
Verified in version
[root@apollo ~]# rpm -q sssd-common
sssd-common-1.13.3-15.el6.x86_64

[root@apollo ~]# rpm -ql sssd-common | grep sss.so
/usr/lib64/libnfsidmap/sss.so

[root@apollo ~]# file /usr/lib64/libnfsidmap/sss.so
/usr/lib64/libnfsidmap/sss.so: ELF 64-bit LSB shared object, x86-64, version 1 (SYSV), dynamically linked, stripped

[root@apollo ~]# cat /etc/redhat-release 
Red Hat Enterprise Linux Server release 6.8 Beta (Santiago)

Comment 14 errata-xmlrpc 2016-05-10 20:21:29 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://rhn.redhat.com/errata/RHBA-2016-0782.html