[Freeipa-devel] [freeipa PR#363][+ack] ipaclient: schema cache: Handle malformed server info data gracefully

2017-01-09 Thread HonzaCholasta
URL: https://github.com/freeipa/freeipa/pull/363 Title: #363: ipaclient: schema cache: Handle malformed server info data gracefully Label: +ack -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://ww

[Freeipa-devel] [freeipa PR#363][closed] ipaclient: schema cache: Handle malformed server info data gracefully

2017-01-09 Thread HonzaCholasta
URL: https://github.com/freeipa/freeipa/pull/363 Author: dkupka Title: #363: ipaclient: schema cache: Handle malformed server info data gracefully Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/363/head:pr363 g

[Freeipa-devel] [freeipa PR#363][+pushed] ipaclient: schema cache: Handle malformed server info data gracefully

2017-01-09 Thread HonzaCholasta
URL: https://github.com/freeipa/freeipa/pull/363 Title: #363: ipaclient: schema cache: Handle malformed server info data gracefully Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http:/

[Freeipa-devel] [freeipa PR#363][comment] ipaclient: schema cache: Handle malformed server info data gracefully

2017-01-09 Thread HonzaCholasta
URL: https://github.com/freeipa/freeipa/pull/363 Title: #363: ipaclient: schema cache: Handle malformed server info data gracefully HonzaCholasta commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/d15ccde20fcc97a597180255ee9f5eb38caa206c """ See the full comment a

[Freeipa-devel] [freeipa PR#181][comment] Tests : User Tracker creation of user with minimal values

2017-01-09 Thread gkaihorodova
URL: https://github.com/freeipa/freeipa/pull/181 Title: #181: Tests : User Tracker creation of user with minimal values gkaihorodova commented: """ Thank you for review! """ See the full comment at https://github.com/freeipa/freeipa/pull/181#issuecomment-271236642 -- Manage your subscription

[Freeipa-devel] [bind-dyndb-ldap PR#7][opened] Added named.conf API transformation script to spec

2017-01-09 Thread tomaskrizek
URL: https://github.com/freeipa/bind-dyndb-ldap/pull/7 Author: tomaskrizek Title: #7: Added named.conf API transformation script to spec Action: opened PR body: """ A script that converts old-style configuration API of named.conf to the new-style API after rpm isntallation was added to contrib

[Freeipa-devel] [bind-dyndb-ldap PR#7][comment] Added named.conf API transformation script to spec

2017-01-09 Thread tomaskrizek
URL: https://github.com/freeipa/bind-dyndb-ldap/pull/7 Title: #7: Added named.conf API transformation script to spec tomaskrizek commented: """ This patch is meant to fix existing IPA installations when they're upgraded to use BIND-9.11. New IPA installations are covered by freeipa/freeipa#351

[Freeipa-devel] [freeipa PR#378][opened] Integrate make check into CI

2017-01-09 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/378 Author: tiran Title: #378: Integrate make check into CI Action: opened PR body: """ make check runs cmocka tests for our C code. The patch also adds some additional files to gitignore and cleanup. Signed-off-by: Christian Heimes """ To pull t

[Freeipa-devel] [freeipa PR#379][opened] Packaging: Add placeholder and IPA commands packages

2017-01-09 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/379 Author: tiran Title: #379: Packaging: Add placeholder and IPA commands packages Action: opened PR body: """ The ipacommands package contains ipa-getkeytab and ipa-rmkeytab for installation in a virtual env. The programs are compiled with distuti

[Freeipa-devel] [freeipa PR#351][synchronized] [fedora-26] named.conf template: update API for bind 9.11

2017-01-09 Thread tomaskrizek
URL: https://github.com/freeipa/freeipa/pull/351 Author: tomaskrizek Title: #351: [fedora-26] named.conf template: update API for bind 9.11 Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/351/head:pr351 git

[Freeipa-devel] [freeipa PR#351][comment] [fedora-26] named.conf template: update API for bind 9.11

2017-01-09 Thread tomaskrizek
URL: https://github.com/freeipa/freeipa/pull/351 Title: #351: [fedora-26] named.conf template: update API for bind 9.11 tomaskrizek commented: """ Required version of BIND is a subject to change. When a version with fixed `named-pkcs11` issue ([BZ 1410433](https://bugzilla.redhat.com/show_bug.

[Freeipa-devel] [freeipa PR#380][opened] Travis CI: actually return non-zero exit status when the test job fails

2017-01-09 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/380 Author: martbab Title: #380: Travis CI: actually return non-zero exit status when the test job fails Action: opened PR body: """ Thanks to @stlaz for catching this. """ To pull the PR as Git branch: git remote add ghfreeipa https://github.com/

[Freeipa-devel] [freeipa PR#347][comment] Improvements in {get|set}_directive functions

2017-01-09 Thread tomaskrizek
URL: https://github.com/freeipa/freeipa/pull/347 Title: #347: Improvements in {get|set}_directive functions tomaskrizek commented: """ Please see my feedback in in-line comments. """ See the full comment at https://github.com/freeipa/freeipa/pull/347#issuecomment-271272632 -- Manage your subs

[Freeipa-devel] [freeipa PR#380][comment] Travis CI: actually return non-zero exit status when the test job fails

2017-01-09 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/380 Title: #380: Travis CI: actually return non-zero exit status when the test job fails stlaz commented: """ It works but for some reason there are many extra newlines in the failure log """ See the full comment at https://github.com/freeipa/freei

[Freeipa-devel] [freeipa PR#380][+ack] Travis CI: actually return non-zero exit status when the test job fails

2017-01-09 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/380 Title: #380: Travis CI: actually return non-zero exit status when the test job fails Label: +ack -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://

[Freeipa-devel] [freeipa PR#380][comment] Travis CI: actually return non-zero exit status when the test job fails

2017-01-09 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/380 Title: #380: Travis CI: actually return non-zero exit status when the test job fails stlaz commented: """ It works but for some reason there are many extra newlines in the failure log """ See the full comment at https://github.com/freeipa/freei

[Freeipa-devel] [freeipa PR#380][comment] Travis CI: actually return non-zero exit status when the test job fails

2017-01-09 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/380 Title: #380: Travis CI: actually return non-zero exit status when the test job fails stlaz commented: """ It works but for some reason there are many extra newlines in the failure log **edit:** nvm, displayes correctly now, apparently it's a Tra

[Freeipa-devel] [freeipa PR#380][synchronized] Travis CI: actually return non-zero exit status when the test job fails

2017-01-09 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/380 Author: martbab Title: #380: Travis CI: actually return non-zero exit status when the test job fails Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/380/hea

[Freeipa-devel] [freeipa PR#380][+pushed] Travis CI: actually return non-zero exit status when the test job fails

2017-01-09 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/380 Title: #380: Travis CI: actually return non-zero exit status when the test job fails Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http

[Freeipa-devel] [freeipa PR#380][comment] Travis CI: actually return non-zero exit status when the test job fails

2017-01-09 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/380 Title: #380: Travis CI: actually return non-zero exit status when the test job fails martbab commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/9b5b7131502a73fa24dc56c72a9648528c5aceee """ See the full comment at h

[Freeipa-devel] [freeipa PR#380][closed] Travis CI: actually return non-zero exit status when the test job fails

2017-01-09 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/380 Author: martbab Title: #380: Travis CI: actually return non-zero exit status when the test job fails Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/380/head:pr38

[Freeipa-devel] [freeipa PR#374][+ack] pytest: set rules to find test files and functions

2017-01-09 Thread apophys
URL: https://github.com/freeipa/freeipa/pull/374 Title: #374: pytest: set rules to find test files and functions 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

[Freeipa-devel] [freeipa PR#179][comment] Fix for handling CalledProcessError in authconfig

2017-01-09 Thread tomaskrizek
URL: https://github.com/freeipa/freeipa/pull/179 Title: #179: Fix for handling CalledProcessError in authconfig tomaskrizek commented: """ I investigated some other options for the displayed error message, but I haven't found anything more appropriate. [Comment#4](https://fedorahosted.org/free

[Freeipa-devel] [freeipa PR#381][opened] disable hostname canonicalization by Kerberos library

2017-01-09 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/381 Author: martbab Title: #381: disable hostname canonicalization by Kerberos library Action: opened PR body: """ By default, Kerberos client library attempts to canonicalize service hostname in TGS requests. This can fail e.g. if hosts file on the

[Freeipa-devel] [freeipa PR#378][comment] Integrate make check into CI

2017-01-09 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/378 Title: #378: Integrate make check into CI martbab commented: """ IIRC cmocka tests are already ran as a part of build process, seee the following excerpt from the build log: ``` PASS: ipa_kdb_tests

[Freeipa-devel] [freeipa PR#382][opened] [WIP] Py3 ipa-server-install fixes

2017-01-09 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/382 Author: mbasti-rh Title: #382: [WIP] Py3 ipa-server-install fixes Action: opened PR body: """ This PR should allow to install server with py3 """ To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fet

[Freeipa-devel] [freeipa PR#383][opened] Remove duplicated step from DS install

2017-01-09 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/383 Author: mbasti-rh Title: #383: Remove duplicated step from DS install Action: opened PR body: """ "Adding SASL mappings.." is duplicated step in __common_setup in DS instance and should be removed. """ To pull the PR as Git branch: git remote a

[Freeipa-devel] [freeipa PR#384][opened] Add fix for user prompt in dnsrecord-add

2017-01-09 Thread Akasurde
URL: https://github.com/freeipa/freeipa/pull/384 Author: Akasurde Title: #384: Add fix for user prompt in dnsrecord-add Action: opened PR body: """ Fix added to skip optional parameter in dnsrecord-add interactive prompts Fixes https://fedorahosted.org/freeipa/ticket/6457 Signed-off-by: Abhi

Re: [Freeipa-devel] CSR autogeneration next steps

2017-01-09 Thread Jan Cholasta
On 19.12.2016 21:59, Ben Lipton wrote: On 12/15/2016 11:11 PM, Ben Lipton wrote: On 12/12/2016 03:52 AM, Jan Cholasta wrote: On 5.12.2016 16:48, Ben Lipton wrote: Hi Jan, thanks for the comments. On 12/05/2016 04:25 AM, Jan Cholasta wrote: Hi Ben, On 3.11.2016 00:12, Ben Lipton wrote: H