Skip to content

spec: clarify description of sssd-idp package#8316

Merged
alexey-tikhonov merged 1 commit intoSSSD:masterfrom
sumit-bose:spec_idp_description
Dec 23, 2025
Merged

spec: clarify description of sssd-idp package#8316
alexey-tikhonov merged 1 commit intoSSSD:masterfrom
sumit-bose:spec_idp_description

Conversation

@sumit-bose
Copy link
Contributor

Make clean the the sssd-idp package now contains SSSD's IdP provider as well.

Resolves: #8022

Copy link

@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 updates the summary and description for the sssd-idp package in the RPM spec file. The changes clarify that the package includes the SSSD IdP provider back end, in addition to the previously mentioned Kerberos plugins and OIDC helper. The new description is more accurate and comprehensive, which is a welcome improvement.

@alexey-tikhonov alexey-tikhonov self-assigned this Dec 22, 2025
@alexey-tikhonov alexey-tikhonov added no-backport This should go to target branch only. Trivial A single reviewer is sufficient to review the Pull Request Accepted labels Dec 22, 2025
Make clean the the sssd-idp package now contains SSSD's IdP provider as
well.

Resolves: SSSD#8022
Reviewed-by: Alexey Tikhonov <atikhono@redhat.com>
@sssd-bot
Copy link
Contributor

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


🟢 CodeFactor (success)
🟢 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)
🟢 Build / freebsd (success)
🟢 Build / make-distcheck (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) (failure)
➖ 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.

@sssd-bot sssd-bot force-pushed the spec_idp_description branch from c5660a9 to 5cdbdc4 Compare December 23, 2025 08:53
@alexey-tikhonov alexey-tikhonov merged commit 9579e08 into SSSD:master Dec 23, 2025
11 of 15 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. Trivial A single reviewer is sufficient to review the Pull Request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

sssd-idp is available but not functional on Fedora 42

3 participants