[Freeipa-devel] [PATCH] 0016..0018 profiles regression fixes

2015-06-05 Thread Fraser Tweedale
Patches 16 and 17 fix regressions in the default profile. Patch 18 fixes the `ipa-replica-install --setup-ca' breakage. Cheers, Fraser From bc2b1d729c50dc1ae88a5e5709f655ea2f5ecd66 Mon Sep 17 00:00:00 2001 From: Fraser Tweedale ftwee...@redhat.com Date: Thu, 4 Jun 2015 22:49:01 -0400 Subject:

Re: [Freeipa-devel] [PATCH] 0016..0018 profiles regression fixes

2015-06-05 Thread Fraser Tweedale
On Fri, Jun 05, 2015 at 01:00:14PM +0200, Martin Basti wrote: On 05/06/15 11:47, Fraser Tweedale wrote: Patches 16 and 17 fix regressions in the default profile. Patch 18 fixes the `ipa-replica-install --setup-ca' breakage. Cheers, Fraser NACK, sorry ./make-lint *

Re: [Freeipa-devel] [PATCH] 0016..0018 profiles regression fixes

2015-06-05 Thread Oleg Fayans
Is it supposeed to fis this issue? 2015-06-05T12:11:57Z DEBUG File /usr/lib/python2.7/site-packages/ipapython/admintool.py, line 171, in execute return_value = self.run() File /usr/lib/python2.7/site-packages/ipaserver/install/ipa_server_upgrade.py, line 44, in run import

Re: [Freeipa-devel] [PATCH 0049] Move CA installation code into single module.

2015-06-05 Thread David Kupka
On 06/03/2015 05:49 PM, David Kupka wrote: Updated patch attached. -- David Kupka From ca004a585f86a5e35d02a90dc9db0753f786b84a Mon Sep 17 00:00:00 2001 From: David Kupka dku...@redhat.com Date: Wed, 3 Jun 2015 17:43:27 +0200 Subject: [PATCH] Move CA installation code into single module. ---

Re: [Freeipa-devel] [PATCH] 828 webui: don't log in back after logout

2015-06-05 Thread Martin Basti
On 07/05/15 19:24, Petr Vobornik wrote: Automatic login attempt is initiated by first failed xhr request which happens in metadata phase. New phase was added before metadata phase. It interrupts UI load and shows login page if it's directly after logout(marked in session storage). Successfull

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

2015-06-05 Thread Jan Cholasta
Dne 3.6.2015 v 14:17 Jan Cholasta napsal(a): Dne 2.6.2015 v 02:00 Endi Sukma Dewata napsal(a): Please take a look at the updated patch. On 5/27/2015 12:39 AM, Jan Cholasta wrote: 21) vault_archive is not a retrieve operation, it should be based on LDAPUpdate instead of LDAPRetrieve. Or

Re: [Freeipa-devel] [PATCH] 0016..0018 profiles regression fixes

2015-06-05 Thread Martin Basti
On 05/06/15 14:16, Oleg Fayans wrote: Is it supposeed to fis this issue? 2015-06-05T12:11:57Z DEBUG File /usr/lib/python2.7/site-packages/ipapython/admintool.py, line 171, in execute return_value = self.run() File

Re: [Freeipa-devel] [PATCH] 831 webui: better error reporting

2015-06-05 Thread Martin Basti
On 15/05/15 11:02, Petr Vobornik wrote: Helps in development and debugging. - ActionDropdownWidget - report error if required action is missing - report build errors to console ACK -- Martin Basti -- Manage your subscription for the Freeipa-devel mailing list:

Re: [Freeipa-devel] [patch 0002] Abstract the HostTracker class from host plugin test

2015-06-05 Thread David Kupka
On 06/02/2015 10:27 AM, Milan Kubik wrote: Hello, this is the (first) patch with the Tracker class implementation based on host plugin test. It is meant to be used as a base class to implement a helper class to write xml-rpc (api) tests for LDAP based plugins and to replace the Declarative

Re: [Freeipa-devel] Suggestion for the A part of IPA

2015-06-05 Thread Innes, Duncan
-Original Message- From: Martin Kosek [mailto:mko...@redhat.com] Sent: 05 June 2015 13:31 To: Innes, Duncan; freeipa-devel@redhat.com; Jakub Hrozek Subject: Re: [Freeipa-devel] Suggestion for the A part of IPA On 06/02/2015 10:29 AM, Innes, Duncan wrote: Just a bit of a head's up

Re: [Freeipa-devel] Password Maxlife 0 causes expiration of 90 days

2015-06-05 Thread Drew Erny
On 06/04/2015 05:41 PM, Alexander Bokovoy wrote: On Thu, 04 Jun 2015, Drew Erny wrote: https://fedorahosted.org/freeipa/ticket/2795 I've tracked down the source of this bug; it's nutty C stuff. So, in daemons/ipa-slapi-plugins/ipa-pwd-extop/common.c, when you update password, the expiration

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

2015-06-05 Thread Gabe Alford
How should ​ https://www.redhat.com/archives/freeipa-users/2015-June/msg00116.html be handled where the user cleared out the db cache? On Fri, Jun 5, 2015 at 9:08 AM, Jakub Hrozek jhro...@redhat.com wrote: On Fri, Jun 05, 2015 at 05:03:08PM +0200, Martin Basti wrote: On 05/06/15 16:13, Gabe

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

2015-06-05 Thread Gabe Alford
Thanks. Updated patch attached. On Fri, Jun 5, 2015 at 9:53 AM, Jakub Hrozek jhro...@redhat.com wrote: On Fri, Jun 05, 2015 at 09:46:05AM -0600, Gabe Alford wrote: How should ​ https://www.redhat.com/archives/freeipa-users/2015-June/msg00116.html be handled where the user cleared out the

Re: [Freeipa-devel] [PATCH 0026-0028] Fix nits in user-visible output

2015-06-05 Thread Petr Vobornik
On 06/03/2015 03:55 PM, Martin Basti wrote: On 14/04/15 09:43, Petr Spacek wrote: On 14.4.2015 09:10, Martin Kosek wrote: On 04/13/2015 05:05 PM, Petr Spacek wrote: Hello, documentation team proposed few changes in user-visible messages so here it is. It was not worth a ticket and related

Re: [Freeipa-devel] Database error on replicas

2015-06-05 Thread thierry bordaz
Hi, So far I am still unable to reproduce the problem. Comparing the errors logs of failing replica vs successful replica they are very similar. Except this failure Failing one ... [03/Jun/2015:03:45:33 -0400] slapd_ldap_sasl_interactive_bind - Error: could not perform interactive

Re: [Freeipa-devel] [PATCH 0001 v2] Migrate now accepts scope as argument

2015-06-05 Thread Petr Vobornik
On 06/05/2015 01:50 PM, Martin Basti wrote: On 04/06/15 21:36, Drew Erny wrote: Whoops, hit the wrong reply. On 06/04/2015 03:34 PM, Drew Erny wrote: This is the same patch sort of manually rebased on the master branch. I couldn't get it to cleanly rebase using tools, so I apply my commit

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

2015-06-05 Thread Jakub Hrozek
On Fri, Jun 05, 2015 at 09:46:05AM -0600, Gabe Alford wrote: How should ​ https://www.redhat.com/archives/freeipa-users/2015-June/msg00116.html be handled where the user cleared out the db cache? Ah, I confused that one with another issue Jan Pazdziora had, which was incidentally about client

Re: [Freeipa-devel] [PATCH] 831 webui: better error reporting

2015-06-05 Thread Petr Vobornik
On 06/05/2015 01:04 PM, Martin Basti wrote: On 15/05/15 11:02, Petr Vobornik wrote: Helps in development and debugging. - ActionDropdownWidget - report error if required action is missing - report build errors to console ACK Pushed to master: 847b5fd8d202e526c9d9599d5671808e9316aab7 --

Re: [Freeipa-devel] [PATCH] 828 webui: don't log in back after logout

2015-06-05 Thread Petr Vobornik
On 06/05/2015 12:39 PM, Martin Basti wrote: On 07/05/15 19:24, Petr Vobornik wrote: Automatic login attempt is initiated by first failed xhr request which happens in metadata phase. New phase was added before metadata phase. It interrupts UI load and shows login page if it's directly after

Re: [Freeipa-devel] [patch 0002] Abstract the HostTracker class from host plugin test

2015-06-05 Thread Petr Vobornik
On 06/05/2015 03:32 PM, David Kupka wrote: On 06/02/2015 10:27 AM, Milan Kubik wrote: Hello, this is the (first) patch with the Tracker class implementation based on host plugin test. It is meant to be used as a base class to implement a helper class to write xml-rpc (api) tests for LDAP

Re: [Freeipa-devel] [PATCH] 0016..0018 profiles regression fixes

2015-06-05 Thread Petr Vobornik
On 06/05/2015 05:09 PM, Martin Basti wrote: On 05/06/15 14:18, Martin Basti wrote: snip NACK, sorry snip Martin Basti Whoops. One wayward comma that should have been a period! Here's an updated 0017..0018. Thanks, Fraser -- Martin Basti ACK 16-1, 17-2, 18-2 pushed to

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

2015-06-05 Thread Rob Crittenden
Jakub Hrozek wrote: On Fri, Jun 05, 2015 at 09:46:05AM -0600, Gabe Alford wrote: How should ​ https://www.redhat.com/archives/freeipa-users/2015-June/msg00116.html be handled where the user cleared out the db cache? Ah, I confused that one with another issue Jan Pazdziora had, which was

Re: [Freeipa-devel] [PATCH 0001 v2] Migrate now accepts scope as argument

2015-06-05 Thread Martin Basti
On 04/06/15 21:36, Drew Erny wrote: Whoops, hit the wrong reply. On 06/04/2015 03:34 PM, Drew Erny wrote: This is the same patch sort of manually rebased on the master branch. I couldn't get it to cleanly rebase using tools, so I apply my commit line-by-line; the only changes I made were

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

2015-06-05 Thread Endi Sukma Dewata
On 6/5/2015 7:13 AM, Jan Cholasta wrote: If KRA is not installed, vault-archive and vault-retrieve fail with internal error. Added a code to check KRA installation in all vault commands. If you know a way not to load the vault plugin if the KRA is not installed please let me know, that's

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

2015-06-05 Thread Martin Basti
On 05/06/15 16:13, Gabe Alford wrote: Hello, Fix for https://fedorahosted.org/freeipa/ticket/5049 Thanks, Gabe Thank you. I dont think we should remove all SSSD caches. SSSD can have configured several providers not just IPA. IMO we should remove only IPA related caches, but wait

Re: [Freeipa-devel] Password Maxlife 0 causes expiration of 90 days

2015-06-05 Thread Simo Sorce
On Fri, 2015-06-05 at 10:37 -0400, Drew Erny wrote: On 06/04/2015 05:41 PM, Alexander Bokovoy wrote: On Thu, 04 Jun 2015, Drew Erny wrote: https://fedorahosted.org/freeipa/ticket/2795 I've tracked down the source of this bug; it's nutty C stuff. So, in

Re: [Freeipa-devel] [PATCH] 0016..0018 profiles regression fixes

2015-06-05 Thread Martin Basti
On 05/06/15 14:18, Martin Basti wrote: On 05/06/15 14:16, Oleg Fayans wrote: Is it supposeed to fis this issue? 2015-06-05T12:11:57Z DEBUG File /usr/lib/python2.7/site-packages/ipapython/admintool.py, line 171, in execute return_value = self.run() File