Re: [Freeipa-devel] Build system refactoring - design document

2016-10-10 Thread Jan Cholasta
Hi, On 7.10.2016 11:56, Petr Spacek wrote: Dear FreeIPA developers and packagers, you can find first version of the Build system refactoring design document on: http://www.freeipa.org/page/V4/Build_system_refactoring If you do not care about implementation details, please be so kind and quickl

[Freeipa-devel] [freeipa PR#148][comment] Unaccessible variable self.attrs in Tracker

2016-10-10 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/148 Title: #148: Unaccessible variable self.attrs in Tracker mbasti-rh commented: """ You can check last step in this howto http://www.freeipa.org/page/Pull_request_on_Github """ See the full comment at https://github.com/freeipa/freeipa/pull/148#i

[Freeipa-devel] [freeipa PR#148][reopened] Unaccessible variable self.attrs in Tracker

2016-10-10 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/148 Author: gkaihorodova Title: #148: Unaccessible variable self.attrs in Tracker Action: reopened To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/148/head:pr148 git checkout pr148 -

[Freeipa-devel] [freeipa PR#149][comment] Tests: Unaccessible variable self.attrs in Tracker

2016-10-10 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/149 Title: #149: Tests: Unaccessible variable self.attrs in Tracker stlaz commented: """ Please fix this in the original PR, it will be a nice training for you ;) """ See the full comment at https://github.com/freeipa/freeipa/pull/149#issuecomment-2

[Freeipa-devel] [freeipa PR#149][closed] Tests: Unaccessible variable self.attrs in Tracker

2016-10-10 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/149 Author: gkaihorodova Title: #149: Tests: Unaccessible variable self.attrs in Tracker Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/149/head:pr149 git checkout pr

[Freeipa-devel] [freeipa PR#149][+rejected] Tests: Unaccessible variable self.attrs in Tracker

2016-10-10 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/149 Title: #149: Tests: Unaccessible variable self.attrs in Tracker Label: +rejected -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/p

[Freeipa-devel] [freeipa PR#148][comment] Unaccessible variable self.attrs in Tracker

2016-10-10 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/148 Title: #148: Unaccessible variable self.attrs in Tracker stlaz commented: """ That's not how pull requests work. You should change the commit and force push it to the branch instead. """ See the full comment at https://github.com/freeipa/freeip

[Freeipa-devel] [freeipa PR#149][opened] Tests: Unaccessible variable self.attrs in Tracker

2016-10-10 Thread gkaihorodova
URL: https://github.com/freeipa/freeipa/pull/149 Author: gkaihorodova Title: #149: Tests: Unaccessible variable self.attrs in Tracker Action: opened PR body: """ In tracker, 'self.attrs' variable is created and filled in track_create method. Some objects are not created but still require acces

[Freeipa-devel] [freeipa PR#148][closed] Unaccessible variable self.attrs in Tracker

2016-10-10 Thread gkaihorodova
URL: https://github.com/freeipa/freeipa/pull/148 Author: gkaihorodova Title: #148: Unaccessible variable self.attrs in Tracker Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/148/head:pr148 git checkout pr148 --

[Freeipa-devel] [freeipa PR#148][comment] Unaccessible variable self.attrs in Tracker

2016-10-10 Thread gkaihorodova
URL: https://github.com/freeipa/freeipa/pull/148 Title: #148: Unaccessible variable self.attrs in Tracker gkaihorodova commented: """ You're right, empty dict is more applicable here. I will close that one and open new with suggested fix. """ See the full comment at https://github.com/freeipa

[Freeipa-devel] [freeipa PR#145][comment] [WIP] Refactoring: LDAP Connection Management

2016-10-10 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/145 Title: #145: [WIP] Refactoring: LDAP Connection Management mbasti-rh commented: """ I did some inline comments, I'm not fully satisfied with implementation of the connection manager, I'll think about it tomorrow. """ See the full comment at htt

[Freeipa-devel] [freeipa PR#10][comment] Client-side CSR autogeneration

2016-10-10 Thread LiptonB
URL: https://github.com/freeipa/freeipa/pull/10 Title: #10: Client-side CSR autogeneration LiptonB commented: """ Thanks, I've updated the code based on your comments (force pushed to fix conflicts with master). And thanks for pointing out that email! I don't know how I missed it, but I will r

[Freeipa-devel] [freeipa PR#145][comment] [WIP] Refactoring: LDAP Connection Management

2016-10-10 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/145 Title: #145: [WIP] Refactoring: LDAP Connection Management mbasti-rh commented: """ Please fix PEP8 ``` ./ipapython/ipaldap.py:766:49: E231 missing whitespace after ',' ./ipapython/ipaldap.py:822:9: E265 block comment should start with '# ' ./ipas

[Freeipa-devel] [freeipa PR#148][comment] Unaccessible variable self.attrs in Tracker

2016-10-10 Thread apophys
URL: https://github.com/freeipa/freeipa/pull/148 Title: #148: Unaccessible variable self.attrs in Tracker apophys commented: """ NACK, comments inline. """ See the full comment at https://github.com/freeipa/freeipa/pull/148#issuecomment-252603864 -- Manage your subscription for the Freeipa-de

[Freeipa-devel] [freeipa PR#147][+rejected] Tests: Unaccessible variable self.attrs in Tracker

2016-10-10 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/147 Title: #147: Tests: Unaccessible variable self.attrs in Tracker Label: +rejected -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/p

[Freeipa-devel] [freeipa PR#148][opened] Unaccessible variable self.attrs in Tracker

2016-10-10 Thread gkaihorodova
URL: https://github.com/freeipa/freeipa/pull/148 Author: gkaihorodova Title: #148: Unaccessible variable self.attrs in Tracker Action: opened PR body: """ In tracker, 'self.attrs' variable is created and filled in track_create method. Some objects are not created but still require access to th

[Freeipa-devel] [freeipa PR#147][closed] Tests: Unaccessible variable self.attrs in Tracker

2016-10-10 Thread gkaihorodova
URL: https://github.com/freeipa/freeipa/pull/147 Author: gkaihorodova Title: #147: Tests: Unaccessible variable self.attrs in Tracker Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/147/head:pr147 git checkout pr

[Freeipa-devel] [freeipa PR#147][opened] Tests: Unaccessible variable self.attrs in Tracker

2016-10-10 Thread gkaihorodova
URL: https://github.com/freeipa/freeipa/pull/147 Author: gkaihorodova Title: #147: Tests: Unaccessible variable self.attrs in Tracker Action: opened PR body: """ https://fedorahosted.org/freeipa/ticket/6125 """ To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa

[Freeipa-devel] [freeipa PR#142][synchronized] CheckedIPAddress: Implement __(g|s)etstate__ and to ensure proper (un)pickling

2016-10-10 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/142 Author: dkupka Title: #142: CheckedIPAddress: Implement __(g|s)etstate__ and to ensure proper (un)pickling Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/1

[Freeipa-devel] [freeipa PR#142][-ack] CheckedIPAddress: Implement __(g|s)etstate__ and to ensure proper (un)pickling

2016-10-10 Thread jcholast
URL: https://github.com/freeipa/freeipa/pull/142 Title: #142: CheckedIPAddress: Implement __(g|s)etstate__ and to ensure proper (un)pickling Label: -ack -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

[Freeipa-devel] [freeipa PR#142][comment] CheckedIPAddress: Implement __(g|s)etstate__ and to ensure proper (un)pickling

2016-10-10 Thread jcholast
URL: https://github.com/freeipa/freeipa/pull/142 Title: #142: CheckedIPAddress: Implement __(g|s)etstate__ and to ensure proper (un)pickling jcholast commented: """ NACK, see inline comments. """ See the full comment at https://github.com/freeipa/freeipa/pull/142#issuecomment-252580195 -- Ma

[Freeipa-devel] [freeipa PR#142][synchronized] CheckedIPAddress: Implement __(g|s)etstate__ and to ensure proper (un)pickling

2016-10-10 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/142 Author: dkupka Title: #142: CheckedIPAddress: Implement __(g|s)etstate__ and to ensure proper (un)pickling Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/1

Re: [Freeipa-devel] [PATCH 0497] Py3: fix unicode/str error in LDAP*ReverseMember

2016-10-10 Thread Martin Basti
On 10.10.2016 07:57, Jan Cholasta wrote: On 7.6.2016 10:35, Martin Basti wrote: On 07.06.2016 10:35, Jan Cholasta wrote: On 7.6.2016 10:29, Martin Basti wrote: On 07.06.2016 09:08, Jan Cholasta wrote: On 6.6.2016 14:33, Martin Basti wrote: https://fedorahosted.org/freeipa/ticket/5923

[Freeipa-devel] Broken IPA installation caused by new python-dns package

2016-10-10 Thread Martin Basti
https://bodhi.fedoraproject.org/updates/FEDORA-2016-1857421df6 Please set karma accordingly Traceback: ... File "/usr/lib/python2.7/site-packages/ipaserver/dns_data_management.py", line 426, in update_dns_records self.update_base_records(), File "/usr/lib/python2.7/site-packages/

Re: [Freeipa-devel] Freeipa-devel Digest, Vol 113, Issue 35

2016-10-10 Thread Alexander Bokovoy
plugin is not >>>>>>> perfect, it would probably be better to subtract the number of >>>>>>> currently >>>>>>> loaded records from the sizelimit, although in the end the >>>>>>> number of >>>>>>> retu

Re: [Freeipa-devel] kinit: Cannot contact any KDC for realm... from Freeipa clinet (Active Directory trust setup)

2016-10-10 Thread Sumit Bose
On Mon, Oct 10, 2016 at 09:43:24AM +0200, rajat gupta wrote: > https://access.redhat.com/documentation/en-US/Red_Hat_ > Enterprise_Linux/7/html/Windows_Integration_Guide/ > trust-requirements.html#trust-req-ports > > these port are required for trust. Is port 88 required to open from ipa > client

Re: [Freeipa-devel] kinit: Cannot contact any KDC for realm... from Freeipa clinet (Active Directory trust setup)

2016-10-10 Thread rajat gupta
https://access.redhat.com/documentation/en-US/Red_Hat_ Enterprise_Linux/7/html/Windows_Integration_Guide/ trust-requirements.html#trust-req-ports these port are required for trust. Is port 88 required to open from ipa client to AD? On Mon, Oct 10, 2016 at 5:23 AM, rajat gupta wrote: > Hi, > >

Re: [Freeipa-devel] Freeipa-devel Digest, Vol 113, Issue 35

2016-10-10 Thread rajat gupta
h > >>>>>>>> it? ... The answer is that it depends on the purpose of each > >>>>>>>> individual LDAP search (like in get_memberindirect() above, we > >>>>>>>> have to > >>>>>>>> do unlimite

[Freeipa-devel] [freeipa PR#146][opened] WebUI: fix API Browser menu label

2016-10-10 Thread pvomacka
URL: https://github.com/freeipa/freeipa/pull/146 Author: pvomacka Title: #146: WebUI: fix API Browser menu label Action: opened PR body: """ The label of API Browser is now in translatable strings and it has uppercase B at the beginnig of second word. https://fedorahosted.org/freeipa/ticket/6