Bug 842498

Summary: ISC-DHCP server with LDAP configuration does not respect FailOver options
Product: [Fedora] Fedora Reporter: William Brown <william>
Component: dhcpAssignee: Jiri Popelka <jpopelka>
Status: CLOSED UPSTREAM QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 17CC: jpopelka, thozza, william
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-05-10 07:19:08 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
Patch to allow FailOverPeer's to be configured with LDAP backend.
none
Patch without the ldif formatted schema changes. none

Description William Brown 2012-07-24 03:36:25 UTC
Created attachment 599901 [details]
Patch to allow FailOverPeer's to be configured with LDAP backend.

Description of problem:
ISC-DHCP server with an LDAP backend does not properly utilise the dhcpFailOverPeer object to generate it's configuration. Additionally, there was no way to specifiy a relationship between a dhcpPool and a dhcpFailOverPeer configuration. The attached patch corrects the issue, and allows dhcpFailOverPeer options to be read and generated. 

The attached patch adds a dhcpFailOverPeer parser, modifies the dhcpPool parser to accept dhcpFailOverDN options, and modifies the LDAP schema to all a dhcpFailOverDN object to exist in a dhcpPool. Note that this dhcpFailOverDN is *NOT* followed by the configuration tool like the dhcpFailOverPeerDN options - This is merely a reference so that the pool can generate the correct peer name. Finally, I have also added a copy of the ldap schema in ldif format for 389ds and OpenLDAP to utilise.

Comment 1 Jiri Popelka 2012-07-24 11:20:29 UTC
Thank you William.

I'm however not sure I want to maintain such a big patch.
As with bug #838400, please send it upstream and let me know what's the assigned ID.

Comment 2 William Brown 2012-07-24 12:06:43 UTC
The majority of this patch is the LDAP ldif formatted schema, not code changes which makes it appear larger than it really is. I can split this patch into 3 patches for each of the modified files if you would prefer.

Comment 3 Jiri Popelka 2012-07-24 12:11:48 UTC
All in one patch is better.

Comment 4 William Brown 2012-07-24 12:23:49 UTC
I have had a quick rethink and have decided that I will submit the change to ldif formatted schema at a later time. Does this patch seem more acceptable with that in mind?

Comment 5 William Brown 2012-07-24 12:24:46 UTC
Created attachment 600007 [details]
Patch without the ldif formatted schema changes.

Comment 6 William Brown 2012-07-25 02:32:08 UTC
[ISC-Bugs #30402]