Re: [Freeipa-devel] links to docs in the messages from code

2016-10-12 Thread Martin Basti
On 12.10.2016 19:56, Petr Spacek wrote: Hello FreeIPA developers, looking at freeipa-users mailing list, a lot of questions could be answered by just quick glance to the docs. I wonder if we should add links HTML version of docs on access.redhat.com to the messages generated by the code. If

[Freeipa-devel] [freeipa PR#160][opened] Reverted the essertion for replica uninstall returncode

2016-10-12 Thread ofayans
URL: https://github.com/freeipa/freeipa/pull/160 Author: ofayans Title: #160: Reverted the essertion for replica uninstall returncode Action: opened PR body: """ As the issue with ipa installer always returning 0 returncode is apparently addressed, the test needs to be made aware of this chang

[Freeipa-devel] [freeipa PR#156][synchronized] cert: add revocation reason back to cert-find output

2016-10-12 Thread jcholast
URL: https://github.com/freeipa/freeipa/pull/156 Author: jcholast Title: #156: cert: add revocation reason back to cert-find output Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/156/head:pr156 git checkou

[Freeipa-devel] [freeipa PR#156][comment] cert: add revocation reason back to cert-find output

2016-10-12 Thread jcholast
URL: https://github.com/freeipa/freeipa/pull/156 Title: #156: cert: add revocation reason back to cert-find output jcholast commented: """ It's intentional. With `--raw`, only data retrieved directly from the backend should be returned. `cert-show` does the same. I should have mentioned it in

[Freeipa-devel] [bind-dyndb-ldap PR#2][opened] fix ldif syntax and add idnsTemplateAttribute

2016-10-12 Thread stutiredboy
URL: https://github.com/freeipa/bind-dyndb-ldap/pull/2 Author: stutiredboy Title: #2: fix ldif syntax and add idnsTemplateAttribute Action: opened PR body: """ schema.ldif lost some white space in the line end. schema.ldif lost the idnsTemplateAttribute definitition """ To pull the PR as Git

Re: [Freeipa-devel] Heimdal Kerberos support for client

2016-10-12 Thread Rob Crittenden
Petr Spacek wrote: Hello list, I just noticed that client/configure.ac contains some checks to detect and support Heimdal Kerberos libraries. Was it tested? Does it work? Can I drop it? :-) Wow, that's some old code. Only Simo would know if it was ever tested or ever worked. I suppose sinc

[Freeipa-devel] Heimdal Kerberos support for client

2016-10-12 Thread Petr Spacek
Hello list, I just noticed that client/configure.ac contains some checks to detect and support Heimdal Kerberos libraries. Was it tested? Does it work? Can I drop it? :-) -- Petr^2 Spacek -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/f

[Freeipa-devel] links to docs in the messages from code

2016-10-12 Thread Petr Spacek
Hello FreeIPA developers, looking at freeipa-users mailing list, a lot of questions could be answered by just quick glance to the docs. I wonder if we should add links HTML version of docs on access.redhat.com to the messages generated by the code. If we really want, we can make these platform-s

[Freeipa-devel] [freeipa PR#158][comment] WebUI: update Patternfly and Bootstrap

2016-10-12 Thread pvoborni
URL: https://github.com/freeipa/freeipa/pull/158 Title: #158: WebUI: update Patternfly and Bootstrap pvoborni commented: """ I don't think the patternfly package can be included in Fedora as is. It internally bundles several packages, some of them already packaged (jquery, fontawesome-fonts, O

[Freeipa-devel] [freeipa PR#158][comment] WebUI: update Patternfly and Bootstrap

2016-10-12 Thread redhatrises
URL: https://github.com/freeipa/freeipa/pull/158 Title: #158: WebUI: update Patternfly and Bootstrap redhatrises commented: """ > @redhatrises Thank you for the comment and the link. I agree that it would be > really nice, but unfortunately there is no PatternFly package in Fedora. @pvomacka

[Freeipa-devel] [freeipa PR#158][comment] WebUI: update Patternfly and Bootstrap

2016-10-12 Thread pvomacka
URL: https://github.com/freeipa/freeipa/pull/158 Title: #158: WebUI: update Patternfly and Bootstrap pvomacka commented: """ @redhatrises Thank you for the comment and the link. I agree that it would be really nice, but unfortunately there is no PatternFly package in Fedora. Anyway, I would

[Freeipa-devel] [freeipa PR#157][comment] git: Add commit template

2016-10-12 Thread mzidek-rh
URL: https://github.com/freeipa/freeipa/pull/157 Title: #157: git: Add commit template mzidek-rh commented: """ Forgot to add a comment. I updated the patch according to jcholast's comments. """ See the full comment at https://github.com/freeipa/freeipa/pull/157#issuecomment-253261173 -- Mana

[Freeipa-devel] [freeipa PR#156][comment] cert: add revocation reason back to cert-find output

2016-10-12 Thread pvomacka
URL: https://github.com/freeipa/freeipa/pull/156 Title: #156: cert: add revocation reason back to cert-find output pvomacka commented: """ I found one difference in output of cert-find command before and after this patch, it behaves differently only with --raw option. In output of the command

[Freeipa-devel] Building refactoring branches in copr

2016-10-12 Thread David Kupka
Hello everyone, as we already agreed we will have branches for our refactoring efforts somewhere and we will build them in copr so then can be easily consumed by CI and also everyone else willing to test them. I already put together simple script that pulls new patches makes srpm and submits

[Freeipa-devel] [freeipa PR#157][synchronized] git: Add commit template

2016-10-12 Thread mzidek-rh
URL: https://github.com/freeipa/freeipa/pull/157 Author: mzidek-rh Title: #157: git: Add commit template Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/157/head:pr157 git checkout pr157 From 6780d0fd78a53a

[Freeipa-devel] [freeipa PR#159][opened] spec file: clean up BuildRequires

2016-10-12 Thread jcholast
URL: https://github.com/freeipa/freeipa/pull/159 Author: jcholast Title: #159: spec file: clean up BuildRequires Action: opened PR body: """ Add missing cyrus-sasl-devel, python-cffi, python-custodia, python-nose, python-paste, python-sssdconfig and systemd-python BuildRequires. Remove unused

Re: [Freeipa-devel] Feature branches for sub-team efforts

2016-10-12 Thread Alexander Bokovoy
On ke, 12 loka 2016, David Kupka wrote: On 11/10/16 16:27, Alexander Bokovoy wrote: On ti, 11 loka 2016, Petr Vobornik wrote: On 10/11/2016 03:50 PM, Alexander Bokovoy wrote: On ti, 11 loka 2016, Petr Vobornik wrote: Hi List, we discussed locally a proposal about creating a feature branch fo

Re: [Freeipa-devel] Feature branches for sub-team efforts

2016-10-12 Thread David Kupka
On 11/10/16 16:27, Alexander Bokovoy wrote: On ti, 11 loka 2016, Petr Vobornik wrote: On 10/11/2016 03:50 PM, Alexander Bokovoy wrote: On ti, 11 loka 2016, Petr Vobornik wrote: Hi List, we discussed locally a proposal about creating a feature branch for each sub-team effort in our main git. C

Re: [Freeipa-devel] [Test][Patch-0047] Added a test for Ticket N 5964

2016-10-12 Thread Martin Basti
1) Can you just turn off dirsrv on replica instead of doing iptables magic? 2) NACK No more eval() ever in code, use 'getattr', 'get' or whatever in the object that can be used. +evalhost = eval("args[0].%s" % host) Martin^2 On 12.10.2016 14:03, Oleg Fayans wrote: Hi Mart

[Freeipa-devel] [freeipa PR#158][comment] WebUI: update Patternfly and Bootstrap

2016-10-12 Thread redhatrises
URL: https://github.com/freeipa/freeipa/pull/158 Title: #158: WebUI: update Patternfly and Bootstrap redhatrises commented: """ @pvomacka should this use the patternfly RPM rather than having the code copied here as well? See https://www.redhat.com/archives/patternfly/2014-July/msg00017.html "

[Freeipa-devel] [freeipa PR#158][comment] WebUI: update Patternfly and Bootstrap

2016-10-12 Thread redhatrises
URL: https://github.com/freeipa/freeipa/pull/158 Title: #158: WebUI: update Patternfly and Bootstrap redhatrises commented: """ @pvomacka should this use the patternfly RPM rather than having the code copied here as well? See https://www.redhat.com/archives/patternfly/2014-July/msg00017.html "

Re: [Freeipa-devel] [Test][Patch-0047] Added a test for Ticket N 5964

2016-10-12 Thread Oleg Fayans
Hi Martin, After extensive discussion with Ludwig, I finally got the clue on how does this feature work. When we uninstall the replica, the master cleans the replication agreements with this replica and automatically cleans all replica's RUVs. If we clean replica's RUVs on master without unins

[Freeipa-devel] [freeipa PR#157][comment] git: Add commit template

2016-10-12 Thread mzidek-rh
URL: https://github.com/freeipa/freeipa/pull/157 Title: #157: git: Add commit template mzidek-rh commented: """ Sure, that was a copy paste mistake in itself :) . Patch updated. """ See the full comment at https://github.com/freeipa/freeipa/pull/157#issuecomment-253193300 -- Manage your subsc

[Freeipa-devel] [freeipa PR#157][synchronized] git: Add commit template

2016-10-12 Thread mzidek-rh
URL: https://github.com/freeipa/freeipa/pull/157 Author: mzidek-rh Title: #157: git: Add commit template Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/157/head:pr157 git checkout pr157 From 3e6b55ffc271c0

[Freeipa-devel] [freeipa PR#157][comment] git: Add commit template

2016-10-12 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/157 Title: #157: git: Add commit template martbab commented: """ Could you please change the link in the template to: https://fedorahosted.org/freeipa/ticket/ just to avoid copy-paste errors when filling in ticket No.? """ See the full comment

[Freeipa-devel] [freeipa PR#157][comment] git: Add commit template

2016-10-12 Thread mzidek-rh
URL: https://github.com/freeipa/freeipa/pull/157 Title: #157: git: Add commit template mzidek-rh commented: """ This is the same commit template we use in SSSD. Maybe it will be helpful for FreeIPA too. """ See the full comment at https://github.com/freeipa/freeipa/pull/157#issuecomment-25319

[Freeipa-devel] [freeipa PR#157][opened] git: Add commit template

2016-10-12 Thread mzidek-rh
URL: https://github.com/freeipa/freeipa/pull/157 Author: mzidek-rh Title: #157: git: Add commit template Action: opened PR body: """ In order to use the commit template, run the following command: git config commit.template .git-commit-template """ To pull the PR as Git branch: git remote add

[Freeipa-devel] [freeipa PR#156][opened] cert: add revocation reason back to cert-find output

2016-10-12 Thread jcholast
URL: https://github.com/freeipa/freeipa/pull/156 Author: jcholast Title: #156: cert: add revocation reason back to cert-find output Action: opened PR body: """ In commit c718ef058847bb39e78236e8af0ad69ac961bbcf some param values were accidentally removed from cert-find output. In commit 22d5f

[Freeipa-devel] [freeipa PR#155][synchronized] Build system cleanup

2016-10-12 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/155 Author: pspacek Title: #155: Build system cleanup Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/155/head:pr155 git checkout pr155 From e641d9cf9b9e596bd335

[Freeipa-devel] [freeipa PR#154][+pushed] [ipa-4-4] Rebase: Tests: Fix cert revocation tests

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/154 Title: #154: [ipa-4-4] Rebase: Tests: Fix cert revocation tests 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/pag

[Freeipa-devel] [freeipa PR#154][closed] [ipa-4-4] Rebase: Tests: Fix cert revocation tests

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/154 Author: mirielka Title: #154: [ipa-4-4] Rebase: Tests: Fix cert revocation tests Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/154/head:pr154 git checkout pr154

[Freeipa-devel] [freeipa PR#154][comment] [ipa-4-4] Rebase: Tests: Fix cert revocation tests

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/154 Title: #154: [ipa-4-4] Rebase: Tests: Fix cert revocation tests mbasti-rh commented: """ Fixed upstream ipa-4-4: https://fedorahosted.org/freeipa/changeset/afabdd365a35e0e454997ff021152422bcbcf785 https://fedorahosted.org/freeipa/changeset/c8cdc6a

[Freeipa-devel] [freeipa PR#154][+ack] [ipa-4-4] Rebase: Tests: Fix cert revocation tests

2016-10-12 Thread pvomacka
URL: https://github.com/freeipa/freeipa/pull/154 Title: #154: [ipa-4-4] Rebase: Tests: Fix cert revocation tests 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/C

Re: [Freeipa-devel] HBAC for AD users Active Directory trust setup

2016-10-12 Thread Alexander Bokovoy
On ke, 12 loka 2016, rajat gupta wrote: Hi, thank you for answering. I this case i need to create multiple group in AD side. like user1 have only "server1.example.com" and "server2.example.com" access and some other user have some other server access. Then only the my HBAC rule will be implem

Re: [Freeipa-devel] HBAC for AD users Active Directory trust setup

2016-10-12 Thread rajat gupta
Hi, thank you for answering. I this case i need to create multiple group in AD side. like user1 have only "server1.example.com" and "server2.example.com" access and some other user have some other server access. Then only the my HBAC rule will be implemented to particular group and every time

[Freeipa-devel] [freeipa PR#155][opened] Build system cleanup

2016-10-12 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/155 Author: pspacek Title: #155: Build system cleanup Action: opened PR body: """ This is first step in build system refactoring effort. This patch set contains cleanup patches for daemons/configure.ac. After the cleanup, the file will be "promoted

[Freeipa-devel] [freeipa PR#153][closed] [ipa-4-4 only] disable warnings reported by pylint-1.6.4-1

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/153 Author: martbab Title: #153: [ipa-4-4 only] disable warnings reported by pylint-1.6.4-1 Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/153/head:pr153 git checkout

[Freeipa-devel] [freeipa PR#153][comment] [ipa-4-4 only] disable warnings reported by pylint-1.6.4-1

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/153 Title: #153: [ipa-4-4 only] disable warnings reported by pylint-1.6.4-1 mbasti-rh commented: """ Fixed upstream ipa-4-4: https://fedorahosted.org/freeipa/changeset/2b2fc1abf1844b807e50b99f0912fa10d9169eca """ See the full comment at https://gith

[Freeipa-devel] [freeipa PR#153][+pushed] [ipa-4-4 only] disable warnings reported by pylint-1.6.4-1

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/153 Title: #153: [ipa-4-4 only] disable warnings reported by pylint-1.6.4-1 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

[Freeipa-devel] [freeipa PR#153][+ack] [ipa-4-4 only] disable warnings reported by pylint-1.6.4-1

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/153 Title: #153: [ipa-4-4 only] disable warnings reported by pylint-1.6.4-1 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.or

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

2016-10-12 Thread Petr Spacek
On 10.10.2016 10:28, Martin Basti wrote: > 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.up

Re: [Freeipa-devel] HBAC for AD users Active Directory trust setup

2016-10-12 Thread Alexander Bokovoy
On ke, 12 loka 2016, rajat gupta wrote: Hi, Normally HBAC for AD users should be done through an external group. You should use freeipa-users@ mailing list for these questions. And start with documentation: https://access.redhat.com/documentation/en-US/Red_Hat_Enterprise_Linux/7/html-single/L

[Freeipa-devel] [freeipa PR#154][opened] [ipa-4-4] Rebase: Tests: Fix cert revocation tests

2016-10-12 Thread mirielka
URL: https://github.com/freeipa/freeipa/pull/154 Author: mirielka Title: #154: [ipa-4-4] Rebase: Tests: Fix cert revocation tests Action: opened PR body: """ """ To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/154/head:pr154

[Freeipa-devel] [freeipa PR#153][opened] [ipa-4-4 only] disable warnings reported by pylint-1.6.4-1

2016-10-12 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/153 Author: martbab Title: #153: [ipa-4-4 only] disable warnings reported by pylint-1.6.4-1 Action: opened PR body: """ Pylint shipped in Fedora 25 reports 'trailing-newlines' and 'consider-iterating-dictionary' warnings which break FreeIPA builds.

[Freeipa-devel] HBAC for AD users Active Directory trust setup

2016-10-12 Thread rajat gupta
Hi, Normally HBAC for AD users should be done through an external group. So for example if we have 500+ users on AD and only 100 user are administrator and they have Linux server access. I want to set the HBAC and sudo rules for users. So user have correct access server access and sudo rights a

[Freeipa-devel] [freeipa PR#137][+pushed] Test: disabled wrong client domain tests for domlevel 0

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/137 Title: #137: Test: disabled wrong client domain tests for domlevel 0 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.or

[Freeipa-devel] [freeipa PR#137][closed] Test: disabled wrong client domain tests for domlevel 0

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/137 Author: ofayans Title: #137: Test: disabled wrong client domain tests for domlevel 0 Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/137/head:pr137 git checkout pr

[Freeipa-devel] [freeipa PR#137][comment] Test: disabled wrong client domain tests for domlevel 0

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/137 Title: #137: Test: disabled wrong client domain tests for domlevel 0 mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/8b0faa25d1c47f605bc6c91933469bb2370276c1 ipa-4-4: https://fedorahosted.org/freeipa/chan

[Freeipa-devel] [freeipa PR#140][closed] Tests: Fix cert revocation tests

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/140 Author: mirielka Title: #140: Tests: Fix cert revocation tests Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/140/head:pr140 git checkout pr140 -- Manage your su

[Freeipa-devel] [freeipa PR#140][+pushed] Tests: Fix cert revocation tests

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/140 Title: #140: Tests: Fix cert revocation tests 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#140][comment] Tests: Fix cert revocation tests

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/140 Title: #140: Tests: Fix cert revocation tests mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/c9c92e3a7f4961d91e0395daf17f5aeb34c20178 https://fedorahosted.org/freeipa/changeset/8f04d1a793b8ff01804bc03eac

[Freeipa-devel] [freeipa PR#140][comment] Tests: Fix cert revocation tests

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/140 Title: #140: Tests: Fix cert revocation tests mbasti-rh commented: """ `Tests: Certificate revocation` doesn't apply to ipa-4-4 branch, please open separate PR against IPA 4.4 """ See the full comment at https://github.com/freeipa/freeipa/pull

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

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/142 Title: #142: CheckedIPAddress: Implement __(g|s)etstate__ and to ensure proper (un)pickling mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/fb85230e25bd37a2a02a9d90793f337aad40a037 ipa-4-4: https://fedor

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

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

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

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

[Freeipa-devel] [freeipa PR#152][+pushed] Fix warnings reported by pylint in rawhide

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/152 Title: #152: Fix warnings reported by pylint in rawhide 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/Contri

[Freeipa-devel] [freeipa PR#152][closed] Fix warnings reported by pylint in rawhide

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/152 Author: martbab Title: #152: Fix warnings reported by pylint in rawhide Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/152/head:pr152 git checkout pr152 -- Manag

[Freeipa-devel] [freeipa PR#152][comment] Fix warnings reported by pylint in rawhide

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/152 Title: #152: Fix warnings reported by pylint in rawhide mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/29829cc55a6be697abf881ea7867ef834bb66be7 https://fedorahosted.org/freeipa/changeset/71f642f75132fe30

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

2016-10-12 Thread mbasti-rh
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#140][comment] Tests: Fix cert revocation tests

2016-10-12 Thread pvomacka
URL: https://github.com/freeipa/freeipa/pull/140 Title: #140: Tests: Fix cert revocation tests pvomacka commented: """ Works correctly. ACK """ See the full comment at https://github.com/freeipa/freeipa/pull/140#issuecomment-253139511 -- Manage your subscription for the Freeipa-devel mailing

[Freeipa-devel] [freeipa PR#140][+ack] Tests: Fix cert revocation tests

2016-10-12 Thread pvomacka
URL: https://github.com/freeipa/freeipa/pull/140 Title: #140: Tests: Fix cert revocation tests 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#152][+ack] Fix warnings reported by pylint in rawhide

2016-10-12 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/152 Title: #152: Fix warnings reported by pylint in rawhide 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/Contribut