Bug 1472957

Summary: [Docs][Admin] Update documentation to direct users to example files for AD configuration
Product: Red Hat Enterprise Virtualization Manager Reporter: Anitha Udgiri <audgiri>
Component: DocumentationAssignee: Tahlia Richardson <trichard>
Status: CLOSED CURRENTRELEASE QA Contact: Byron Gravenorst <bgraveno>
Severity: high Docs Contact:
Priority: medium    
Version: 4.2.3CC: audgiri, lbopf, lsurette, mkalinin, omachace, rbalakri, Rhev-m-bugs, srevivo
Target Milestone: ovirt-4.2.5Keywords: Triaged
Target Release: ---Flags: lsvaty: testing_plan_complete-
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: docs-accepted
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2018-07-12 01:14:48 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Docs RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Anitha Udgiri 2017-07-19 16:38:27 UTC
Description of problem:
With this version of RHV, ad.properties is using LDAP_MATCHING_RULE_IN_CHAIN to retrieve groups, and it doesn't  fetch domain local groups from different domains.

So, for multi domain AD environments, Customers need to change 
the 'include = <ad.properties>' to 'include = <ad-recursive.properties>'.


Please provide appropriate documentation in Customer facing docs for this change.

Comment 2 Martin Perina 2017-07-20 10:27:17 UTC
Anitha, this is strange because we have not supported using domain local groups in multi-domain forrest at all before BZ1336707, and BZ1336707 was fixed in aaa-ldap contained in RHV 4.1.0. So:

1. If customers used domain local groups in managed-domains and they've only moved to aaa-ldap with no AD changes (especially manage-domains didn't support forrest only single domain, so they have configured aaa-ldap only to correct single domain servers), they should not face the issue

2. Customers used only single domain setup and after upgrade to 4.1 now they have changed their AD to multi-domain forrest. If so then they faced that issue.

Anyway it needs to mentioned that using domain local groups for group membership inside multi-domains forrest in not recommended by Microsoft.

Ondro, could you please provide correct steps to alter configuration and also please update DocText also in BZ1336707.

Comment 3 Lucy Bopf 2017-08-08 00:49:25 UTC
Ondra, can you please provide the steps as requested by Martin in comment 2?

Comment 4 Ondra Machacek 2017-08-08 07:59:10 UTC
The difference is described in profile itself here:

https://github.com/oVirt/ovirt-engine-extension-aaa-ldap/blob/master/profiles/ad.properties#L22

If you need any more information, please let me know.

Comment 5 Lucy Bopf 2017-08-29 00:36:03 UTC
Anitha, can you take a look at the link Ondra provided in comment 4, and let us know whether that information resolves your questions. If not, what else is required?

Comment 6 Anitha Udgiri 2017-10-02 14:40:07 UTC
(In reply to Lucy Bopf from comment #5)
> Anitha, can you take a look at the link Ondra provided in comment 4, and let
> us know whether that information resolves your questions. If not, what else
> is required?

Lucy,
    Apologies for the delay in responding here. The information in the link is what exactly customers need to know. I doubt if customers ever open this file to peek inside to read this information. 
The best thing would be to either direct customers to read this info or get this info into the published documentation.

Comment 7 Lucy Bopf 2017-10-31 01:05:48 UTC
Thanks, Anitha! That's very helpful.

I discussed this with Derek, and we agreed that pointing users to the file would be the most achievable solution, and would mean the file can be updated without requiring a change in the documentation.

Updating the summary to reflect the new action.

Comment 11 Lucy Bopf 2018-06-29 04:08:11 UTC
Apologies for the delay. I realized that the BZ was not on the 'Documentation' component, so I have now moved it.

Accepting into the 4.2.5 program, and assigning to Tahlia for review.

Comment 13 Byron Gravenorst 2018-07-12 00:00:42 UTC
Reviewed and merged.