On 17.6.2016 13:58, Martin Sehnoutka wrote:
> ACK

Thanks, pushed to master:
75d1dbabc3893deac89d36d56c9ff86d88ed3b55 Make task selection in
syncrepl_update() deterministic.
431b8c47b4c6a27a933c842ed875cb5155f968fb Avoid redundant sync_task_add() calls.
62b19b4d423164e4a0dfb00a13358611c86ed789 Rework hack for synchronous event
processing to make it reliable.
ca9551ac12d597867870332f667ee872eacd5749 Remove misleading zonemgr check from
task selection mechanism.
c24cae2626ca36909022a05d361996584ac82493 Remove unused
inst->local_settings("base") lookup from ldap_sync_search_entry.
cfde79fec8f6fc14b10fabed6280777b4a0de525 Support default TTL configuration at
zone level: dNSdefaultTTL attribute.
b1d31eeacd438950dfde38e55a284dfbc119bce8 Optimize idnsTemplateAttribute
processing.
3780a448e403ac525de322901c09cec83d49bfbc Detect unsupported combination of
idnsTemplateObject with other object classes.
0c7560b921805581b6a98e72c8bb0fa3672c82d8 Support dynamic record generation
using idnsTemplateAttribute.
73ef87f498839ea264b93513f6539618c3419376 Revive str.c:str_cat_char_len
function from git history.
adb38c70803401b8b7c28e4ff08696d5bb0ae67c Refactor ldap_parse_rrentry to make
settings system available to it.
3d3fbc6ef1fdba2e9184379ce28aefaadcbb03b5 Read
idnsSubstitutionVariable;ipalocation variable from server's config in LDAP.
aa9bed122a59064c9d0885e1f7401863335f6af2 Support fake_mname option in
per-server configuration in LDAP.
97e9d93cb0e7d0f72fe130c6bfa6a886bc609928 Read configuration from LDAP first to
guarantee its availability.
112f50a6d9d746753bb793437975fb895c1f0606 Refactor LDAP SyncRepl connection
handling to make it reusable.
a47a591225593243c89da7df1cf2a58cffc37653 Do not log "connection to the LDAP
server was lost" on forced reconnects.
9864fadfeecd6b13d7c98bd752b78fce039224c4 Fix interaction between root zone and
global forwarders.
ad9bb03a45f84b523ba5721998698c5f905f34d7 Parse idnsServerConfigObject and use
its values for forwarder configuration.
6cc8e7264cdf46242c8be06d7d6ed7519ac26e78 Add LDAP schema for per-server config
in LDAP.
5dd20013d1fd1ce964485b279f78c50ce7bbb416 Add server_ldap_settings layer to
tree of setting objects.
fdfb1a635824fe45bd7a8deb52442fc8cfad891d Do not apply forwarding configuration
for disabled master zones.
759f3d97329faf08447cc3195b29cb87c95c4e56 Untangle forwarder configuration from
LDAP code.
ee433742457950c138127e90fdbb7f614a3b43a9 Settings: Expose API for
setting_unset and setting_find to other modules.
6c40c6067340e845eccebb73793f43574d698997 Settings: Support enum
description->value mapping.
6a1e2512668e59f434e86cddc3f9320557723e1d Separate BIND config utilities from
ACL parsing.
f0de8168979520541eeb3186227bd4a4470e91e5 Fix in destroy_ldap_instance() caused
by uninitialized MetaLDAP.
f5b1da78950dc6a28934b2778f5a6853ba37019f Initialize empty zone iterator to
avoid potential crashes.

Petr^2 Spacek

> 
> Martin Sehnoutka
> 
> ----- Original Message -----
> From: "Petr Spacek" <pspa...@redhat.com>
> To: msehn...@redhat.com
> Sent: Wednesday, June 15, 2016 4:25:15 PM
> Subject: bind-dyndb-ldap reviews
> 
> Hello,
> 
> please review patches mentioned below.
> 
> Do not hesitate to contact me whenever you have a question or anything is 
> unclear.
> 
> 
> Project home page:
> https://fedorahosted.org/bind-dyndb-ldap/
> 
> README file in Git:
> http://git.fedorahosted.org/cgit/bind-dyndb-ldap.git/plain/README
> 
> Design pages for new features:
> https://fedorahosted.org/bind-dyndb-ldap/wiki/Design/PerServerConfigInLDAP
> https://fedorahosted.org/bind-dyndb-ldap/wiki/Design/RecordGenerator
> 
> Notes regarding current state:
> https://www.redhat.com/archives/freeipa-devel/2016-June/msg00395.html
> 
> Patches waiting for review on the mailing list:
> https://www.redhat.com/archives/freeipa-devel/2016-June/msg00203.html
> https://www.redhat.com/archives/freeipa-devel/2016-June/msg00205.html
> https://www.redhat.com/archives/freeipa-devel/2016-June/msg00329.html
> https://www.redhat.com/archives/freeipa-devel/2016-June/msg00207.html
> https://www.redhat.com/archives/freeipa-devel/2016-June/msg00343.html
> https://www.redhat.com/archives/freeipa-devel/2016-June/msg00331.html
> https://www.redhat.com/archives/freeipa-devel/2016-June/msg00332.html
> 
> Git repo with all patches:
> https://github.com/pspacek/bind-dyndb-ldap/tree/server_config_in_ldap4
> 
> COPR build:
> https://copr.fedorainfracloud.org/coprs/pspacek/bind-dyndb-ldap/build/342711/
> 
> Lab user guide:
> https://mojo.redhat.com/docs/DOC-1005343
> 
> 
> Thank you very much for your time.

-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to