[Freeipa-devel] [freeipa PR#650][comment] CA-less installation fix

2017-04-03 Thread HonzaCholasta
URL: https://github.com/freeipa/freeipa/pull/650 Title: #650: CA-less installation fix HonzaCholasta commented: """ master: * 8c87014e199b3dbe885c69d40a01d2723f813c3e Get correct CA cert nickname in CA-less * aae9a918b68dc4f9a7b4fb9abf1bb4d26673109d Remove publish_ca_cert() method from NSSDat

[Freeipa-devel] [freeipa PR#650][comment] CA-less installation fix

2017-04-03 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/650 Title: #650: CA-less installation fix stlaz commented: """ Done in https://github.com/freeipa/freeipa/pull/685 """ See the full comment at https://github.com/freeipa/freeipa/pull/650#issuecomment-291131467 -- Manage your subscription for the Fr

[Freeipa-devel] [freeipa PR#650][comment] CA-less installation fix

2017-04-03 Thread HonzaCholasta
URL: https://github.com/freeipa/freeipa/pull/650 Title: #650: CA-less installation fix HonzaCholasta commented: """ @stlaz, please also provide a version of this PR rebased on ipa-4-5. """ See the full comment at https://github.com/freeipa/freeipa/pull/650#issuecomment-291122587 -- Manage you

[Freeipa-devel] [freeipa PR#650][comment] CA-less installation fix

2017-04-03 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/650 Title: #650: CA-less installation fix stlaz commented: """ Sorry, must have screwed up the rebase. """ See the full comment at https://github.com/freeipa/freeipa/pull/650#issuecomment-291109172 -- Manage your subscription for the Freeipa-devel

[Freeipa-devel] [freeipa PR#650][comment] CA-less installation fix

2017-04-03 Thread HonzaCholasta
URL: https://github.com/freeipa/freeipa/pull/650 Title: #650: CA-less installation fix HonzaCholasta commented: """ Actually, there is a pylint failure introduced by this PR: ``` * Module ipapython.certdb ipapython/certdb.py:579: [C0305(trailing-newlines), ] Trailing newlines) ``` ""

[Freeipa-devel] [freeipa PR#650][comment] CA-less installation fix

2017-04-03 Thread HonzaCholasta
URL: https://github.com/freeipa/freeipa/pull/650 Title: #650: CA-less installation fix HonzaCholasta commented: """ I found additional bugs in CA-less (replica) install, but with this PR, `publish_ca_cert` does not fail anymore. """ See the full comment at https://github.com/freeipa/freeipa/p

[Freeipa-devel] [freeipa PR#650][comment] CA-less installation fix

2017-04-03 Thread HonzaCholasta
URL: https://github.com/freeipa/freeipa/pull/650 Title: #650: CA-less installation fix HonzaCholasta commented: """ @stlaz, please rebase. """ See the full comment at https://github.com/freeipa/freeipa/pull/650#issuecomment-291080182 -- Manage your subscription for the Freeipa-devel mailing l

[Freeipa-devel] [freeipa PR#650][comment] CA-less installation fix

2017-03-27 Thread HonzaCholasta
URL: https://github.com/freeipa/freeipa/pull/650 Title: #650: CA-less installation fix HonzaCholasta commented: """ @stlaz, `NSSDatabase.publish_ca_cert()` and `CertDB.publish_ca_cert()` become unused after your changes, could we remove them? """ See the full comment at https://github.com/fre

[Freeipa-devel] [freeipa PR#650][comment] CA-less installation fix

2017-03-24 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/650 Title: #650: CA-less installation fix stlaz commented: """ Fixed according to the comments, thanks. """ See the full comment at https://github.com/freeipa/freeipa/pull/650#issuecomment-289014989 -- Manage your subscription for the Freeipa-devel