Skip to content

[autobackport: sssd-2-9] Remove obsolete entry_cache_computer_timeout option - #9221

Merged
alexey-tikhonov merged 1 commit into
SSSD:sssd-2-9from
sssd-bot:SSSD-sssd-backport-pr9088-to-sssd-2-9
Sep 4, 2026
Merged

[autobackport: sssd-2-9] Remove obsolete entry_cache_computer_timeout option#9221
alexey-tikhonov merged 1 commit into
SSSD:sssd-2-9from
sssd-bot:SSSD-sssd-backport-pr9088-to-sssd-2-9

Conversation

@sssd-bot

@sssd-bot sssd-bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

This is an automatic backport of PR#9088 Remove obsolete entry_cache_computer_timeout option to branch sssd-2-9, created by @asakure.

Please make sure this backport is correct.

Note

The commits were cherry-picked without conflicts.

You can push changes to this pull request

git remote add sssd-bot git@github.com:sssd-bot/sssd.git
git fetch sssd-bot refs/heads/SSSD-sssd-backport-pr9088-to-sssd-2-9
git checkout SSSD-sssd-backport-pr9088-to-sssd-2-9
git push sssd-bot SSSD-sssd-backport-pr9088-to-sssd-2-9 --force

Original commits
78ea2f0 - Remove obsolete entry_cache_computer_timeout option

Backported commits

  • 5092b95 - Remove obsolete entry_cache_computer_timeout option

Original Pull Request Body

This patch removes option entry_cache_computer_timeout and all its references from the code (from sssd.conf.5.xml, cfg_rules.ini, confdb.c, confdb.h) since it is not used anymore. entry_cache_computer_timeout parameter was originally introduced with a2e7f68 but its usage was later removed with c02e09a.

@alexey-tikhonov
alexey-tikhonov removed the request for review from sumit-bose September 4, 2026 13:15
@alexey-tikhonov alexey-tikhonov added the no-backport This should go to target branch only. label Sep 4, 2026

@sumit-bose sumit-bose left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi,

backport matches original patch, ACK.

bye,
Sumit

@alexey-tikhonov

Copy link
Copy Markdown
Member
FAILED tests/test_failover.py::test_failover__connect_using_ipv4_second_family (ipa) - AssertionError: testuser was not found, SSSD did not switch to IPv4 family!
assert None is not None
FAILED tests/test_access_control_simple.py::test_access_control_simple__deny_group_supersedes_allow_group (ldap) - sssd_test_framework.misc.errors.ExpectScriptError: Unexpected end of file
FAILED tests/test_authentication.py::test_authentication__user_login[root-ssh] (ldap) - sssd_test_framework.misc.errors.ExpectScriptError: Unexpected end of file

are unrelated.

This patch removes option entry_cache_computer_timeout
and all its references from the code (from sssd.conf.5.xml,
cfg_rules.ini. confdb.c, confdb.h) since it is not used anymore.
entry_cache_computer_timeout parameter was originally introduced
with a2e7f68 but its usage was later removed with c02e09a.

Signed-off-by: Akshay Sakure <asakure@redhat.com>
Reviewed-by: Alexey Tikhonov <atikhono@redhat.com>
Reviewed-by: Sumit Bose <sbose@redhat.com>
(cherry picked from commit 78ea2f0)
@sssd-bot

sssd-bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor Author

The pull request was accepted by @alexey-tikhonov with the following PR CI status:


🟢 CodeQL (success)
🟢 rpm-build:centos-stream-9-x86_64:upstream (success)
🟢 testing-farm:centos-stream-9-x86_64:upstream (success)
🟢 Build / make-distcheck (success)
🟢 ci / prepare (success)
🔴 ci / system (centos-9) (failure)
🟢 Static code analysis / codeql (success)
🟢 Static code analysis / files-changed (success)
🟢 Static code analysis / pre-commit (success)
➖ Static code analysis / python-system-tests (skipped)


There are unsuccessful or unfinished checks. Make sure that the failures are not related to this pull request before merging.

@sssd-bot
sssd-bot force-pushed the SSSD-sssd-backport-pr9088-to-sssd-2-9 branch from 5092b95 to cb28700 Compare September 4, 2026 14:54
@alexey-tikhonov
alexey-tikhonov merged commit c773786 into SSSD:sssd-2-9 Sep 4, 2026
8 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Accepted no-backport This should go to target branch only.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants