[Freeipa-devel] Re: add Dogtag 10.4 builds to FreeIPA COPRs

2017-06-08 Thread Fraser Tweedale via FreeIPA-devel
On Thu, Jun 08, 2017 at 05:13:43PM +0200, Martin Bašti wrote: > > > On 08.06.2017 09:08, Martin Bašti via FreeIPA-devel wrote: > > > > > > On 08.06.2017 02:43, Fraser Tweedale via FreeIPA-devel wrote: > > > My PR https://github.com/freeipa/freeipa/pull/859 bumps the pki-core > > > dependency

[Freeipa-devel] [freeipa PR#859][comment] Add CommonNameToSANDefault to default cert profile

2017-06-08 Thread frasertweedale via FreeIPA-devel
URL: https://github.com/freeipa/freeipa/pull/859 Title: #859: Add CommonNameToSANDefault to default cert profile frasertweedale commented: """ ipa-4-5 PR: https://github.com/freeipa/freeipa/pull/863 """ See the full comment at https://github.com/freeipa/freeipa/pull/859#issuecomment-307287760

[Freeipa-devel] [freeipa PR#845][comment] ipadiscovery: Return realm as a string

2017-06-08 Thread MartinBasti via FreeIPA-devel
URL: https://github.com/freeipa/freeipa/pull/845 Title: #845: ipadiscovery: Return realm as a string MartinBasti commented: """ LGTM, we anyway assume inside framework that everything is in utf-8 """ See the full comment at https://github.com/freeipa/freeipa/pull/845#issuecomment-307142144

[Freeipa-devel] [freeipa PR#849][+ack] session_storage: Correctly handle string/byte types

2017-06-08 Thread MartinBasti via FreeIPA-devel
URL: https://github.com/freeipa/freeipa/pull/849 Title: #849: session_storage: Correctly handle string/byte types Label: +ack ___ FreeIPA-devel mailing list -- freeipa-devel@lists.fedorahosted.org To unsubscribe send an email to

[Freeipa-devel] Re: add Dogtag 10.4 builds to FreeIPA COPRs

2017-06-08 Thread Martin Bašti via FreeIPA-devel
On 08.06.2017 09:08, Martin Bašti via FreeIPA-devel wrote: On 08.06.2017 02:43, Fraser Tweedale via FreeIPA-devel wrote: My PR https://github.com/freeipa/freeipa/pull/859 bumps the pki-core dependency to >= 10.4. This patch is intended for master and 4.5 branches. Could someone with the

[Freeipa-devel] [freeipa PR#857][closed] server.py: Removes dns-server configuration from ldap

2017-06-08 Thread MartinBasti via FreeIPA-devel
URL: https://github.com/freeipa/freeipa/pull/857 Author: Tiboris Title: #857: server.py: Removes dns-server configuration from ldap Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/857/head:pr857 git checkout

[Freeipa-devel] [freeipa PR#857][+pushed] server.py: Removes dns-server configuration from ldap

2017-06-08 Thread MartinBasti via FreeIPA-devel
URL: https://github.com/freeipa/freeipa/pull/857 Title: #857: server.py: Removes dns-server configuration from ldap Label: +pushed ___ FreeIPA-devel mailing list -- freeipa-devel@lists.fedorahosted.org To unsubscribe send an email to

[Freeipa-devel] [freeipa PR#857][comment] server.py: Removes dns-server configuration from ldap

2017-06-08 Thread MartinBasti via FreeIPA-devel
URL: https://github.com/freeipa/freeipa/pull/857 Title: #857: server.py: Removes dns-server configuration from ldap MartinBasti commented: """ master: * 063211d665d02fc343952f5b158fd8d89223fbc9 server.py: Removes dns-server configuration from ldap ipa-4-5: *

[Freeipa-devel] [freeipa PR#848][closed] sssd.py: Deprecating no-sssd option.

2017-06-08 Thread MartinBasti via FreeIPA-devel
URL: https://github.com/freeipa/freeipa/pull/848 Author: Tiboris Title: #848: sssd.py: Deprecating no-sssd option. Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/848/head:pr848 git checkout pr848

[Freeipa-devel] [freeipa PR#848][comment] sssd.py: Deprecating no-sssd option.

2017-06-08 Thread MartinBasti via FreeIPA-devel
URL: https://github.com/freeipa/freeipa/pull/848 Title: #848: sssd.py: Deprecating no-sssd option. MartinBasti commented: """ master: * dfc271fdf4514481c11c342fabda135feeb44de6 sssd.py: Deprecating no-sssd option. ipa-4-5: * f984cef6ed49e04a4e3754d2f3214d64715d26df sssd.py: Deprecating

[Freeipa-devel] [freeipa PR#829][+pushed] client.py: Replace hardcoded 'admin' with options.principal

2017-06-08 Thread MartinBasti via FreeIPA-devel
URL: https://github.com/freeipa/freeipa/pull/829 Title: #829: client.py: Replace hardcoded 'admin' with options.principal Label: +pushed ___ FreeIPA-devel mailing list -- freeipa-devel@lists.fedorahosted.org To unsubscribe send an email to

[Freeipa-devel] [freeipa PR#840][comment] Add Role 'Client Administrator'

2017-06-08 Thread abbra via FreeIPA-devel
URL: https://github.com/freeipa/freeipa/pull/840 Title: #840: Add Role 'Client Administrator' abbra commented: """ I think "Enrollment Administrator" would be fine. On the "IT Specialists" and "Host Enrollment" -- why not, it seems logical. """ See the full comment at

[Freeipa-devel] [freeipa PR#840][comment] Add Role 'Client Administrator'

2017-06-08 Thread MartinBasti via FreeIPA-devel
URL: https://github.com/freeipa/freeipa/pull/840 Title: #840: Add Role 'Client Administrator' MartinBasti commented: """ Off Topic: Shouldn't have "IT specialists" also 'Host Enrollment' privilege, because they have 'Host administrators' already and this should close the circle. "Client

[Freeipa-devel] [freeipa PR#829][comment] client.py: Replace hardcoded 'admin' with options.principal

2017-06-08 Thread Tiboris via FreeIPA-devel
URL: https://github.com/freeipa/freeipa/pull/829 Title: #829: client.py: Replace hardcoded 'admin' with options.principal Tiboris commented: """ Hi @flo-renaud Thanks too. """ See the full comment at https://github.com/freeipa/freeipa/pull/829#issuecomment-307088893

[Freeipa-devel] [freeipa PR#829][comment] client.py: Replace hardcoded 'admin' with options.principal

2017-06-08 Thread flo-renaud via FreeIPA-devel
URL: https://github.com/freeipa/freeipa/pull/829 Title: #829: client.py: Replace hardcoded 'admin' with options.principal flo-renaud commented: """ Hi @Tiboris thank you, works as expected. """ See the full comment at https://github.com/freeipa/freeipa/pull/829#issuecomment-307085504

[Freeipa-devel] [freeipa PR#829][+ack] client.py: Replace hardcoded 'admin' with options.principal

2017-06-08 Thread flo-renaud via FreeIPA-devel
URL: https://github.com/freeipa/freeipa/pull/829 Title: #829: client.py: Replace hardcoded 'admin' with options.principal Label: +ack ___ FreeIPA-devel mailing list -- freeipa-devel@lists.fedorahosted.org To unsubscribe send an email to

[Freeipa-devel] [freeipa PR#857][edited] server.py: Removes dns-server configuration from ldap

2017-06-08 Thread Tiboris via FreeIPA-devel
URL: https://github.com/freeipa/freeipa/pull/857 Author: Tiboris Title: #857: server.py: Removes dns-server configuration from ldap Action: edited Changed field: title Original value: """ baseldap.py: Removes dns-server configuration from ldap """

[Freeipa-devel] [freeipa PR#857][synchronized] baseldap.py: Removes dns-server configuration from ldap

2017-06-08 Thread Tiboris via FreeIPA-devel
URL: https://github.com/freeipa/freeipa/pull/857 Author: Tiboris Title: #857: baseldap.py: Removes dns-server configuration from ldap Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/857/head:pr857 git

[Freeipa-devel] [freeipa PR#857][comment] baseldap.py: Removes dns-server configuration from ldap

2017-06-08 Thread MartinBasti via FreeIPA-devel
URL: https://github.com/freeipa/freeipa/pull/857 Title: #857: baseldap.py: Removes dns-server configuration from ldap MartinBasti commented: """ Actually NACK until you fix commit message, this is not related to baseldap.py """ See the full comment at