Re: [Freeipa-devel] [PATCH 0288] ipa-replica-manage: Allow value 'no' for replica-certify-all attr in abort-clean-ruv subcommand

2015-07-17 Thread Petr Vobornik
On 07/16/2015 06:22 PM, Martin Basti wrote: On 16/07/15 17:07, Rob Crittenden wrote: Martin Basti wrote: https://fedorahosted.org/freeipa/ticket/4988 Patch attached. IMHO this should be mentioned in the man page. rob Updated patch attached. ACK Pushed to: master:

Re: [Freeipa-devel] [PATCH 0057] Do not use anonymous bind in migration UI.

2015-07-17 Thread Petr Vobornik
On 07/17/2015 10:31 AM, Petr Vobornik wrote: On 07/17/2015 07:18 AM, Alexander Bokovoy wrote: On Fri, 17 Jul 2015, Jan Cholasta wrote: Dne 16.7.2015 v 12:16 David Kupka napsal(a): On 15/07/15 16:04, David Kupka wrote: On 15/07/15 15:34, Jan Cholasta wrote: Dne 15.7.2015 v 15:21 David Kupka

Re: [Freeipa-devel] [PATCH 0286] Sysrestore: copy files instead of moving them to avoid SELinux issues

2015-07-17 Thread Petr Vobornik
On 07/17/2015 01:44 PM, Alexander Bokovoy wrote: On Fri, 17 Jul 2015, Martin Basti wrote: From b05f4a2e17ae00e5c20e5eb7bd046472f100e0ad Mon Sep 17 00:00:00 2001 From: Martin Basti mba...@redhat.com Date: Wed, 15 Jul 2015 16:20:59 +0200 Subject: [PATCH] sysrestore: copy files instead of moving

Re: [Freeipa-devel] [PATCH 013] Fix minor typos

2015-07-17 Thread Petr Vobornik
On 07/17/2015 01:17 PM, Petr Vobornik wrote: On 07/17/2015 12:21 PM, Christian Heimes wrote: This patch from Yuri Chornoivan fixes some typos. All fixes look ok to me. +1, one minor issue though. Btw, the Reviewed-By line is added by ipatool utility on push.

Re: [Freeipa-devel] [PATCH 0286] Sysrestore: copy files instead of moving them to avoid SELinux issues

2015-07-17 Thread Alexander Bokovoy
On Fri, 17 Jul 2015, Martin Basti wrote: From b05f4a2e17ae00e5c20e5eb7bd046472f100e0ad Mon Sep 17 00:00:00 2001 From: Martin Basti mba...@redhat.com Date: Wed, 15 Jul 2015 16:20:59 +0200 Subject: [PATCH] sysrestore: copy files instead of moving them to avoind SELinux issues Copying files

Re: [Freeipa-devel] [PATCH 0286] Sysrestore: copy files instead of moving them to avoid SELinux issues

2015-07-17 Thread Martin Basti
On 17/07/15 13:57, Petr Vobornik wrote: On 07/17/2015 01:46 PM, Petr Vobornik wrote: On 07/17/2015 01:44 PM, Alexander Bokovoy wrote: On Fri, 17 Jul 2015, Martin Basti wrote: From b05f4a2e17ae00e5c20e5eb7bd046472f100e0ad Mon Sep 17 00:00:00 2001 From: Martin Basti mba...@redhat.com Date: Wed,

Re: [Freeipa-devel] [PATCHES 0279-0280] Backport index fixes into IPA 4.1

2015-07-17 Thread Petr Vobornik
On 07/17/2015 12:55 PM, Martin Babinsky wrote: On 07/09/2015 01:50 PM, Martin Basti wrote: Backport following commits into IPA 4-1: 57fba7a56f88c517b3ebb03842f1cc18bc129ebb 16f47ed4520d4f89db39d1dc58be7a8efb1d8612 Patches attached. ACK pushed to ipa-4-1: *

Re: [Freeipa-devel] [PATCH 0274] DNS: Check if dns package is installed

2015-07-17 Thread Martin Basti
On 03/07/15 09:03, Tomas Babej wrote: On 07/02/2015 02:03 PM, Petr Spacek wrote: On 2.7.2015 13:54, Jan Cholasta wrote: Dne 2.7.2015 v 13:34 Petr Spacek napsal(a): On 2.7.2015 12:57, Tomas Babej wrote: On 07/02/2015 08:50 AM, Petr Spacek wrote: On 1.7.2015 20:29, Tomas Babej wrote: On

Re: [Freeipa-devel] [PATCH 0285] DNS: consolidate RR types in API and LDAP schema

2015-07-17 Thread Petr Spacek
On 17.7.2015 13:00, Martin Basti wrote: On 15/07/15 18:13, Petr Spacek wrote: On 15.7.2015 17:39, Martin Basti wrote: On 15/07/15 17:38, Petr Spacek wrote: On 15.7.2015 17:33, Martin Basti wrote: On 15/07/15 16:03, Martin Basti wrote: On 15/07/15 15:39, Petr Vobornik wrote: On 07/15/2015

Re: [Freeipa-devel] [PATCH 0286] Sysrestore: copy files instead of moving them to avoid SELinux issues

2015-07-17 Thread Petr Vobornik
On 07/17/2015 01:46 PM, Petr Vobornik wrote: On 07/17/2015 01:44 PM, Alexander Bokovoy wrote: On Fri, 17 Jul 2015, Martin Basti wrote: From b05f4a2e17ae00e5c20e5eb7bd046472f100e0ad Mon Sep 17 00:00:00 2001 From: Martin Basti mba...@redhat.com Date: Wed, 15 Jul 2015 16:20:59 +0200 Subject:

Re: [Freeipa-devel] [PATCH 0052] Create server-dns sub-package

2015-07-17 Thread Simo Sorce
- Original Message - From: Jan Cholasta jchol...@redhat.com To: Simo Sorce s...@redhat.com, Petr Spacek pspa...@redhat.com Cc: freeipa-devel@redhat.com, Alexander Bokovoy aboko...@redhat.com Sent: Thursday, July 16, 2015 2:08:09 AM Subject: Re: [Freeipa-devel] [PATCH 0052] Create

[Freeipa-devel] [PATCH 0289] [Py3] Replace tabs with space

2015-07-17 Thread Martin Basti
I accidentally found several tabs in code, so I fixed it, as py3 does not allow to mix tabs and spaces. Patch attached. -- Martin Basti From 74e0ac4f24e3bc329abb2d08c3729945324d945d Mon Sep 17 00:00:00 2001 From: Martin Basti mba...@redhat.com Date: Fri, 17 Jul 2015 13:25:32 +0200 Subject:

Re: [Freeipa-devel] [PATCH 0052] Create server-dns sub-package

2015-07-17 Thread Simo Sorce
- Original Message - From: Petr Spacek pspa...@redhat.com To: Alexander Bokovoy aboko...@redhat.com, Jan Cholasta jchol...@redhat.com Cc: Simo Sorce s...@redhat.com, freeipa-devel@redhat.com Sent: Thursday, July 16, 2015 7:47:57 AM Subject: Re: [Freeipa-devel] [PATCH 0052] Create

Re: [Freeipa-devel] [PATCH 0339] trusts: Check for AD root domain among our trusted domains

2015-07-17 Thread Petr Vobornik
On 07/15/2015 02:41 PM, Tomas Babej wrote: On 07/15/2015 02:31 PM, Alexander Bokovoy wrote: On Wed, 15 Jul 2015, Tomas Babej wrote: Hi, Check for the presence of the forest root DNS domain of the AD realm among the IPA realm domains prior to esablishing the trust. This prevents creation of

Re: [Freeipa-devel] right way to connect to DS with Directory Manager creds from the test

2015-07-17 Thread Martin Basti
On 17/07/15 17:03, Oleg Fayans wrote: Hi everybody, What is the right way to connect to the Directory Server using Directory Manager credentials from the autotest? I need to execute something like ldapsearch -D cn=Directory Manager -w 'directory_manager_password' -b cn=IPA Topology

Re: [Freeipa-devel] [PATCH 0289] [Py3] Replace tabs with space

2015-07-17 Thread Petr Vobornik
On 07/17/2015 03:58 PM, Christian Heimes wrote: On 2015-07-17 15:51, Martin Basti wrote: I accidentally found several tabs in code, so I fixed it, as py3 does not allow to mix tabs and spaces. ACK Pushed to: master: c6c84faecf5b7017c0d648d76ba0db4a2eba2f03 ipa-4-2:

Re: [Freeipa-devel] [PATCH 0057] Do not use anonymous bind in migration UI.

2015-07-17 Thread Petr Vobornik
On 07/17/2015 07:18 AM, Alexander Bokovoy wrote: On Fri, 17 Jul 2015, Jan Cholasta wrote: Dne 16.7.2015 v 12:16 David Kupka napsal(a): On 15/07/15 16:04, David Kupka wrote: On 15/07/15 15:34, Jan Cholasta wrote: Dne 15.7.2015 v 15:21 David Kupka napsal(a):

Re: [Freeipa-devel] [PATCH 0052] Create server-dns sub-package

2015-07-17 Thread Tomas Babej
On 07/17/2015 05:46 AM, Jan Cholasta wrote: Dne 16.7.2015 v 16:50 Alexander Bokovoy napsal(a): On Thu, 16 Jul 2015, Petr Spacek wrote: Third version of the patch is attached, please view. Behavior: - freeipa-server package continues to exist and does not include DNS dependencies -

Re: [Freeipa-devel] [PATCH 0286] Sysrestore: copy files instead of moving them to avoid SELinux issues

2015-07-17 Thread Martin Basti
On 17/07/15 13:04, Alexander Bokovoy wrote: On Wed, 15 Jul 2015, Martin Basti wrote: On 15/07/15 18:01, Alexander Bokovoy wrote: On Wed, 15 Jul 2015, Martin Basti wrote: Moved files temporarily exist without a proper SElinux context which causes issues when running SSSD/ntpd tries to work

Re: [Freeipa-devel] [PATCH 013] Fix minor typos

2015-07-17 Thread Petr Vobornik
On 07/17/2015 12:21 PM, Christian Heimes wrote: This patch from Yuri Chornoivan fixes some typos. All fixes look ok to me. +1, one minor issue though. Btw, the Reviewed-By line is added by ipatool utility on push. https://fedorahosted.org/freeipa/ticket/5109

[Freeipa-devel] [PATCH 013] Fix minor typos

2015-07-17 Thread Christian Heimes
This patch from Yuri Chornoivan fixes some typos. All fixes look ok to me. https://fedorahosted.org/freeipa/ticket/5109 https://bugzilla.redhat.com/show_bug.cgi?id=1239132 From 9554caac8965671840e2361cbb62884333c12e3c Mon Sep 17 00:00:00 2001 From: Yuri Chornoivan yurc...@ukr.net Date: Fri, 3 Jul

Re: [Freeipa-devel] [PATCHES 0279-0280] Backport index fixes into IPA 4.1

2015-07-17 Thread Martin Babinsky
On 07/09/2015 01:50 PM, Martin Basti wrote: Backport following commits into IPA 4-1: 57fba7a56f88c517b3ebb03842f1cc18bc129ebb 16f47ed4520d4f89db39d1dc58be7a8efb1d8612 Patches attached. ACK -- Martin^3 Babinsky -- Manage your subscription for the Freeipa-devel mailing list:

Re: [Freeipa-devel] [PATCH 0285] DNS: consolidate RR types in API and LDAP schema

2015-07-17 Thread Martin Basti
On 15/07/15 18:13, Petr Spacek wrote: On 15.7.2015 17:39, Martin Basti wrote: On 15/07/15 17:38, Petr Spacek wrote: On 15.7.2015 17:33, Martin Basti wrote: On 15/07/15 16:03, Martin Basti wrote: On 15/07/15 15:39, Petr Vobornik wrote: On 07/15/2015 02:42 PM, Martin Basti wrote: Patch

Re: [Freeipa-devel] [PATCH 0286] Sysrestore: copy files instead of moving them to avoid SELinux issues

2015-07-17 Thread Alexander Bokovoy
On Wed, 15 Jul 2015, Martin Basti wrote: On 15/07/15 18:01, Alexander Bokovoy wrote: On Wed, 15 Jul 2015, Martin Basti wrote: Moved files temporarily exist without a proper SElinux context which causes issues when running SSSD/ntpd tries to work with files.

Re: [Freeipa-devel] [PATCH 0286] Sysrestore: copy files instead of moving them to avoid SELinux issues

2015-07-17 Thread Alexander Bokovoy
On Fri, 17 Jul 2015, Martin Basti wrote: On 17/07/15 13:04, Alexander Bokovoy wrote: On Wed, 15 Jul 2015, Martin Basti wrote: On 15/07/15 18:01, Alexander Bokovoy wrote: On Wed, 15 Jul 2015, Martin Basti wrote: Moved files temporarily exist without a proper SElinux context which causes

Re: [Freeipa-devel] [PATCH 0286] Sysrestore: copy files instead of moving them to avoid SELinux issues

2015-07-17 Thread Martin Basti
On 17/07/15 13:25, Alexander Bokovoy wrote: On Fri, 17 Jul 2015, Martin Basti wrote: On 17/07/15 13:04, Alexander Bokovoy wrote: On Wed, 15 Jul 2015, Martin Basti wrote: On 15/07/15 18:01, Alexander Bokovoy wrote: On Wed, 15 Jul 2015, Martin Basti wrote: Moved files temporarily exist without