updating subid test case to test provider_ldap config

[danlavu]

No description provided.

[gemini-code-assist]

Code Review

The pull request updates the sssd-test-framework dependency to a specific branch and enhances an existing IPA subid test case. The test case test_ipa__subids_configured is now parameterized to cover both provider=ipa and provider=ldap configurations, ensuring that SSSD can correctly read subid ranges in both scenarios. This improves the test coverage for subid functionality.

[danlavu]

Depends on SSSD/sssd-test-framework@0b97f84

[alexey-tikhonov]

ldap-provider doesn't support subid ranges in sssd-2-11.
Also added 'blocked' while there is tmp commit.

[alexey-tikhonov]

ImportError while loading conftest '/home/runner/work/sssd/sssd/sssd/src/tests/system/conftest.py'.
conftest.py:6: in <module>
    from sssd_test_framework.config import SSSDMultihostConfig
.venv/lib/python3.14/site-packages/sssd_test_framework/config.py:7: in <module>
    from .topology import SSSDTopologyMark
.venv/lib/python3.14/site-packages/sssd_test_framework/topology.py:125: in <module>
    class KnownTopology(KnownTopologyBase):
.venv/lib/python3.14/site-packages/sssd_test_framework/topology.py:139: in KnownTopology
    Client = SSSDTopologyMark(
.venv/lib/python3.14/site-packages/sssd_test_framework/topology.py:61: in __init__
    super().__init__(name, topology, fixtures)
E   TypeError: TopologyMark.__init__() takes 3 positional arguments but 4 were given
Error: Process completed with exit code 4.

[alexey-tikhonov]

ACK, but needs framework PR in.

[jakub-vavra-cz]

LGTM, but let's wait for green ci.

[alexey-tikhonov]

mypy is red. Could you please check it, @danlavu?

[danlavu]

mypy is red. Could you please check it, @danlavu?

tests/test_ipa.py:1096: error: "SSSDCommonConfiguration" has no attribute "ldap_provider" [attr-defined]

Re ran it, green now.

[sssd-bot]

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

---

🟢 CodeQL (success)
🟢 osh-diff-scan:fedora-rawhide-x86_64:upstream (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 (success)
🟢 rpm-build:fedora-rawhide-x86_64:upstream (success)
🟢 Analyze (target) / cppcheck (success)
🟢 ci / intgcheck (centos-10) (success)
🟢 ci / intgcheck (fedora-42) (success)
🟢 ci / intgcheck (fedora-43) (success)
🟢 ci / intgcheck (fedora-44) (success)
🟢 ci / prepare (success)
🟢 ci / system (centos-10) (success)
🟢 ci / system (fedora-42) (success)
🟢 ci / system (fedora-43) (success)
🟢 ci / system (fedora-44) (success)
➖ Coverity scan / coverity (skipped)
🟢 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.