Re: [SSSD] [PATCHES] AD: support for AD site override

2015-01-26 Thread Jakub Hrozek
On Fri, Jan 23, 2015 at 10:14:56PM +0100, Jakub Hrozek wrote: > On Fri, Jan 23, 2015 at 12:23:43PM +0100, Pavel Reichl wrote: > > > > On 01/22/2015 05:58 PM, Stephen Gallagher wrote: > > > > > > > > >On Mon, 2015-01-19 at 09:58 +0100, Jakub Hrozek wrote: > > >>On Mon, Jan 19, 2015 at 09:39:41AM +0

Re: [SSSD] [PATCHES] AD: support for AD site override

2015-01-23 Thread Jakub Hrozek
On Fri, Jan 23, 2015 at 12:23:43PM +0100, Pavel Reichl wrote: > > On 01/22/2015 05:58 PM, Stephen Gallagher wrote: > > > > > >On Mon, 2015-01-19 at 09:58 +0100, Jakub Hrozek wrote: > >>On Mon, Jan 19, 2015 at 09:39:41AM +0100, Pavel Reichl wrote: > >man page: > >Specify AD site client shou

Re: [SSSD] [PATCHES] AD: support for AD site override

2015-01-23 Thread Pavel Reichl
On 01/22/2015 05:58 PM, Stephen Gallagher wrote: On Mon, 2015-01-19 at 09:58 +0100, Jakub Hrozek wrote: On Mon, Jan 19, 2015 at 09:39:41AM +0100, Pavel Reichl wrote: man page: Specify AD site client should try to connect to. Specify AD site to which client should try to connect. Which one s

Re: [SSSD] [PATCHES] AD: support for AD site override

2015-01-23 Thread Dan Lavu
27;ll review the man text. Dan Lavu Red Hat, Inc. Senior Quality Engineer 571.449.RMRF (7673) mobile d...@redhat.com - Original Message - From: "Pavel Reichl" To: sssd-devel@lists.fedorahosted.org, d...@redhat.com Sent: Monday, January 19, 2015 4:07:01 AM Subject: Re: [SSSD

Re: [SSSD] [PATCHES] AD: support for AD site override

2015-01-22 Thread Stephen Gallagher
On Mon, 2015-01-19 at 09:58 +0100, Jakub Hrozek wrote: > On Mon, Jan 19, 2015 at 09:39:41AM +0100, Pavel Reichl wrote: > > >>man page: > > >>Specify AD site client should try to connect to. > > >>Specify AD site to which client should try to connect. > > >> > > >>Which one sounds better? > > >I

Re: [SSSD] [PATCHES] AD: support for AD site override

2015-01-19 Thread Dan Lavu
To: sssd-devel@lists.fedorahosted.org, d...@redhat.com Sent: Monday, January 19, 2015 4:07:01 AM Subject: Re: [SSSD] [PATCHES] AD: support for AD site override On 01/19/2015 09:58 AM, Jakub Hrozek wrote: > On Mon, Jan 19, 2015 at 09:39:41AM +0100, Pavel Reichl wrote: >>>> man page: >>>

Re: [SSSD] [PATCHES] AD: support for AD site override

2015-01-19 Thread Pavel Reichl
On 01/19/2015 09:58 AM, Jakub Hrozek wrote: On Mon, Jan 19, 2015 at 09:39:41AM +0100, Pavel Reichl wrote: man page: Specify AD site client should try to connect to. Specify AD site to which client should try to connect. Which one sounds better? I don't care much, maybe the second one. Should

Re: [SSSD] [PATCHES] AD: support for AD site override

2015-01-19 Thread Jakub Hrozek
On Mon, Jan 19, 2015 at 09:39:41AM +0100, Pavel Reichl wrote: > >>man page: > >>Specify AD site client should try to connect to. > >>Specify AD site to which client should try to connect. > >> > >>Which one sounds better? > >I don't care much, maybe the second one. Should I prepare a new patch or >

Re: [SSSD] [PATCHES] AD: support for AD site override

2015-01-19 Thread Pavel Reichl
On 01/15/2015 03:57 PM, Pavel Reichl wrote: On 01/15/2015 11:21 AM, Pavel Březina wrote: On 01/09/2015 04:56 PM, Pavel Reichl wrote: On 01/08/2015 08:36 PM, Pavel Březina wrote: On 01/07/2015 03:28 PM, Pavel Reichl wrote: Please see attached patches. Thanks. Hi, I know there is a desig

Re: [SSSD] [PATCHES] AD: support for AD site override

2015-01-15 Thread Pavel Reichl
On 01/15/2015 11:21 AM, Pavel Březina wrote: On 01/09/2015 04:56 PM, Pavel Reichl wrote: On 01/08/2015 08:36 PM, Pavel Březina wrote: On 01/07/2015 03:28 PM, Pavel Reichl wrote: Please see attached patches. Thanks. Hi, I know there is a design page I did not comment on but seeing the pat

Re: [SSSD] [PATCHES] AD: support for AD site override

2015-01-15 Thread Pavel Březina
On 01/09/2015 04:56 PM, Pavel Reichl wrote: On 01/08/2015 08:36 PM, Pavel Březina wrote: On 01/07/2015 03:28 PM, Pavel Reichl wrote: Please see attached patches. Thanks. Hi, I know there is a design page I did not comment on but seeing the patch now, does the option really have to be names

Re: [SSSD] [PATCHES] AD: support for AD site override

2015-01-12 Thread Jakub Hrozek
On Fri, Jan 09, 2015 at 04:56:46PM +0100, Pavel Reichl wrote: > > On 01/08/2015 08:36 PM, Pavel Březina wrote: > >On 01/07/2015 03:28 PM, Pavel Reichl wrote: > >>Please see attached patches. > >> > >>Thanks. > > > >Hi, > > > >I know there is a design page I did not comment on but seeing the patch

Re: [SSSD] [PATCHES] AD: support for AD site override

2015-01-09 Thread Lukas Slebodnik
On (09/01/15 10:51), Jakub Hrozek wrote: >On Thu, Jan 08, 2015 at 08:36:03PM +0100, Pavel Březina wrote: >> On 01/07/2015 03:28 PM, Pavel Reichl wrote: >> >Please see attached patches. >> > >> >Thanks. >> >> Hi, >> >> I know there is a design page I did not comment on but seeing the patch now, >>

Re: [SSSD] [PATCHES] AD: support for AD site override

2015-01-09 Thread Pavel Reichl
On 01/08/2015 08:36 PM, Pavel Březina wrote: On 01/07/2015 03:28 PM, Pavel Reichl wrote: Please see attached patches. Thanks. Hi, I know there is a design page I did not comment on but seeing the patch now, does the option really have to be names ad_site_override? Wouldn't simple ad_site

Re: [SSSD] [PATCHES] AD: support for AD site override

2015-01-09 Thread Jakub Hrozek
On Thu, Jan 08, 2015 at 08:36:03PM +0100, Pavel Březina wrote: > On 01/07/2015 03:28 PM, Pavel Reichl wrote: > >Please see attached patches. > > > >Thanks. > > Hi, > > I know there is a design page I did not comment on but seeing the patch now, > does the option really have to be names ad_site_o

Re: [SSSD] [PATCHES] AD: support for AD site override

2015-01-08 Thread Pavel Březina
On 01/07/2015 03:28 PM, Pavel Reichl wrote: Please see attached patches. Thanks. Hi, I know there is a design page I did not comment on but seeing the patch now, does the option really have to be names ad_site_override? Wouldn't simple ad_site be a better name? I'd change the description

[SSSD] [PATCHES] AD: support for AD site override

2015-01-07 Thread Pavel Reichl
Please see attached patches. Thanks. >From 6978e8a2f98ddaca310cc421942f77a88deaec37 Mon Sep 17 00:00:00 2001 From: Pavel Reichl Date: Wed, 7 Jan 2015 09:40:45 + Subject: [PATCH 1/2] AD: add new option ad_site_override This option overrides a result of the automatic site discovery. Resolves