URL: https://github.com/SSSD/sssd/pull/414 Author: lslebodn Title: #414: NSS: Add a way to disable memory cache Action: opened
PR body: """ It is a temporary workaround for case where you hit corner case when restarting sssd very often and different process open different different memory cache.
Experimental features are enabled by default in upstream.
Resolves: https://pagure.io/SSSD/sssd/issue/3496 """
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/414/head:pr414 git checkout pr414
URL: https://github.com/SSSD/sssd/pull/414 Title: #414: NSS: Add a way to disable memory cache
fidencio commented: """ @lslebodn, let's go for @mzidek-rh's approach as discussed in our meeting Today, thus I'm closing this PR as rejected. """
See the full comment at https://github.com/SSSD/sssd/pull/414#issuecomment-338021333
URL: https://github.com/SSSD/sssd/pull/414 Title: #414: NSS: Add a way to disable memory cache
Label: +Rejected
URL: https://github.com/SSSD/sssd/pull/414 Author: lslebodn Title: #414: NSS: Add a way to disable memory cache Action: closed
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/414/head:pr414 git checkout pr414
URL: https://github.com/SSSD/sssd/pull/414 Author: lslebodn Title: #414: NSS: Add a way to disable memory cache Action: reopened
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/414/head:pr414 git checkout pr414
URL: https://github.com/SSSD/sssd/pull/414 Title: #414: NSS: Add a way to disable memory cache
Label: -Rejected
URL: https://github.com/SSSD/sssd/pull/414 Title: #414: NSS: Add a way to disable memory cache
lslebodn commented: """ Disabling memory cache have a huge performance degradation and not only in parallel case. https://pagure.io/SSSD/sssd/issue/3520#comment-467642 In case of parallel asses (e.g. multithreaded application it would be even worse)
Fedora 26 switched order of nsswitch modules due to memory cache. http://fedoraproject.org/wiki/Releases/26/ChangeSet#SSSD_fast_cache_for_loca...
There is not any valid technical reason to disable memory cache globally therefore this code is conditional and enabled by default in upstream spec file
"""
See the full comment at https://github.com/SSSD/sssd/pull/414#issuecomment-338129881
URL: https://github.com/SSSD/sssd/pull/414 Title: #414: NSS: Add a way to disable memory cache
pbrezina commented: """ We agreed yesterday that it will not be conditional. """
See the full comment at https://github.com/SSSD/sssd/pull/414#issuecomment-338131455
URL: https://github.com/SSSD/sssd/pull/414 Title: #414: NSS: Add a way to disable memory cache
lslebodn commented: """
We agreed yesterday that it will not be conditional.
As you wish but it is a terrible mistake in upstream and there is not any valid technical reason to support such options. And I won't support in in fedora. """
See the full comment at https://github.com/SSSD/sssd/pull/414#issuecomment-338133783
URL: https://github.com/SSSD/sssd/pull/414 Title: #414: NSS: Add a way to disable memory cache
fidencio commented: """ How come that we're *again* have this very same discussion? We democratically agreed on the solution and, as far as I understand, we agreed on what would be on Fedora.
@lslebodn, sorry, but the project is not only yours and doing what you're doing is disrespectful to all other team members who voted for this decision. Please, try to be reasonable here. """
See the full comment at https://github.com/SSSD/sssd/pull/414#issuecomment-338137745
URL: https://github.com/SSSD/sssd/pull/414 Title: #414: NSS: Add a way to disable memory cache
fidencio commented: """ How come that we're *again* having this very same discussion? We democratically agreed on the solution and, as far as I understand, we agreed on what would be on Fedora.
@lslebodn, sorry, but the project is not only yours and doing what you're doing is disrespectful to all other team members who voted for this decision. Please, try to be reasonable here. """
See the full comment at https://github.com/SSSD/sssd/pull/414#issuecomment-338137745
URL: https://github.com/SSSD/sssd/pull/414 Title: #414: NSS: Add a way to disable memory cache
fidencio commented: """ I'm closing this PR again per https://github.com/SSSD/sssd/pull/414#issuecomment-338021333 """
See the full comment at https://github.com/SSSD/sssd/pull/414#issuecomment-338152144
URL: https://github.com/SSSD/sssd/pull/414 Title: #414: NSS: Add a way to disable memory cache
Label: +Rejected
URL: https://github.com/SSSD/sssd/pull/414 Author: lslebodn Title: #414: NSS: Add a way to disable memory cache Action: closed
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/414/head:pr414 git checkout pr414
URL: https://github.com/SSSD/sssd/pull/414 Title: #414: NSS: Add a way to disable memory cache
lslebodn commented: """ Reopening per comment https://github.com/SSSD/sssd/pull/416#issuecomment-338128727 """
See the full comment at https://github.com/SSSD/sssd/pull/414#issuecomment-338160131
URL: https://github.com/SSSD/sssd/pull/414 Title: #414: NSS: Add a way to disable memory cache
Label: -Rejected
URL: https://github.com/SSSD/sssd/pull/414 Title: #414: NSS: Add a way to disable memory cache
lslebodn commented: """ I cannot see conditional build in https://github.com/lslebodn/sssd/commit/606045d73cf81f81c6ddb6e5ca146213e756... """
See the full comment at https://github.com/SSSD/sssd/pull/414#issuecomment-338160535
sssd-devel@lists.fedorahosted.org
-
fidencio -
lslebodn
-
pbrezina