Re: [Freeipa-devel] MD5 certificate fingerprints removal

2017-02-23 Thread Jan Cholasta
On 23.2.2017 19:06, Martin Basti wrote: On 23.02.2017 15:09, Tomas Krizek wrote: On 02/22/2017 01:44 PM, Fraser Tweedale wrote: On Wed, Feb 22, 2017 at 01:41:22PM +0100, Tomas Krizek wrote: On 02/22/2017 12:28 AM, Fraser Tweedale wrote: On Tue, Feb 21, 2017 at 05:23:07PM +0100, Standa

Re: [Freeipa-devel] MD5 certificate fingerprints removal

2017-02-23 Thread Standa Laznicka
On 02/24/2017 08:29 AM, Jan Cholasta wrote: On 23.2.2017 19:06, Martin Basti wrote: On 23.02.2017 15:09, Tomas Krizek wrote: On 02/22/2017 01:44 PM, Fraser Tweedale wrote: On Wed, Feb 22, 2017 at 01:41:22PM +0100, Tomas Krizek wrote: On 02/22/2017 12:28 AM, Fraser Tweedale wrote: On Tue,

Re: [Freeipa-devel] MD5 certificate fingerprints removal

2017-02-23 Thread Tomas Krizek
On 02/24/2017 08:34 AM, Standa Laznicka wrote: > On 02/24/2017 08:29 AM, Jan Cholasta wrote: >> On 23.2.2017 19:06, Martin Basti wrote: >>> >>> >>> On 23.02.2017 15:09, Tomas Krizek wrote: On 02/22/2017 01:44 PM, Fraser Tweedale wrote: > On Wed, Feb 22, 2017 at 01:41:22PM +0100, Tomas

