[SSSD] Do not start multiple backends for the same domain

Pavel Reichl preichl at redhat.com
Mon Jan 20 17:22:51 UTC 2014


On Mon, 2014-01-20 at 18:07 +0100, Pavel Reichl wrote:
> On Fri, 2014-01-17 at 19:47 +0100, Jakub Hrozek wrote:
> > On Fri, Jan 17, 2014 at 03:40:35PM +0100, Pavel Reichl wrote:
> > > +                DEBUG(SSSDBG_FATAL_FAILURE,
> > > +                      ("Domain '%s' has too similar name to domain '%s'.\n",
> > > +                       domlist[i], domain->name));
> > > +                sss_log(SSS_LOG_CRIT,
> > > +                        "Domain '%s' has too similar name to domain '%s'.\n",
> > > +                        domlist[i], domain->name);
> > 
> > I think we can just say that 'dom X differs from dom Y in case only'
> > unless we plan on adding more similar checks later.
> > _______________________________________________
> > sssd-devel mailing list
> > sssd-devel at lists.fedorahosted.org
> > https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
> 
> Hello Jakub,
> 
> I updated the string as you suggested.
> 
> BTW I was wondering if it is OK if I stored this string into local
> constant variable and used it in both DEBUG and sss_log as I hate code
> duplication on every level:-).
> 
> PR
> _______________________________________________
> sssd-devel mailing list
> sssd-devel at lists.fedorahosted.org
> https://lists.fedorahosted.org/mailman/listinfo/sssd-devel

Sorry I attached previously wrong patch.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-CONFDB-fail-if-there-are-domains-with-same-name.patch
Type: text/x-patch
Size: 1663 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20140120/432ecc8b/attachment.bin>


More information about the sssd-devel mailing list