[Freeipa-devel] [freeipa PR#534][closed] Move csrgen templates into ipaclient package

2017-03-08 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/534 Author: tiran Title: #534: Move csrgen templates into ipaclient package Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/534/head:pr534 git checkout pr534 --

[Freeipa-devel] [freeipa PR#549][closed] WebUI: certmap match

2017-03-08 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/549 Author: pvomacka Title: #549: WebUI: certmap match Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/549/head:pr549 git checkout pr549 -- Manage your subscription

[Freeipa-devel] [freeipa PR#549][+pushed] WebUI: certmap match

2017-03-08 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/549 Title: #549: WebUI: certmap match 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#549][comment] WebUI: certmap match

2017-03-08 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/549 Title: #549: WebUI: certmap match dkupka commented: """ master: * 6be32edde0ae16473d4d109747adae78f9d725e4 WebUI: Add possibility to turn of autoload when details.load is called * 1d6cc35c03669ea67d9e9ee9ca0ff62401d1b157 WebUI: Possibility to

[Freeipa-devel] [freeipa PR#554][+pushed] webui: fixes normalization of value in attributes widget

2017-03-08 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/554 Title: #554: webui: fixes normalization of value in attributes widget 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#554][closed] webui: fixes normalization of value in attributes widget

2017-03-08 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/554 Author: pvoborni Title: #554: webui: fixes normalization of value in attributes widget Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/554/head:pr554 git checkout

[Freeipa-devel] [freeipa PR#554][comment] webui: fixes normalization of value in attributes widget

2017-03-08 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/554 Title: #554: webui: fixes normalization of value in attributes widget dkupka commented: """ master: * 56a2642af0a29328df4defef138b9fa65624335a webui: fixes normalization of value in attributes widget """ See the full comment at

[Freeipa-devel] [freeipa PR#556][opened] Don't allow standalone KRA uninstalls

2017-03-08 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/556 Author: stlaz Title: #556: Don't allow standalone KRA uninstalls Action: opened PR body: """ KRA uninstallation is very likely to break the user's setup. Don't allow it at least till we can be safely sure we are able to remove it in a

[Freeipa-devel] [freeipa PR#420][+pushed] Allow login to WebUI using Kerberos aliases/enterprise principals

2017-03-08 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/420 Title: #420: Allow login to WebUI using Kerberos aliases/enterprise principals 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#420][comment] Allow login to WebUI using Kerberos aliases/enterprise principals

2017-03-08 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/420 Title: #420: Allow login to WebUI using Kerberos aliases/enterprise principals dkupka commented: """ master: * f8d7e37a091c1df4c989b80b8d19e12ab35533c8 Allow login to WebUI using Kerberos aliases/enterprise principals """ See the full comment

[Freeipa-devel] [freeipa PR#551][comment] config: re-add `init_config` and `config`

2017-03-08 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/551 Title: #551: config: re-add `init_config` and `config` dkupka commented: """ master: * 0c7ca279c78bc23d45582e92bb1638865ec3059e config: re-add `init_config` and `config` """ See the full comment at

[Freeipa-devel] [freeipa PR#551][closed] config: re-add `init_config` and `config`

2017-03-08 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/551 Author: HonzaCholasta Title: #551: config: re-add `init_config` and `config` Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/551/head:pr551 git checkout pr551 --

[Freeipa-devel] [freeipa PR#548][closed] ipa-server-install: add --setup-kra option

2017-03-08 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/548 Author: MartinBasti Title: #548: ipa-server-install: add --setup-kra option Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/548/head:pr548 git checkout pr548 --

[Freeipa-devel] [freeipa PR#548][comment] ipa-server-install: add --setup-kra option

2017-03-08 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/548 Title: #548: ipa-server-install: add --setup-kra option dkupka commented: """ master: * 4006cbbc02c368ac9e5e3721613158decb34fd37 KRA: add --setup-kra to ipa-server-install * 25fa2bb6c9fa1b498330b13c9a6116b646eb75ba tests: use --setup-kra in

[Freeipa-devel] [freeipa PR#549][synchronized] WebUI: certmap match

2017-03-08 Thread pvomacka
URL: https://github.com/freeipa/freeipa/pull/549 Author: pvomacka Title: #549: WebUI: certmap match Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/549/head:pr549 git checkout pr549 From

[Freeipa-devel] [freeipa PR#548][+pushed] ipa-server-install: add --setup-kra option

2017-03-08 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/548 Title: #548: ipa-server-install: add --setup-kra 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#555][opened] ipa-managed-entries: use server-mode API

2017-03-08 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/555 Author: martbab Title: #555: ipa-managed-entries: use server-mode API Action: opened PR body: """ During LDAP connection management refactoring the ad-hoc ldap connection in `ipa-managed-entries` was replaced by calls to ldap2 backend without

[Freeipa-devel] [freeipa PR#553][comment] Add check for removing last KRA server

2017-03-08 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/553 Title: #553: Add check for removing last KRA server stlaz commented: """ Split done. """ See the full comment at https://github.com/freeipa/freeipa/pull/553#issuecomment-285077007 -- Manage your subscription for the Freeipa-devel mailing list:

[Freeipa-devel] [freeipa PR#553][synchronized] Add check for removing last KRA server

2017-03-08 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/553 Author: stlaz Title: #553: Add check for removing last KRA server Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/553/head:pr553 git checkout pr553 From

[Freeipa-devel] [freeipa PR#535][comment] add whoami command

2017-03-08 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/535 Title: #535: add whoami command stlaz commented: """ I believe that in CLI `ipa whoami` should actually output the output of the command it found with the arguments and options it found since in WebUI this is eventually done as well. I can go

[Freeipa-devel] [freeipa PR#551][+ack] config: re-add `init_config` and `config`

2017-03-08 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/551 Title: #551: config: re-add `init_config` and `config` 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#551][comment] config: re-add `init_config` and `config`

2017-03-08 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/551 Title: #551: config: re-add `init_config` and `config` MartinBasti commented: """ Works for me I was able to install Ipsilon using this: https://ipsilon-project.org/doc/quickstart-ipa.html """ See the full comment at

[Freeipa-devel] [freeipa PR#535][comment] add whoami command

2017-03-08 Thread abbra
URL: https://github.com/freeipa/freeipa/pull/535 Title: #535: add whoami command abbra commented: """ We can disable it for CLI, that's not a problem. """ See the full comment at https://github.com/freeipa/freeipa/pull/535#issuecomment-285085254 -- Manage your subscription for the

[Freeipa-devel] [freeipa PR#549][comment] WebUI: certmap match

2017-03-08 Thread pvomacka
URL: https://github.com/freeipa/freeipa/pull/549 Title: #549: WebUI: certmap match pvomacka commented: """ @pvoborni Yes, we should make a lint rule for leading spaces. """ See the full comment at https://github.com/freeipa/freeipa/pull/549#issuecomment-285061561 -- Manage your subscription

[Freeipa-devel] [freeipa PR#505][+pushed] dns: fix `dnsrecord_add` interactive mode

2017-03-08 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/505 Title: #505: dns: fix `dnsrecord_add` interactive mode 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#555][synchronized] ipa-managed-entries: use server-mode API

2017-03-08 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/555 Author: martbab Title: #555: ipa-managed-entries: use server-mode API Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/555/head:pr555 git checkout pr555 From

[Freeipa-devel] [freeipa PR#544][synchronized] Don't use weak ciphers for client HTTPS connections

2017-03-08 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/544 Author: stlaz Title: #544: Don't use weak ciphers for client HTTPS connections Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/544/head:pr544 git checkout

[Freeipa-devel] [freeipa PR#539][comment] Define errors_by_code in ipalib.errors

2017-03-08 Thread HonzaCholasta
URL: https://github.com/freeipa/freeipa/pull/539 Title: #539: Define errors_by_code in ipalib.errors HonzaCholasta commented: """ Seems like an overkill for the ~2 types of errors which the code uses. Anyway, I would rather wait before making any kind of decision based on cert-request code

[Freeipa-devel] [freeipa PR#539][comment] Define errors_by_code in ipalib.errors

2017-03-08 Thread frasertweedale
URL: https://github.com/freeipa/freeipa/pull/539 Title: #539: Define errors_by_code in ipalib.errors frasertweedale commented: """ Righto. I'll withdraw this PR for now and it will make a comeback closer to landing the gssapi work. """ See the full comment at

[Freeipa-devel] [freeipa PR#539][closed] Define errors_by_code in ipalib.errors

2017-03-08 Thread frasertweedale
URL: https://github.com/freeipa/freeipa/pull/539 Author: frasertweedale Title: #539: Define errors_by_code in ipalib.errors Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/539/head:pr539 git checkout pr539 --

[Freeipa-devel] [freeipa PR#557][opened] certmap: load certificate from file in certmap-match CLI

2017-03-08 Thread HonzaCholasta
URL: https://github.com/freeipa/freeipa/pull/557 Author: HonzaCholasta Title: #557: certmap: load certificate from file in certmap-match CLI Action: opened PR body: """ Load the certificate from a file specified in the first argument. Raw certificate value can be specified using

[Freeipa-devel] [freeipa PR#516][comment] IdM Server: list all Employees with matching Smart Card

2017-03-08 Thread HonzaCholasta
URL: https://github.com/freeipa/freeipa/pull/516 Title: #516: IdM Server: list all Employees with matching Smart Card HonzaCholasta commented: """ I forgot to say that in the CLI, the certificate should be specified using a file. PR #557 implements this. """ See the full comment at

[Freeipa-devel] [freeipa PR#548][+ack] ipa-server-install: add --setup-kra option

2017-03-08 Thread tomaskrizek
URL: https://github.com/freeipa/freeipa/pull/548 Title: #548: ipa-server-install: add --setup-kra 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#331][comment] WebUI: don't change casing of Auth Indicators values

2017-03-08 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/331 Title: #331: WebUI: don't change casing of Auth Indicators values dkupka commented: """ master: * 0220fc8986e4fef017185bde675dc9cf0f90afd8 WebUI: Allow disabling lowering text in custom_checkbox_widget * ad3451067ad474ea52872913d6789b1652f9a9c4

[Freeipa-devel] [freeipa PR#331][closed] WebUI: don't change casing of Auth Indicators values

2017-03-08 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/331 Author: pvomacka Title: #331: WebUI: don't change casing of Auth Indicators values Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/331/head:pr331 git checkout

[Freeipa-devel] [freeipa PR#549][+ack] WebUI: certmap match

2017-03-08 Thread pvoborni
URL: https://github.com/freeipa/freeipa/pull/549 Title: #549: WebUI: certmap match 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#420][closed] Allow login to WebUI using Kerberos aliases/enterprise principals

2017-03-08 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/420 Author: martbab Title: #420: Allow login to WebUI using Kerberos aliases/enterprise principals Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/420/head:pr420 git

[Freeipa-devel] [freeipa PR#535][comment] add whoami command

2017-03-08 Thread abbra
URL: https://github.com/freeipa/freeipa/pull/535 Title: #535: add whoami command abbra commented: """ Uhm, no, I don't want that. It makes the command behaving differently depending on a context and that would be broken. For client-side plugin that would also be an abuse of interface, I'd

[Freeipa-devel] [freeipa PR#534][+pushed] Move csrgen templates into ipaclient package

2017-03-08 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/534 Title: #534: Move csrgen templates into ipaclient package 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#534][comment] Move csrgen templates into ipaclient package

2017-03-08 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/534 Title: #534: Move csrgen templates into ipaclient package dkupka commented: """ master: * 80be18162921268be9c8981495c9e8a4de0c85cd Move csrgen templates into ipaclient package * 177f07e163d6d591a1e609d35e0a6f6f5347551e Chain CSR generator file

[Freeipa-devel] [freeipa PR#553][comment] Add check for removing last KRA server

2017-03-08 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/553 Title: #553: Add check for removing last KRA server MartinBasti commented: """ JFTR: KRA uninstall commit is here #556 """ See the full comment at https://github.com/freeipa/freeipa/pull/553#issuecomment-285093976 -- Manage your subscription

[Freeipa-devel] [freeipa PR#553][comment] Add check for removing last KRA server

2017-03-08 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/553 Title: #553: Add check for removing last KRA server MartinBasti commented: """ Probably you we should fix this before we double number of alerts https://pagure.io/freeipa/issue/6598 """ See the full comment at

[Freeipa-devel] [freeipa PR#544][+ack] Don't use weak ciphers for client HTTPS connections

2017-03-08 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/544 Title: #544: Don't use weak ciphers for client HTTPS connections 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#420][synchronized] Allow login to WebUI using Kerberos aliases/enterprise principals

2017-03-08 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/420 Author: martbab Title: #420: Allow login to WebUI using Kerberos aliases/enterprise principals 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#331][synchronized] WebUI: don't change casing of Auth Indicators values

2017-03-08 Thread pvomacka
URL: https://github.com/freeipa/freeipa/pull/331 Author: pvomacka Title: #331: WebUI: don't change casing of Auth Indicators values Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/331/head:pr331 git

[Freeipa-devel] [freeipa PR#552][+ack] man: add missing --setup-adtrust option to manpage

2017-03-08 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/552 Title: #552: man: add missing --setup-adtrust option to manpage 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#552][comment] man: add missing --setup-adtrust option to manpage

2017-03-08 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/552 Title: #552: man: add missing --setup-adtrust option to manpage martbab commented: """ master: * 6c95f33d37a2c346fc56d9890d594f1e40029c77 man: add missing --setup-adtrust option to manpage """ See the full comment at

[Freeipa-devel] [freeipa PR#552][closed] man: add missing --setup-adtrust option to manpage

2017-03-08 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/552 Author: MartinBasti Title: #552: man: add missing --setup-adtrust option to manpage Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/552/head:pr552 git checkout

[Freeipa-devel] [freeipa PR#552][+pushed] man: add missing --setup-adtrust option to manpage

2017-03-08 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/552 Title: #552: man: add missing --setup-adtrust option to manpage 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#553][opened] Add check for removing last KRA server

2017-03-08 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/553 Author: stlaz Title: #553: Add check for removing last KRA server Action: opened PR body: """ This patchset adds a check for removal of a last KRA server + adds a message about there only being one KRA to WebUI. """ To pull the PR as Git

[Freeipa-devel] [freeipa PR#300][synchronized] WebUI: Add support for custom table pagination size

2017-03-08 Thread pvomacka
URL: https://github.com/freeipa/freeipa/pull/300 Author: pvomacka Title: #300: WebUI: Add support for custom table pagination size Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/300/head:pr300 git

[Freeipa-devel] [freeipa PR#300][comment] WebUI: Add support for custom table pagination size

2017-03-08 Thread pvomacka
URL: https://github.com/freeipa/freeipa/pull/300 Title: #300: WebUI: Add support for custom table pagination size pvomacka commented: """ @pvoborni Thank you for review. Proposed changes fixed. """ See the full comment at https://github.com/freeipa/freeipa/pull/300#issuecomment-285025154 --

[Freeipa-devel] [freeipa PR#549][comment] WebUI: certmap match

2017-03-08 Thread pvomacka
URL: https://github.com/freeipa/freeipa/pull/549 Title: #549: WebUI: certmap match pvomacka commented: """ In last sync I changed string of clear button title. """ See the full comment at https://github.com/freeipa/freeipa/pull/549#issuecomment-285025740 -- Manage your subscription for the

[Freeipa-devel] [freeipa PR#516][comment] IdM Server: list all Employees with matching Smart Card

2017-03-08 Thread flo-renaud
URL: https://github.com/freeipa/freeipa/pull/516 Title: #516: IdM Server: list all Employees with matching Smart Card flo-renaud commented: """ @dkupka I added the following explanation in the doc for certmap_match: """ Search for users matching the provided certificate. This

[Freeipa-devel] [freeipa PR#548][synchronized] ipa-server-install: add --setup-kra option

2017-03-08 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/548 Author: MartinBasti Title: #548: ipa-server-install: add --setup-kra option Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/548/head:pr548 git checkout

[Freeipa-devel] [freeipa PR#552][opened] man: add missing --setup-adtrust option to manpage

2017-03-08 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/552 Author: MartinBasti Title: #552: man: add missing --setup-adtrust option to manpage Action: opened PR body: """ ipa-server-install and ipa-replica-install manpages miss --setup-adtrust options https://pagure.io/freeipa/issue/6630 """ To pull

[Freeipa-devel] [freeipa PR#549][synchronized] WebUI: certmap match

2017-03-08 Thread pvomacka
URL: https://github.com/freeipa/freeipa/pull/549 Author: pvomacka Title: #549: WebUI: certmap match Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/549/head:pr549 git checkout pr549 From

[Freeipa-devel] [freeipa PR#516][synchronized] IdM Server: list all Employees with matching Smart Card

2017-03-08 Thread flo-renaud
URL: https://github.com/freeipa/freeipa/pull/516 Author: flo-renaud Title: #516: IdM Server: list all Employees with matching Smart Card Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/516/head:pr516 git

[Freeipa-devel] [freeipa PR#548][comment] ipa-server-install: add --setup-kra option

2017-03-08 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/548 Title: #548: ipa-server-install: add --setup-kra option MartinBasti commented: """ Given that there is no time, I dropped commit you NACKed as it unneeded for this PR, but please note my disagreement about a way how `kra.py` handles

[Freeipa-devel] [freeipa PR#551][+pushed] config: re-add `init_config` and `config`

2017-03-08 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/551 Title: #551: config: re-add `init_config` and `config` 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#555][+ack] ipa-managed-entries: use server-mode API

2017-03-08 Thread tomaskrizek
URL: https://github.com/freeipa/freeipa/pull/555 Title: #555: ipa-managed-entries: use server-mode API 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#505][+ack] dns: fix `dnsrecord_add` interactive mode

2017-03-08 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/505 Title: #505: dns: fix `dnsrecord_add` interactive mode 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#420][comment] Allow login to WebUI using Kerberos aliases/enterprise principals

2017-03-08 Thread abbra
URL: https://github.com/freeipa/freeipa/pull/420 Title: #420: Allow login to WebUI using Kerberos aliases/enterprise principals abbra commented: """ Thanks. LGTM and works for me with IPA user, IPA host principal, and AD user. The latter cannot yet actually use Web UI but that is a separate

[Freeipa-devel] [freeipa PR#420][+ack] Allow login to WebUI using Kerberos aliases/enterprise principals

2017-03-08 Thread abbra
URL: https://github.com/freeipa/freeipa/pull/420 Title: #420: Allow login to WebUI using Kerberos aliases/enterprise principals 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#331][+pushed] WebUI: don't change casing of Auth Indicators values

2017-03-08 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/331 Title: #331: WebUI: don't change casing of Auth Indicators values 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#534][+ack] Move csrgen templates into ipaclient package

2017-03-08 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/534 Title: #534: Move csrgen templates into ipaclient package 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#505][closed] dns: fix `dnsrecord_add` interactive mode

2017-03-08 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/505 Author: HonzaCholasta Title: #505: dns: fix `dnsrecord_add` interactive mode Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/505/head:pr505 git checkout pr505 --

[Freeipa-devel] [freeipa PR#505][comment] dns: fix `dnsrecord_add` interactive mode

2017-03-08 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/505 Title: #505: dns: fix `dnsrecord_add` interactive mode dkupka commented: """ master: * 1e912f5b83166154806e0382f3f028d0eac81731 dns: fix `dnsrecord_add` interactive mode """ See the full comment at

[Freeipa-devel] [freeipa PR#535][comment] add whoami command

2017-03-08 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/535 Title: #535: add whoami command stlaz commented: """ Ok. It just doesn't seem right to have a command which shows something that's not immediately useful to the user. I am not sure whether we should have it enabled for CLI. """ See the full

[Freeipa-devel] [freeipa PR#300][+ack] WebUI: Add support for custom table pagination size

2017-03-08 Thread pvoborni
URL: https://github.com/freeipa/freeipa/pull/300 Title: #300: WebUI: Add support for custom table pagination size 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#331][+ack] WebUI: don't change casing of Auth Indicators values

2017-03-08 Thread pvoborni
URL: https://github.com/freeipa/freeipa/pull/331 Title: #331: WebUI: don't change casing of Auth Indicators values 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#331][comment] WebUI: don't change casing of Auth Indicators values

2017-03-08 Thread pvoborni
URL: https://github.com/freeipa/freeipa/pull/331 Title: #331: WebUI: don't change casing of Auth Indicators values pvoborni commented: """ ACK but I've find out that the change is not enough because of existing bug. See pr #554 """ See the full comment at

[Freeipa-devel] [freeipa PR#553][comment] Add check for removing last KRA server

2017-03-08 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/553 Title: #553: Add check for removing last KRA server stlaz commented: """ Hm, I forgot that KRA is the only IPA component that has a standalone uninstaller, this is therefore only a partial fix. """ See the full comment at

[Freeipa-devel] [freeipa PR#553][comment] Add check for removing last KRA server

2017-03-08 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/553 Title: #553: Add check for removing last KRA server MartinBasti commented: """ @stlaz I wrote it to ticket """ See the full comment at https://github.com/freeipa/freeipa/pull/553#issuecomment-285033127 -- Manage your subscription for the

[Freeipa-devel] [freeipa PR#554][opened] webui: fixes normalization of value in attributes widget

2017-03-08 Thread pvoborni
URL: https://github.com/freeipa/freeipa/pull/554 Author: pvoborni Title: #554: webui: fixes normalization of value in attributes widget Action: opened PR body: """ Fix is in checkboxes widget but the only affected one is attributes widget. Reproduction: 1. Add permission with attribute with

[Freeipa-devel] [freeipa PR#553][synchronized] Add check for removing last KRA server

2017-03-08 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/553 Author: stlaz Title: #553: Add check for removing last KRA server Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/553/head:pr553 git checkout pr553 From

[Freeipa-devel] [freeipa PR#553][comment] Add check for removing last KRA server

2017-03-08 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/553 Title: #553: Add check for removing last KRA server stlaz commented: """ @MartinBasti ah, sorry, I completely overlooked it. The current PR version implements your suggestion. """ See the full comment at

[Freeipa-devel] [freeipa PR#310][comment] WIP: CLI testing

2017-03-08 Thread pvoborni
URL: https://github.com/freeipa/freeipa/pull/310 Title: #310: WIP: CLI testing pvoborni commented: """ Marking as postponed. We cannot expect the changes to be addressed by @mirielka any time soon. And CLI testing might need design discussion. """ See the full comment at

[Freeipa-devel] [freeipa PR#310][+postponed] WIP: CLI testing

2017-03-08 Thread pvoborni
URL: https://github.com/freeipa/freeipa/pull/310 Title: #310: WIP: CLI testing Label: +postponed -- 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#300][+pushed] WebUI: Add support for custom table pagination size

2017-03-08 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/300 Title: #300: WebUI: Add support for custom table pagination size 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#300][closed] WebUI: Add support for custom table pagination size

2017-03-08 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/300 Author: pvomacka Title: #300: WebUI: Add support for custom table pagination size Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/300/head:pr300 git checkout

[Freeipa-devel] [freeipa PR#300][comment] WebUI: Add support for custom table pagination size

2017-03-08 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/300 Title: #300: WebUI: Add support for custom table pagination size martbab commented: """ master: * 7b699105a52d4d8c26a73044ba182d752b4a9833 Add javascript integer validator * f78cc8932626de667c6e3a4461141a10a5d9c2e6 Make singleton from config

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

2017-03-08 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/397 Title: #397: Improve wheel building and provide ipaserver wheel for local testing MartinBasti commented: """ needs rebase """ See the full comment at https://github.com/freeipa/freeipa/pull/397#issuecomment-285048889 -- Manage your

[Freeipa-devel] [freeipa PR#516][+ack] IdM Server: list all Employees with matching Smart Card

2017-03-08 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/516 Title: #516: IdM Server: list all Employees with matching Smart Card 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#516][comment] IdM Server: list all Employees with matching Smart Card

2017-03-08 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/516 Title: #516: IdM Server: list all Employees with matching Smart Card dkupka commented: """ @flo-renaud Thank you. """ See the full comment at https://github.com/freeipa/freeipa/pull/516#issuecomment-285049667 -- Manage your subscription for

[Freeipa-devel] [freeipa PR#516][comment] IdM Server: list all Employees with matching Smart Card

2017-03-08 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/516 Title: #516: IdM Server: list all Employees with matching Smart Card dkupka commented: """ master: * ea34e17a46a60efb9c4dc81dab919a1639dec73b IdM Server: list all Employees with matching Smart Card """ See the full comment at

[Freeipa-devel] [freeipa PR#516][+pushed] IdM Server: list all Employees with matching Smart Card

2017-03-08 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/516 Title: #516: IdM Server: list all Employees with matching Smart Card 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#516][closed] IdM Server: list all Employees with matching Smart Card

2017-03-08 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/516 Author: flo-renaud Title: #516: IdM Server: list all Employees with matching Smart Card Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/516/head:pr516 git

[Freeipa-devel] [freeipa PR#553][comment] Add check for removing last KRA server

2017-03-08 Thread MartinBasti
URL: https://github.com/freeipa/freeipa/pull/553 Title: #553: Add check for removing last KRA server MartinBasti commented: """ Please create a separate commit for KRA Uninstall """ See the full comment at https://github.com/freeipa/freeipa/pull/553#issuecomment-285050412 -- Manage your

[Freeipa-devel] [freeipa PR#554][+ack] webui: fixes normalization of value in attributes widget

2017-03-08 Thread pvomacka
URL: https://github.com/freeipa/freeipa/pull/554 Title: #554: webui: fixes normalization of value in attributes widget 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#549][edited] T6601 certmap match

2017-03-08 Thread pvomacka
URL: https://github.com/freeipa/freeipa/pull/549 Author: pvomacka Title: #549: T6601 certmap match Action: edited Changed field: title Original value: """ T6601 certmap match """ -- Manage your subscription for the Freeipa-devel mailing list:

Re: [Freeipa-devel] Please review: V4/AD user short names design draft

2017-03-08 Thread Martin Babinsky
On Tue, Feb 28, 2017 at 01:29:50PM +0100, Martin Babinsky wrote: >Hello list, > >I have put together a draft of design page describing server-side >implementation of user short name -> fully-qualified name resolution.[1] > >In the end I have taken the liberty to change a few aspects of the design

[Freeipa-devel] [freeipa PR#542][comment] Implementation independent interface for CSR generation

2017-03-08 Thread HonzaCholasta
URL: https://github.com/freeipa/freeipa/pull/542 Title: #542: Implementation independent interface for CSR generation HonzaCholasta commented: """ I would rather make things simple and remove the abstraction. We can support NSS databases by PKCS#12 export/import until we have first-class

Re: [Freeipa-devel] Please review: V4/AD user short names design draft

2017-03-08 Thread Jan Cholasta
On 8.3.2017 10:30, Martin Babinsky wrote: On Tue, Feb 28, 2017 at 01:29:50PM +0100, Martin Babinsky wrote: Hello list, I have put together a draft of design page describing server-side implementation of user short name -> fully-qualified name resolution.[1] In the end I have taken the liberty

[Freeipa-devel] [freeipa PR#516][comment] IdM Server: list all Employees with matching Smart Card

2017-03-08 Thread sumit-bose
URL: https://github.com/freeipa/freeipa/pull/516 Title: #516: IdM Server: list all Employees with matching Smart Card sumit-bose commented: """ I agree, it would be good if the help text can mention that cached data is used and maybe even mention the sss_cache utility to invalidate the entry.

[Freeipa-devel] [freeipa PR#550][opened] install: fix help

2017-03-08 Thread HonzaCholasta
URL: https://github.com/freeipa/freeipa/pull/550 Author: HonzaCholasta Title: #550: install: fix help Action: opened PR body: """ This PR fixes the known issue in the installer refactoring PR #232 and concludes https://pagure.io/freeipa/issue/6392. **server install: remove duplicate -w

[Freeipa-devel] [freeipa PR#545][comment] install_check: require IPv6 stack to be enabled

2017-03-08 Thread HonzaCholasta
URL: https://github.com/freeipa/freeipa/pull/545 Title: #545: install_check: require IPv6 stack to be enabled HonzaCholasta commented: """ Nitpick: the "install_check" prefix in the subject does not say anything, there are currently 7 `install_check`s in IPA. Please use something more

[Freeipa-devel] [freeipa PR#548][comment] ipa-server-install: add --setup-kra option

2017-03-08 Thread HonzaCholasta
URL: https://github.com/freeipa/freeipa/pull/548 Title: #548: ipa-server-install: add --setup-kra option HonzaCholasta commented: """ NACK on the "KRA: run install and install_check only when KRA should be installed" commit. The end goal for all component installers is to make them isolated

[Freeipa-devel] [freeipa PR#516][comment] IdM Server: list all Employees with matching Smart Card

2017-03-08 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/516 Title: #516: IdM Server: list all Employees with matching Smart Card dkupka commented: """ @sumit-bose I agree. If this is in help text we can also display it in WebUI. @flo-renaud Please add description and explanation of this behaviour into

[Freeipa-devel] [freeipa PR#400][+ack] WebUI: Certificate Mapping

2017-03-08 Thread pvoborni
URL: https://github.com/freeipa/freeipa/pull/400 Title: #400: WebUI: Certificate Mapping 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

  1   2   >