[PATCH] Add methods for activating and deactivating services to SSSDConfig

[Jan Cholasta]

Hi,

I have noticed that there are no methods for manipulating the services 
option of [sssd] in SSSDConfig. I have created a patch that adds these 
methods, see attachment.

Honza

[Jakub Hrozek]

On Thu, Feb 09, 2012 at 12:03:18PM +0100, Jan Cholasta wrote:
> Hi,
>
> I have noticed that there are no methods for manipulating the
> services option of [sssd] in SSSDConfig. I have created a patch that
> adds these methods, see attachment.
>
> Honza

I think it would also be nice to add a list_active_services() method. At
least for the sake of tests, currently they don't really test anything.

[Jan Cholasta]

On 9.2.2012 14:29, Jakub Hrozek wrote:
> On Thu, Feb 09, 2012 at 12:03:18PM +0100, Jan Cholasta wrote:
> > Hi,
> >
> > I have noticed that there are no methods for manipulating the
> > services option of [sssd] in SSSDConfig. I have created a patch that
> > adds these methods, see attachment.
> >
> > Honza
>
> I think it would also be nice to add a list_active_services() method. At
> least for the sake of tests, currently they don't really test anything.

Added.

[Jakub Hrozek]

On Thu, Feb 09, 2012 at 03:27:15PM +0100, Jan Cholasta wrote:
> On 9.2.2012 14:29, Jakub Hrozek wrote:
> > On Thu, Feb 09, 2012 at 12:03:18PM +0100, Jan Cholasta wrote:
> > > Hi,
> > >
> > > I have noticed that there are no methods for manipulating the
> > > services option of [sssd] in SSSDConfig. I have created a patch that
> > > adds these methods, see attachment.
> > >
> > > Honza
> >
> > I think it would also be nice to add a list_active_services() method. At
> > least for the sake of tests, currently they don't really test anything.
>
> Added.

Ack

[Stephen Gallagher]

On Thu, 2012-02-09 at 17:07 +0100, Jakub Hrozek wrote:
> On Thu, Feb 09, 2012 at 03:27:15PM +0100, Jan Cholasta wrote:
> > On 9.2.2012 14:29, Jakub Hrozek wrote:
> > > On Thu, Feb 09, 2012 at 12:03:18PM +0100, Jan Cholasta wrote:
> > > > Hi,
> > > >
> > > > I have noticed that there are no methods for manipulating the
> > > > services option of [sssd] in SSSDConfig. I have created a patch that
> > > > adds these methods, see attachment.
> > > >
> > > > Honza
> > >
> > > I think it would also be nice to add a list_active_services() method. At
> > > least for the sake of tests, currently they don't really test anything.
> >
> > Added.
>
> Ack

Pushed to master and sssd-1-8