[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

[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

[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:

[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-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

[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#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

[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

[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

[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 '# '

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

2016-10-10 Thread rajat gupta
o you do something else > >>>>>>>> with > >>>>>>>> it? ... The answer is that it depends on the purpose of each > >>>>>>>> individual LDAP search (like in get_memberindirect() above, we > >>>>>

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

2016-10-10 Thread Petr Spacek
On 10.10.2016 05:23, rajat gupta wrote: > Hi, > > I am trying to setup the freeipa Active Directory trust setup and i am > following > the http://www.freeipa.org/page/Active_Directory_trust_setup documentation. > > I am able to login on freeipa Server with AD users. > > But when i am trying to

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

[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

[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.

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

2016-10-10 Thread Alexander Bokovoy
t;>> I do realize that the proposed fix for the permission 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 >>>>

[Freeipa-devel] [freeipa PR#134][comment] DNS URI support

2016-10-10 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/134 Title: #134: DNS URI support pspacek commented: """ I thought that messing with API numbers in VERSION is not be necessary anymore after we introduced thin client. @jcholast ? """ See the full comment at

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

Re: [Freeipa-devel] [PATCH 0058] Make get_entries not ignore its size_limit argument

2016-10-10 Thread Standa Laznicka
On 10/10/2016 07:53 AM, Jan Cholasta wrote: On 7.10.2016 12:23, Standa Laznicka wrote: On 10/07/2016 08:31 AM, Jan Cholasta wrote: On 17.8.2016 13:47, Stanislav Laznicka wrote: On 08/11/2016 02:59 PM, Stanislav Laznicka wrote: On 08/11/2016 07:49 AM, Jan Cholasta wrote: On 2.8.2016 13:47,

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] [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 --

[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][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