[Freeipa-devel] [freeipa PR#100][comment] Ipa 4 4

2016-09-22 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/100 Title: #100: Ipa 4 4 dkupka commented: """ > hi, my issue is not in deployment my issue is a bug in the software itself i > have already submitted a ticket with redhat support which is taking too much > time so i hoped you guys can see this sinc

[Freeipa-devel] [freeipa PR#102][+ack] Updated ipa-client-install info about hostname

2016-09-22 Thread tomaskrizek
URL: https://github.com/freeipa/freeipa/pull/102 Title: #102: Updated ipa-client-install info about hostname Label: +ack -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/page/Contr

[Freeipa-devel] [freeipa PR#84][synchronized] Fix update_from_dict function testing

2016-09-22 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/84 Author: stlaz Title: #84: Fix update_from_dict function testing Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/84/head:pr84 git checkout pr84 From 45b739afe8

[Freeipa-devel] [freeipa PR#84][comment] Fix update_from_dict function testing

2016-09-22 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/84 Title: #84: Fix update_from_dict function testing stlaz commented: """ Removed the method and its respective test again. """ See the full comment at https://github.com/freeipa/freeipa/pull/84#issuecomment-248832033 -- Manage your subscription fo

[Freeipa-devel] [freeipa PR#84][edited] Removed update_from_dict function from ldapupdate

2016-09-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/84 Author: stlaz Title: #84: Removed update_from_dict function from ldapupdate Action: edited To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/84/head:pr84 git checkout pr84 -- Manag

[Freeipa-devel] [freeipa PR#102][closed] Updated ipa-client-install info about hostname

2016-09-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/102 Author: stlaz Title: #102: Updated ipa-client-install info about hostname Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/102/head:pr102 git checkout pr102 -- Man

[Freeipa-devel] [freeipa PR#102][+pushed] Updated ipa-client-install info about hostname

2016-09-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/102 Title: #102: Updated ipa-client-install info about hostname Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/page/Co

[Freeipa-devel] [freeipa PR#102][comment] Updated ipa-client-install info about hostname

2016-09-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/102 Title: #102: Updated ipa-client-install info about hostname mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/2e0afab5f2a47149580b4bc79093cdbb77f489c3 ipa-4-4: https://fedorahosted.org/freeipa/changeset/e2a

[Freeipa-devel] [freeipa PR#97][comment] Pylint fixes

2016-09-22 Thread tomaskrizek
URL: https://github.com/freeipa/freeipa/pull/97 Title: #97: Pylint fixes tomaskrizek commented: """ NACK, please see inline comments """ See the full comment at https://github.com/freeipa/freeipa/pull/97#issuecomment-248867576 -- Manage your subscription for the Freeipa-devel mailing list: ht

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

2016-09-22 Thread Oleg Fayans
Fixed patch N 41 On 09/21/2016 04:21 PM, Oleg Fayans wrote: Patch-0076 rebased to current master On 09/21/2016 02:41 PM, Oleg Fayans wrote: Hi David, As per your comments the patches were once again refactored. I am attaching the full set of them, please ignore any previous versions The patch

[Freeipa-devel] [freeipa PR#84][comment] Removed update_from_dict function from ldapupdate

2016-09-22 Thread abbra
URL: https://github.com/freeipa/freeipa/pull/84 Title: #84: Removed update_from_dict function from ldapupdate abbra commented: """ LGTM. Thanks. """ See the full comment at https://github.com/freeipa/freeipa/pull/84#issuecomment-248869910 -- Manage your subscription for the Freeipa-devel mail

[Freeipa-devel] [freeipa PR#84][+ack] Removed update_from_dict function from ldapupdate

2016-09-22 Thread abbra
URL: https://github.com/freeipa/freeipa/pull/84 Title: #84: Removed update_from_dict function from ldapupdate Label: +ack -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/page/Cont

[Freeipa-devel] [freeipa PR#84][comment] Removed update_from_dict function from ldapupdate

2016-09-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/84 Title: #84: Removed update_from_dict function from ldapupdate mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/330a3ca93101bcec82ec5d3add14586871864bdd ipa-4-4: https://fedorahosted.org/freeipa/changeset/fd

[Freeipa-devel] [freeipa PR#84][closed] Removed update_from_dict function from ldapupdate

2016-09-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/84 Author: stlaz Title: #84: Removed update_from_dict function from ldapupdate Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/84/head:pr84 git checkout pr84 -- Manag

[Freeipa-devel] [freeipa PR#84][+pushed] Removed update_from_dict function from ldapupdate

2016-09-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/84 Title: #84: Removed update_from_dict function from ldapupdate Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/page/C

[Freeipa-devel] [freeipa PR#104][synchronized] Backport XMLRPC test fixes to ipa-4-3 branch

2016-09-22 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/104 Author: martbab Title: #104: Backport XMLRPC test fixes to ipa-4-3 branch Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/104/head:pr104 git checkout pr104 F

[Freeipa-devel] [freeipa PR#97][synchronized] Pylint fixes

2016-09-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/97 Author: mbasti-rh Title: #97: Pylint fixes Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/97/head:pr97 git checkout pr97 From 9c4208a527c22685fa79c32080fb036

Re: [Freeipa-devel] What would break if loopback addresses were allowed for IPA server?

2016-09-22 Thread Martin Basti
On 21.09.2016 12:01, Jan Pazdziora wrote: Hello, I've recently hit again the situation of IPA installer not happy about the provided IP address not being local to it, this time in containerized environment: https://bugzilla.redhat.com/show_bug.cgi?id=1377973 During the discussion, we

[Freeipa-devel] Suspicious IPA cert test fail after upgrade to pki-ca-10.3.5-6

2016-09-22 Thread Martin Basti
Hello all, Following test is failing: test_cert_find.test_0007_find_revocation_reason_0 self = 0x7f1bf4532f90> def test_00

Re: [Freeipa-devel] Suspicious IPA cert test fail after upgrade to pki-ca-10.3.5-6

2016-09-22 Thread Martin Babinsky
On 09/22/2016 01:41 PM, Martin Basti wrote: Hello all, Following test is failing: test_cert_find.test_0007_find_revocation_reason_0

Re: [Freeipa-devel] Suspicious IPA cert test fail after upgrade to pki-ca-10.3.5-6

2016-09-22 Thread Martin Basti
On 22.09.2016 13:56, Martin Babinsky wrote: On 09/22/2016 01:41 PM, Martin Basti wrote: Hello all, Following test is failing: test_cert_find.test_0007_find_revocation_reason_0 __

[Freeipa-devel] [freeipa PR#97][comment] Pylint fixes

2016-09-22 Thread tomaskrizek
URL: https://github.com/freeipa/freeipa/pull/97 Title: #97: Pylint fixes tomaskrizek commented: """ Please fix the typo. """ See the full comment at https://github.com/freeipa/freeipa/pull/97#issuecomment-248884825 -- Manage your subscription for the Freeipa-devel mailing list: https://www.re

[Freeipa-devel] [freeipa PR#105][opened] Test: dont use global variable for iteration in test_cert_plugin

2016-09-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/105 Author: mbasti-rh Title: #105: Test: dont use global variable for iteration in test_cert_plugin Action: opened PR body: """ Iteration over global variable causes unwanted value changes outside method https://fedorahosted.org/freeipa/ticket/5755

[Freeipa-devel] [freeipa PR#97][synchronized] Pylint fixes

2016-09-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/97 Author: mbasti-rh Title: #97: Pylint fixes Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/97/head:pr97 git checkout pr97 From fc4c31912df4d004f04700fa2a4f4ef

[Freeipa-devel] [freeipa PR#97][synchronized] Pylint fixes

2016-09-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/97 Author: mbasti-rh Title: #97: Pylint fixes Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/97/head:pr97 git checkout pr97 From fc4c31912df4d004f04700fa2a4f4ef

[Freeipa-devel] [freeipa PR#105][+ack] Test: dont use global variable for iteration in test_cert_plugin

2016-09-22 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/105 Title: #105: Test: dont use global variable for iteration in test_cert_plugin Label: +ack -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.free

[Freeipa-devel] [freeipa PR#105][+pushed] Test: dont use global variable for iteration in test_cert_plugin

2016-09-22 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/105 Title: #105: Test: dont use global variable for iteration in test_cert_plugin Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.f

[Freeipa-devel] [freeipa PR#105][closed] Test: dont use global variable for iteration in test_cert_plugin

2016-09-22 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/105 Author: mbasti-rh Title: #105: Test: dont use global variable for iteration in test_cert_plugin Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/105/head:pr105 git

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

2016-09-22 Thread David Kupka
Hi Oleg! As we discussed this morning there're still some issues [1][2] but I think the patch set is in reasonably good state and I'm not sure that the issues are bugs in IPA or in tests. This can be investigated and fixed later. ACK. Pushed to master: * bbac233b5ee487ab0e035cf0b861144769a0

[Freeipa-devel] [freeipa PR#105][comment] Test: dont use global variable for iteration in test_cert_plugin

2016-09-22 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/105 Title: #105: Test: dont use global variable for iteration in test_cert_plugin martbab commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/929086e0992cc32a654b4dfa435f536ecb0c665b """ See the full comment at https://g

[Freeipa-devel] [freeipa PR#97][+ack] Pylint fixes

2016-09-22 Thread tomaskrizek
URL: https://github.com/freeipa/freeipa/pull/97 Title: #97: Pylint fixes Label: +ack -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#97][synchronized] Pylint fixes

2016-09-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/97 Author: mbasti-rh Title: #97: Pylint fixes Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/97/head:pr97 git checkout pr97 From 6f16460d67397c2b0aa5fd5d7066c5c

[Freeipa-devel] [freeipa PR#97][comment] Pylint fixes

2016-09-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/97 Title: #97: Pylint fixes mbasti-rh commented: """ I had to rebase, there were conflicts due the latest pushed patches """ See the full comment at https://github.com/freeipa/freeipa/pull/97#issuecomment-248913108 -- Manage your subscription for t

[Freeipa-devel] pylint: remove unused variables

2016-09-22 Thread Martin Basti
Hello all, In 4.5, I would like to remove all unused variables from code and enable pylint check. Due to big amount of unused variables in the code this will be longterm effort. Why this?: * better code readability * removing dead code * unused variable may uncover potential bug It is cl

[Freeipa-devel] [freeipa PR#97][closed] Pylint fixes

2016-09-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/97 Author: mbasti-rh Title: #97: Pylint fixes Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/97/head:pr97 git checkout pr97 -- Manage your subscription for the Freei

[Freeipa-devel] [freeipa PR#106][opened] Pylint: enable additional checks

2016-09-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/106 Author: mbasti-rh Title: #106: Pylint: enable additional checks Action: opened PR body: """ Enabling: * cyclic-imports * global-variable-not-assigned """ To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa

[Freeipa-devel] [freeipa PR#97][comment] Pylint fixes

2016-09-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/97 Title: #97: Pylint fixes mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/36484e8672f5ee1fdc2bd57622e330ab8dbb7671 https://fedorahosted.org/freeipa/changeset/275e85d076607ce317b3aeca467167fac55bf396 https:/

[Freeipa-devel] [freeipa PR#97][+pushed] Pylint fixes

2016-09-22 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/97 Title: #97: Pylint fixes Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#107][opened] Update man/help for --server option

2016-09-22 Thread tomaskrizek
URL: https://github.com/freeipa/freeipa/pull/107 Author: tomaskrizek Title: #107: Update man/help for --server option Action: opened PR body: """ The --server option now specifically mentions that it expects the FQDN of the IPA server. https://fedorahosted.org/freeipa/ticket/6202 """ To pull

Re: [Freeipa-devel] pylint: remove unused variables

2016-09-22 Thread Tomas Krizek
On 09/22/2016 04:39 PM, Martin Basti wrote: Hello all, In 4.5, I would like to remove all unused variables from code and enable pylint check. Due to big amount of unused variables in the code this will be longterm effort. Why this?: * better code readability * removing dead code * unused

Re: [Freeipa-devel] pylint: remove unused variables

2016-09-22 Thread Martin Basti
On 22.09.2016 17:59, Tomas Krizek wrote: On 09/22/2016 04:39 PM, Martin Basti wrote: Hello all, In 4.5, I would like to remove all unused variables from code and enable pylint check. Due to big amount of unused variables in the code this will be longterm effort. Why this?: * better code

Re: [Freeipa-devel] pylint: remove unused variables

2016-09-22 Thread Tomas Krizek
On 09/22/2016 06:00 PM, Martin Basti wrote: On 22.09.2016 17:59, Tomas Krizek wrote: On 09/22/2016 04:39 PM, Martin Basti wrote: Hello all, In 4.5, I would like to remove all unused variables from code and enable pylint check. Due to big amount of unused variables in the code this will be

[Freeipa-devel] FedoraHosted.org sunset

2016-09-22 Thread Petr Vobornik
Hi all, As you know, FedoraHosted.org will be decommissioned. https://communityblog.fedoraproject.org/fedorahosted-sunset-2017-02-28/ We use Trac instance there. Let's discuss where we should migrate and what are our requirements. Then put results on one place. For that I've created: http://ww

Re: [Freeipa-devel] pylint: remove unused variables

2016-09-22 Thread Martin Basti
On 22.09.2016 18:05, Tomas Krizek wrote: On 09/22/2016 06:00 PM, Martin Basti wrote: On 22.09.2016 17:59, Tomas Krizek wrote: On 09/22/2016 04:39 PM, Martin Basti wrote: Hello all, In 4.5, I would like to remove all unused variables from code and enable pylint check. Due to big amount of

Re: [Freeipa-devel] [PATCH] 0097 Add options to write lightweight CA cert or chain to file

2016-09-22 Thread Fraser Tweedale
Bump for review. Rebased patches attached (there was a trivial conflict in imports). Thanks, Fraser On Tue, Sep 06, 2016 at 02:05:06AM +1000, Fraser Tweedale wrote: > On Fri, Aug 26, 2016 at 10:28:58AM +0200, Jan Cholasta wrote: > > On 19.8.2016 13:11, Fraser Tweedale wrote: > > > Bump for revie

Re: [Freeipa-devel] [PATCH] 0107 Fix cert revocation when removing all certs via host/service-mod

2016-09-22 Thread Fraser Tweedale
Bump for review. On Wed, Sep 07, 2016 at 04:06:25PM +0700, Fraser Tweedale wrote: > Attached patch fixes https://fedorahosted.org/freeipa/ticket/6305 > > Thanks, > Fraser > From d4d7e77795f96a4970058e61d99c70522689b22d Mon Sep 17 00:00:00 2001 > From: Fraser Tweedale > Date: Wed, 7 Sep 2016 19:

Re: [Freeipa-devel] pylint: remove unused variables

2016-09-22 Thread Jan Cholasta
On 22.9.2016 16:39, Martin Basti wrote: Hello all, In 4.5, I would like to remove all unused variables from code and enable pylint check. Due to big amount of unused variables in the code this will be longterm effort. Why this?: * better code readability * removing dead code * unused variabl

Re: [Freeipa-devel] [PATCH] 0107 Fix cert revocation when removing all certs via host/service-mod

2016-09-22 Thread Jan Cholasta
On 23.9.2016 05:30, Fraser Tweedale wrote: Bump for review. Works for me, ACK. Pushed to master: 97d4ffc2dc5db00fd7ed10b0b290cc97a506d0ef -- Jan Cholasta -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIP

[Freeipa-devel] [freeipa PR#106][+ack] Pylint: enable additional checks

2016-09-22 Thread tomaskrizek
URL: https://github.com/freeipa/freeipa/pull/106 Title: #106: Pylint: enable additional checks Label: +ack -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#108][opened] https://fedorahosted.org/freeipa/ticket/6256

2016-09-22 Thread frasertweedale
URL: https://github.com/freeipa/freeipa/pull/108 Author: frasertweedale Title: #108: https://fedorahosted.org/freeipa/ticket/6256 Action: opened PR body: """ None """ To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/108/head:pr

[Freeipa-devel] [freeipa PR#108][edited] Bump pki min version and add commentary about sub-CA revocation on delete

2016-09-22 Thread frasertweedale
URL: https://github.com/freeipa/freeipa/pull/108 Author: frasertweedale Title: #108: Bump pki min version and add commentary about sub-CA revocation on delete Action: edited To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/108/

[Freeipa-devel] [freeipa PR#108][edited] Bump pki min version and add commentary about sub-CA revocation on delete

2016-09-22 Thread frasertweedale
URL: https://github.com/freeipa/freeipa/pull/108 Author: frasertweedale Title: #108: Bump pki min version and add commentary about sub-CA revocation on delete Action: edited To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/108/

Re: [Freeipa-devel] [PATCH 0060] Add --force-join option to ipa-replica-install

2016-09-22 Thread Jan Cholasta
On 25.8.2016 15:31, Martin Basti wrote: On 10.08.2016 07:53, Stanislav Laznicka wrote: On 08/10/2016 07:31 AM, Jan Cholasta wrote: On 9.8.2016 18:52, Petr Vobornik wrote: On 08/09/2016 04:18 PM, Martin Basti wrote: On 09.08.2016 16:07, Stanislav Laznicka wrote: https://fedorahosted.org/f

[Freeipa-devel] [freeipa PR#109][opened] sudorule: add SELinux transition examples to plugin doc

2016-09-22 Thread frasertweedale
URL: https://github.com/freeipa/freeipa/pull/109 Author: frasertweedale Title: #109: sudorule: add SELinux transition examples to plugin doc Action: opened PR body: """ It is not obvious how to add SELinux type and role transitions to a Sudo rule. Update the 'sudorule' plugin documentation wi

Re: [Freeipa-devel] [PATCH] 0091 Allow full customisability of CA subject name

2016-09-22 Thread Jan Cholasta
On 25.8.2016 12:08, Jan Cholasta wrote: On 22.8.2016 07:00, Fraser Tweedale wrote: On Fri, Aug 19, 2016 at 08:09:33PM +1000, Fraser Tweedale wrote: On Mon, Aug 15, 2016 at 10:54:25PM +1000, Fraser Tweedale wrote: On Mon, Aug 15, 2016 at 02:08:54PM +0200, Jan Cholasta wrote: On 19.7.2016 12:05