Re: [SSSD] [PATCH]-Bad debug message when adding services without explicit dns_discovery_domain

[Ariel Barria]

thanks for comments.

From: [e-mail]
To: [e-mail]
Date: Wed, 13 Jun 2012 07:27:54 -0400
Subject: Re: [SSSD] [PATCH]-Bad debug message when adding services without explicit dns_discovery_domain

On Tue, 2012-06-12 at 23:20 -0500, Ariel Barria wrote:
> thanks for comments
> > From: [e-mail]
> > To: [e-mail]
> > Date: Tue, 12 Jun 2012 08:30:07 -0400
> > Subject: Re: [SSSD] [PATCH]-Bad debug message when adding services
>
> without explicit dns_discovery_domain
> > On Tue, 2012-06-12 at 11:11 +0200, Jakub Hrozek wrote:
> > > On Tue, Jun 12, 2012 at 11:04:28AM +0200, Jan Zelený wrote:
> > > > > https://fedorahosted.org/sssd/ticket/920
> > > >
> > > > Nack,
> > > > please follow our coding guidelines:
> > > >
> > > > http://www.freeipa.org/page/Coding_Style
> > > >
> > > > In particular, I'm referring to the line length.
> > > >
> > > > Thanks
> > > > Jan
> > >
> > > Please also change the debug message to appropriate symbolic
>
> constant.
> > s/message/level/ I guess :)

Nack

First of all, you're still not following our style guidelines. Please do
not use tabs anywhere, keep your lines to 79 charcacters and do not have
trailing spaces in the lines.

Now, for the meat of the problem. The message that you're printing is
incorrect. We don't want to print the literal strings "example.com" and
"DOMAINNAME".

"DOMAINNAME" should be replaced by the input  string sssd_domain.

Unfortunately, at this stage of evaluation we do not have the actual DNS
domain to print, so we'll have to replace "example.com" with "the
machine's DNS domain name".

_______________________________________________
sssd-devel mailing list
[e-mail]
https://fedorahosted.org/mailman/listinfo/sssd-devel