Re: [Freeipa-devel] [PATCHES] 0578-0579 Convert Host default permissions to managed

2014-06-23 Thread Petr Viktorin
On 06/20/2014 10:35 PM, Martin Kosek wrote: [...] Everything worked as expected, I tested both enrollments with privileged user and setting the OTP/class. I have just one request (you will not like this) - before pushing please also fix casing for the new host permissions to match others: +

Re: [Freeipa-devel] [PATCHES] 0578-0579 Convert Host default permissions to managed

2014-06-20 Thread Martin Kosek
On 06/19/2014 01:41 PM, Petr Viktorin wrote: On 06/18/2014 05:46 PM, Martin Kosek wrote: On 06/11/2014 06:39 PM, Petr Viktorin wrote: Patch 0578 does the conversion Patch 0579 fixes https://fedorahosted.org/freeipa/ticket/4252 and provides permissions needed for automatic enrollment (from http

Re: [Freeipa-devel] [PATCHES] 0578-0579 Convert Host default permissions to managed

2014-06-19 Thread Petr Viktorin
On 06/18/2014 05:46 PM, Martin Kosek wrote: On 06/11/2014 06:39 PM, Petr Viktorin wrote: Patch 0578 does the conversion Patch 0579 fixes https://fedorahosted.org/freeipa/ticket/4252 and provides permissions needed for automatic enrollment (from http://projects.theforeman.org/projects/foreman/wi

Re: [Freeipa-devel] [PATCHES] 0578-0579 Convert Host default permissions to managed

2014-06-18 Thread Martin Kosek
On 06/11/2014 06:39 PM, Petr Viktorin wrote: > Patch 0578 does the conversion > > Patch 0579 fixes https://fedorahosted.org/freeipa/ticket/4252 and provides > permissions needed for automatic enrollment (from > http://projects.theforeman.org/projects/foreman/wiki/IPASmartProxyUser) 1) Inconsisten

Re: [Freeipa-devel] [PATCHES] 0578-0579 Convert Host default permissions to managed

2014-06-18 Thread Petr Viktorin
On 06/11/2014 06:39 PM, Petr Viktorin wrote: Patch 0578 does the conversion Patch 0579 fixes https://fedorahosted.org/freeipa/ticket/4252 and provides permissions needed for automatic enrollment (from http://projects.theforeman.org/projects/foreman/wiki/IPASmartProxyUser) Rebasing to current m