[Freeipa-devel] [freeipa PR#404][comment] tests: Add LDAP URI to ldappasswd explicitly

2017-01-26 Thread tiran
  URL: https://github.com/freeipa/freeipa/pull/404
Title: #404: tests: Add LDAP URI to ldappasswd explicitly

tiran commented:
"""
I stand corrected! Thanks David.
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/404#issuecomment-275367590
-- 
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#404][comment] tests: Add LDAP URI to ldappasswd explicitly

2017-01-26 Thread dkupka
  URL: https://github.com/freeipa/freeipa/pull/404
Title: #404: tests: Add LDAP URI to ldappasswd explicitly

dkupka commented:
"""
@tiran After a closer look I believe that it's handled correctly in 
ipaserver/install/service.py
https://github.com/freeipa/freeipa/blob/master/ipaserver/install/service.py#L205:L208
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/404#issuecomment-275366693
-- 
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#404][comment] tests: Add LDAP URI to ldappasswd explicitly

2017-01-25 Thread dkupka
  URL: https://github.com/freeipa/freeipa/pull/404
Title: #404: tests: Add LDAP URI to ldappasswd explicitly

dkupka commented:
"""
@tiran Nice catch, I've added it to ipaserver/install/service.py, too. but I 
would rather not extend this outside of tests.
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/404#issuecomment-275153569
-- 
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#404][comment] tests: Add LDAP URI to ldappasswd explicitly

2017-01-25 Thread dkupka
  URL: https://github.com/freeipa/freeipa/pull/404
Title: #404: tests: Add LDAP URI to ldappasswd explicitly

dkupka commented:
"""
@tiran Nice catch, I've added it to ipaserver/install/service.py, too. but I 
would rather not extend this outside of tests.
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/404#issuecomment-275153569
-- 
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