Re: [Freeipa-devel] [PATCH] [DOC] Remove SELinux user paragraph replacement

2013-10-11 Thread Martin Kosek
On 10/10/2013 06:37 PM, Jérôme Fenal wrote: > Attached. > Replaced the dodgy sentence with Martin's one. > > Regards, > > J. Thanks Jérôme for the patch, I have few comments though: 1) One more note for patch format, please use the following command to extract your patch from git: $ git format

Re: [Freeipa-devel] [PATCH] 426, 430 Winsync re-initialize should not run memberOf fixup task

2013-10-11 Thread Martin Kosek
On 10/10/2013 05:45 PM, Jan Cholasta wrote: > Hi, > > On 9.10.2013 13:31, Martin Kosek wrote: >> Change re-initialize command to consider memberOf fixup task only >> for non-winsync replication agreements. This patch also includes >> few fixes for DsInstance to properly set realm and fqdn properti

Re: [Freeipa-devel] [PATCH] 427 Remove --no-serial-autoincrement

2013-10-11 Thread Martin Kosek
On 10/10/2013 02:18 PM, Jan Cholasta wrote: > On 10.10.2013 14:02, Martin Kosek wrote: >> On 10/10/2013 01:47 PM, Jan Cholasta wrote: >>> Hi, >>> >>> On 9.10.2013 16:21, Martin Kosek wrote: Deprecate this option and do not offer it in installation tools. Without this option enabled, advan

Re: [Freeipa-devel] [PATCH] 428 PKI installation on replica failing due to missing proxy conf

2013-10-11 Thread Martin Kosek
On 10/10/2013 03:24 PM, Jan Cholasta wrote: > Hi, > > On 9.10.2013 22:38, Martin Kosek wrote: >> Proxy configuration was not detected correctly. Both >> ipa-pki-proxy.conf and ipa.conf need to be in place and httpd >> restarted to be able to check it's status. >> >> https://fedorahosted.org/freeip

Re: [Freeipa-devel] [PATCH] 426, 430 Winsync re-initialize should not run memberOf fixup task

2013-10-11 Thread Jan Cholasta
On 11.10.2013 09:47, Martin Kosek wrote: On 10/10/2013 05:45 PM, Jan Cholasta wrote: Hi, On 9.10.2013 13:31, Martin Kosek wrote: Change re-initialize command to consider memberOf fixup task only for non-winsync replication agreements. This patch also includes few fixes for DsInstance to proper

Re: [Freeipa-devel] [PATCH] 426, 430 Winsync re-initialize should not run memberOf fixup task

2013-10-11 Thread Martin Kosek
On 10/11/2013 10:00 AM, Jan Cholasta wrote: > On 11.10.2013 09:47, Martin Kosek wrote: >> On 10/10/2013 05:45 PM, Jan Cholasta wrote: >>> Hi, >>> >>> On 9.10.2013 13:31, Martin Kosek wrote: Change re-initialize command to consider memberOf fixup task only for non-winsync replication agree

Re: [Freeipa-devel] [PATCH][DOC] 0007 Removed trailing whitespaces

2013-10-11 Thread Petr Vobornik
On 10/01/2013 03:49 PM, Martin Basti wrote: Patch removes trailing whitespaces in user guide from chapter 6 to end Whitespaces was removed by: sed 's/[ \t]\+$//' ACK and pushed to master -- Petr Vobornik ___ Freeipa-devel mailing list Freeipa-de

Re: [Freeipa-devel] [PATCH 0118] Do not create separate ranges for subdomains in case of POSIX trust

2013-10-11 Thread Alexander Bokovoy
On Thu, 10 Oct 2013, Tomas Babej wrote: Hi, This is a fix for a bug I found related to the subdomains code while working on the AD Continuous Integration testing. Thanks. ACK, now I can see no ranges created for the subdomains if there is POSIX range in use for the forest root. -- / Alexande

Re: [Freeipa-devel] [PATCHES 114-117] Do proper cleanup after ipa-adtrust-install

2013-10-11 Thread Alexander Bokovoy
On Wed, 09 Oct 2013, Tomas Babej wrote: Hi, the following patchset contains fixes for https://fedorahosted.org/freeipa/ticket/3479 . With it applied, you should be able to establish a trust on a reinstalled IPA master that had a trust established. ACK. Small comment -- not "that had a tru

Re: [Freeipa-devel] [PATCH] [DOC] Remove SELinux user paragraph replacement

2013-10-11 Thread Jérôme Fenal
2013/10/11 Martin Kosek : > On 10/10/2013 06:37 PM, Jérôme Fenal wrote: >> Attached. >> Replaced the dodgy sentence with Martin's one. >> >> Regards, >> >> J. > > Thanks Jérôme for the patch, I have few comments though: > > 1) One more note for patch format, please use the following command to extr

[Freeipa-devel] [PATCH 0197-0200]

2013-10-11 Thread Petr Spacek
Hello, update documentation and schema files for upcoming version 4.0. This fixes typo in schema file: https://fedorahosted.org/bind-dyndb-ldap/ticket/121 Have a nice weekend! -- Petr^2 Spacek From 7964ddadbf03e8340faa8ff1175bd75f7cfb624c Mon Sep 17 00:00:00 2001 From: Petr Spacek Date: Fri,

[Freeipa-devel] [PATCH 0197-0200] Preparation for bind-dyndb-ldap release 4.0

2013-10-11 Thread Petr Spacek
Hello, update documentation and schema files for upcoming version 4.0. This fixes typo in schema file: https://fedorahosted.org/bind-dyndb-ldap/ticket/121 Have a nice weekend! -- Petr^2 Spacek From 7964ddadbf03e8340faa8ff1175bd75f7cfb624c Mon Sep 17 00:00:00 2001 From: Petr Spacek Date: Fri,

[Freeipa-devel] [PATCH] Split out function

2013-10-11 Thread Simo Sorce
During the Kerberos interop we found out we are still missing to generate a Ms-PAc field the MS testsuite wants to see. I did some work in the past to create the UPN_DNS_INFO buffer in the samba IDL files but it has been recently reverted upstream due to some issues it created for Samba4 and is be