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

Stephen Gallagher sgallagh at redhat.com
Wed Jun 13 11:27:54 UTC 2012


On Tue, 2012-06-12 at 23:20 -0500, Ariel Barria wrote:
> thanks for comments
> 
> 
> > From: simo at redhat.com
> > To: sssd-devel at lists.fedorahosted.org
> > 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".

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20120613/dc519b6b/attachment.sig>


More information about the sssd-devel mailing list