[Freeipa-devel] [freeipa PR#502][opened] Make pylint and jsl optional

2017-02-23 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/502 Author: tiran Title: #502: Make pylint and jsl optional Action: opened PR body: """ ./configure no longer fails when pylint or jsl are not available. The make targets for pylint and jsl are no longer defined without the tools. Rational: pylint

[Freeipa-devel] [freeipa PR#503][opened] [WIP] Update testcase for cert plugin

2017-02-23 Thread Akasurde
URL: https://github.com/freeipa/freeipa/pull/503 Author: Akasurde Title: #503: [WIP] Update testcase for cert plugin Action: opened PR body: """ Fixes https://fedorahosted.org/freeipa/ticket/6275 Signed-off-by: Abhijeet Kasurde """ To pull the PR as Git branch: git

[Freeipa-devel] [freeipa PR#501][opened] C compilation fixes and hardening

2017-02-23 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/501 Author: tiran Title: #501: C compilation fixes and hardening Action: opened PR body: """ Fix "implicit declaration of function ‘strlen’" in ipa_pwd_ntlm.c, credits to Lukas. Add -Werror=implicit-function-declaration to CFLAGS to point

[Freeipa-devel] [freeipa PR#495][comment] Fix ipa-server-upgrade

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/495 Title: #495: Fix ipa-server-upgrade MartinBasti commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/32076df10231b381a80c9ef850c2c31d7a25feb8 """ See the full comment at

[Freeipa-devel] [freeipa PR#472][synchronized] Packaging: Add placeholder packages

2017-02-23 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/472 Author: tiran Title: #472: Packaging: Add placeholder packages Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/472/head:pr472 git checkout pr472 From

[Freeipa-devel] [freeipa PR#499][comment] added help about default value for --external-ca-type option

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/499 Title: #499: added help about default value for --external-ca-type option MartinBasti commented: """ Thanks! Next time please use 'git push --force' to update current pull request instead a creating a new one. """ See the full comment at

[Freeipa-devel] [freeipa PR#499][comment] added help about default value for --external-ca-type option

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/499 Title: #499: added help about default value for --external-ca-type option MartinBasti commented: """ Thanks! Next time please use `git push --force` to update current pull request instead creating a new one. """ See the full comment at

[Freeipa-devel] [freeipa PR#497][+rejected] added more meaningful help for --external-ca-type option

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/497 Title: #497: added more meaningful help for --external-ca-type option 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#484][closed] FIPS: Remove pkispawn cruft

2017-02-23 Thread pvoborni
URL: https://github.com/freeipa/freeipa/pull/484 Author: stlaz Title: #484: FIPS: Remove pkispawn cruft Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/484/head:pr484 git checkout pr484 -- Manage your

[Freeipa-devel] [freeipa PR#499][+ack] added help about default value for --external-ca-type option

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/499 Title: #499: added help about default value for --external-ca-type option 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#497][comment] added more meaningful help for --external-ca-type option

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/497 Title: #497: added more meaningful help for --external-ca-type option MartinBasti commented: """ Replaced by #499 """ See the full comment at https://github.com/freeipa/freeipa/pull/497#issuecomment-281995516 -- Manage your subscription for

Re: [Freeipa-devel] MD5 certificate fingerprints removal

2017-02-23 Thread Tomas Krizek
On 02/22/2017 01:44 PM, Fraser Tweedale wrote: > On Wed, Feb 22, 2017 at 01:41:22PM +0100, Tomas Krizek wrote: >> On 02/22/2017 12:28 AM, Fraser Tweedale wrote: >>> On Tue, Feb 21, 2017 at 05:23:07PM +0100, Standa Laznicka wrote: On 02/21/2017 04:24 PM, Tomas Krizek wrote: > On 02/21/2017

[Freeipa-devel] [freeipa PR#484][comment] FIPS: Remove pkispawn cruft

2017-02-23 Thread pvoborni
URL: https://github.com/freeipa/freeipa/pull/484 Title: #484: FIPS: Remove pkispawn cruft pvoborni commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/728a6bd4229ba170b2e94f216127b19d5d94e2ba

[Freeipa-devel] [freeipa PR#484][+pushed] FIPS: Remove pkispawn cruft

2017-02-23 Thread pvoborni
URL: https://github.com/freeipa/freeipa/pull/484 Title: #484: FIPS: Remove pkispawn cruft 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#472][synchronized] Packaging: Add placeholder packages

2017-02-23 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/472 Author: tiran Title: #472: Packaging: Add placeholder packages Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/472/head:pr472 git checkout pr472 From

Re: [Freeipa-devel] python-ipaserver & freeipa-server-trust-ad split

2017-02-23 Thread Timo Aaltonen
On 20.02.2017 20:24, Alexander Bokovoy wrote: > On la, 18 helmi 2017, Timo Aaltonen wrote: >> >> Hi, >> >> So Fedora puts all of dist-packages/ipaserver/* in python-ipaserver, >> but dcerpc.py imports python-samba which -ipaserver does not depend on. >> So I've kept dcerpc.py and

[Freeipa-devel] [freeipa PR#472][comment] Packaging: Add placeholder packages

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/472 Title: #472: Packaging: Add placeholder packages MartinBasti commented: """ @tiran I thought we agreed on having `with_wheels` in specfile and install dependencies only when you want to build wheel packages, what is not the case of RHEL. So

[Freeipa-devel] [freeipa PR#476][synchronized] vault: cache the transport certificate on client

2017-02-23 Thread HonzaCholasta
URL: https://github.com/freeipa/freeipa/pull/476 Author: HonzaCholasta Title: #476: vault: cache the transport certificate on client Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/476/head:pr476 git

[Freeipa-devel] [freeipa PR#497][comment] added more meaningful help for --external-ca-type option

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/497 Title: #497: added more meaningful help for --external-ca-type option MartinBasti commented: """ This is already fixed in master, however it missed what is the default value ``` --external-ca-type={generic,ms-cs} Type

[Freeipa-devel] [freeipa PR#498][opened] compat: fix `Any` params in `batch` and `dnsrecord`

2017-02-23 Thread HonzaCholasta
URL: https://github.com/freeipa/freeipa/pull/498 Author: HonzaCholasta Title: #498: compat: fix `Any` params in `batch` and `dnsrecord` Action: opened PR body: """ The `methods` argument of `batch` and `dnsrecords` attribute of `dnsrecord` were incorrectly defined as `Str` instead of `Any`.

[Freeipa-devel] [freeipa PR#493][comment] Update Contributors.txt

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/493 Title: #493: Update Contributors.txt MartinBasti commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/b367c3a622c8c8b96777e4ce50334d2a4477bbe7 """ See the full comment at

[Freeipa-devel] [freeipa PR#493][closed] Update Contributors.txt

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/493 Author: mkosek Title: #493: Update Contributors.txt Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/493/head:pr493 git checkout pr493 -- Manage your subscription

[Freeipa-devel] [freeipa PR#472][comment] Packaging: Add placeholder packages

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/472 Title: #472: Packaging: Add placeholder packages MartinBasti commented: """ The old pylint issue is back ``` * Module ipaserver.install.installutils ipaserver/install/installutils.py:1209: [E1101(no-member), store_version] Module

[Freeipa-devel] [freeipa PR#484][comment] FIPS: Remove pkispawn cruft

2017-02-23 Thread tomaskrizek
URL: https://github.com/freeipa/freeipa/pull/484 Title: #484: FIPS: Remove pkispawn cruft tomaskrizek commented: """ Works as expected. """ See the full comment at https://github.com/freeipa/freeipa/pull/484#issuecomment-281979229 -- Manage your subscription for the Freeipa-devel mailing

[Freeipa-devel] [freeipa PR#497][closed] added more meaningful help for --external-ca-type option

2017-02-23 Thread tscherf
URL: https://github.com/freeipa/freeipa/pull/497 Author: tscherf Title: #497: added more meaningful help for --external-ca-type option Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/497/head:pr497 git checkout

[Freeipa-devel] [freeipa PR#493][comment] Update Contributors.txt

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/493 Title: #493: Update Contributors.txt MartinBasti commented: """ If script for generating contributors file was published it may be documented in releasing page and it may be part of each release. """ See the full comment at

[Freeipa-devel] [freeipa PR#472][comment] Packaging: Add placeholder packages

2017-02-23 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/472 Title: #472: Packaging: Add placeholder packages tiran commented: """ @MartinBasti I dropped the last commit. make check no longer checks wheel packages. I'm going to open a new ticket for @martbab and ask him to add to add a proper test for

Re: [Freeipa-devel] Release: script for updating contributors

2017-02-23 Thread Martin Basti
On 23.02.2017 12:31, Martin Kosek wrote: Hi all, Based on my recent Contributors.txt update and on Martin Basti's request in the pull request: https://github.com/freeipa/freeipa/pull/493#issuecomment-281938080 I added my (hacky) script for updating the file in the freeipa-tools repo and

[Freeipa-devel] [freeipa PR#398][comment] Support for Certificate Identity Mapping

2017-02-23 Thread sumit-bose
URL: https://github.com/freeipa/freeipa/pull/398 Title: #398: Support for Certificate Identity Mapping sumit-bose commented: """ Ok, sorry for the noise, I tested on a fresh install again and now it is working as expected. I guess I shouldn't have tried to update from an older version of your

[Freeipa-devel] [freeipa PR#497][opened] added more meaningful help for --external-ca-type option

2017-02-23 Thread tscherf
URL: https://github.com/freeipa/freeipa/pull/497 Author: tscherf Title: #497: added more meaningful help for --external-ca-type option Action: opened PR body: """ """ To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa

[Freeipa-devel] [freeipa PR#412][comment] Define template version in certmap.conf

2017-02-23 Thread tomaskrizek
URL: https://github.com/freeipa/freeipa/pull/412 Title: #412: Define template version in certmap.conf tomaskrizek commented: """ Works as expected. """ See the full comment at https://github.com/freeipa/freeipa/pull/412#issuecomment-28194 -- Manage your subscription for the Freeipa-devel

[Freeipa-devel] [freeipa PR#412][+ack] Define template version in certmap.conf

2017-02-23 Thread tomaskrizek
URL: https://github.com/freeipa/freeipa/pull/412 Title: #412: Define template version in certmap.conf 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#472][comment] Packaging: Add placeholder packages

2017-02-23 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/472 Title: #472: Packaging: Add placeholder packages tiran commented: """ Commit 1f8326aa fixes an issue in ```Makefile.python.am```. I think the issue caused ```ipaplatform``` and ```pypi/ipaplatform``` to cross streams. """ See the full comment

[Freeipa-devel] [freeipa PR#483][comment] lite-server: validate LDAP connection and cache schema

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/483 Title: #483: lite-server: validate LDAP connection and cache schema MartinBasti commented: """ Code looks good to me, as far as this is just code for developers I assume that @tiran tested it enough, so ACK. """ See the full comment at

[Freeipa-devel] [freeipa PR#483][+ack] lite-server: validate LDAP connection and cache schema

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/483 Title: #483: lite-server: validate LDAP connection and cache schema 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#493][+pushed] Update Contributors.txt

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/493 Title: #493: Update Contributors.txt 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#472][synchronized] Packaging: Add placeholder packages

2017-02-23 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/472 Author: tiran Title: #472: Packaging: Add placeholder packages Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/472/head:pr472 git checkout pr472 From

[Freeipa-devel] [freeipa PR#492][synchronized] [WIP] config: remove meaningless defaults

2017-02-23 Thread HonzaCholasta
URL: https://github.com/freeipa/freeipa/pull/492 Author: HonzaCholasta Title: #492: [WIP] config: remove meaningless defaults Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/492/head:pr492 git checkout

[Freeipa-devel] [freeipa PR#497][comment] added more meaningful help for --external-ca-type option

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/497 Title: #497: added more meaningful help for --external-ca-type option MartinBasti commented: """ This is already fixed in master, however it missed what is the default value ``` --external-ca-type={generic,ms-cs} Type

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

2017-02-23 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/379 Title: #379: Packaging: Add placeholder and IPA commands packages Label: +postponed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

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

2017-02-23 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/379 Title: #379: Packaging: Add placeholder and IPA commands packages tiran commented: """ I have postponed the ```ipacommands``` part. Placeholders are covered by #472. """ See the full comment at

[Freeipa-devel] [freeipa PR#482][+ack] Remove MD5 certificate fingerprints

2017-02-23 Thread tomaskrizek
URL: https://github.com/freeipa/freeipa/pull/482 Title: #482: Remove MD5 certificate fingerprints 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#482][comment] Remove MD5 certificate fingerprints

2017-02-23 Thread tomaskrizek
URL: https://github.com/freeipa/freeipa/pull/482 Title: #482: Remove MD5 certificate fingerprints tomaskrizek commented: """ ACK, there is no disagreement on the freeipa-devel. I'm already working on replacing SHA1 with SHA256 given the [recent

[Freeipa-devel] [freeipa PR#472][synchronized] Packaging: Add placeholder packages

2017-02-23 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/472 Author: tiran Title: #472: Packaging: Add placeholder packages Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/472/head:pr472 git checkout pr472 From

[Freeipa-devel] [freeipa PR#397][synchronized] Improve wheel building and provide ipaserver wheel for local testing

2017-02-23 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/397 Author: tiran Title: #397: Improve wheel building and provide ipaserver wheel for local testing 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#367][comment] Remove nsslib from IPA

2017-02-23 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/367 Title: #367: Remove nsslib from IPA stlaz commented: """ Some more fixes for Travis to check. """ See the full comment at https://github.com/freeipa/freeipa/pull/367#issuecomment-281950085 -- Manage your subscription for the Freeipa-devel

[Freeipa-devel] Release: script for updating contributors

2017-02-23 Thread Martin Kosek
Hi all, Based on my recent Contributors.txt update and on Martin Basti's request in the pull request: https://github.com/freeipa/freeipa/pull/493#issuecomment-281938080 I added my (hacky) script for updating the file in the freeipa-tools repo and updated our Release page:

[Freeipa-devel] make causes unsolicited changes to PO files

2017-02-23 Thread Christian Heimes
Hi, for a while make causes unsolicited modifications to all translation files. I have to reset all PO files a couple of times a day during development: git checkout -- po/*.po It's slowly wearing me off. I opened ticket https://fedorahosted.org/freeipa/ticket/6605 a while ago. It contains

[Freeipa-devel] [freeipa PR#484][+ack] FIPS: Remove pkispawn cruft

2017-02-23 Thread tomaskrizek
URL: https://github.com/freeipa/freeipa/pull/484 Title: #484: FIPS: Remove pkispawn cruft 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#499][opened] added help about default value for --external-ca-type option

2017-02-23 Thread tscherf
URL: https://github.com/freeipa/freeipa/pull/499 Author: tscherf Title: #499: added help about default value for --external-ca-type option Action: opened PR body: """ """ To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa

[Freeipa-devel] [freeipa PR#498][comment] compat: fix `Any` params in `batch` and `dnsrecord`

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/498 Title: #498: compat: fix `Any` params in `batch` and `dnsrecord` MartinBasti commented: """ Works for me """ See the full comment at https://github.com/freeipa/freeipa/pull/498#issuecomment-282068296 -- Manage your subscription for the

[Freeipa-devel] [freeipa PR#498][+ack] compat: fix `Any` params in `batch` and `dnsrecord`

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/498 Title: #498: compat: fix `Any` params in `batch` and `dnsrecord` 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#498][closed] compat: fix `Any` params in `batch` and `dnsrecord`

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/498 Author: HonzaCholasta Title: #498: compat: fix `Any` params in `batch` and `dnsrecord` Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/498/head:pr498 git checkout

[Freeipa-devel] [freeipa PR#498][comment] compat: fix `Any` params in `batch` and `dnsrecord`

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/498 Title: #498: compat: fix `Any` params in `batch` and `dnsrecord` MartinBasti commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/19060db1b8fa9d1d3e8f3ac3fcd1f387e9a39c94 """ See the full comment at

[Freeipa-devel] [freeipa PR#499][comment] added help about default value for --external-ca-type option

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/499 Title: #499: added help about default value for --external-ca-type option MartinBasti commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/573a0f1ffe5035b75fb88ee7752029e34c6b37af """ See the full comment at

[Freeipa-devel] [freeipa PR#499][+pushed] added help about default value for --external-ca-type option

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/499 Title: #499: added help about default value for --external-ca-type option Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

[Freeipa-devel] [freeipa PR#482][closed] Remove MD5 certificate fingerprints

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/482 Author: stlaz Title: #482: Remove MD5 certificate fingerprints Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/482/head:pr482 git checkout pr482 -- Manage your

[Freeipa-devel] [freeipa PR#500][opened] Replace sha1 fingerprints with sha256

2017-02-23 Thread tomaskrizek
URL: https://github.com/freeipa/freeipa/pull/500 Author: tomaskrizek Title: #500: Replace sha1 fingerprints with sha256 Action: opened PR body: """ - we probably want to keep SHA1 for DNS SSHFP (along with SHA256) - removing SHA1 from RSA-OAEP probably doesn't have any benefits

[Freeipa-devel] [freeipa PR#499][closed] added help about default value for --external-ca-type option

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/499 Author: tscherf Title: #499: added help about default value for --external-ca-type option Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/499/head:pr499 git

[Freeipa-devel] [freeipa PR#498][+pushed] compat: fix `Any` params in `batch` and `dnsrecord`

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/498 Title: #498: compat: fix `Any` params in `batch` and `dnsrecord` Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

[Freeipa-devel] [freeipa PR#483][closed] lite-server: validate LDAP connection and cache schema

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/483 Author: tiran Title: #483: lite-server: validate LDAP connection and cache schema Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/483/head:pr483 git checkout

[Freeipa-devel] [freeipa PR#483][+pushed] lite-server: validate LDAP connection and cache schema

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/483 Title: #483: lite-server: validate LDAP connection and cache schema Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

[Freeipa-devel] [freeipa PR#412][comment] Define template version in certmap.conf

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/412 Title: #412: Define template version in certmap.conf MartinBasti commented: """ needs rebase """ See the full comment at https://github.com/freeipa/freeipa/pull/412#issuecomment-282069402 -- Manage your subscription for the Freeipa-devel

[Freeipa-devel] [freeipa PR#482][comment] Remove MD5 certificate fingerprints

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/482 Title: #482: Remove MD5 certificate fingerprints MartinBasti commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/e2d1b21c5049f68d0336dcaf3f8657b214a34e2b """ See the full comment at

[Freeipa-devel] [freeipa PR#482][+pushed] Remove MD5 certificate fingerprints

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/482 Title: #482: Remove MD5 certificate fingerprints Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

Re: [Freeipa-devel] MD5 certificate fingerprints removal

2017-02-23 Thread Martin Basti
On 23.02.2017 15:09, Tomas Krizek wrote: On 02/22/2017 01:44 PM, Fraser Tweedale wrote: On Wed, Feb 22, 2017 at 01:41:22PM +0100, Tomas Krizek wrote: On 02/22/2017 12:28 AM, Fraser Tweedale wrote: On Tue, Feb 21, 2017 at 05:23:07PM +0100, Standa Laznicka wrote: On 02/21/2017 04:24 PM,

[Freeipa-devel] [freeipa PR#483][comment] lite-server: validate LDAP connection and cache schema

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/483 Title: #483: lite-server: validate LDAP connection and cache schema MartinBasti commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/dcb618152572ca013a447336e13d24399b5f7960 """ See the full comment at

[Freeipa-devel] [freeipa PR#367][comment] Remove nsslib from IPA

2017-02-23 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/367 Title: #367: Remove nsslib from IPA stlaz commented: """ Hopefully all issues were addressed + `radb` removed. If the Travis check passes then this is ready for review again. """ See the full comment at

[Freeipa-devel] [freeipa PR#139][comment] WebUI: Vault Management

2017-02-23 Thread pvomacka
URL: https://github.com/freeipa/freeipa/pull/139 Title: #139: WebUI: Vault Management pvomacka commented: """ @tiran Yes, rebased. """ See the full comment at https://github.com/freeipa/freeipa/pull/139#issuecomment-282060928 -- Manage your subscription for the Freeipa-devel mailing list:

[Freeipa-devel] [freeipa PR#139][synchronized] WebUI: Vault Management

2017-02-23 Thread pvomacka
URL: https://github.com/freeipa/freeipa/pull/139 Author: pvomacka Title: #139: WebUI: Vault Management Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/139/head:pr139 git checkout pr139 From

[Freeipa-devel] [freeipa PR#479][synchronized] Merge AD trust installer into composite ones

2017-02-23 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/479 Author: martbab Title: #479: Merge AD trust installer into composite ones Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/479/head:pr479 git checkout pr479

[Freeipa-devel] [freeipa PR#139][comment] WebUI: Vault Management

2017-02-23 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/139 Title: #139: WebUI: Vault Management tiran commented: """ @MartinBasti you approved this PR a month ago but it has neither the ACK flag nor was it merged. @pvomacka Your work would be useful for my Custodia Vault work. Can you rebase this PR

[Freeipa-devel] [freeipa PR#472][synchronized] Packaging: Add placeholder packages

2017-02-23 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/472 Author: tiran Title: #472: Packaging: Add placeholder packages Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/472/head:pr472 git checkout pr472 From

[Freeipa-devel] [freeipa PR#472][comment] Packaging: Add placeholder packages

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/472 Title: #472: Packaging: Add placeholder packages MartinBasti commented: """ Is really needed to have this is make check? It makes build dependencies of wheel mandratory not optional ``` error: invalid command 'bdist_wheel' make[5]: ***

[Freeipa-devel] [freeipa PR#472][comment] Packaging: Add placeholder packages

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/472 Title: #472: Packaging: Add placeholder packages MartinBasti commented: """ Is really needed to have this is make check? It makes build dependencies of wheel mandratory not optional ``` error: invalid command 'bdist_wheel' make[5]: ***

[Freeipa-devel] [freeipa PR#496][+ack] Use newer Certificate.serial_number in krainstance.py

2017-02-23 Thread flo-renaud
URL: https://github.com/freeipa/freeipa/pull/496 Title: #496: Use newer Certificate.serial_number in krainstance.py 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#496][comment] Use newer Certificate.serial_number in krainstance.py

2017-02-23 Thread flo-renaud
URL: https://github.com/freeipa/freeipa/pull/496 Title: #496: Use newer Certificate.serial_number in krainstance.py flo-renaud commented: """ Hi @stlaz , the warning `/usr/lib/python2.7/site-packages/ipaserver/install/krainstance.py:316: DeprecationWarning: Certificate serial is deprecated,

[Freeipa-devel] [freeipa PR#472][comment] Packaging: Add placeholder packages

2017-02-23 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/472 Title: #472: Packaging: Add placeholder packages tiran commented: """ I'm open to suggestions here, but I like to have automatic validation of packaging. """ See the full comment at

[Freeipa-devel] [freeipa PR#472][comment] Packaging: Add placeholder packages

2017-02-23 Thread lslebodn
URL: https://github.com/freeipa/freeipa/pull/472 Title: #472: Packaging: Add placeholder packages lslebodn commented: """ I agree with @MartinBasti it is not a unit test. IMHO better approach is to test it in CI/travis/... """ See the full comment at

[Freeipa-devel] [freeipa PR#472][comment] Packaging: Add placeholder packages

2017-02-23 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/472 Title: #472: Packaging: Add placeholder packages tiran commented: """ That's not the point here. We are arguing about a new build dependency (python-wheel). """ See the full comment at

[Freeipa-devel] [freeipa PR#412][comment] Define template version in certmap.conf

2017-02-23 Thread flo-renaud
URL: https://github.com/freeipa/freeipa/pull/412 Title: #412: Define template version in certmap.conf flo-renaud commented: """ Bump for review """ See the full comment at https://github.com/freeipa/freeipa/pull/412#issuecomment-281931336 -- Manage your subscription for the Freeipa-devel

[Freeipa-devel] [freeipa PR#397][comment] Improve wheel building and provide ipaserver wheel for local testing

2017-02-23 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/397 Title: #397: Improve wheel building and provide ipaserver wheel for local testing tiran commented: """ I didn't know about the ```SkipPluginModule``` feature. I agree with you, your solution is more elegant. """ See the full comment at

[Freeipa-devel] [freeipa PR#472][comment] Packaging: Add placeholder packages

2017-02-23 Thread lslebodn
URL: https://github.com/freeipa/freeipa/pull/472 Title: #472: Packaging: Add placeholder packages lslebodn commented: """ FYI: At the moment, `make check` run just C-based unit test and all of them are optional. If required dependency is not found at configure time then test is not

[Freeipa-devel] [freeipa PR#493][comment] Update Contributors.txt

2017-02-23 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/493 Title: #493: Update Contributors.txt stlaz commented: """ I don't give two poops and a popsicle about the order of names in the Contributors.txt file. ACK. """ See the full comment at

[Freeipa-devel] [freeipa PR#493][+ack] Update Contributors.txt

2017-02-23 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/493 Title: #493: Update Contributors.txt 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#472][comment] Packaging: Add placeholder packages

2017-02-23 Thread pvoborni
URL: https://github.com/freeipa/freeipa/pull/472 Title: #472: Packaging: Add placeholder packages pvoborni commented: """ Some distros like RHEL doesn't have python-wheel packaged. It can be disabled by downstream patch, but better would be to remove it or make it configurable. """ See the

[Freeipa-devel] [freeipa PR#472][comment] Packaging: Add placeholder packages

2017-02-23 Thread lslebodn
URL: https://github.com/freeipa/freeipa/pull/472 Title: #472: Packaging: Add placeholder packages lslebodn commented: """ NACK for downstream patch. The intentin of build system refactoring was make packaging in downstream simpler. """ See the full comment at

[Freeipa-devel] [freeipa PR#460][+postponed] [Py3] ipa-server-install, ipa-server-upgrade fixes

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/460 Title: #460: [Py3] ipa-server-install, ipa-server-upgrade fixes Label: +postponed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

[Freeipa-devel] [freeipa PR#439][+postponed] [WIP] [Py3] testing both py2/py3 in travis

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/439 Title: #439: [WIP] [Py3] testing both py2/py3 in travis Label: +postponed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

[Freeipa-devel] [freeipa PR#472][comment] Packaging: Add placeholder packages

2017-02-23 Thread lslebodn
URL: https://github.com/freeipa/freeipa/pull/472 Title: #472: Packaging: Add placeholder packages lslebodn commented: """ NACK for downstream patch. The intentin of build system refactoring was make packaging in downstream simpler. """ See the full comment at

[Freeipa-devel] [freeipa PR#495][+pushed] Fix ipa-server-upgrade

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/495 Title: #495: Fix ipa-server-upgrade 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#496][closed] Use newer Certificate.serial_number in krainstance.py

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/496 Author: stlaz Title: #496: Use newer Certificate.serial_number in krainstance.py Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/496/head:pr496 git checkout pr496

[Freeipa-devel] [freeipa PR#495][closed] Fix ipa-server-upgrade

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/495 Author: stlaz Title: #495: Fix ipa-server-upgrade Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/495/head:pr495 git checkout pr495 -- Manage your subscription

[Freeipa-devel] [freeipa PR#496][comment] Use newer Certificate.serial_number in krainstance.py

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/496 Title: #496: Use newer Certificate.serial_number in krainstance.py MartinBasti commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/8c2cd66269d7be976ae7fc990343ed8e9b5282a3 """ See the full comment at

[Freeipa-devel] [freeipa PR#496][+pushed] Use newer Certificate.serial_number in krainstance.py

2017-02-23 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/496 Title: #496: Use newer Certificate.serial_number in krainstance.py Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: