Re: [Freeipa-devel] [PATCH] Make enabling the autofs service more robust

2012-11-14 Thread Martin Kosek
On 11/13/2012 11:33 AM, Jakub Hrozek wrote: On Mon, Nov 12, 2012 at 05:54:22PM +0100, Jan Cholasta wrote: On 9.11.2012 16:06, Jakub Hrozek wrote: On Tue, Nov 06, 2012 at 08:57:06AM +0100, Jan Cholasta wrote: On 29.10.2012 17:21, Jakub Hrozek wrote: An improvement based on Honza's suggestion.

Re: [Freeipa-devel] [PATCH] Make enabling the autofs service more robust

2012-11-13 Thread Jakub Hrozek
On Mon, Nov 12, 2012 at 05:54:22PM +0100, Jan Cholasta wrote: On 9.11.2012 16:06, Jakub Hrozek wrote: On Tue, Nov 06, 2012 at 08:57:06AM +0100, Jan Cholasta wrote: On 29.10.2012 17:21, Jakub Hrozek wrote: An improvement based on Honza's suggestion. You might want to remove the try block

Re: [Freeipa-devel] [PATCH] Make enabling the autofs service more robust

2012-11-12 Thread Jan Cholasta
On 9.11.2012 16:06, Jakub Hrozek wrote: On Tue, Nov 06, 2012 at 08:57:06AM +0100, Jan Cholasta wrote: On 29.10.2012 17:21, Jakub Hrozek wrote: An improvement based on Honza's suggestion. You might want to remove the try block around sssdconfig.activate_service('autofs'). If it throws

Re: [Freeipa-devel] [PATCH] Make enabling the autofs service more robust

2012-11-09 Thread Jakub Hrozek
On Tue, Nov 06, 2012 at 08:57:06AM +0100, Jan Cholasta wrote: On 29.10.2012 17:21, Jakub Hrozek wrote: An improvement based on Honza's suggestion. You might want to remove the try block around sssdconfig.activate_service('autofs'). If it throws NoServiceError at this point, there is

Re: [Freeipa-devel] [PATCH] Make enabling the autofs service more robust

2012-11-07 Thread Martin Kosek
On 11/07/2012 09:14 AM, Jakub Hrozek wrote: On Tue, Nov 06, 2012 at 08:57:06AM +0100, Jan Cholasta wrote: On 29.10.2012 17:21, Jakub Hrozek wrote: An improvement based on Honza's suggestion. You might want to remove the try block around sssdconfig.activate_service('autofs'). If it throws

Re: [Freeipa-devel] [PATCH] Make enabling the autofs service more robust

2012-11-05 Thread Jan Cholasta
On 29.10.2012 17:21, Jakub Hrozek wrote: An improvement based on Honza's suggestion. You might want to remove the try block around sssdconfig.activate_service('autofs'). If it throws NoServiceError at this point, there is something really broken in SSSDConfig, in which case it is better to