Re: [Freeipa-devel] [PATCH] 550 webui-css: improve radio, checkbox keyboard support and color

2014-03-20 Thread Petr Vobornik
On 19.3.2014 16:32, Misnyovszki Adam wrote: On Fri, 14 Mar 2014 18:39:14 +0100 Petr Vobornik pvobo...@redhat.com wrote: On 13.3.2014 16:55, Petr Vobornik wrote: On 7.3.2014 15:34, Petr Vobornik wrote: checkboxes and radio buttons: - do not change color on hover when disabled - are focusable

Re: [Freeipa-devel] [PATCH] 552-557 Permissions v2 Web UI

2014-03-20 Thread Petr Vobornik
On 19.3.2014 17:03, Misnyovszki Adam wrote: On Wed, 19 Mar 2014 10:52:10 +0100 Petr Viktorin pvikt...@redhat.com wrote: On 03/18/2014 04:56 PM, Petr Vobornik wrote: On 18.3.2014 15:07, Petr Viktorin wrote: On 03/18/2014 01:09 PM, Petr Vobornik wrote: New revision for patch patch #557

Re: [Freeipa-devel] [PATCH] 559 webui: update license information of used third party code

2014-03-20 Thread Petr Vobornik
On 19.3.2014 16:19, Simo Sorce wrote: On Wed, 2014-03-19 at 16:02 +0100, Petr Vobornik wrote: I have some cleanup and refactoring patches which were created along with RCUE login effort. They are not that much connected so I'm sending them now (in 4 mails) to make the subsequent review

[Freeipa-devel] [PATCH] 464 Proxy PKI clone /ca/ee/ca/profileSubmit URI

2014-03-20 Thread Martin Kosek
PKI change done in ticket https://fedorahosted.org/pki/ticket/816 requires the PKI Clone's SSL Server certificate to be issued by it's associated PKI master. Allow this call on IPA master. https://fedorahosted.org/freeipa/ticket/4265 --- We will need this change in upcoming FreeIPA 3.3.5 which

Re: [Freeipa-devel] [PATCH] Review: rga-0005 Fix order of synchronizing time when running ipa-client-install

2014-03-20 Thread Petr Viktorin
On 03/18/2014 04:17 PM, Gabe Alford wrote: Patch was posted for review on Feb 19th, but may have missed due to volume, priorities, etc. We (freeipa commiters) try to give priority to reviewing patches, but this one fell through the cracks. Sorry for that! Please feel free ask for status or

Re: [Freeipa-devel] Talking json/rpc with java client

2014-03-20 Thread Massimiliano Perrone (tirasa.net)
On 03/18/2014 05:26 PM, Alexander Bokovoy wrote: On Tue, 18 Mar 2014, Massimiliano Perrone (example.com) wrote: The difference between the two calls is on the last TGS_REQ; because the first one is on ldap/olmo.example@example.com and it's OK whereas the second one is on

Re: [Freeipa-devel] Talking json/rpc with java client

2014-03-20 Thread Alexander Bokovoy
On Thu, 20 Mar 2014, Massimiliano Perrone (tirasa.net) wrote: On 03/18/2014 05:26 PM, Alexander Bokovoy wrote: On Tue, 18 Mar 2014, Massimiliano Perrone (example.com) wrote: The difference between the two calls is on the last TGS_REQ; because the first one is on

Re: [Freeipa-devel] [PATCH] [DOC] login as another user in the WebUI

2014-03-20 Thread Petr Vobornik
On 20.3.2014 00:03, Gabe Alford wrote: Thanks for the review. Here's an updated patch. Gabe ACK Pushed to master: befe9632c4636a151f5e4ed11a58e7e48ba8c877 On Wed, Mar 19, 2014 at 3:35 AM, Petr Vobornik pvobo...@redhat.com wrote: On 19.3.2014 02:29, Gabe Alford wrote: All, Just

Re: [Freeipa-devel] Talking json/rpc with java client

2014-03-20 Thread Rob Crittenden
Alexander Bokovoy wrote: On Thu, 20 Mar 2014, Massimiliano Perrone (tirasa.net) wrote: On 03/18/2014 05:26 PM, Alexander Bokovoy wrote: On Tue, 18 Mar 2014, Massimiliano Perrone (example.com) wrote: The difference between the two calls is on the last TGS_REQ; because the first one is on

Re: [Freeipa-devel] Talking json/rpc with java client

2014-03-20 Thread Alexander Bokovoy
On Thu, 20 Mar 2014, Rob Crittenden wrote: Alexander Bokovoy wrote: On Thu, 20 Mar 2014, Massimiliano Perrone (tirasa.net) wrote: On 03/18/2014 05:26 PM, Alexander Bokovoy wrote: On Tue, 18 Mar 2014, Massimiliano Perrone (example.com) wrote: The difference between the two calls is on the

[Freeipa-devel] [PATCH 0159] ipatests: test_trust: Change expected home directories for

2014-03-20 Thread Tomas Babej
Hi, Information from the AD about the home directories is not leveraged at all, but is generated from the username and domain. Fix the assumptions in the tests. Also changes 'Subdomain Test User' to 'Subdomaintest User' to be more consistent. https://fedorahosted.org/freeipa/ticket/4184 --

Re: [Freeipa-devel] Talking json/rpc with java client

2014-03-20 Thread Simo Sorce
On Thu, 2014-03-20 at 14:47 +0200, Alexander Bokovoy wrote: On Thu, 20 Mar 2014, Rob Crittenden wrote: Alexander Bokovoy wrote: On Thu, 20 Mar 2014, Massimiliano Perrone (tirasa.net) wrote: On 03/18/2014 05:26 PM, Alexander Bokovoy wrote: On Tue, 18 Mar 2014, Massimiliano Perrone

Re: [Freeipa-devel] [DOC] 0006 Chapter 5 update

2014-03-20 Thread Petr Viktorin
On 03/19/2014 01:43 PM, Martin Basti wrote: On Tue, 2014-03-18 at 14:34 +0100, Petr Viktorin wrote: On 09/26/2013 06:52 PM, Martin Basti wrote: Patch for chapter 5 Contains patch for ticket https://fedorahosted.org/freeipa/ticket/3842 Hello, Thanks for the patches! Next time, when you do

Re: [Freeipa-devel] [PATCH][RFC] 7 automember rebuild nowait feature added

2014-03-20 Thread Misnyovszki Adam
On Fri, 14 Mar 2014 13:26:15 -0400 Rob Crittenden rcrit...@redhat.com wrote: Misnyovszki Adam wrote: Hi, automember-rebuild uses asynchronous 389 task, and returned success even if the task didn't run. This patch fixes this issue adding a --nowait parameter to 'ipa automember-rebuild',

[Freeipa-devel] [PATCH] 0503 test_integration.tasks: Do not fail cleanup if backup directory does not exist

2014-03-20 Thread Petr Viktorin
Hello, This should fix a bug in integration tests. See commit message. -- PetrĀ³ From bef311329726c4625b61d003c3632538dbb15691 Mon Sep 17 00:00:00 2001 From: Petr Viktorin pvikt...@redhat.com Date: Thu, 20 Mar 2014 11:45:13 +0100 Subject: [PATCH] test_integration.tasks: Do not fail cleanup if

Re: [Freeipa-devel] Talking json/rpc with java client

2014-03-20 Thread Massimiliano Perrone (tirasa.net)
On 03/20/2014 01:29 PM, Rob Crittenden wrote: Alexander Bokovoy wrote: On Thu, 20 Mar 2014, Massimiliano Perrone (tirasa.net) wrote: On 03/18/2014 05:26 PM, Alexander Bokovoy wrote: On Tue, 18 Mar 2014, Massimiliano Perrone (example.com) wrote: The difference between the two calls is on the

Re: [Freeipa-devel] [PATCH] 0503 test_integration.tasks: Do not fail cleanup if backup directory does not exist

2014-03-20 Thread Martin Kosek
On 03/20/2014 02:26 PM, Petr Viktorin wrote: Hello, This should fix a bug in integration tests. See commit message. Looks good to me. Should fix I was seeing in Jenkins. ACK. Pushed to master, ipa-3-3. Martin ___ Freeipa-devel mailing list

Re: [Freeipa-devel] [PATCH][RFC] 7 automember rebuild nowait feature added

2014-03-20 Thread Misnyovszki Adam
On Thu, 20 Mar 2014 14:19:51 +0100 Misnyovszki Adam amisn...@redhat.com wrote: On Fri, 14 Mar 2014 13:26:15 -0400 Rob Crittenden rcrit...@redhat.com wrote: Misnyovszki Adam wrote: Hi, automember-rebuild uses asynchronous 389 task, and returned success even if the task didn't

Re: [Freeipa-devel] [PATCH] 560 webui: rename domNode to dom_node

2014-03-20 Thread Misnyovszki Adam
On Wed, 19 Mar 2014 16:02:12 +0100 Petr Vobornik pvobo...@redhat.com wrote: - unites domNode and dom_node usage to dom_node Nack, install/ui/test/details_tests.js:236 install/ui/test/details_tests.js:242

Re: [Freeipa-devel] [PATCHES] 172-196 Refactor certificate renewal code

2014-03-20 Thread Petr Spacek
On 19.3.2014 14:33, Jan Cholasta wrote: freeipa-jcholast-190.2-Store-information-about-which-CA-server-is-master-in.patch From 147ab524032902f29b8c3048cdaf21c5606f2274 Mon Sep 17 00:00:00 2001 From: Jan Cholastajchol...@redhat.com Date: Wed, 16 Oct 2013 08:51:06 + Subject: [PATCH 17/23]

Re: [Freeipa-devel] [PATCH 0159] ipatests: test_trust: Change expected home directories for

2014-03-20 Thread Jakub Hrozek
On Thu, Mar 20, 2014 at 01:49:30PM +0100, Tomas Babej wrote: Hi, Information from the AD about the home directories is not leveraged at all, but is generated from the username and domain. Fix the assumptions in the tests. Right, until we enhance the extop plugin to send the full set of

Re: [Freeipa-devel] [PATCHES] 0499-0502 permission CLI: rename --permissions to --right

2014-03-20 Thread Misnyovszki Adam
On Tue, 18 Mar 2014 12:02:06 +0100 Petr Viktorin pvikt...@redhat.com wrote: Hello, This renames --permissions to --right. The old name is kept as a deprecated alias. FreeIPA didn't have a mechanism for doing this, so I added one. Also, while I was digging around in this part, I made the new