#2082 Implement new API for DNS records
Closed: Fixed None Opened 12 years ago by mkosek.

https://bugzilla.redhat.com/show_bug.cgi?id=766073

This ticket should track an implementation of new per record interface for DNS plugin. The interface will be based on a proposal acked on the freeipa-devel list.

The design proposal was tracked in ticket #1766.


Patch freeipa-mkosek-172-rework-dnsrecord-processing.patch sent for review

Patch freeipa-mkosek-173-create-per-type-dns-api.patch sent for review

I created a design page fpr new DNS API on freeipa.org space: http://www.freeipa.org/page/IPAv3_DNS_Interface_Design

Patch freeipa-mkosek-182-create-per-type-dns-api.patch sent for review

Patch 182 implements a different approach for DNS API based on new options instead of new commands. Details of this approach were sent to freeipa-devel mailing list:

http://www.redhat.com/archives/freeipa-devel/2011-December/msg00278.html

The patches are not reviewed yet, moving to next milestone.

master:[[BR]]
52ea3a6[[BR]]
1f36ab1[[BR]]
2188dae[[BR]]
a22620c[[BR]]

ipa-2-2:[[BR]]
f8881bb[[BR]]
bff9101[[BR]]
688f630[[BR]]
5539577[[BR]]

Fixed a build on 32b systems:

master: 485c3bb[[BR]]
ipa-2-2: e640370

Patch freeipa-mkosek-191-fix-maxvalue-in-dns-plugin.patch sent for review

Reviewed these changes:
http://pvoborni.fedorapeople.org/dns/#dns=dnsrecord&identity=dns&navigation=identity&dnsrecord-facet=default&dnsrecord-pkey=_kerberos&dnszone-pkey=test.com

All changes look solid. The only feedback I have is the 'edit' column in the tables should be moved to be a single button in-line with the add/delete controls.

Metadata Update from @mkosek:
- Issue assigned to mkosek
- Issue set to the milestone: FreeIPA 2.2 Core Effort - 2012/01

7 years ago

Login to comment on this ticket.

Metadata