[Freeipa-devel] [PATCH] - Add DRM to IPA

Martin Kosek mkosek at redhat.com
Thu Aug 21 19:52:54 UTC 2014


On 08/21/2014 05:27 PM, Petr Viktorin wrote:
> On 08/21/2014 03:48 PM, Ade Lee wrote:
>> As agreed on #irc, disabling uninstallation for now.
>> Please apply this new patch on top of the big one.
>
>
> I'm fine with pushing a patch with incomplete functionality, after all I did
> this all the time with permissions.
>
> The incomplete parts (apart from the plugin which is entirely out of scope) are:
> - The agent PEM issue (will be sorted out as the plugin is implemented)
> - Missing man page (will be written before the plugin is implemented)
> - Uninstall (will be fixed on Dogtag side, re-tested and enabled)
>
> I'll open tickets for these before pushing.
>
> ACK from me if Rob agrees. On IRC, Rob said he'd rather delay pushing until the
> man page is written, but delegated the decision to Martin.
> So, Martin, can we push now and trust Ade's promise that he'll write the docs?

Yes, if you open ticket(s) for all missing parts and put them in the same 
milestone. I would rather have the patches in than waiting for man page and 
then have a conflict and postpone the patch set.

I trust Ade to provide the man page later, I am sure he does not want to meet 
with my whip otherwise :-)

Martin




More information about the Freeipa-devel mailing list