[Freeipa-devel] [freeipa PR#872][comment] Add IPA-specific bind unit file

2017-06-19 Thread MartinBasti via FreeIPA-devel
  URL: https://github.com/freeipa/freeipa/pull/872
Title: #872: Add IPA-specific bind unit file

MartinBasti commented:
"""
I checked BZ, this may not be worth fixing as those fails are just during 
upgrade but at the end named is working. For sure this huge change cannot go to 
ipa-4-4 or ipa-4-5
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/872#issuecomment-309485173
___
FreeIPA-devel mailing list -- freeipa-devel@lists.fedorahosted.org
To unsubscribe send an email to freeipa-devel-le...@lists.fedorahosted.org


[Freeipa-devel] [freeipa PR#872][comment] Add IPA-specific bind unit file

2017-06-15 Thread stlaz via FreeIPA-devel
  URL: https://github.com/freeipa/freeipa/pull/872
Title: #872: Add IPA-specific bind unit file

stlaz commented:
"""
The last update should sort the masking of the named-pkcs11 service along the 
previously handled named. I will update the release notes once we settle on the 
form of the patch.
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/872#issuecomment-308668223
___
FreeIPA-devel mailing list -- freeipa-devel@lists.fedorahosted.org
To unsubscribe send an email to freeipa-devel-le...@lists.fedorahosted.org


[Freeipa-devel] [freeipa PR#872][comment] Add IPA-specific bind unit file

2017-06-14 Thread MartinBasti via FreeIPA-devel
  URL: https://github.com/freeipa/freeipa/pull/872
Title: #872: Add IPA-specific bind unit file

MartinBasti commented:
"""
I have a few comments:

* named-pkcs11 should be masked in installer and upgrader to avoid issues when 
users start incorrect named service manually
* please update release notes, this is a quite big change that we changed name 
of service


"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/872#issuecomment-308362410
___
FreeIPA-devel mailing list -- freeipa-devel@lists.fedorahosted.org
To unsubscribe send an email to freeipa-devel-le...@lists.fedorahosted.org