Re: [Freeipa-devel] [PATCH 016, 024, 025] First part of the replica promotion tests + testplan

2016-03-03 Thread Martin Basti
On 04.03.2016 08:37, Martin Basti wrote: On 03.03.2016 18:38, Martin Basti wrote: On 02.03.2016 13:47, Oleg Fayans wrote: Hi Martin, I've made the requested changes. The full set of necessary patches is attached. On 03/02/2016 10:05 AM, Martin Basti wrote: On 02.03.2016 00:12, Oleg

Re: [Freeipa-devel] [PATCH 016, 024, 025] First part of the replica promotion tests + testplan

2016-03-03 Thread Martin Basti
On 03.03.2016 18:38, Martin Basti wrote: On 02.03.2016 13:47, Oleg Fayans wrote: Hi Martin, I've made the requested changes. The full set of necessary patches is attached. On 03/02/2016 10:05 AM, Martin Basti wrote: On 02.03.2016 00:12, Oleg Fayans wrote: Hi Martin, On 03/01/2016

Re: [Freeipa-devel] [PATCH 016, 024, 025] First part of the replica promotion tests + testplan

2016-03-03 Thread Martin Basti
On 02.03.2016 13:47, Oleg Fayans wrote: Hi Martin, I've made the requested changes. The full set of necessary patches is attached. On 03/02/2016 10:05 AM, Martin Basti wrote: On 02.03.2016 00:12, Oleg Fayans wrote: Hi Martin, On 03/01/2016 07:04 PM, Martin Basti wrote: On 01.03.2016

Re: [Freeipa-devel] [PATCH 0426] spec: add missing requires to python*-ipalib package

2016-03-03 Thread Martin Basti
On 26.02.2016 07:25, Jan Cholasta wrote: Hi, On 25.2.2016 18:05, Martin Basti wrote: https://fedorahosted.org/freeipa/ticket/5680 Patch attached. NACK. For python 3, the ldap module is provided by python3-pyldap. Any reason for the random ordering? The requires are not alphabetically

Re: [Freeipa-devel] [patch 0033] spec file: update the python-polib dependency name to python2-polib

2016-03-03 Thread Martin Basti
On 25.02.2016 11:07, Milan Kubík wrote: On 02/25/2016 11:07 AM, Jan Cholasta wrote: On 25.2.2016 11:03, Milan Kubík wrote: On 02/15/2016 05:39 PM, Lukas Slebodnik wrote: On (15/02/16 17:00), Petr Vobornik wrote: On 02/15/2016 04:37 PM, Milan Kubík wrote: Reflect the updated name of the

Re: [Freeipa-devel] host-del & client uninstall: additional discussion related to DNS needed

2016-03-03 Thread Martin Basti
On 03.03.2016 17:36, Petr Vobornik wrote: On 03/03/2016 03:52 PM, Martin Basti wrote: Hello all, related tickets: https://fedorahosted.org/freeipa/ticket/5676 https://fedorahosted.org/freeipa/ticket/5675 https://fedorahosted.org/freeipa/ticket/5715 I'm trying to implement both tickets, but

Re: [Freeipa-devel] [python-pytest-multihost] Request to add support to specify username/password for each host

2016-03-03 Thread Niranjan
Petr Viktorin wrote: > On 03/01/2016 05:18 PM, Petr Viktorin wrote: > > On 02/29/2016 05:57 PM, Niranjan wrote: > >> Greetings, > >> > >> Attached wrong patch in my last mail. Request to review the patch > >> attached to this email > >> > > > > Hello, > > I've generalized the patch a bit to make

Re: [Freeipa-devel] host-del & client uninstall: additional discussion related to DNS needed

2016-03-03 Thread Petr Vobornik
On 03/03/2016 03:52 PM, Martin Basti wrote: Hello all, related tickets: https://fedorahosted.org/freeipa/ticket/5676 https://fedorahosted.org/freeipa/ticket/5675 https://fedorahosted.org/freeipa/ticket/5715 I'm trying to implement both tickets, but I don't like the way we decided on devel

Re: [Freeipa-devel] [PATCH] 951 webui: fail nicely if cookies are disabled

2016-03-03 Thread Tomas Babej
On 01/28/2016 04:25 PM, Petr Vobornik wrote: > On 01/28/2016 04:23 PM, Tomas Babej wrote: >> >> >> On 01/28/2016 04:15 PM, Petr Vobornik wrote: >>> Reworks also sessionStorage test because disablement of cookies might be >>> connected with sessionStorage and localStorage. E.g. Chrome raises >>>

Re: [Freeipa-devel] [PATCH 0422] CI: allow customized DS install test to run under different domain levels

2016-03-03 Thread Martin Basti
On 03.03.2016 17:26, Milan Kubík wrote: On 03/02/2016 04:50 PM, Martin Basti wrote: On 24.02.2016 19:01, Martin Basti wrote: https://fedorahosted.org/freeipa/ticket/5606 Patch attached. Bump for review Works for me, ACK. -- Milan Kubik Pushed to: master:

Re: [Freeipa-devel] [PATCH 0422] CI: allow customized DS install test to run under different domain levels

2016-03-03 Thread Milan Kubík
On 03/02/2016 04:50 PM, Martin Basti wrote: On 24.02.2016 19:01, Martin Basti wrote: https://fedorahosted.org/freeipa/ticket/5606 Patch attached. Bump for review Works for me, ACK. -- Milan Kubik -- Manage your subscription for the Freeipa-devel mailing list:

Re: [Freeipa-devel] [pytest-multihost-devel][PATCH] Add reset_connection to BaseHost class

2016-03-03 Thread Petr Viktorin
On 02/12/2016 09:23 PM, Scott Poore wrote: > Hi, > > Please review attached patch for pytest-multihost plugin. > > Fixes: https://fedorahosted.org/python-pytest-multihost/ticket/4 Hi, Sorry for the delay (and thanks for pinging me privately). I've released v1.0 with this patch. It should be

Re: [Freeipa-devel] host-del & client uninstall: additional discussion related to DNS needed

2016-03-03 Thread Jan Cholasta
Hi, On 3.3.2016 15:52, Martin Basti wrote: Hello all, related tickets: https://fedorahosted.org/freeipa/ticket/5676 https://fedorahosted.org/freeipa/ticket/5675 https://fedorahosted.org/freeipa/ticket/5715 I'm trying to implement both tickets, but I don't like the way we decided on devel

Re: [Freeipa-devel] [python-pytest-multihost] Request to add support to specify username/password for each host

2016-03-03 Thread Petr Viktorin
On 03/01/2016 05:18 PM, Petr Viktorin wrote: > On 02/29/2016 05:57 PM, Niranjan wrote: >> Greetings, >> >> Attached wrong patch in my last mail. Request to review the patch >> attached to this email >> > > Hello, > I've generalized the patch a bit to make it use existing host > attributes, and I

Re: [Freeipa-devel] [PATCH 0400] l10n: Remove Transifex configuration

2016-03-03 Thread Martin Basti
On 25.02.2016 11:40, Tomas Babej wrote: Hi, We're not using Transifex to manage our translations anymore. Tomas ACK Pushed to master: 61e627b4b3b9a9547128f128410d24b3cbe81b13 -- Manage your subscription for the Freeipa-devel mailing list:

Re: [Freeipa-devel] [PATCH 0400] l10n: Remove Transifex configuration

2016-03-03 Thread Martin Basti
On 25.02.2016 11:40, Tomas Babej wrote: Hi, We're not using Transifex to manage our translations anymore. Tomas ACK Pushed to master: 61e627b4b3b9a9547128f128410d24b3cbe81b13 -- Manage your subscription for the Freeipa-devel mailing list:

Re: [Freeipa-devel] [PATCH 0403] adtrustinstance: Make sure smb.conf exists

2016-03-03 Thread Martin Basti
On 26.02.2016 14:32, Tomas Babej wrote: Hi, The 'net' command fails unless smb.conf exists. Touch the file prior to any 'net' call to make sure we do not crash for this very reason. I couldn't find the aforementioned Samba bug either in the RH/Samba bugzilla, despite spending non-trivial

[Freeipa-devel] host-del & client uninstall: additional discussion related to DNS needed

2016-03-03 Thread Martin Basti
Hello all, related tickets: https://fedorahosted.org/freeipa/ticket/5676 https://fedorahosted.org/freeipa/ticket/5675 https://fedorahosted.org/freeipa/ticket/5715 I'm trying to implement both tickets, but I don't like the way we decided on devel meeting anymore.

Re: [Freeipa-devel] [PATCHES] 0772-0774 Python3 fixes in for client installation

2016-03-03 Thread Jan Cholasta
On 2.3.2016 14:58, Petr Viktorin wrote: On 03/01/2016 08:57 AM, Jan Cholasta wrote: On 29.2.2016 17:14, Petr Viktorin wrote: On 02/29/2016 08:12 AM, Jan Cholasta wrote: Hi, On 26.2.2016 17:14, Petr Viktorin wrote: On 02/22/2016 12:37 PM, Petr Viktorin wrote: Hello, These fixes are needed

Re: [Freeipa-devel] [PATCHES 548-549] ipalib: add convenient Command method for adding messages

2016-03-03 Thread Martin Basti
On 02.03.2016 15:35, Jan Cholasta wrote: Hi, these patches are for master only, see commit messages for details. Honza I love these patches. ACK, it works! Pushed to master: 3c57c305add17b95d4fb962efd9e5dfc9bd35efe -- Manage your subscription for the Freeipa-devel mailing list:

Re: [Freeipa-devel] [bind-dyndb-ldap] [PATCH] SPEC: Add missing build dependency

2016-03-03 Thread Lukas Slebodnik
On (01/03/16 14:35), Petr Spacek wrote: >On 1.3.2016 12:06, Lukas Slebodnik wrote: >> On (25/02/16 15:57), Petr Spacek wrote: >>> On 19.2.2016 13:55, Petr Spacek wrote: Hello, Fix build with GCC 4.9+. GCC 4.9+ is too aggressive when optimizing functions with nonnull

Re: [Freeipa-devel] [PATCH 0390] Fix build with GCC 4.9+

2016-03-03 Thread Lukas Slebodnik
On (01/03/16 14:35), Petr Spacek wrote: >On 1.3.2016 12:06, Lukas Slebodnik wrote: >> On (25/02/16 15:57), Petr Spacek wrote: >>> On 19.2.2016 13:55, Petr Spacek wrote: Hello, Fix build with GCC 4.9+. GCC 4.9+ is too aggressive when optimizing functions with nonnull