[Freeipa-devel] [PATCH] otptoken: use ipapython.nsslib instead of Python's ssl module

2015-07-07 Thread Christian Heimes
Hello, the patch removes the dependency on Python's ssl module and python-backports-ssl_match_hostname. https://fedorahosted.org/freeipa/ticket/5068 Open question - Is paths.IPA_NSSDB_DIR the correct NSSDB? Christian From 976427e5c448093131a99bdf77fc9d23c2d87883 Mon Sep 17 00:00:00

Re: [Freeipa-devel] [PATCH 0334] Hide topology and domainlevel features

2015-07-07 Thread Martin Basti
On 07/07/15 10:33, Tomas Babej wrote: Hi, * Hide topology and domainlevel commands in the CLI * Hide topology and domainlevel in the WebUI * Set maximum allowed domain level to 0 * Do not configure and enable the topology plugin https://fedorahosted.org/freeipa/ticket/5097 ACK -- Martin

Re: [Freeipa-devel] [PATCH 0055] ipa-replica-prepare: Do not create DNS zone it automatically.

2015-07-07 Thread Martin Basti
On 03/07/15 06:17, David Kupka wrote: Since ipa-replica-* tools will be soon removed I think this simple check should be enough. ACK -- Martin Basti -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to

Re: [Freeipa-devel] [PATCH] 0024..0025 Add missing certprofile features

2015-07-07 Thread Martin Basti
On 04/07/15 16:58, Fraser Tweedale wrote: On Fri, Jul 03, 2015 at 10:34:07PM +1000, Fraser Tweedale wrote: On Thu, Jul 02, 2015 at 08:12:12PM +1000, Fraser Tweedale wrote: On Thu, Jul 02, 2015 at 11:23:49AM +0200, Jan Cholasta wrote: Hi, Dne 2.7.2015 v 11:15 Fraser Tweedale napsal(a):

Re: [Freeipa-devel] [PATCH 0046] add option to skip client API version check and proceed at user's own risk

2015-07-07 Thread Martin Basti
On 03/07/15 16:41, Martin Babinsky wrote: On 07/02/2015 01:58 PM, Martin Babinsky wrote: First attempt at https://fedorahosted.org/freeipa/ticket/4768 Attaching reworked patch. ACK -- Martin Basti -- Manage your subscription for the Freeipa-devel mailing list:

[Freeipa-devel] [PATCH 0276] Fix: ipa-dns-install will add CA records if CA is installed

2015-07-07 Thread Martin Basti
https://fedorahosted.org/freeipa/ticket/5101 Patch attached. -- Martin Basti From f5de8e7a9ecd8f8220bd542d9ff264ce7917a829 Mon Sep 17 00:00:00 2001 From: Martin Basti mba...@redhat.com Date: Tue, 7 Jul 2015 16:28:48 +0200 Subject: [PATCH] Fix regression: ipa-dns-install will add CA records if

Re: [Freeipa-devel] [MAN] [PATCH] 0004 Fix phrasing in man page for stageuser.py

2015-07-07 Thread Tomas Babej
On 07/04/2015 02:03 PM, Jérôme Fenal wrote: Hi all, A quick patch to the man page part of stageuser to avoid ambiguity in the phrasing, spotted while translating the page. Regards, J. Thanks, ACK. I will not push this patch to master until we branch off 4.2 development branch

Re: [Freeipa-devel] [PATCHES 0252-0253, 268, 50 - 51] DNSSEC: allow to move DNSSEC key master to another IPA server

2015-07-07 Thread Tomas Babej
On 07/01/2015 12:47 PM, Petr Spacek wrote: On 1.7.2015 12:35, Martin Basti wrote: On 30/06/15 22:09, Petr Spacek wrote: On 30.6.2015 16:04, Martin Basti wrote: On 30/06/15 10:25, Martin Basti wrote: On 29/06/15 15:16, Martin Basti wrote: On 25/06/15 13:46, Petr Spacek wrote: On 17.6.2015

Re: [Freeipa-devel] [PATCH] Password vault

2015-07-07 Thread Jan Cholasta
Dne 3.7.2015 v 15:44 Endi Sukma Dewata napsal(a): Here is the rebased patch for vault access control. LGTM, except: @@ -356,6 +386,13 @@ class vault(LDAPObject): { 'objectclass': ['nsContainer'], 'cn': rdn['cn'], +

Re: [Freeipa-devel] [PATCH] Password vault

2015-07-07 Thread Jan Cholasta
Dne 3.7.2015 v 14:23 Endi Sukma Dewata napsal(a): On 7/1/2015 1:53 AM, Jan Cholasta wrote: I think it would be better to use a new attribute type which inherits from ipaPublicKey (ipaVaultPublicKey?) rather than ipaPublicKey directly for assymetric vault public keys, so that assymetric public

Re: [Freeipa-devel] [PATCH] 892 webui: add mangedby tab to otptoken

2015-07-07 Thread Tomas Babej
On 07/03/2015 02:49 PM, Martin Babinsky wrote: On 07/01/2015 06:59 PM, Petr Vobornik wrote: Added managedby_user tab to manage users who can manage the token. https://fedorahosted.org/freeipa/ticket/5003 Nathaniel, I could not reproduce the following part of the ticket: Careful

[Freeipa-devel] [PATCH 0334] Hide topology and domainlevel features

2015-07-07 Thread Tomas Babej
Hi, * Hide topology and domainlevel commands in the CLI * Hide topology and domainlevel in the WebUI * Set maximum allowed domain level to 0 * Do not configure and enable the topology plugin https://fedorahosted.org/freeipa/ticket/5097 From 8cdc723d334540258fdc408933b3f47ccebf5b53 Mon Sep 17

Re: [Freeipa-devel] [PATCHES 330-331] Update translations and introduce Zanata configuration

2015-07-07 Thread Tomas Babej
On 07/07/2015 09:09 AM, Tomas Babej wrote: On 06/24/2015 04:33 PM, Tomas Babej wrote: On 06/24/2015 04:29 PM, Martin Basti wrote: On 24/06/15 14:39, Tomas Babej wrote: +msgid Automount location name. +msgstr Job Title + in german po file +msgid Automount location name. +msgstr Job

Re: [Freeipa-devel] [PATCH 0055] ipa-replica-prepare: Do not create DNS zone it automatically.

2015-07-07 Thread David Kupka
On 03/07/15 06:17, David Kupka wrote: Since ipa-replica-* tools will be soon removed I think this simple check should be enough. Updated patch attached. -- David Kupka From 3df59261538f6b28e158802d8f6e4a47dadeab84 Mon Sep 17 00:00:00 2001 From: David Kupka dku...@redhat.com Date: Fri, 3 Jul

Re: [Freeipa-devel] [PATCH 0276] Fix: ipa-dns-install will add CA records if CA is installed

2015-07-07 Thread Tomas Babej
On 07/07/2015 07:36 PM, Martin Basti wrote: https://fedorahosted.org/freeipa/ticket/5101 Patch attached. ACK. Pushed to master: 1d9bdb240943527e1e19704acd183eae254267ae -- Manage your subscription for the Freeipa-devel mailing list:

Re: [Freeipa-devel] [PATCH 0046] add option to skip client API version check and proceed at user's own risk

2015-07-07 Thread Tomas Babej
On 07/07/2015 07:49 PM, Martin Basti wrote: On 03/07/15 16:41, Martin Babinsky wrote: On 07/02/2015 01:58 PM, Martin Babinsky wrote: First attempt at https://fedorahosted.org/freeipa/ticket/4768 Attaching reworked patch. ACK -- Martin Basti Pushed to master:

Re: [Freeipa-devel] [PATCH 0334] Hide topology and domainlevel features

2015-07-07 Thread Tomas Babej
On 07/07/2015 07:16 PM, Martin Basti wrote: On 07/07/15 10:33, Tomas Babej wrote: Hi, * Hide topology and domainlevel commands in the CLI * Hide topology and domainlevel in the WebUI * Set maximum allowed domain level to 0 * Do not configure and enable the topology plugin

Re: [Freeipa-devel] [PATCHES 145-148] ipa-kdb: add unit-test for filter_logon_info()

2015-07-07 Thread Tomas Babej
On 07/07/2015 03:49 PM, Sumit Bose wrote: On Tue, May 26, 2015 at 01:36:35PM +0200, Martin Kosek wrote: On 05/26/2015 01:33 PM, Sumit Bose wrote: Hi, these patches add some unit tests and some additional improvements related to the issues described in

Re: [Freeipa-devel] [PATCH 144] extdom: add unit-test for get_user_grouplist()

2015-07-07 Thread Tomas Babej
On 07/07/2015 03:47 PM, Sumit Bose wrote: On Tue, May 26, 2015 at 02:47:02PM +0300, Alexander Bokovoy wrote: On Tue, 26 May 2015, Sumit Bose wrote: On Tue, May 26, 2015 at 01:24:30PM +0200, Petr Vobornik wrote: On 05/26/2015 01:21 PM, Sumit Bose wrote: Hi, this tests should have gone

Re: [Freeipa-devel] [PATCH] 0180-0190 oneway trust and other trust-related patches

2015-07-07 Thread Tomas Babej
On 07/07/2015 04:31 PM, Alexander Bokovoy wrote: On Tue, 07 Jul 2015, Alexander Bokovoy wrote: Hi, attached are patches to introduce one-way trust support and few more to fix currently outstanding trust-related bugs. More details are in the commit messages. For oddjobd-activated helper,

Re: [Freeipa-devel] [PATCH] 0024..0025 Add missing certprofile features

2015-07-07 Thread Tomas Babej
On 07/07/2015 07:30 PM, Martin Basti wrote: On 04/07/15 16:58, Fraser Tweedale wrote: On Fri, Jul 03, 2015 at 10:34:07PM +1000, Fraser Tweedale wrote: On Thu, Jul 02, 2015 at 08:12:12PM +1000, Fraser Tweedale wrote: On Thu, Jul 02, 2015 at 11:23:49AM +0200, Jan Cholasta wrote: Hi, Dne

Re: [Freeipa-devel] [PATCH 0055] ipa-replica-prepare: Do not create DNS zone it automatically.

2015-07-07 Thread Tomas Babej
On 07/07/2015 07:16 PM, Martin Basti wrote: On 03/07/15 06:17, David Kupka wrote: Since ipa-replica-* tools will be soon removed I think this simple check should be enough. ACK -- Martin Basti Pushed to master: 6a91893ff50fee6d7c71d9bc982d85a3ec8b7583 -- Manage your

Re: [Freeipa-devel] [PATCHES 330-331] Update translations and introduce Zanata configuration

2015-07-07 Thread Tomas Babej
On 07/07/2015 11:48 AM, Martin Basti wrote: On 07/07/15 10:37, Tomas Babej wrote: On 07/07/2015 09:09 AM, Tomas Babej wrote: On 06/24/2015 04:33 PM, Tomas Babej wrote: On 06/24/2015 04:29 PM, Martin Basti wrote: On 24/06/15 14:39, Tomas Babej wrote: +msgid Automount location name.

Re: [Freeipa-devel] [PATCH] 0180-0190 oneway trust and other trust-related patches

2015-07-07 Thread Alexander Bokovoy
On Tue, 07 Jul 2015, Alexander Bokovoy wrote: From 4a856d8ff597ec516cc1eb05f06e062bb4ecca5b Mon Sep 17 00:00:00 2001 From: Alexander Bokovoy aboko...@redhat.com Date: Thu, 28 May 2015 11:49:58 + Subject: [PATCH 05/11] trusts: pass AD DC hostname if specified explicitly Fixes

Re: [Freeipa-devel] error handling in httpd.service and ipa-httpd-kdcproxy

2015-07-07 Thread Nathaniel McCallum
On Jul 6, 2015, at 11:35 AM, Christian Heimes chei...@redhat.com wrote: Hello, I like to ask for your opinion regarding the pre-exec hook 'ipa-httpd-kdcproxy' in httpd.service. Alex has asked me to handle error cases like LDAP connection timeout more gracefully. At the moment any error

Re: [Freeipa-devel] [PATCH] 0180-0190 oneway trust and other trust-related patches

2015-07-07 Thread Alexander Bokovoy
On Tue, 07 Jul 2015, Alexander Bokovoy wrote: From b7a3b206deb3257b3a78939f0d2a6a114e48b758 Mon Sep 17 00:00:00 2001 From: Alexander Bokovoy aboko...@redhat.com Date: Thu, 26 Mar 2015 14:34:06 +0200 Subject: [PATCH 01/11] add one-way trust support to ipasam When trust is established, ipasam

Re: [Freeipa-devel] [PATCHES 330-331] Update translations and introduce Zanata configuration

2015-07-07 Thread Martin Basti
On 07/07/15 10:37, Tomas Babej wrote: On 07/07/2015 09:09 AM, Tomas Babej wrote: On 06/24/2015 04:33 PM, Tomas Babej wrote: On 06/24/2015 04:29 PM, Martin Basti wrote: On 24/06/15 14:39, Tomas Babej wrote: +msgid Automount location name. +msgstr Job Title + in german po file +msgid

[Freeipa-devel] [PATCH] 0180-0190 oneway trust and other trust-related patches

2015-07-07 Thread Alexander Bokovoy
Hi, attached are patches to introduce one-way trust support and few more to fix currently outstanding trust-related bugs. More details are in the commit messages. For oddjobd-activated helper, if you want to test the one-way trust setup, you need to put SELinux into permissive. We have bugs

Re: [Freeipa-devel] [PATCH] 0180-0190 oneway trust and other trust-related patches

2015-07-07 Thread Alexander Bokovoy
On Tue, 07 Jul 2015, Alexander Bokovoy wrote: Hi, attached are patches to introduce one-way trust support and few more to fix currently outstanding trust-related bugs. More details are in the commit messages. For oddjobd-activated helper, if you want to test the one-way trust setup, you need

Re: [Freeipa-devel] [PATCH] 0180-0190 oneway trust and other trust-related patches

2015-07-07 Thread Alexander Bokovoy
On Tue, 07 Jul 2015, Alexander Bokovoy wrote: From a4e2034028d64a8b2b533af9541e698a68388fb2 Mon Sep 17 00:00:00 2001 From: Alexander Bokovoy aboko...@redhat.com Date: Thu, 4 Jun 2015 21:29:36 + Subject: [PATCH 07/11] ipa-adtrust-install: allow configuring of trust agents Trust agents are

Re: [Freeipa-devel] [PATCH] Password vault

2015-07-07 Thread Martin Kosek
On 07/07/2015 10:51 AM, Jan Cholasta wrote: Dne 3.7.2015 v 15:44 Endi Sukma Dewata napsal(a): Here is the rebased patch for vault access control. LGTM, except: @@ -356,6 +386,13 @@ class vault(LDAPObject): { 'objectclass': ['nsContainer'],

Re: [Freeipa-devel] error handling in httpd.service and ipa-httpd-kdcproxy

2015-07-07 Thread Christian Heimes
On 2015-07-07 15:41, Simo Sorce wrote: On Tue, 2015-07-07 at 08:48 -0400, Nathaniel McCallum wrote: On Jul 6, 2015, at 11:35 AM, Christian Heimes chei...@redhat.com wrote: Hello, I like to ask for your opinion regarding the pre-exec hook 'ipa-httpd-kdcproxy' in httpd.service. Alex has asked

Re: [Freeipa-devel] [PATCH] 004 Improve error handling in ipa-httpd-kdcproxy

2015-07-07 Thread Nathaniel McCallum
This LGTM. However, I’ll let Alexander give the ACK. On Jul 7, 2015, at 10:11 AM, Christian Heimes chei...@redhat.com wrote: Hi, the patch addresses the error handling of ipa-httpd-kdcproxy as discussed in the other thread. Christian

Re: [Freeipa-devel] error handling in httpd.service and ipa-httpd-kdcproxy

2015-07-07 Thread Simo Sorce
On Tue, 2015-07-07 at 08:48 -0400, Nathaniel McCallum wrote: On Jul 6, 2015, at 11:35 AM, Christian Heimes chei...@redhat.com wrote: Hello, I like to ask for your opinion regarding the pre-exec hook 'ipa-httpd-kdcproxy' in httpd.service. Alex has asked me to handle error cases like

Re: [Freeipa-devel] [PATCHES 145-148] ipa-kdb: add unit-test for filter_logon_info()

2015-07-07 Thread Sumit Bose
On Tue, May 26, 2015 at 01:36:35PM +0200, Martin Kosek wrote: On 05/26/2015 01:33 PM, Sumit Bose wrote: Hi, these patches add some unit tests and some additional improvements related to the issues described in https://bugzilla.redhat.com/show_bug.cgi?id=1222475 . The original issue is

[Freeipa-devel] [PATCH] 004 Improve error handling in ipa-httpd-kdcproxy

2015-07-07 Thread Christian Heimes
Hi, the patch addresses the error handling of ipa-httpd-kdcproxy as discussed in the other thread. Christian From 85dc0cc3f597accdee6f6de9d7b4d41b2173a8d9 Mon Sep 17 00:00:00 2001 From: Christian Heimes chei...@redhat.com Date: Tue, 7 Jul 2015 16:05:48 +0200 Subject: [PATCH] Improve error

Re: [Freeipa-devel] [PATCH 144] extdom: add unit-test for get_user_grouplist()

2015-07-07 Thread Sumit Bose
On Tue, May 26, 2015 at 02:47:02PM +0300, Alexander Bokovoy wrote: On Tue, 26 May 2015, Sumit Bose wrote: On Tue, May 26, 2015 at 01:24:30PM +0200, Petr Vobornik wrote: On 05/26/2015 01:21 PM, Sumit Bose wrote: Hi, this tests should have gone together with

Re: [Freeipa-devel] [PATCH 0046] add option to skip client API version check and proceed at user's own risk

2015-07-07 Thread Jan Cholasta
Dne 8.7.2015 v 00:37 Tomas Babej napsal(a): On 07/07/2015 07:49 PM, Martin Basti wrote: On 03/07/15 16:41, Martin Babinsky wrote: On 07/02/2015 01:58 PM, Martin Babinsky wrote: First attempt at https://fedorahosted.org/freeipa/ticket/4768 Attaching reworked patch. ACK -- Martin

Re: [Freeipa-devel] [PATCH 0051] Clear SSSD caches when uninstalling the client

2015-07-07 Thread Tomas Babej
On 06/30/2015 05:40 PM, Simo Sorce wrote: On Tue, 2015-06-30 at 16:10 +0200, Martin Basti wrote: On 30/06/15 15:18, Martin Basti wrote: On 30/06/15 14:47, Simo Sorce wrote: On Tue, 2015-06-30 at 13:19 +0200, Tomas Babej wrote: On 06/30/2015 01:08 PM, Martin Basti wrote: On 30/06/15 13:00,

Re: [Freeipa-devel] [PATCH] 004 Improve error handling in ipa-httpd-kdcproxy

2015-07-07 Thread Alexander Bokovoy
On Tue, 07 Jul 2015, Nathaniel McCallum wrote: This LGTM. However, I’ll let Alexander give the ACK. Looks good for me too. -- / Alexander Bokovoy signature.asc Description: PGP signature -- Manage your subscription for the Freeipa-devel mailing list:

Re: [Freeipa-devel] [PATCH] 0180-0190 oneway trust and other trust-related patches

2015-07-07 Thread Alexander Bokovoy
On Tue, 07 Jul 2015, Alexander Bokovoy wrote: Hi, attached are patches to introduce one-way trust support and few more to fix currently outstanding trust-related bugs. More details are in the commit messages. For oddjobd-activated helper, if you want to test the one-way trust setup, you need

Re: [Freeipa-devel] [PATCH] 004 Improve error handling in ipa-httpd-kdcproxy

2015-07-07 Thread Tomas Babej
On 07/07/2015 04:28 PM, Alexander Bokovoy wrote: On Tue, 07 Jul 2015, Nathaniel McCallum wrote: This LGTM. However, I’ll let Alexander give the ACK. Looks good for me too. Pushed to master: 25d1afdc54284c6bcf1caf08beae2e66ceb7f4e8 -- Manage your subscription for the Freeipa-devel

Re: [Freeipa-devel] [PATCH] Password vault

2015-07-07 Thread Endi Sukma Dewata
- Original Message - On 07/07/2015 10:51 AM, Jan Cholasta wrote: Dne 3.7.2015 v 15:44 Endi Sukma Dewata napsal(a): Here is the rebased patch for vault access control. LGTM, except: @@ -356,6 +386,13 @@ class vault(LDAPObject): {

Re: [Freeipa-devel] [PATCH 0054] cermonger: Use private unix socket when DBus SystemBus is not, available.

2015-07-07 Thread David Kupka
On 03/07/15 08:46, Martin Kosek wrote: On 07/03/2015 08:41 AM, Jan Cholasta wrote: Dne 2.7.2015 v 14:34 David Kupka napsal(a): On 01/07/15 16:31, David Kupka wrote: Updated patch attached. Client install works, but uninstall does not: # ipa-client-install --uninstall -U certmonger