URL:
https://github.com/SSSD/sssd/pull/390
Title: #390: NSS: Add option to disable memcache
fidencio commented:
"""
@pbrezina, @mzidek-rh:
Although I do believe that Pavel's suggestion is reasonable, I can see some problems
with that, which I will try to describe below:
- option and env-var are evaluated in different parts of the code: I see this as a
possible limitation for @pbrezina's suggestion, although it could be changed;
- env-var is only checked for NO: In case we want it to have priority, we have to have the
code changed in a way that we also would evaluate YES and "NOT PRESENT" and
based on this we could decide whether to use or not the memcache
- having two methods for doing the very same thing is not so nice (as then we start
dealing with priorities): So, here is more like a question than a suggestion ... can't
we just use this option from 2.0 (where we'll be breaking compats anyways)?
Enough "bla bla bla" .... so, summing up I guess @mzidek-rh's approach is
simpler while also being functional. So, I'd go for it, at least till some customer
complains about the approach.
"""
See the full comment at
https://github.com/SSSD/sssd/pull/390#issuecomment-331416745