Skip to content

[autobackport: sssd-2-11] removing unstable topologies from memecache tests#8625

Merged
spoore1 merged 1 commit intoSSSD:sssd-2-11from
sssd-bot:SSSD-sssd-backport-pr8624-to-sssd-2-11
Apr 24, 2026
Merged

[autobackport: sssd-2-11] removing unstable topologies from memecache tests#8625
spoore1 merged 1 commit intoSSSD:sssd-2-11from
sssd-bot:SSSD-sssd-backport-pr8624-to-sssd-2-11

Conversation

@sssd-bot
Copy link
Copy Markdown
Contributor

This is an automatic backport of PR#8624 removing unstable topologies from memecache tests to branch sssd-2-11, created by @danlavu.

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-pr8624-to-sssd-2-11
git checkout SSSD-sssd-backport-pr8624-to-sssd-2-11
git push sssd-bot SSSD-sssd-backport-pr8624-to-sssd-2-11 --force

Original commits
6726f5a - removing unstable topologies from memecache tests

Backported commits

  • 2c5a616 - removing unstable topologies from memecache tests

Original Pull Request Body

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request simplifies the test configuration in src/tests/system/tests/test_memcache.py by replacing the combination of KnownTopologyGroup.AnyProvider and KnownTopology.LDAP with a direct KnownTopology.LDAP mark across multiple test cases. Additionally, the unused KnownTopologyGroup import has been removed. I have no feedback to provide.

@danlavu danlavu added no-backport This should go to target branch only. Waiting for review Tests Trivial A single reviewer is sufficient to review the Pull Request labels Apr 22, 2026
@danlavu
Copy link
Copy Markdown

danlavu commented Apr 23, 2026

Backport to 2.11 all the failed distros run sssd-2.12 or higher, the failures are expected.

Copy link
Copy Markdown
Contributor

@spoore1 spoore1 left a comment

Choose a reason for hiding this comment

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

LGTM. check failures unrelated to PR changes and expected.

Reviewed-by: Scott Poore <spoore@redhat.com>
(cherry picked from commit 6726f5a)
@sssd-bot
Copy link
Copy Markdown
Contributor Author

The pull request was accepted by @spoore1 with the following PR CI status:


🟢 CodeQL (success)
🟢 rpm-build:centos-stream-10-x86_64:upstream (success)
🟢 rpm-build:fedora-42-x86_64:upstream (success)
🔴 rpm-build:fedora-43-x86_64:upstream (failure)
🔴 rpm-build:fedora-44-x86_64:upstream (failure)
🔴 rpm-build:fedora-rawhide-x86_64:upstream (failure)
🟢 ci / intgcheck (centos-10) (success)
🟢 ci / intgcheck (fedora-42) (success)
🟢 ci / intgcheck (fedora-43) (success)
🟢 ci / intgcheck (fedora-44) (success)
🔴 ci / intgcheck (fedora-45) (failure)
🟢 ci / prepare (success)
🟢 ci / system (centos-10) (success)
🟢 ci / system (fedora-42) (success)
🔴 ci / system (fedora-43) (failure)
🔴 ci / system (fedora-44) (failure)
🔴 ci / system (fedora-45) (failure)
🟢 Static code analysis / codeql (success)
🟢 Static code analysis / pre-commit (success)
🟢 Static code analysis / python-system-tests (success)


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-pr8624-to-sssd-2-11 branch from 2c5a616 to 5189388 Compare April 24, 2026 19:25
@spoore1 spoore1 merged commit 5dc1dbe into SSSD:sssd-2-11 Apr 24, 2026
10 of 13 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. Tests Trivial A single reviewer is sufficient to review the Pull Request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants