Re: [Freeipa-devel] [patch 0032] ipatests: add missing certprofile fixture

2016-02-11 Thread Martin Babinsky
On 02/09/2016 04:06 PM, Milan Kubík wrote: On 02/09/2016 02:37 PM, Milan Kubík wrote: Fixes the CA ACL tests broken by removed import. This patch doesn't rely on undocumented behavior of pytest. The patch invalidates patch 133 by Martin Babinsky. Patch updated with trac link -- Milan

Re: [Freeipa-devel] [PATCH 0030] Modernize mod_nss's cipher suites

2016-02-11 Thread Martin Basti
On 03.02.2016 15:35, Christian Heimes wrote: On 2016-01-29 15:05, Martin Basti wrote: On 29.01.2016 14:42, Christian Heimes wrote: On 2016-01-28 09:47, Martin Basti wrote: On 22.01.2016 12:32, Martin Kosek wrote: On 01/21/2016 04:21 PM, Christian Heimes wrote: The list of supported TLS

Re: [Freeipa-devel] [PATCH 0407] make lint: migration to config file and pylint plugin due pylint 1.5.2

2016-02-11 Thread Martin Basti
On 09.02.2016 15:26, David Kupka wrote: On 09/02/16 11:28, Martin Basti wrote: On 03.02.2016 10:19, David Kupka wrote: On 01/02/16 14:18, Martin Basti wrote: On 26.01.2016 14:16, Martin Basti wrote: On 20.01.2016 14:38, Jan Cholasta wrote: Hi, On 19.1.2016 13:43, Martin Basti

[Freeipa-devel] [PATCH] 0005 webui: topology graph: canvas resizes itself according to the window size

2016-02-11 Thread Pavel Vomacka
Hello, The canvas of the graph had static size. This patch fixes this issue and from now the graph canvas is resized according to the window size. Pavel Vomacka >From 648037bf3952d6cb55c1951d55fad6b05f11f4b7 Mon Sep 17 00:00:00 2001 From: Pavel Vomacka Date: Tue, 9 Feb

Re: [Freeipa-devel] [patch 0032] ipatests: add missing certprofile fixture

2016-02-11 Thread Milan Kubík
On 02/11/2016 10:00 AM, Martin Babinsky wrote: On 02/09/2016 04:06 PM, Milan Kubík wrote: On 02/09/2016 02:37 PM, Milan Kubík wrote: Fixes the CA ACL tests broken by removed import. This patch doesn't rely on undocumented behavior of pytest. The patch invalidates patch 133 by Martin Babinsky.

[Freeipa-devel] INFO: changes to pylint

2016-02-11 Thread Martin Basti
Please note that current implementation of pylint check has been changed due to upcoming changes in pylint 1.5.x ./make-lint command has been removed To do lint check, please use * make lint * Details https://fedorahosted.org/freeipa/ticket/5615 -- Manage your subscription for the

[Freeipa-devel] [TESTS][PATCH 0009] WebUI tests fix

2016-02-11 Thread Lenka Doudova
Hi all, most of webUI tests fail with AssertionError: Can't click on checkbox label: table.table Message: Element is not clickable at point (37, 340.338964844). Other element would receive the click: The problem seems to be that the tests attempt to click on a checkbox label that is no

Re: [Freeipa-devel] [patch 0032] ipatests: add missing certprofile fixture

2016-02-11 Thread Martin Babinsky
On 02/11/2016 02:08 PM, Milan Kubík wrote: On 02/11/2016 10:00 AM, Martin Babinsky wrote: On 02/09/2016 04:06 PM, Milan Kubík wrote: On 02/09/2016 02:37 PM, Milan Kubík wrote: Fixes the CA ACL tests broken by removed import. This patch doesn't rely on undocumented behavior of pytest. The

Re: [Freeipa-devel] [PATCH] 0007 Refactor test_sudocmd_plugin

2016-02-11 Thread Aleš Mareček
NACK. "create_sudocmd" and "delete_sudocmd" should be placed in Tracker. So this patch should create the tracker as well. - Original Message - > From: "Filip Skola" > To: freeipa-devel@redhat.com > Sent: Monday, January 25, 2016 3:57:25 PM > Subject: [Freeipa-devel]

Re: [Freeipa-devel] [patch 0032] ipatests: add missing certprofile fixture

2016-02-11 Thread Martin Basti
On 11.02.2016 14:12, Martin Babinsky wrote: On 02/11/2016 02:08 PM, Milan Kubík wrote: On 02/11/2016 10:00 AM, Martin Babinsky wrote: On 02/09/2016 04:06 PM, Milan Kubík wrote: On 02/09/2016 02:37 PM, Milan Kubík wrote: Fixes the CA ACL tests broken by removed import. This patch doesn't

Re: [Freeipa-devel] [PATCH] 0008 Refactor test_sudocmdgroup_plugin, create SudoCmdGroupTracker

2016-02-11 Thread Aleš Mareček
NACK. "create_sudocmd" and "delete_sudocmd" should be imported from Tracker, not from the previous test (sudocmd_plugin). - alich - - Original Message - > From: "Filip Skola" > To: freeipa-devel@redhat.com > Sent: Thursday, January 28, 2016 12:49:17 PM > Subject:

Re: [Freeipa-devel] [PATCH 0030] Modernize mod_nss's cipher suites

2016-02-11 Thread Martin Kosek
On 02/11/2016 10:45 AM, Martin Basti wrote: > > > On 03.02.2016 15:35, Christian Heimes wrote: >> On 2016-01-29 15:05, Martin Basti wrote: >>> >>> On 29.01.2016 14:42, Christian Heimes wrote: On 2016-01-28 09:47, Martin Basti wrote: > On 22.01.2016 12:32, Martin Kosek wrote: >> On

Re: [Freeipa-devel] [PATCH 0411] upgrade: log to ipaupgrade.log if ipa is not installed

2016-02-11 Thread Martin Babinsky
On 01/29/2016 10:48 AM, Martin Basti wrote: Missing record in ipaupgrade.log that upgrade failed because IPA is not installed, causes harder time to debugging upgrade from log. Patch attached. ACK. -- Martin^3 Babinsky -- Manage your subscription for the Freeipa-devel mailing list:

Re: [Freeipa-devel] [PATCH 0411] upgrade: log to ipaupgrade.log if ipa is not installed

2016-02-11 Thread Martin Basti
On 11.02.2016 18:34, Martin Babinsky wrote: On 01/29/2016 10:48 AM, Martin Basti wrote: Missing record in ipaupgrade.log that upgrade failed because IPA is not installed, causes harder time to debugging upgrade from log. Patch attached. ACK. Pushed to master:

Re: [Freeipa-devel] [PATCH 544] replica promotion: fix AVC denials in remote connection check

2016-02-11 Thread Martin Basti
On 04.02.2016 11:18, Martin Babinsky wrote: On 02/04/2016 09:40 AM, Jan Cholasta wrote: Hi, the attached patch fixes . Honza Connection check for replica/CA install now works in enforcing mode, ACK. Pushed to: master:

Re: [Freeipa-devel] [PATCH 0407] make lint: migration to config file and pylint plugin due pylint 1.5.2

2016-02-11 Thread Martin Basti
On 11.02.2016 11:08, Martin Basti wrote: On 09.02.2016 15:26, David Kupka wrote: On 09/02/16 11:28, Martin Basti wrote: On 03.02.2016 10:19, David Kupka wrote: On 01/02/16 14:18, Martin Basti wrote: On 26.01.2016 14:16, Martin Basti wrote: On 20.01.2016 14:38, Jan Cholasta wrote: