Bug 1432518 - AD with external auth, When doing group lookup for user group SID number is displayed instead of Group name
Summary: AD with external auth, When doing group lookup for user group SID number is d...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: Appliance
Version: 5.6.0
Hardware: Unspecified
OS: Unspecified
medium
high
Target Milestone: GA
: 5.9.0
Assignee: Satoe Imaishi
QA Contact: Matt Pusateri
URL:
Whiteboard: auth:externalauth:ad
Depends On: 1449729
Blocks: 1477194 1477195
TreeView+ depends on / blocked
 
Reported: 2017-03-15 14:42 UTC by Matt Pusateri
Modified: 2021-03-11 15:03 UTC (History)
16 users (show)

Fixed In Version: 5.9.0.1
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1477194 1477195 (view as bug list)
Environment:
Last Closed: 2018-03-06 15:05:41 UTC
Category: ---
Cloudforms Team: ---
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
AD numeric group sid (154.90 KB, image/png)
2017-03-15 14:42 UTC, Matt Pusateri
no flags Details
Sid's displayed in 5.7.2.0 (205.77 KB, image/png)
2017-03-29 14:02 UTC, Matt Pusateri
no flags Details

Description Matt Pusateri 2017-03-15 14:42:13 UTC
Created attachment 1263350 [details]
AD numeric group sid

Description of problem:

When doing group lookup for user group SID number is displayed instead of Group name.  We noticed this in the UI and in dbus-send, then it eventually works. We duplicated this on multiple appliances(which are gone)  Must be a race condition, Alberto debugged with me and witnessed it.  But it fixed itself without any changes.  Capturing this for future reference. 

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

How reproducible:


Steps to Reproduce:
1. Configure external auth for AD.
2. See Groups are numeric sid, not name
3.

Actual results:


Expected results:


Additional info:

Comment 2 Matt Pusateri 2017-03-29 14:01:54 UTC
Verified still an issue in 5.7.2.0

Comment 3 Matt Pusateri 2017-03-29 14:02:38 UTC
Created attachment 1267308 [details]
Sid's displayed in 5.7.2.0

Comment 11 Joe Vlcek 2017-04-24 18:59:08 UTC
Because this issue is not reproducible by engineering and dbus-send produces the same failure it is likely an SSSD issue.

A quick search of Bugzilla produces some possible causes, including this one:
https://bugzilla.redhat.com/show_bug.cgi?id=1072995

and this Fedora upstream issue:
https://pagure.io/SSSD/sssd/issue/2274

The best way to proceed would be to enable debug logging for SSSD, by adding:
"debug_level = 9" to each section of the sssd.conf file on a machine where this
issue is observed.

If this is an SSSD timing related issue the SSSD cache would need to be cleared
between SSSD restarts.

I full sssd restart "flush" can be achieved with the following commands:

