Re: [Freeipa-devel] [PATCH] Normalize manager name in user-add

2015-11-05 Thread Martin Basti
On 05.11.2015 07:58, Abhijeet Kasurde wrote: Hi All, Please find the patch for https://fedorahosted.org/freeipa/ticket/5387 Thanks, Abhijeet Kasurde Hello, I'm sorry I forgot to mark "patch posted for review", but the ticket is marked as assigned, that means the somebody (me in this

Re: [Freeipa-devel] [PATCH 505] install: fix command line option validation

2015-11-05 Thread Jan Cholasta
On 3.11.2015 15:03, Martin Babinsky wrote: On 11/03/2015 09:41 AM, Jan Cholasta wrote: Hi, the attached patch fixes: Honza Once more for the whole

Re: [Freeipa-devel] [PATCH] SPEC: Drop sssd from BuildRequires

2015-11-05 Thread Martin Basti
On 05.11.2015 08:40, Lukas Slebodnik wrote: On (05/11/15 09:24), Alexander Bokovoy wrote: On Thu, 05 Nov 2015, Lukas Slebodnik wrote: On (04/11/15 18:12), Alexander Bokovoy wrote: On Tue, 03 Nov 2015, Lukas Slebodnik wrote: >From c466be0e769a80fe204fc70675b8fa0ea3efb7b2 Mon Sep 17 00:00:00

[Freeipa-devel] is installation broken in upstream?

2015-11-05 Thread Oleg Fayans
Hi all, Having built the latest upstream code locally I am getting the following weird messages at the attempt to install the bits: $ dnf install freeipa-* -y Last metadata expiration check performed 1 day, 4:09:14 ago on Wed Nov 4 07:13:28 2015. Error: nothing provides installed needed by

Re: [Freeipa-devel] [PATCH 0060] Incomplete ports for IPA AD Trust

2015-11-05 Thread Tomas Babej
On 10/31/2015 11:38 AM, Alexander Bokovoy wrote: > On Fri, 30 Oct 2015, Gabe Alford wrote: >> Okay. Added the port range to ipa-adtrust-install and updated the man >> page >> to reflect firewall requirements. >> The firewall section seems a little rough, so let me know what you >> think it >>

Re: [Freeipa-devel] is installation broken in upstream?

2015-11-05 Thread Martin Basti
On 05.11.2015 11:28, Oleg Fayans wrote: Hi all, Having built the latest upstream code locally I am getting the following weird messages at the attempt to install the bits: $ dnf install freeipa-* -y Last metadata expiration check performed 1 day, 4:09:14 ago on Wed Nov 4 07:13:28 2015.

Re: [Freeipa-devel] [PATCH] Normalize manager name in user-add

2015-11-05 Thread Abhijeet Kasurde
On 11/05/2015 03:25 PM, Martin Basti wrote: On 05.11.2015 07:58, Abhijeet Kasurde wrote: Hi All, Please find the patch for https://fedorahosted.org/freeipa/ticket/5387 Thanks, Abhijeet Kasurde Hello, I'm sorry I forgot to mark "patch posted for review", but the ticket is marked as

Re: [Freeipa-devel] [PATCH 0062] custodia: ipa-upgrade failed on replica

2015-11-05 Thread Martin Basti
On 05.11.2015 03:48, Gabe Alford wrote: Hello, Fix for https://fedorahosted.org/freeipa/ticket/5374. I could reproduce it as the Custodia update file was missing from the updates Makefile which in turn was not being packaged into the rpms. Thanks, Gabe Thank you! ACK I synced my

Re: [Freeipa-devel] [PATCH 0062] custodia: ipa-upgrade failed on replica

2015-11-05 Thread Martin Basti
On 05.11.2015 11:34, Martin Basti wrote: On 05.11.2015 03:48, Gabe Alford wrote: Hello, Fix for https://fedorahosted.org/freeipa/ticket/5374. I could reproduce it as the Custodia update file was missing from the updates Makefile which in turn was not being packaged into the rpms.

Re: [Freeipa-devel] is installation broken in upstream?

2015-11-05 Thread Oleg Fayans
Hi Martin, On 11/05/2015 11:36 AM, Martin Basti wrote: On 05.11.2015 11:28, Oleg Fayans wrote: Hi all, Having built the latest upstream code locally I am getting the following weird messages at the attempt to install the bits: $ dnf install freeipa-* -y Last metadata expiration check

Re: [Freeipa-devel] [Update]Time-Based Account Policies

2015-11-05 Thread Petr Spacek
On 4.11.2015 15:20, Martin Basti wrote: > > > On 04.11.2015 13:46, Stanislav Laznicka wrote: >> Hi, >> >> The fixed patches to Martin^2's and Jakub's reviews are almost ready, there >> are just a few things left. Martin B. mentioned in his review that '~' might >> not be the best delimiter for

Re: [Freeipa-devel] [PATCH 0092] ipa-replica-prepare: more robust and concise check for supported domain level

2015-11-05 Thread Petr Vobornik
On 10/29/2015 03:01 PM, Martin Babinsky wrote: On 10/29/2015 01:28 PM, Tomas Babej wrote: On 10/29/2015 01:10 PM, Petr Vobornik wrote: On 10/29/2015 11:19 AM, Martin Babinsky wrote: Petr^2 and Tomas were not happy by the way https://fedorahosted.org/freeipa/ticket/5175 was handled

Re: [Freeipa-devel] [PATCH 0338] Drop configure.jar file

2015-11-05 Thread Petr Vobornik
On 10/30/2015 09:31 AM, Martin Basti wrote: https://fedorahosted.org/freeipa/ticket/5144 Patch attached. configure.jar is basically a signed zip which contains preferences.html therefore also all references to preferences.html should be removed. ssbrowser.html: Also whole section

Re: [Freeipa-devel] [patch 0023] Applied tier0 and tier1 marks on unit tests and xmlrpc tests

2015-11-05 Thread Aleš Mareček
Looks good, ACK. - Original Message - > From: "Milan Kubík" > To: freeipa-devel@redhat.com > Sent: Thursday, November 5, 2015 12:20:29 PM > Subject: Re: [Freeipa-devel] [patch 0023] Applied tier0 and tier1 marks on > unit tests and xmlrpc tests > > On 11/05/2015

Re: [Freeipa-devel] [PATCH] ca-less tests updated - POC

2015-11-05 Thread Oleg Fayans
Patch 0014 updated and passes lint On 11/05/2015 03:41 PM, Oleg Fayans wrote: Wait a bit, the patch has problems with pylint: it does not build :) The updated version (without the setupmaster nonsense) is being tested now. On 11/05/2015 08:45 AM, Oleg Fayans wrote: Hi Jan, Could you take a

Re: [Freeipa-devel] [PATCH 0090] show optionally configured components in server-find/show command output

2015-11-05 Thread Petr Vobornik
On 11/02/2015 12:37 PM, Martin Kosek wrote: On 11/02/2015 06:10 AM, Jan Cholasta wrote: Hi, On 22.10.2015 10:44, Martin Babinsky wrote: https://fedorahosted.org/freeipa/ticket/5181 This should be handled by a separate object plugin: $ ipa servercomponent-find master.ipa.test

Re: [Freeipa-devel] [PATCH] ca-less tests updated - POC

2015-11-05 Thread Jan Cholasta
Hi Oleg, I think you can just remove TestCertinstall.test_{http,ds}_intermediate_ca, the certificates are imported correctly in this case and I didn't see anything break. Honza On 5.11.2015 20:20, Oleg Fayans wrote: Patch 0014 updated and passes lint On 11/05/2015 03:41 PM, Oleg Fayans

Re: [Freeipa-devel] [BLOG] FreeIPA PKI: current plans and a future vision

2015-11-05 Thread Simo Sorce
On 04/11/15 00:14, Fraser Tweedale wrote: I have been alluding for a while about my ideas for future FreeIPA/Dogtag PKI integration; I finally put the ideas down in a blog post. If you are interested in this aspect of IdM please read it; all feedback is welcome!

Re: [Freeipa-devel] [PATCH] ca-less tests updated - POC

2015-11-05 Thread Oleg Fayans
Wait a bit, the patch has problems with pylint: it does not build :) The updated version (without the setupmaster nonsense) is being tested now. On 11/05/2015 08:45 AM, Oleg Fayans wrote: Hi Jan, Could you take a look at these, whenever you are free? On 10/30/2015 02:57 PM, Oleg Fayans wrote: