MONITOR: Add disable_netlink sssd.conf option

[Justin Stephenson]

Hello,

The attached patches resolve https://fedorahosted.org/sssd/ticket/3142

However, I am having difficult with the man page addition to 
'src/man/sssd.conf.5.xml' for this new option. I have stared at the open 
and close xml tags(for far too long) and it looks correct but when I 
build sssd I never see the sssd.conf man page inclusion. Could anyone 
tell me what I am missing here?

If you feel there is better wording for the description please let me know.

Kind regards,
Justin Stephenson

[Petr Cech]

On 08/27/2016 06:54 PM, Justin Stephenson wrote:
> Hello,
>
> The attached patches resolve https://fedorahosted.org/sssd/ticket/3142
>
> However, I am having difficult with the man page addition to
> 'src/man/sssd.conf.5.xml' for this new option. I have stared at the open
> and close xml tags(for far too long) and it looks correct but when I
> build sssd I never see the sssd.conf man page inclusion. Could anyone
> tell me what I am missing here?
>
> If you feel there is better wording for the description please let me know.
>
> Kind regards,
> Justin Stephenson

Hello Justin,

CI passed:
http://sssd-ci.duckdns.org/logs/job/52/72/summary.html

I have one little comment about coding style. See below.
> 0001-MONITOR-Remove-disable-netlink-command-line-option.patch
>
> From 0552c199dd37c7e280304b9bc92ff44a8a1a6d57 Mon Sep 17 00:00:00 2001
> From: Justin Stephenson 
> Date: Fri, 26 Aug 2016 15:15:32 -0400
> Subject: [PATCH 1/2]     MONITOR: Remove --disable-netlink command-line option
> Removing monitor command-line option, to be superceded by
> sssd.conf option

ACK
> 0002-MONITOR-Add-disable_netlink-option.patch
>
> From c52c0c1a520cdf8509baaaaac00fa3c7bec0dd73 Mon Sep 17 00:00:00 2001
> From: Justin Stephenson 
> Date: Fri, 26 Aug 2016 17:43:25 -0400
> Subject: [PATCH 2/2]     MONITOR: Add disable_netlink option
> Adding a new monitor boolean option to disable netlink support.
> This will give users more control over sssd state changes without
> having to modify systemd unit files.
>
> Resolves:
> https://fedorahosted.org/sssd/ticket/3142

[...]
>  /* Set up the environment variable for the Kerberos Replay Cache */
>
> @@ -2471,14 +2472,28 @@ static int monitor_process_init(struct mt_ctx *ctx,
>          return ret;
>      }
> ret = setup_netlink(ctx, ctx->ev, network_status_change_cb,
>                    ctx, &ctx->nlctx);
> ret = confdb_get_bool(ctx->cdb,
>                      CONFDB_MONITOR_CONF_ENTRY,
>                      CONFDB_MONITOR_DISABLE_NETLINK,
>                      false, &disable_netlink);
> if (ret != EOK) {
>        DEBUG(SSSDBG_OP_FAILURE,
>          "Cannot set up listening for network notifications\n");
>        "Failed to read disable_netlink from confdb: [%d] %s\n",

^ --- this is right indentation
>        ret, sss_strerror(ret));

^ --- this is right indentation

Please, fix this little nitpicking.

I am not native speaker, I am not able check
text in man page. (I guess you are.)

The first patch ACKed, the second needs
little work.

Regards

[Jakub Hrozek]

On Sat, Aug 27, 2016 at 12:54:53PM -0400, Justin Stephenson wrote:
> Hello,
>
> The attached patches resolve https://fedorahosted.org/sssd/ticket/3142
>
> However, I am having difficult with the man page addition to
> 'src/man/sssd.conf.5.xml' for this new option. I have stared at the open and
> close xml tags(for far too long) and it looks correct but when I build sssd
> I never see the sssd.conf man page inclusion. Could anyone tell me what I am
> missing here?
>
> If you feel there is better wording for the description please let me know.
>
> Kind regards,
> Justin Stephenson
> From 0552c199dd37c7e280304b9bc92ff44a8a1a6d57 Mon Sep 17 00:00:00 2001
> From: Justin Stephenson 
> Date: Fri, 26 Aug 2016 15:15:32 -0400
> Subject: [PATCH 1/2]     MONITOR: Remove --disable-netlink command-line option

I'm not sure I like removing the netlink option w/o letting admins who
use it at least know what happened. Could we keep the option in the popt
option list, but use the HIDDEN argument so that it doesn't show up in
--help output and print a loud warning that the option was removed in
favor of a sssd.conf option?

I already know of two people from sssd-users list who might be using
this feature. On the other hand, it was just introduced in the last
version and not in any enterprise distro, so just printing a warning and
removing even that warning in the next version would be fine for me..
> From c52c0c1a520cdf8509baaaaac00fa3c7bec0dd73 Mon Sep 17 00:00:00 2001
> From: Justin Stephenson 
> Date: Fri, 26 Aug 2016 17:43:25 -0400
> Subject: [PATCH 2/2]     MONITOR: Add disable_netlink option

LGTM, untested, though.

[Justin Stephenson]

On 08/30/2016 03:54 AM, Jakub Hrozek wrote:
> On Sat, Aug 27, 2016 at 12:54:53PM -0400, Justin Stephenson wrote:
> > Hello,
> >
> > The attached patches resolve https://fedorahosted.org/sssd/ticket/3142
> >
> > However, I am having difficult with the man page addition to
> > 'src/man/sssd.conf.5.xml' for this new option. I have stared at the open and
> > close xml tags(for far too long) and it looks correct but when I build sssd
> > I never see the sssd.conf man page inclusion. Could anyone tell me what I am
> > missing here?
> >
> > If you feel there is better wording for the description please let me know.
> >
> > Kind regards,
> > Justin Stephenson
> > From 0552c199dd37c7e280304b9bc92ff44a8a1a6d57 Mon Sep 17 00:00:00 2001
> > From: Justin Stephenson 
> > Date: Fri, 26 Aug 2016 15:15:32 -0400
> > Subject: [PATCH 1/2]     MONITOR: Remove --disable-netlink command-line option
>
> I'm not sure I like removing the netlink option w/o letting admins who
> use it at least know what happened. Could we keep the option in the popt
> option list, but use the HIDDEN argument so that it doesn't show up in
> --help output and print a loud warning that the option was removed in
> favor of a sssd.conf option?
>
> I already know of two people from sssd-users list who might be using
> this feature. On the other hand, it was just introduced in the last
> version and not in any enterprise distro, so just printing a warning and
> removing even that warning in the next version would be fine for me..

Agreed, please see updated patches also with Petr's corrections. Once 
this fix is pushed I can respond to the email and at least let these 
users know.

I am still having trouble with the man page addition to sssd.conf not 
showing, any ideas why?

diff --git a/src/man/sssd.conf.5.xml b/src/man/sssd.conf.5.xml
index 
ae291e0fc8f2f9afabcdf32f18a5ec12252bbbbf..6f231b8ab8fc078d83331bb7ef5b980528a30bd6 
100644
--- a/src/man/sssd.conf.5.xml
+++ b/src/man/sssd.conf.5.xml
@@ -482,6 +482,24 @@
                              </para>
                          </listitem>
                      </varlistentry>
+                    <varlistentry>
+                        <term>disable_netlink (boolean)</term>
+                        <listitem>
+                            <para>
+                                SSSD hooks into the netlink interface to
+                                monitor changes to routes, addresses, links
+                                and trigger certain actions.
+                            </para>
+                            <para>
+                                The SSSD state changes caused by netlink
+                                events may be undesirable and can be 
disabled
+                                by setting this option to 'true'
+                            </para>
+                            <para>
+                                Default: false (netlink changes are 
detected)
+                            </para>
+                        </listitem>
+                    </varlistentry>
                  </variablelist>
              </para>
          </refsect2>

Kind regards,
Justin Stephenson
> > From c52c0c1a520cdf8509baaaaac00fa3c7bec0dd73 Mon Sep 17 00:00:00 2001
> > From: Justin Stephenson 
> > Date: Fri, 26 Aug 2016 17:43:25 -0400
> > Subject: [PATCH 2/2]     MONITOR: Add disable_netlink option
>
> LGTM, untested, though.
> _______________________________________________
> sssd-devel mailing list
> [e-mail]
> https://lists.fedorahosted.org/admin/lists/[e-mail]

[Petr Cech]

On 09/01/2016 03:36 PM, Justin Stephenson wrote:
> On 08/30/2016 03:54 AM, Jakub Hrozek wrote:
> > On Sat, Aug 27, 2016 at 12:54:53PM -0400, Justin Stephenson wrote:
> > > Hello,
> > >
> > > The attached patches resolve https://fedorahosted.org/sssd/ticket/3142
> > >
> > > However, I am having difficult with the man page addition to
> > > 'src/man/sssd.conf.5.xml' for this new option. I have stared at the
> > > open and
> > > close xml tags(for far too long) and it looks correct but when I
> > > build sssd
> > > I never see the sssd.conf man page inclusion. Could anyone tell me
> > > what I am
> > > missing here?
> > >
> > > If you feel there is better wording for the description please let me
> > > know.
> > >
> > > Kind regards,
> > > Justin Stephenson
> > > From 0552c199dd37c7e280304b9bc92ff44a8a1a6d57 Mon Sep 17 00:00:00 2001
> > > From: Justin Stephenson 
> > > Date: Fri, 26 Aug 2016 15:15:32 -0400
> > > Subject: [PATCH 1/2]     MONITOR: Remove --disable-netlink
> > > command-line option
> >
> > I'm not sure I like removing the netlink option w/o letting admins who
> > use it at least know what happened. Could we keep the option in the popt
> > option list, but use the HIDDEN argument so that it doesn't show up in
> > --help output and print a loud warning that the option was removed in
> > favor of a sssd.conf option?
> >
> > I already know of two people from sssd-users list who might be using
> > this feature. On the other hand, it was just introduced in the last
> > version and not in any enterprise distro, so just printing a warning and
> > removing even that warning in the next version would be fine for me..
>
> Agreed, please see updated patches also with Petr's corrections. Once
> this fix is pushed I can respond to the email and at least let these
> users know.
>
> I am still having trouble with the man page addition to sssd.conf not
> showing, any ideas why?
>
> diff --git a/src/man/sssd.conf.5.xml b/src/man/sssd.conf.5.xml
> index
> ae291e0fc8f2f9afabcdf32f18a5ec12252bbbbf..6f231b8ab8fc078d83331bb7ef5b980528a30bd6
> 100644
> --- a/src/man/sssd.conf.5.xml
> +++ b/src/man/sssd.conf.5.xml
> @@ -482,6 +482,24 @@
>                              </para>
>                          </listitem>
>                      </varlistentry>
>                <varlistentry>
>                    <term>disable_netlink (boolean)</term>
>                    <listitem>
>                        <para>
>                            SSSD hooks into the netlink interface to
>                            monitor changes to routes, addresses,
>
> links
>                            and trigger certain actions.
>                        </para>
>                        <para>
>                            The SSSD state changes caused by netlink
>                            events may be undesirable and can be
>
> disabled
>                            by setting this option to 'true'
>                        </para>
>                        <para>
>                            Default: false (netlink changes are
>
> detected)
>                        </para>
>                    </listitem>
>                </varlistentry>
>            </variablelist>
>        </para>
>    </refsect2>
>
> Kind regards,
> Justin Stephenson
> > > From c52c0c1a520cdf8509baaaaac00fa3c7bec0dd73 Mon Sep 17 00:00:00 2001
> > > From: Justin Stephenson 
> > > Date: Fri, 26 Aug 2016 17:43:25 -0400
> > > Subject: [PATCH 2/2]     MONITOR: Add disable_netlink option
> >
> > LGTM, untested, though.

Hello Justin and Jakub,

I tested it:
sssd --help ... option is gone

/sbin/sssd --disable-netlink
Option --disable-netlink has been removed and replaced as a Monitor 
option in sssd.conf

I see disable-netlink in man sssd.conf.
Justin, I run 'make rpms' and reinstall all,
so man pages were reinstall too.

LGTM

I just wait for CI :-)

Regards

[Petr Cech]

On 09/02/2016 11:23 AM, Petr Cech wrote:
> On 09/01/2016 03:36 PM, Justin Stephenson wrote:
> > On 08/30/2016 03:54 AM, Jakub Hrozek wrote:
> > > On Sat, Aug 27, 2016 at 12:54:53PM -0400, Justin Stephenson wrote:
> > > > Hello,
> > > >
> > > > The attached patches resolve https://fedorahosted.org/sssd/ticket/3142
> > > >
> > > > However, I am having difficult with the man page addition to
> > > > 'src/man/sssd.conf.5.xml' for this new option. I have stared at the
> > > > open and
> > > > close xml tags(for far too long) and it looks correct but when I
> > > > build sssd
> > > > I never see the sssd.conf man page inclusion. Could anyone tell me
> > > > what I am
> > > > missing here?
> > > >
> > > > If you feel there is better wording for the description please let me
> > > > know.
> > > >
> > > > Kind regards,
> > > > Justin Stephenson
> > > > From 0552c199dd37c7e280304b9bc92ff44a8a1a6d57 Mon Sep 17 00:00:00 2001
> > > > From: Justin Stephenson 
> > > > Date: Fri, 26 Aug 2016 15:15:32 -0400
> > > > Subject: [PATCH 1/2]     MONITOR: Remove --disable-netlink
> > > > command-line option
> > >
> > > I'm not sure I like removing the netlink option w/o letting admins who
> > > use it at least know what happened. Could we keep the option in the popt
> > > option list, but use the HIDDEN argument so that it doesn't show up in
> > > --help output and print a loud warning that the option was removed in
> > > favor of a sssd.conf option?
> > >
> > > I already know of two people from sssd-users list who might be using
> > > this feature. On the other hand, it was just introduced in the last
> > > version and not in any enterprise distro, so just printing a warning and
> > > removing even that warning in the next version would be fine for me..
> >
> > Agreed, please see updated patches also with Petr's corrections. Once
> > this fix is pushed I can respond to the email and at least let these
> > users know.
> >
> > I am still having trouble with the man page addition to sssd.conf not
> > showing, any ideas why?
> >
> > diff --git a/src/man/sssd.conf.5.xml b/src/man/sssd.conf.5.xml
> > index
> > ae291e0fc8f2f9afabcdf32f18a5ec12252bbbbf..6f231b8ab8fc078d83331bb7ef5b980528a30bd6
> >
> > 100644
> > --- a/src/man/sssd.conf.5.xml
> > +++ b/src/man/sssd.conf.5.xml
> > @@ -482,6 +482,24 @@
> >                              </para>
> >                          </listitem>
> >                      </varlistentry>
> >                <varlistentry>
> >                    <term>disable_netlink (boolean)</term>
> >                    <listitem>
> >                        <para>
> >                            SSSD hooks into the netlink interface to
> >                            monitor changes to routes, addresses,
> >
> > links
> >                            and trigger certain actions.
> >                        </para>
> >                        <para>
> >                            The SSSD state changes caused by netlink
> >                            events may be undesirable and can be
> >
> > disabled
> >                            by setting this option to 'true'
> >                        </para>
> >                        <para>
> >                            Default: false (netlink changes are
> >
> > detected)
> >                        </para>
> >                    </listitem>
> >                </varlistentry>
> >            </variablelist>
> >        </para>
> >    </refsect2>
> >
> > Kind regards,
> > Justin Stephenson
> > > > From c52c0c1a520cdf8509baaaaac00fa3c7bec0dd73 Mon Sep 17 00:00:00 2001
> > > > From: Justin Stephenson 
> > > > Date: Fri, 26 Aug 2016 17:43:25 -0400
> > > > Subject: [PATCH 2/2]     MONITOR: Add disable_netlink option
> > >
> > > LGTM, untested, though.
>
> Hello Justin and Jakub,
>
> I tested it:
> sssd --help ... option is gone
>
> /sbin/sssd --disable-netlink
> Option --disable-netlink has been removed and replaced as a Monitor
> option in sssd.conf
>
> I see disable-netlink in man sssd.conf.
> Justin, I run 'make rpms' and reinstall all,
> so man pages were reinstall too.
>
> LGTM
>
> I just wait for CI :-)

CI passed:
http://sssd-ci.duckdns.org/logs/job/52/96/summary.html

=> ACK

Regards

[Justin Stephenson]

On 09/02/2016 05:23 AM, Petr Cech wrote:
> On 09/01/2016 03:36 PM, Justin Stephenson wrote:
> > On 08/30/2016 03:54 AM, Jakub Hrozek wrote:
> > > On Sat, Aug 27, 2016 at 12:54:53PM -0400, Justin Stephenson wrote:
> > > > Hello,
> > > >
> > > > The attached patches resolve https://fedorahosted.org/sssd/ticket/3142
> > > >
> > > > However, I am having difficult with the man page addition to
> > > > 'src/man/sssd.conf.5.xml' for this new option. I have stared at the
> > > > open and
> > > > close xml tags(for far too long) and it looks correct but when I
> > > > build sssd
> > > > I never see the sssd.conf man page inclusion. Could anyone tell me
> > > > what I am
> > > > missing here?
> > > >
> > > > If you feel there is better wording for the description please let me
> > > > know.
> > > >
> > > > Kind regards,
> > > > Justin Stephenson
> > > > From 0552c199dd37c7e280304b9bc92ff44a8a1a6d57 Mon Sep 17 00:00:00 2001
> > > > From: Justin Stephenson 
> > > > Date: Fri, 26 Aug 2016 15:15:32 -0400
> > > > Subject: [PATCH 1/2]     MONITOR: Remove --disable-netlink
> > > > command-line option
> > >
> > > I'm not sure I like removing the netlink option w/o letting admins who
> > > use it at least know what happened. Could we keep the option in the popt
> > > option list, but use the HIDDEN argument so that it doesn't show up in
> > > --help output and print a loud warning that the option was removed in
> > > favor of a sssd.conf option?
> > >
> > > I already know of two people from sssd-users list who might be using
> > > this feature. On the other hand, it was just introduced in the last
> > > version and not in any enterprise distro, so just printing a warning and
> > > removing even that warning in the next version would be fine for me..
> >
> > Agreed, please see updated patches also with Petr's corrections. Once
> > this fix is pushed I can respond to the email and at least let these
> > users know.
> >
> > I am still having trouble with the man page addition to sssd.conf not
> > showing, any ideas why?
> >
> > diff --git a/src/man/sssd.conf.5.xml b/src/man/sssd.conf.5.xml
> > index
> > ae291e0fc8f2f9afabcdf32f18a5ec12252bbbbf..6f231b8ab8fc078d83331bb7ef5b980528a30bd6
> >
> > 100644
> > --- a/src/man/sssd.conf.5.xml
> > +++ b/src/man/sssd.conf.5.xml
> > @@ -482,6 +482,24 @@
> >                              </para>
> >                          </listitem>
> >                      </varlistentry>
> >                <varlistentry>
> >                    <term>disable_netlink (boolean)</term>
> >                    <listitem>
> >                        <para>
> >                            SSSD hooks into the netlink interface to
> >                            monitor changes to routes, addresses,
> >
> > links
> >                            and trigger certain actions.
> >                        </para>
> >                        <para>
> >                            The SSSD state changes caused by netlink
> >                            events may be undesirable and can be
> >
> > disabled
> >                            by setting this option to 'true'
> >                        </para>
> >                        <para>
> >                            Default: false (netlink changes are
> >
> > detected)
> >                        </para>
> >                    </listitem>
> >                </varlistentry>
> >            </variablelist>
> >        </para>
> >    </refsect2>
> >
> > Kind regards,
> > Justin Stephenson
> > > > From c52c0c1a520cdf8509baaaaac00fa3c7bec0dd73 Mon Sep 17 00:00:00 2001
> > > > From: Justin Stephenson 
> > > > Date: Fri, 26 Aug 2016 17:43:25 -0400
> > > > Subject: [PATCH 2/2]     MONITOR: Add disable_netlink option
> > >
> > > LGTM, untested, though.
>
> Hello Justin and Jakub,
>
> I tested it:
> sssd --help ... option is gone
>
> /sbin/sssd --disable-netlink
> Option --disable-netlink has been removed and replaced as a Monitor
> option in sssd.conf
>
> I see disable-netlink in man sssd.conf.
> Justin, I run 'make rpms' and reinstall all,
> so man pages were reinstall too.

Thanks Petr, I was using the steps in the Contribute wiki 'reconfig && 
chmake' then 'sssinstall' but I guess that did not update the man pages 
from my commit as expected.

-Justin
> LGTM
>
> I just wait for CI :-)
>
> Regards

[Petr Cech]

On 09/02/2016 03:31 PM, Justin Stephenson wrote:
> On 09/02/2016 05:23 AM, Petr Cech wrote:
> > On 09/01/2016 03:36 PM, Justin Stephenson wrote:
> > > On 08/30/2016 03:54 AM, Jakub Hrozek wrote:
> > > > On Sat, Aug 27, 2016 at 12:54:53PM -0400, Justin Stephenson wrote:
> > > > > Hello,
> > > > >
> > > > > The attached patches resolve https://fedorahosted.org/sssd/ticket/3142
> > > > >
> > > > > However, I am having difficult with the man page addition to
> > > > > 'src/man/sssd.conf.5.xml' for this new option. I have stared at the
> > > > > open and
> > > > > close xml tags(for far too long) and it looks correct but when I
> > > > > build sssd
> > > > > I never see the sssd.conf man page inclusion. Could anyone tell me
> > > > > what I am
> > > > > missing here?
> > > > >
> > > > > If you feel there is better wording for the description please let me
> > > > > know.
> > > > >
> > > > > Kind regards,
> > > > > Justin Stephenson
> > > > > From 0552c199dd37c7e280304b9bc92ff44a8a1a6d57 Mon Sep 17 00:00:00 2001
> > > > > From: Justin Stephenson 
> > > > > Date: Fri, 26 Aug 2016 15:15:32 -0400
> > > > > Subject: [PATCH 1/2]     MONITOR: Remove --disable-netlink
> > > > > command-line option
> > > >
> > > > I'm not sure I like removing the netlink option w/o letting admins who
> > > > use it at least know what happened. Could we keep the option in the
> > > > popt
> > > > option list, but use the HIDDEN argument so that it doesn't show up in
> > > > --help output and print a loud warning that the option was removed in
> > > > favor of a sssd.conf option?
> > > >
> > > > I already know of two people from sssd-users list who might be using
> > > > this feature. On the other hand, it was just introduced in the last
> > > > version and not in any enterprise distro, so just printing a warning
> > > > and
> > > > removing even that warning in the next version would be fine for me..
> > >
> > > Agreed, please see updated patches also with Petr's corrections. Once
> > > this fix is pushed I can respond to the email and at least let these
> > > users know.
> > >
> > > I am still having trouble with the man page addition to sssd.conf not
> > > showing, any ideas why?
> > >
> > > diff --git a/src/man/sssd.conf.5.xml b/src/man/sssd.conf.5.xml
> > > index
> > > ae291e0fc8f2f9afabcdf32f18a5ec12252bbbbf..6f231b8ab8fc078d83331bb7ef5b980528a30bd6
> > >
> > > 100644
> > > --- a/src/man/sssd.conf.5.xml
> > > +++ b/src/man/sssd.conf.5.xml
> > > @@ -482,6 +482,24 @@
> > >                              </para>
> > >                          </listitem>
> > >                      </varlistentry>
> > >                <varlistentry>
> > >                    <term>disable_netlink (boolean)</term>
> > >                    <listitem>
> > >                        <para>
> > >                            SSSD hooks into the netlink
> > >
> > > interface to
> > >                            monitor changes to routes, addresses,
> > >
> > > links
> > >                            and trigger certain actions.
> > >                        </para>
> > >                        <para>
> > >                            The SSSD state changes caused by
> > >
> > > netlink
> > >                            events may be undesirable and can be
> > >
> > > disabled
> > >                            by setting this option to 'true'
> > >                        </para>
> > >                        <para>
> > >                            Default: false (netlink changes are
> > >
> > > detected)
> > >                        </para>
> > >                    </listitem>
> > >                </varlistentry>
> > >            </variablelist>
> > >        </para>
> > >    </refsect2>
> > >
> > > Kind regards,
> > > Justin Stephenson
> > > > > From c52c0c1a520cdf8509baaaaac00fa3c7bec0dd73 Mon Sep 17 00:00:00 2001
> > > > > From: Justin Stephenson 
> > > > > Date: Fri, 26 Aug 2016 17:43:25 -0400
> > > > > Subject: [PATCH 2/2]     MONITOR: Add disable_netlink option
> > > >
> > > > LGTM, untested, though.
> >
> > Hello Justin and Jakub,
> >
> > I tested it:
> > sssd --help ... option is gone
> >
> > /sbin/sssd --disable-netlink
> > Option --disable-netlink has been removed and replaced as a Monitor
> > option in sssd.conf
> >
> > I see disable-netlink in man sssd.conf.
> > Justin, I run 'make rpms' and reinstall all,
> > so man pages were reinstall too.
>
> Thanks Petr, I was using the steps in the Contribute wiki 'reconfig &&
> chmake' then 'sssinstall' but I guess that did not update the man pages
> from my commit as expected.

So far as I know, 'sssinstall' isn't good for man pages. But 'make rpms' 
build whole all necessary things.

'sssinstall' is good for common developing process.
> -Justin
> > LGTM
> >
> > I just wait for CI :-)
> >
> > Regards
>
> sssd-devel mailing list
> [e-mail]
> https://lists.fedorahosted.org/admin/lists/[e-mail]

[Lukas Slebodnik]

On (02/09/16 15:34), Petr Cech wrote:
> On 09/02/2016 03:31 PM, Justin Stephenson wrote:
> > On 09/02/2016 05:23 AM, Petr Cech wrote:
> > > On 09/01/2016 03:36 PM, Justin Stephenson wrote:
> > > > On 08/30/2016 03:54 AM, Jakub Hrozek wrote:
> > > > > On Sat, Aug 27, 2016 at 12:54:53PM -0400, Justin Stephenson wrote:
> > > > > > Hello,
> > > > > >
> > > > > > The attached patches resolve https://fedorahosted.org/sssd/ticket/3142
> > > > > >
> > > > > > However, I am having difficult with the man page addition to
> > > > > > 'src/man/sssd.conf.5.xml' for this new option. I have stared at the
> > > > > > open and
> > > > > > close xml tags(for far too long) and it looks correct but when I
> > > > > > build sssd
> > > > > > I never see the sssd.conf man page inclusion. Could anyone tell me
> > > > > > what I am
> > > > > > missing here?
> > > > > >
> > > > > > If you feel there is better wording for the description please let me
> > > > > > know.
> > > > > >
> > > > > > Kind regards,
> > > > > > Justin Stephenson
> > > > > > From 0552c199dd37c7e280304b9bc92ff44a8a1a6d57 Mon Sep 17 00:00:00 2001
> > > > > > From: Justin Stephenson 
> > > > > > Date: Fri, 26 Aug 2016 15:15:32 -0400
> > > > > > Subject: [PATCH 1/2]     MONITOR: Remove --disable-netlink
> > > > > > command-line option
> > > > >
> > > > > I'm not sure I like removing the netlink option w/o letting admins who
> > > > > use it at least know what happened. Could we keep the option in the
> > > > > popt
> > > > > option list, but use the HIDDEN argument so that it doesn't show up in
> > > > > --help output and print a loud warning that the option was removed in
> > > > > favor of a sssd.conf option?
> > > > >
> > > > > I already know of two people from sssd-users list who might be using
> > > > > this feature. On the other hand, it was just introduced in the last
> > > > > version and not in any enterprise distro, so just printing a warning
> > > > > and
> > > > > removing even that warning in the next version would be fine for me..
> > > >
> > > > Agreed, please see updated patches also with Petr's corrections. Once
> > > > this fix is pushed I can respond to the email and at least let these
> > > > users know.
> > > >
> > > > I am still having trouble with the man page addition to sssd.conf not
> > > > showing, any ideas why?
> > > >
> > > > diff --git a/src/man/sssd.conf.5.xml b/src/man/sssd.conf.5.xml
> > > > index
> > > > ae291e0fc8f2f9afabcdf32f18a5ec12252bbbbf..6f231b8ab8fc078d83331bb7ef5b980528a30bd6
> > > >
> > > > 100644
> > > > --- a/src/man/sssd.conf.5.xml
> > > > +++ b/src/man/sssd.conf.5.xml
> > > > @@ -482,6 +482,24 @@
> > > >                              </para>
> > > >                          </listitem>
> > > >                      </varlistentry>
> > > >                <varlistentry>
> > > >                    <term>disable_netlink (boolean)</term>
> > > >                    <listitem>
> > > >                        <para>
> > > >                            SSSD hooks into the netlink
> > > >
> > > > interface to
> > > >                            monitor changes to routes, addresses,
> > > >
> > > > links
> > > >                            and trigger certain actions.
> > > >                        </para>
> > > >                        <para>
> > > >                            The SSSD state changes caused by
> > > >
> > > > netlink
> > > >                            events may be undesirable and can be
> > > >
> > > > disabled
> > > >                            by setting this option to 'true'
> > > >                        </para>
> > > >                        <para>
> > > >                            Default: false (netlink changes are
> > > >
> > > > detected)
> > > >                        </para>
> > > >                    </listitem>
> > > >                </varlistentry>
> > > >            </variablelist>
> > > >        </para>
> > > >    </refsect2>
> > > >
> > > > Kind regards,
> > > > Justin Stephenson
> > > > > > From c52c0c1a520cdf8509baaaaac00fa3c7bec0dd73 Mon Sep 17 00:00:00 2001
> > > > > > From: Justin Stephenson 
> > > > > > Date: Fri, 26 Aug 2016 17:43:25 -0400
> > > > > > Subject: [PATCH 2/2]     MONITOR: Add disable_netlink option
> > > > >
> > > > > LGTM, untested, though.
> > >
> > > Hello Justin and Jakub,
> > >
> > > I tested it:
> > > sssd --help ... option is gone
> > >
> > > /sbin/sssd --disable-netlink
> > > Option --disable-netlink has been removed and replaced as a Monitor
> > > option in sssd.conf
> > >
> > > I see disable-netlink in man sssd.conf.
> > > Justin, I run 'make rpms' and reinstall all,
> > > so man pages were reinstall too.
> >
> > Thanks Petr, I was using the steps in the Contribute wiki 'reconfig &&
> > chmake' then 'sssinstall' but I guess that did not update the man pages
> > from my commit as expected.
>
> So far as I know, 'sssinstall' isn't good for man pages. But 'make rpms'
> build whole all necessary things.

Could you elaborate?
Why 'sssinstall is not good for man pages?

LS

[Petr Cech]

On 09/05/2016 09:45 AM, Lukas Slebodnik wrote:
> On (02/09/16 15:34), Petr Cech wrote:
> > On 09/02/2016 03:31 PM, Justin Stephenson wrote:
> > > On 09/02/2016 05:23 AM, Petr Cech wrote:
> > > > On 09/01/2016 03:36 PM, Justin Stephenson wrote:
> > > > > On 08/30/2016 03:54 AM, Jakub Hrozek wrote:
> > > > > > On Sat, Aug 27, 2016 at 12:54:53PM -0400, Justin Stephenson wrote:
> > > > > > > Hello,
> > > > > > >
> > > > > > > The attached patches resolve https://fedorahosted.org/sssd/ticket/3142
> > > > > > >
> > > > > > > However, I am having difficult with the man page addition to
> > > > > > > 'src/man/sssd.conf.5.xml' for this new option. I have stared at the
> > > > > > > open and
> > > > > > > close xml tags(for far too long) and it looks correct but when I
> > > > > > > build sssd
> > > > > > > I never see the sssd.conf man page inclusion. Could anyone tell me
> > > > > > > what I am
> > > > > > > missing here?
> > > > > > >
> > > > > > > If you feel there is better wording for the description please let me
> > > > > > > know.
> > > > > > >
> > > > > > > Kind regards,
> > > > > > > Justin Stephenson
> > > > > >
> > > > > > > From 0552c199dd37c7e280304b9bc92ff44a8a1a6d57 Mon Sep 17 00:00:00 2001
> > > > > > > From: Justin Stephenson 
> > > > > > > Date: Fri, 26 Aug 2016 15:15:32 -0400
> > > > > > > Subject: [PATCH 1/2]     MONITOR: Remove --disable-netlink
> > > > > > > command-line option
> > > > > >
> > > > > > I'm not sure I like removing the netlink option w/o letting admins who
> > > > > > use it at least know what happened. Could we keep the option in the
> > > > > > popt
> > > > > > option list, but use the HIDDEN argument so that it doesn't show up in
> > > > > > --help output and print a loud warning that the option was removed in
> > > > > > favor of a sssd.conf option?
> > > > > >
> > > > > > I already know of two people from sssd-users list who might be using
> > > > > > this feature. On the other hand, it was just introduced in the last
> > > > > > version and not in any enterprise distro, so just printing a warning
> > > > > > and
> > > > > > removing even that warning in the next version would be fine for me..
> > > > >
> > > > > Agreed, please see updated patches also with Petr's corrections. Once
> > > > > this fix is pushed I can respond to the email and at least let these
> > > > > users know.
> > > > >
> > > > > I am still having trouble with the man page addition to sssd.conf not
> > > > > showing, any ideas why?
> > > > >
> > > > > diff --git a/src/man/sssd.conf.5.xml b/src/man/sssd.conf.5.xml
> > > > > index
> > > > > ae291e0fc8f2f9afabcdf32f18a5ec12252bbbbf..6f231b8ab8fc078d83331bb7ef5b980528a30bd6
> > > > >
> > > > > 100644
> > > > > --- a/src/man/sssd.conf.5.xml
> > > > > +++ b/src/man/sssd.conf.5.xml
> > > > > @@ -482,6 +482,24 @@
> > > > >                              </para>
> > > > >                          </listitem>
> > > > >                      </varlistentry>
> > > > >                <varlistentry>
> > > > >                    <term>disable_netlink (boolean)</term>
> > > > >                    <listitem>
> > > > >                        <para>
> > > > >                            SSSD hooks into the netlink
> > > > >
> > > > > interface to
> > > > >                            monitor changes to routes, addresses,
> > > > >
> > > > > links
> > > > >                            and trigger certain actions.
> > > > >                        </para>
> > > > >                        <para>
> > > > >                            The SSSD state changes caused by
> > > > >
> > > > > netlink
> > > > >                            events may be undesirable and can be
> > > > >
> > > > > disabled
> > > > >                            by setting this option to 'true'
> > > > >                        </para>
> > > > >                        <para>
> > > > >                            Default: false (netlink changes are
> > > > >
> > > > > detected)
> > > > >                        </para>
> > > > >                    </listitem>
> > > > >                </varlistentry>
> > > > >            </variablelist>
> > > > >        </para>
> > > > >    </refsect2>
> > > > >
> > > > > Kind regards,
> > > > > Justin Stephenson
> > > > > > > From c52c0c1a520cdf8509baaaaac00fa3c7bec0dd73 Mon Sep 17 00:00:00 2001
> > > > > > > From: Justin Stephenson 
> > > > > > > Date: Fri, 26 Aug 2016 17:43:25 -0400
> > > > > > > Subject: [PATCH 2/2]     MONITOR: Add disable_netlink option
> > > > > >
> > > > > > LGTM, untested, though.
> > > >
> > > > Hello Justin and Jakub,
> > > >
> > > > I tested it:
> > > > sssd --help ... option is gone
> > > >
> > > > /sbin/sssd --disable-netlink
> > > > Option --disable-netlink has been removed and replaced as a Monitor
> > > > option in sssd.conf
> > > >
> > > > I see disable-netlink in man sssd.conf.
> > > > Justin, I run 'make rpms' and reinstall all,
> > > > so man pages were reinstall too.
> > >
> > > Thanks Petr, I was using the steps in the Contribute wiki 'reconfig &&
> > > chmake' then 'sssinstall' but I guess that did not update the man pages
> > > from my commit as expected.
> >
> > So far as I know, 'sssinstall' isn't good for man pages. But 'make rpms'
> > build whole all necessary things.
>
> Could you elaborate?
> Why 'sssinstall is not good for man pages?
>
> LS

Hi Lukas,

if I understand correctly, command 'reconfig'
prepare build environment to no producing man pages.
See contrib/fedora/bashrc_sssd:53:
         ${SSSD_NO_MANPAGES-} \
So if someone run reconfig before sssinstall it will
not install recent man pages.

Or did I understand it wrong way?

Regards

[Lukas Slebodnik]

On (05/09/16 15:54), Petr Cech wrote:
> On 09/05/2016 09:45 AM, Lukas Slebodnik wrote:
> > On (02/09/16 15:34), Petr Cech wrote:
> > > On 09/02/2016 03:31 PM, Justin Stephenson wrote:
> > > > On 09/02/2016 05:23 AM, Petr Cech wrote:
> > > > > On 09/01/2016 03:36 PM, Justin Stephenson wrote:
> > > > > > On 08/30/2016 03:54 AM, Jakub Hrozek wrote:
> > > > > > > On Sat, Aug 27, 2016 at 12:54:53PM -0400, Justin Stephenson wrote:
> > > > > > > > Hello,
> > > > > > > > 
> > > > > > > > The attached patches resolve https://fedorahosted.org/sssd/ticket/3142
> > > > > > > > 
> > > > > > > > However, I am having difficult with the man page addition to
> > > > > > > > 'src/man/sssd.conf.5.xml' for this new option. I have stared at the
> > > > > > > > open and
> > > > > > > > close xml tags(for far too long) and it looks correct but when I
> > > > > > > > build sssd
> > > > > > > > I never see the sssd.conf man page inclusion. Could anyone tell me
> > > > > > > > what I am
> > > > > > > > missing here?
> > > > > > > > 
> > > > > > > > If you feel there is better wording for the description please let me
> > > > > > > > know.
> > > > > > > > 
> > > > > > > > Kind regards,
> > > > > > > > Justin Stephenson
> > > > > > > 
> > > > > > > > From 0552c199dd37c7e280304b9bc92ff44a8a1a6d57 Mon Sep 17 00:00:00 2001
> > > > > > > > From: Justin Stephenson 
> > > > > > > > Date: Fri, 26 Aug 2016 15:15:32 -0400
> > > > > > > > Subject: [PATCH 1/2]     MONITOR: Remove --disable-netlink
> > > > > > > > command-line option
> > > > > > > 
> > > > > > > I'm not sure I like removing the netlink option w/o letting admins who
> > > > > > > use it at least know what happened. Could we keep the option in the
> > > > > > > popt
> > > > > > > option list, but use the HIDDEN argument so that it doesn't show up in
> > > > > > > --help output and print a loud warning that the option was removed in
> > > > > > > favor of a sssd.conf option?
> > > > > > > 
> > > > > > > I already know of two people from sssd-users list who might be using
> > > > > > > this feature. On the other hand, it was just introduced in the last
> > > > > > > version and not in any enterprise distro, so just printing a warning
> > > > > > > and
> > > > > > > removing even that warning in the next version would be fine for me..
> > > > > >
> > > > > > Agreed, please see updated patches also with Petr's corrections. Once
> > > > > > this fix is pushed I can respond to the email and at least let these
> > > > > > users know.
> > > > > >
> > > > > > I am still having trouble with the man page addition to sssd.conf not
> > > > > > showing, any ideas why?
> > > > > >
> > > > > > diff --git a/src/man/sssd.conf.5.xml b/src/man/sssd.conf.5.xml
> > > > > > index
> > > > > > ae291e0fc8f2f9afabcdf32f18a5ec12252bbbbf..6f231b8ab8fc078d83331bb7ef5b980528a30bd6
> > > > > >
> > > > > > 100644
> > > > > > --- a/src/man/sssd.conf.5.xml
> > > > > > +++ b/src/man/sssd.conf.5.xml
> > > > > > @@ -482,6 +482,24 @@
> > > > > >                              </para>
> > > > > >                          </listitem>
> > > > > >                      </varlistentry>
> > > > > >                <varlistentry>
> > > > > >                    <term>disable_netlink (boolean)</term>
> > > > > >                    <listitem>
> > > > > >                        <para>
> > > > > >                            SSSD hooks into the netlink
> > > > > >
> > > > > > interface to
> > > > > >                            monitor changes to routes, addresses,
> > > > > >
> > > > > > links
> > > > > >                            and trigger certain actions.
> > > > > >                        </para>
> > > > > >                        <para>
> > > > > >                            The SSSD state changes caused by
> > > > > >
> > > > > > netlink
> > > > > >                            events may be undesirable and can be
> > > > > >
> > > > > > disabled
> > > > > >                            by setting this option to 'true'
> > > > > >                        </para>
> > > > > >                        <para>
> > > > > >                            Default: false (netlink changes are
> > > > > >
> > > > > > detected)
> > > > > >                        </para>
> > > > > >                    </listitem>
> > > > > >                </varlistentry>
> > > > > >            </variablelist>
> > > > > >        </para>
> > > > > >    </refsect2>
> > > > > >
> > > > > > Kind regards,
> > > > > > Justin Stephenson
> > > > > >
> > > > > > > 
> > > > > > > > From c52c0c1a520cdf8509baaaaac00fa3c7bec0dd73 Mon Sep 17 00:00:00 2001
> > > > > > > > From: Justin Stephenson 
> > > > > > > > Date: Fri, 26 Aug 2016 17:43:25 -0400
> > > > > > > > Subject: [PATCH 2/2]     MONITOR: Add disable_netlink option
> > > > > > > 
> > > > > > > LGTM, untested, though.
> > > > >
> > > > > Hello Justin and Jakub,
> > > > >
> > > > > I tested it:
> > > > > sssd --help ... option is gone
> > > > >
> > > > > /sbin/sssd --disable-netlink
> > > > > Option --disable-netlink has been removed and replaced as a Monitor
> > > > > option in sssd.conf
> > > > >
> > > > > I see disable-netlink in man sssd.conf.
> > > > > Justin, I run 'make rpms' and reinstall all,
> > > > > so man pages were reinstall too.
> > > >
> > > > Thanks Petr, I was using the steps in the Contribute wiki 'reconfig &&
> > > > chmake' then 'sssinstall' but I guess that did not update the man pages
> > > > from my commit as expected.
> > >
> > > So far as I know, 'sssinstall' isn't good for man pages. But 'make rpms'
> > > build whole all necessary things.
> >
> > Could you elaborate?
> > Why 'sssinstall is not good for man pages?
> >
> > LS
>
> Hi Lukas,
>
> if I understand correctly, command 'reconfig'
> prepare build environment to no producing man pages.
> See contrib/fedora/bashrc_sssd:53:
>        ${SSSD_NO_MANPAGES-} \
> So if someone run reconfig before sssinstall it will
> not install recent man pages.
> Or did I understand it wrong way?

I have no idea what is a purpose of "SSSD_NO_MANPAGES"
It is not used anywhere in sssd.

$ source contrib/fedora/bashrc_sssd
$ reconfig
$ chmake
$ make install DESTDIR=$PWD/_inst
$ find $PWD/_inst -type f | grep man |wc -l
30

ans sssinstall does not do any magic with man pages.

sssinstall()
{
    sudo make -j$PROCESSORS install \
        && sudo rm -f $SSS_LIBDIR/ldb/modules/ldb/memberof.la \
        && sudo restorecon -v /$SSS_LIB/libnss_sss.so.2 \
                              /$SSS_LIB/security/pam_sss.so
}

Summary:
man pages should be properly installed with sssinstall
If no please file a bug.

LS

[Justin Stephenson]

On 09/05/2016 10:23 AM, Lukas Slebodnik wrote:
> On (05/09/16 15:54), Petr Cech wrote:
> > On 09/05/2016 09:45 AM, Lukas Slebodnik wrote:
> > > On (02/09/16 15:34), Petr Cech wrote:
> > > > On 09/02/2016 03:31 PM, Justin Stephenson wrote:
> > > > > On 09/02/2016 05:23 AM, Petr Cech wrote:
> > > > > > On 09/01/2016 03:36 PM, Justin Stephenson wrote:
> > > > > > > On 08/30/2016 03:54 AM, Jakub Hrozek wrote:
> > > > > > >> On Sat, Aug 27, 2016 at 12:54:53PM -0400, Justin Stephenson wrote:
> > > > > > >>> Hello,
> > > > > > >>>
> > > > > > >>> The attached patches resolve https://fedorahosted.org/sssd/ticket/3142
> > > > > > >>>
> > > > > > >>> However, I am having difficult with the man page addition to
> > > > > > >>> 'src/man/sssd.conf.5.xml' for this new option. I have stared at the
> > > > > > >>> open and
> > > > > > >>> close xml tags(for far too long) and it looks correct but when I
> > > > > > >>> build sssd
> > > > > > >>> I never see the sssd.conf man page inclusion. Could anyone tell me
> > > > > > >>> what I am
> > > > > > >>> missing here?
> > > > > > >>>
> > > > > > >>> If you feel there is better wording for the description please let me
> > > > > > >>> know.
> > > > > > >>>
> > > > > > >>> Kind regards,
> > > > > > >>> Justin Stephenson
> > > > > > >>
> > > > > > >>> From 0552c199dd37c7e280304b9bc92ff44a8a1a6d57 Mon Sep 17 00:00:00 2001
> > > > > > >>> From: Justin Stephenson 
> > > > > > >>> Date: Fri, 26 Aug 2016 15:15:32 -0400
> > > > > > >>> Subject: [PATCH 1/2]     MONITOR: Remove --disable-netlink
> > > > > > >>> command-line option
> > > > > > >>
> > > > > > >> I'm not sure I like removing the netlink option w/o letting admins who
> > > > > > >> use it at least know what happened. Could we keep the option in the
> > > > > > >> popt
> > > > > > >> option list, but use the HIDDEN argument so that it doesn't show up in
> > > > > > >> --help output and print a loud warning that the option was removed in
> > > > > > >> favor of a sssd.conf option?
> > > > > > >>
> > > > > > >> I already know of two people from sssd-users list who might be using
> > > > > > >> this feature. On the other hand, it was just introduced in the last
> > > > > > >> version and not in any enterprise distro, so just printing a warning
> > > > > > >> and
> > > > > > >> removing even that warning in the next version would be fine for me..
> > > > > > >
> > > > > > > Agreed, please see updated patches also with Petr's corrections. Once
> > > > > > > this fix is pushed I can respond to the email and at least let these
> > > > > > > users know.
> > > > > > >
> > > > > > > I am still having trouble with the man page addition to sssd.conf not
> > > > > > > showing, any ideas why?
> > > > > > >
> > > > > > > diff --git a/src/man/sssd.conf.5.xml b/src/man/sssd.conf.5.xml
> > > > > > > index
> > > > > > > ae291e0fc8f2f9afabcdf32f18a5ec12252bbbbf..6f231b8ab8fc078d83331bb7ef5b980528a30bd6
> > > > > > >
> > > > > > >
> > > > > > > 100644
> > > > > > > --- a/src/man/sssd.conf.5.xml
> > > > > > > +++ b/src/man/sssd.conf.5.xml
> > > > > > > @@ -482,6 +482,24 @@
> > > > > > >                              </para>
> > > > > > >                          </listitem>
> > > > > > >                      </varlistentry>
> > > > > > > +                    <varlistentry>
> > > > > > > +                        <term>disable_netlink (boolean)</term>
> > > > > > > +                        <listitem>
> > > > > > > +                            <para>
> > > > > > > +                                SSSD hooks into the netlink
> > > > > > > interface to
> > > > > > > +                                monitor changes to routes, addresses,
> > > > > > > links
> > > > > > > +                                and trigger certain actions.
> > > > > > > +                            </para>
> > > > > > > +                            <para>
> > > > > > > +                                The SSSD state changes caused by
> > > > > > > netlink
> > > > > > > +                                events may be undesirable and can be
> > > > > > > disabled
> > > > > > > +                                by setting this option to 'true'
> > > > > > > +                            </para>
> > > > > > > +                            <para>
> > > > > > > +                                Default: false (netlink changes are
> > > > > > > detected)
> > > > > > > +                            </para>
> > > > > > > +                        </listitem>
> > > > > > > +                    </varlistentry>
> > > > > > >                  </variablelist>
> > > > > > >              </para>
> > > > > > >          </refsect2>
> > > > > > >
> > > > > > > Kind regards,
> > > > > > > Justin Stephenson
> > > > > > >
> > > > > > >>
> > > > > > >>> From c52c0c1a520cdf8509baaaaac00fa3c7bec0dd73 Mon Sep 17 00:00:00 2001
> > > > > > >>> From: Justin Stephenson 
> > > > > > >>> Date: Fri, 26 Aug 2016 17:43:25 -0400
> > > > > > >>> Subject: [PATCH 2/2]     MONITOR: Add disable_netlink option
> > > > > > >>
> > > > > > >> LGTM, untested, though.
> > > > > >
> > > > > > Hello Justin and Jakub,
> > > > > >
> > > > > > I tested it:
> > > > > > sssd --help ... option is gone
> > > > > >
> > > > > > /sbin/sssd --disable-netlink
> > > > > > Option --disable-netlink has been removed and replaced as a Monitor
> > > > > > option in sssd.conf
> > > > > >
> > > > > > I see disable-netlink in man sssd.conf.
> > > > > > Justin, I run 'make rpms' and reinstall all,
> > > > > > so man pages were reinstall too.
> > > > >
> > > > > Thanks Petr, I was using the steps in the Contribute wiki 'reconfig &&
> > > > > chmake' then 'sssinstall' but I guess that did not update the man pages
> > > > > from my commit as expected.
> > > >
> > > > So far as I know, 'sssinstall' isn't good for man pages. But 'make rpms'
> > > > build whole all necessary things.
> > >
> > > Could you elaborate?
> > > Why 'sssinstall is not good for man pages?
> > >
> > > LS
> >
> > Hi Lukas,
> >
> > if I understand correctly, command 'reconfig'
> > prepare build environment to no producing man pages.
> > See contrib/fedora/bashrc_sssd:53:
> >        ${SSSD_NO_MANPAGES-} \
> > So if someone run reconfig before sssinstall it will
> > not install recent man pages.
> > Or did I understand it wrong way?
>
> I have no idea what is a purpose of "SSSD_NO_MANPAGES"
> It is not used anywhere in sssd.
>
> $ source contrib/fedora/bashrc_sssd
> $ reconfig
> $ chmake
> $ make install DESTDIR=$PWD/_inst
> $ find $PWD/_inst -type f | grep man |wc -l
> 30
>
> ans sssinstall does not do any magic with man pages.
>
> sssinstall()
> {
>     sudo make -j$PROCESSORS install \
>         && sudo rm -f $SSS_LIBDIR/ldb/modules/ldb/memberof.la \
>         && sudo restorecon -v /$SSS_LIB/libnss_sss.so.2 \
>                               /$SSS_LIB/security/pam_sss.so
> }
>
> Summary:
> man pages should be properly installed with sssinstall
> If no please file a bug.

I noticed the man pages were updated correctly with 'sssinstall' on 
previous patches I worked on, but for some reason it did not when I was 
adding the sssd.conf.5.xml for this ticket.

I will file a bug if it happens again, perhaps I was just doing 
something incorrectly in this instance.

Anyway, thanks for the review of these patches.

Kind regards,
Justin Stephenson
> LS
> _______________________________________________
> sssd-devel mailing list
> [e-mail]
> https://lists.fedorahosted.org/admin/lists/[e-mail]

[Jakub Hrozek]

Hi,

sorry to come late, but I have one more request (last one, I promise..)

On Thu, Sep 01, 2016 at 09:36:32AM -0400, Justin Stephenson wrote:
> From f647e732c2a5b8727376dded962766fb03bb5ea8 Mon Sep 17 00:00:00 2001
> From: Justin Stephenson 
> Date: Fri, 26 Aug 2016 15:15:32 -0400
> Subject: [PATCH 1/3]     MONITOR: Remove --disable-netlink command-line option
> Removing monitor command-line option, to be superceded by
> sssd.conf option

[...]
> if (opt_netlinkoff) {
>    fprintf(stderr, "Option --disable-netlink has been removed and "
>            "replaced as a Monitor option in sssd.conf\n");
>    poptPrintUsage(pc, stderr, 0);
>    return 1;

I would prefer if we were a little more graceful here and only called
DEBUG and sss_log() but did not abort the startup. I think it's better
to start SSSD in a degraded mode than not at all..
> }
> if (!opt_daemon && !opt_interactive && !opt_genconf) {
>        opt_daemon = 1;
>    }

[Justin Stephenson]

On 09/07/2016 02:24 PM, Jakub Hrozek wrote:
> Hi,
>
> sorry to come late, but I have one more request (last one, I promise..)
>
> On Thu, Sep 01, 2016 at 09:36:32AM -0400, Justin Stephenson wrote:
> > From f647e732c2a5b8727376dded962766fb03bb5ea8 Mon Sep 17 00:00:00 2001
> > From: Justin Stephenson 
> > Date: Fri, 26 Aug 2016 15:15:32 -0400
> > Subject: [PATCH 1/3]     MONITOR: Remove --disable-netlink command-line option
> > Removing monitor command-line option, to be superceded by
> > sssd.conf option
>
> [...]
> > if (opt_netlinkoff) {
> >    fprintf(stderr, "Option --disable-netlink has been removed and "
> >            "replaced as a Monitor option in sssd.conf\n");
> >    poptPrintUsage(pc, stderr, 0);
> >    return 1;
>
> I would prefer if we were a little more graceful here and only called
> DEBUG and sss_log() but did not abort the startup. I think it's better
> to start SSSD in a degraded mode than not at all..

Hi Jakub,

No problem, patch updated.
> > }
> > if (!opt_daemon && !opt_interactive && !opt_genconf) {
> >        opt_daemon = 1;
> >    }

[Jakub Hrozek]

On Thu, Sep 08, 2016 at 01:57:29PM -0400, Justin Stephenson wrote:
> On 09/07/2016 02:24 PM, Jakub Hrozek wrote:
> > Hi,
> >
> > sorry to come late, but I have one more request (last one, I promise..)
> >
> > On Thu, Sep 01, 2016 at 09:36:32AM -0400, Justin Stephenson wrote:
> > > From f647e732c2a5b8727376dded962766fb03bb5ea8 Mon Sep 17 00:00:00 2001
> > > From: Justin Stephenson 
> > > Date: Fri, 26 Aug 2016 15:15:32 -0400
> > > Subject: [PATCH 1/3]     MONITOR: Remove --disable-netlink command-line option
> > > Removing monitor command-line option, to be superceded by
> > > sssd.conf option
> >
> > [...]
> > > if (opt_netlinkoff) {
> > >    fprintf(stderr, "Option --disable-netlink has been removed and "
> > >            "replaced as a Monitor option in sssd.conf\n");
> > >    poptPrintUsage(pc, stderr, 0);
> > >    return 1;
> >
> > I would prefer if we were a little more graceful here and only called
> > DEBUG and sss_log() but did not abort the startup. I think it's better
> > to start SSSD in a degraded mode than not at all..
>
> Hi Jakub,
>
> No problem, patch updated.

ACK

[Jakub Hrozek]

On Mon, Sep 12, 2016 at 12:21:26AM +0200, Jakub Hrozek wrote:
> On Thu, Sep 08, 2016 at 01:57:29PM -0400, Justin Stephenson wrote:
> > On 09/07/2016 02:24 PM, Jakub Hrozek wrote:
> > > Hi,
> > >
> > > sorry to come late, but I have one more request (last one, I promise..)
> > >
> > > On Thu, Sep 01, 2016 at 09:36:32AM -0400, Justin Stephenson wrote:
> > > > From f647e732c2a5b8727376dded962766fb03bb5ea8 Mon Sep 17 00:00:00 2001
> > > > From: Justin Stephenson 
> > > > Date: Fri, 26 Aug 2016 15:15:32 -0400
> > > > Subject: [PATCH 1/3]     MONITOR: Remove --disable-netlink command-line option
> > > > Removing monitor command-line option, to be superceded by
> > > > sssd.conf option
> > >
> > > [...]
> > > > if (opt_netlinkoff) {
> > > >    fprintf(stderr, "Option --disable-netlink has been removed and "
> > > >            "replaced as a Monitor option in sssd.conf\n");
> > > >    poptPrintUsage(pc, stderr, 0);
> > > >    return 1;
> > >
> > > I would prefer if we were a little more graceful here and only called
> > > DEBUG and sss_log() but did not abort the startup. I think it's better
> > > to start SSSD in a degraded mode than not at all..
> >
> > Hi Jakub,
> >
> > No problem, patch updated.
>
> ACK

CI: http://sssd-ci.duckdns.org/logs/job/53/18/summary.html

* master:
    * 632fc5d8991d167eea20769c823163551c3f1d8c
    * 081c6d8c7c8e75487d1c4e42862964be1e85b575

[Jakub Hrozek]

On Mon, Sep 12, 2016 at 10:31:24AM +0200, Jakub Hrozek wrote:
> On Mon, Sep 12, 2016 at 12:21:26AM +0200, Jakub Hrozek wrote:
> > On Thu, Sep 08, 2016 at 01:57:29PM -0400, Justin Stephenson wrote:
> > > On 09/07/2016 02:24 PM, Jakub Hrozek wrote:
> > > > Hi,
> > > >
> > > > sorry to come late, but I have one more request (last one, I promise..)
> > > >
> > > > On Thu, Sep 01, 2016 at 09:36:32AM -0400, Justin Stephenson wrote:
> > > > > From f647e732c2a5b8727376dded962766fb03bb5ea8 Mon Sep 17 00:00:00 2001
> > > > > From: Justin Stephenson 
> > > > > Date: Fri, 26 Aug 2016 15:15:32 -0400
> > > > > Subject: [PATCH 1/3]     MONITOR: Remove --disable-netlink command-line option
> > > > > Removing monitor command-line option, to be superceded by
> > > > > sssd.conf option
> > > >
> > > > [...]
> > > > > if (opt_netlinkoff) {
> > > > >    fprintf(stderr, "Option --disable-netlink has been removed and "
> > > > >            "replaced as a Monitor option in sssd.conf\n");
> > > > >    poptPrintUsage(pc, stderr, 0);
> > > > >    return 1;
> > > >
> > > > I would prefer if we were a little more graceful here and only called
> > > > DEBUG and sss_log() but did not abort the startup. I think it's better
> > > > to start SSSD in a degraded mode than not at all..
> > >
> > > Hi Jakub,
> > >
> > > No problem, patch updated.
> >
> > ACK
>
> CI: http://sssd-ci.duckdns.org/logs/job/53/18/summary.html
> master:
> 632fc5d8991d167eea20769c823163551c3f1d8c
> 081c6d8c7c8e75487d1c4e42862964be1e85b575

Oh, fsck..I pushed wrong patch files..

I guess it's cleaner to revert and push the correct ones than to push
interdiff right?

This is exactly why I prefer pull requests over patches to mailing
lists..

[Jakub Hrozek]

On Mon, Sep 12, 2016 at 10:40:21AM +0200, Jakub Hrozek wrote:
> On Mon, Sep 12, 2016 at 10:31:24AM +0200, Jakub Hrozek wrote:
> > On Mon, Sep 12, 2016 at 12:21:26AM +0200, Jakub Hrozek wrote:
> > > On Thu, Sep 08, 2016 at 01:57:29PM -0400, Justin Stephenson wrote:
> > > > On 09/07/2016 02:24 PM, Jakub Hrozek wrote:
> > > > > Hi,
> > > > >
> > > > > sorry to come late, but I have one more request (last one, I promise..)
> > > > >
> > > > > On Thu, Sep 01, 2016 at 09:36:32AM -0400, Justin Stephenson wrote:
> > > > > > From f647e732c2a5b8727376dded962766fb03bb5ea8 Mon Sep 17 00:00:00 2001
> > > > > > From: Justin Stephenson 
> > > > > > Date: Fri, 26 Aug 2016 15:15:32 -0400
> > > > > > Subject: [PATCH 1/3]     MONITOR: Remove --disable-netlink command-line option
> > > > > > Removing monitor command-line option, to be superceded by
> > > > > > sssd.conf option
> > > > >
> > > > > [...]
> > > > > > if (opt_netlinkoff) {
> > > > > >    fprintf(stderr, "Option --disable-netlink has been removed and "
> > > > > >            "replaced as a Monitor option in sssd.conf\n");
> > > > > >    poptPrintUsage(pc, stderr, 0);
> > > > > >    return 1;
> > > > >
> > > > > I would prefer if we were a little more graceful here and only called
> > > > > DEBUG and sss_log() but did not abort the startup. I think it's better
> > > > > to start SSSD in a degraded mode than not at all..
> > > >
> > > > Hi Jakub,
> > > >
> > > > No problem, patch updated.
> > >
> > > ACK
> >
> > CI: http://sssd-ci.duckdns.org/logs/job/53/18/summary.html
> > master:
> > 632fc5d8991d167eea20769c823163551c3f1d8c
> > 081c6d8c7c8e75487d1c4e42862964be1e85b575
>
> Oh, fsck..I pushed wrong patch files..

Actually no, they were correct, sorry. The alarm is off :)