systemctl stop sssd
sss_cache -E
rm -f /var/lib/sss/db/*
rm -f /var/log/sssd/sssd*.log
systemctl restart sssd

Please provide sssd logs on a system where this is observed.

Comment 13 Matt Pusateri 2017-05-02 19:16:30 UTC
[root@dhcp-8-197-223 sssd]# dbus-send --print-reply --system --dest=org.freedesktop.sssd.infopipe /org/freedesktop/sssd/infopipe org.freedesktop.sssd.infopipe.GetUserGroups string:test-user1
method return sender=:1.375 -> dest=:1.378 reply_serial=2
   array [
      string "s-1-5-21-2347313050-3178207562-1725954654-513.bos.redhat.com"
      string "s-1-5-21-2347313050-3178207562-1725954654-1140.bos.redhat.com"
      string "s-1-5-21-2347313050-3178207562-1725954654-1157.bos.redhat.com"
      string "s-1-5-21-2347313050-3178207562-1725954654-1176.bos.redhat.com"
   ]


Just happened on  Version: 5.8.0.12-rc1

Comment 15 Joe Vlcek 2017-05-02 19:57:38 UTC
Seems like we are hitting an SSSD issue

Comment 18 Sumit Bose 2017-05-03 14:24:48 UTC
Yes, this might be an SSSD issue. At some point group might be written to the cache of SSSD with the SID as name. But at the same time the object should be marked as expired to make sure the right name is looked up before sending a result to the client.

Please save the cache file /var/lib/sss/db/cache_* when you see the output with the SIDs and attach it to this ticket for further inspection.

HTH

bye,
Sumit

Comment 19 Jakub Hrozek 2017-05-03 14:28:24 UTC
(In reply to Sumit Bose from comment #18)
> Yes, this might be an SSSD issue. At some point group might be written to
> the cache of SSSD with the SID as name. But at the same time the object
> should be marked as expired to make sure the right name is looked up before
> sending a result to the client.
> 
> Please save the cache file /var/lib/sss/db/cache_* when you see the output
> with the SIDs and attach it to this ticket for further inspection.
> 
> HTH
> 
> bye,
> Sumit

btw I can reproduce this locally.

Joe, since this is an SSSD issue, is it OK to move this bugzilla to the sssd component or should we file a new one against sssd and make this one blocked by the sssd one?

Comment 22 Joe Vlcek 2017-05-03 15:32:07 UTC
Sumit and Jakub, Thank you for the help! JoeV

Comment 23 Jakub Hrozek 2017-05-09 11:22:36 UTC
There is an upstream ticket now:
https://pagure.io/SSSD/sssd/issue/3392
and proposed patches:
https://github.com/SSSD/sssd/compare/master...jhrozek:ifp_groupnames

If you can let me know what sssd version you are running (rpm -q sssd-common) then I can build you a test package.

Comment 24 Jakub Hrozek 2017-05-10 16:25:02 UTC
btw this is also tracked in bugzilla with https://bugzilla.redhat.com/show_bug.cgi?id=1449729

Comment 25 Matt Pusateri 2017-05-15 20:53:51 UTC
sssd-common-1.14.0-43.el7_3.14.x86_64

Comment 26 Jakub Hrozek 2017-05-18 10:56:48 UTC
(In reply to Matt Pusateri from comment #25)
> sssd-common-1.14.0-43.el7_3.14.x86_64

(Sorry for the late reply)

This is the 7.3 baseline. Since the underlying code changed quite a bit in 7.4, the patches don't apply. But since the CFME-related enhancements were coded up on the 7.4 baseline, I would like to ask for testing using the following 7.4 test build instead of spending time on backporting the patches:
https://brewweb.engineering.redhat.com/brew/taskinfo?taskID=13225562

Comment 27 Joe Vlcek 2017-06-06 15:31:52 UTC
(In reply to Jakub Hrozek from comment #26)
> (In reply to Matt Pusateri from comment #25)
> > sssd-common-1.14.0-43.el7_3.14.x86_64
> 
> (Sorry for the late reply)
> 
> This is the 7.3 baseline. Since the underlying code changed quite a bit in
> 7.4, the patches don't apply. But since the CFME-related enhancements were
> coded up on the 7.4 baseline, I would like to ask for testing using the
> following 7.4 test build instead of spending time on backporting the patches:
> https://brewweb.engineering.redhat.com/brew/taskinfo?taskID=13225562

Matt,

Can you please confirm this test build so we can close this BZ out if it works?

Thank you, JoeV

Comment 28 Matt Pusateri 2017-06-19 13:08:15 UTC
Confirmed it resolves the issue.

Comment 29 Joe Vlcek 2017-06-19 13:49:00 UTC
(In reply to Matt Pusateri from comment #28)
> Confirmed it resolves the issue.

Can this be move to "Verified" then?

Comment 30 Matt Pusateri 2017-06-19 13:55:45 UTC
How are we going to track that it actually makes it into the appliance?  I realize you don't want this in your queue.  But for me it's not verified until it's actually in a build.  I've verified the upstream bug.

Comment 32 Felix Dewaleyne 2017-07-31 12:19:30 UTC
attaching another affected customer - we also identified a sssd bug - sssd 1.15 seems like it would resolve it. It's in the beta channel only however right now.

bz#1449729 seems to cover the sssd issue

Comment 39 Matt Pusateri 2017-10-25 15:50:15 UTC
Verified on 5.9.0.2 - External Auth - AD


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