Re: [Freeipa-devel] [PATCH 0031] provide a dedicated ccache file to httpd

2015-05-12 Thread Martin Babinsky
On 05/12/2015 07:50 AM, Alexander Bokovoy wrote: On Mon, 04 May 2015, Martin Babinsky wrote: On 04/30/2015 08:23 AM, Alexander Bokovoy wrote: On Thu, 30 Apr 2015, Jan Cholasta wrote: Hi, Dne 29.4.2015 v 19:42 Martin Babinsky napsal(a): The attached patch is a merge of PATCHES 0031-0032

[Freeipa-devel] [PATCH 0245] Fix uniqueness plugins vol. 2

2015-05-12 Thread Martin Basti
https://fedorahosted.org/freeipa/ticket/4921 To test this, the mkosek/freeipa-master copr repo with 389-ds-base 1.3.4.0 is needed. All previous changes to uniqueness plugins were made just in master branch so upgrade will not work correctly from master to newer master. From IPA 4.1 to

Re: [Freeipa-devel] [PATCH 0031] provide a dedicated ccache file to httpd

2015-05-12 Thread Alexander Bokovoy
On Tue, 12 May 2015, Petr Vobornik wrote: On 05/12/2015 11:22 AM, Alexander Bokovoy wrote: On Tue, 12 May 2015, Martin Babinsky wrote: %attr(644,root,root) %{_unitdir}/ipa-ods-exporter.service +%attr(644,root,root) %{etc_systemd_dir}/httpd.service There is a minor issue: a lack of Requires:

Re: [Freeipa-devel] [PATCH 0245] Fix uniqueness plugins vol. 2

2015-05-12 Thread Martin Basti
On 12/05/15 18:23, Martin Basti wrote: https://fedorahosted.org/freeipa/ticket/4921 To test this, the mkosek/freeipa-master copr repo with 389-ds-base 1.3.4.0 is needed. All previous changes to uniqueness plugins were made just in master branch so upgrade will not work correctly from master

Re: [Freeipa-devel] [PATCH] 819-820 jQuery.ordered_map: faster creation

2015-05-12 Thread Petr Vobornik
On 05/11/2015 11:47 AM, Martin Babinsky wrote: On 04/17/2015 04:31 PM, Petr Vobornik wrote: Improve performance of Web UI with very large user groups or any other usage which adds a lot of values into jQuery.ordered_map. ACK master: * 68f04643d6022707e8ccac9f8b817542d9022119

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

2015-05-12 Thread Endi Sukma Dewata
Please take a look at the attached patch (#353-9). It obsoletes all previous patches. See comments below. On 4/20/2015 1:12 AM, Jan Cholasta wrote: I'm planning to merge the vault and vault container object and use the vault type attribute to distinguish between the two. See more discussion

Re: [Freeipa-devel] [PATCH 0238] Server Upgrade: fix broken memberUid index

2015-05-12 Thread Petr Vobornik
On 05/07/2015 01:48 PM, Martin Babinsky wrote: On 05/06/2015 01:23 PM, Martin Basti wrote: On 06/05/15 13:22, Martin Basti wrote: https://fedorahosted.org/freeipa/ticket/5007 Patch attached. Requires patch mbasti-231-4 -- Martin Basti ACK Pushed to master:

Re: [Freeipa-devel] [PATCH 0031] provide a dedicated ccache file to httpd

2015-05-12 Thread Alexander Bokovoy
On Tue, 12 May 2015, Martin Babinsky wrote: %attr(644,root,root) %{_unitdir}/ipa-ods-exporter.service +%attr(644,root,root) %{etc_systemd_dir}/httpd.service There is a minor issue: a lack of Requires: /etc/systemd/system which is needed because of /etc/systemd/system directory owned by a

Re: [Freeipa-devel] [PATCH 0031] provide a dedicated ccache file to httpd

2015-05-12 Thread Petr Vobornik
On 05/12/2015 11:22 AM, Alexander Bokovoy wrote: On Tue, 12 May 2015, Martin Babinsky wrote: %attr(644,root,root) %{_unitdir}/ipa-ods-exporter.service +%attr(644,root,root) %{etc_systemd_dir}/httpd.service There is a minor issue: a lack of Requires: /etc/systemd/system which is needed

Re: [Freeipa-devel] [PATCH] 0048 Remove unused enable() method from DogtagInstance.

2015-05-12 Thread Martin Basti
On 12/05/15 13:53, David Kupka wrote: DogtagInstance.enable() overrides Service.enable() and does nothing usefulll. Also removing it solves bug discovered recently in uninstall procedure. NACK 1) this is used in step() call, which means the tomcatd will be enabled in systemd, and we do not

Re: [Freeipa-devel] [PATCH] 801-806 webui-ci: otptoken tests

2015-05-12 Thread Petr Vobornik
On 05/11/2015 01:25 PM, Milan Kubik wrote: On 05/07/2015 01:38 PM, Petr Vobornik wrote: On 02/19/2015 03:51 PM, Petr Vobornik wrote: https://fedorahosted.org/freeipa/ticket/4307 For ipa-4-1 apply: - patch 800 (different thread) - patches 801-806 For master apply: - patch 800 (different

[Freeipa-devel] [PATCH] 0048 Remove unused enable() method from DogtagInstance.

2015-05-12 Thread David Kupka
DogtagInstance.enable() overrides Service.enable() and does nothing usefulll. Also removing it solves bug discovered recently in uninstall procedure. -- David Kupka From dbf020b1703a9cf2f4d3614f4c9caa83f340f571 Mon Sep 17 00:00:00 2001 From: David Kupka dku...@redhat.com Date: Tue, 12 May 2015

[Freeipa-devel] [PATCHES 0239-0243] Server Upgrade: minor fixes

2015-05-12 Thread Martin Basti
https://fedorahosted.org/freeipa/ticket/4904 Patches attached. -- Martin Basti From 3cf02ba87d2e64e04d1b4aaeca616d63cb2149d9 Mon Sep 17 00:00:00 2001 From: Martin Basti mba...@redhat.com Date: Thu, 7 May 2015 11:03:27 +0200 Subject: [PATCH 1/5] Server Upgrade: Do not start DS if it was stopped

[Freeipa-devel] [PATCH 0244] DNSSEC: fix traceback in ipa-dnskeysyncd during shutdown phase

2015-05-12 Thread Martin Basti
https://fedorahosted.org/freeipa/ticket/4657 Patch attached. -- Martin Basti From bfb0788c170238a4e1326c4859246099b591dcb8 Mon Sep 17 00:00:00 2001 From: Martin Basti mba...@redhat.com Date: Mon, 11 May 2015 14:04:29 +0200 Subject: [PATCH] DNSSEC: fix traceback during shutdown phase

Re: [Freeipa-devel] [PATCH] 0178 Fix AD trusts in Fedora 22

2015-05-12 Thread Alexander Bokovoy
On Tue, 12 May 2015, Alexander Bokovoy wrote: On Fri, 08 May 2015, Alexander Bokovoy wrote: Hi, attached patch fixes issues with Samba 4.2 in Fedora 22. See commit message for the details. Note that you'll also need Samba fixes from https://bugzilla.redhat.com/show_bug.cgi?id=1219832 to test

Re: [Freeipa-devel] [PATCH] 814-818 migrate-ds: optimize adding users to default group

2015-05-12 Thread Petr Vobornik
On 05/07/2015 07:01 PM, Martin Basti wrote: On 07/05/15 13:17, Petr Vobornik wrote: On 05/06/2015 04:49 PM, Martin Basti wrote: On 05/05/15 10:59, Petr Vobornik wrote: On 04/30/2015 03:53 PM, Martin Basti wrote: On 10/04/15 12:55, Petr Vobornik wrote: The essential patch is 814. snip

Re: [Freeipa-devel] [PATCH] 825 webui-ci: fix type error in host_tasks inicializations

2015-05-12 Thread Petr Vobornik
On 05/12/2015 12:30 PM, Martin Babinsky wrote: On 05/06/2015 01:37 PM, Petr Vobornik wrote: host_tasks initializations were not modified along with pytest migration ACK Pushed to master: 5fadee969cae3b66b986fb9e28fdbf3bb5ec2f60 -- Petr Vobornik -- Manage your subscription for the

Re: [Freeipa-devel] [PATCH] 0178 Fix AD trusts in Fedora 22

2015-05-12 Thread Alexander Bokovoy
On Tue, 12 May 2015, Alexander Bokovoy wrote: On Tue, 12 May 2015, Alexander Bokovoy wrote: On Fri, 08 May 2015, Alexander Bokovoy wrote: Hi, attached patch fixes issues with Samba 4.2 in Fedora 22. See commit message for the details. Note that you'll also need Samba fixes from

Re: [Freeipa-devel] [PATCH] 825 webui-ci: fix type error in host_tasks inicializations

2015-05-12 Thread Martin Babinsky
On 05/06/2015 01:37 PM, Petr Vobornik wrote: host_tasks initializations were not modified along with pytest migration ACK -- Martin^3 Babinsky -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

Re: [Freeipa-devel] [PATCH] 0048 Remove unused enable() method from DogtagInstance.

2015-05-12 Thread David Kupka
On 05/12/2015 02:16 PM, Martin Basti wrote: On 12/05/15 13:53, David Kupka wrote: DogtagInstance.enable() overrides Service.enable() and does nothing usefulll. Also removing it solves bug discovered recently in uninstall procedure. NACK 1) this is used in step() call, which means the tomcatd

[Freeipa-devel] [PATCH] 829 webui: option to not create user private group

2015-05-12 Thread Petr Vobornik
Web UI was not able to create a user without a private group. New field added to user adder dialog to allow that. https://fedorahosted.org/freeipa/ticket/4986 -- Petr Vobornik From 3d6114b820d60d97392e8a014f2fcfa239d69cad Mon Sep 17 00:00:00 2001 From: Petr Vobornik pvobo...@redhat.com Date: