Skip to content

[autobackport: sssd-2-12] Add support for Plasma Login Manager as a supported PAM service#8509

Merged
alexey-tikhonov merged 1 commit intoSSSD:sssd-2-12from
sssd-bot:SSSD-sssd-backport-pr8506-to-sssd-2-12
Mar 11, 2026
Merged

[autobackport: sssd-2-12] Add support for Plasma Login Manager as a supported PAM service#8509
alexey-tikhonov merged 1 commit intoSSSD:sssd-2-12from
sssd-bot:SSSD-sssd-backport-pr8506-to-sssd-2-12

Conversation

@sssd-bot
Copy link
Copy Markdown
Contributor

This is an automatic backport of PR#8506 Add support for Plasma Login Manager as a supported PAM service to branch sssd-2-12, created by @Conan-Kudo.

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

Original commits
5df3bff - Add support for Plasma Login Manager as a supported PAM service

Backported commits

  • 5c3aaa8 - Add support for Plasma Login Manager as a supported PAM service

Original Pull Request Body

Plasma Login Manager is the new recommended login manager for KDE Plasma, and is used as the login manager for Fedora Linux 44 and higher for KDE Plasma.

Reference: https://fedoraproject.org/wiki/Changes/PlasmaLoginManager

Resolves: #8490

@alexey-tikhonov alexey-tikhonov added the no-backport This should go to target branch only. label Mar 10, 2026
@alexey-tikhonov
Copy link
Copy Markdown
Member

alexey-tikhonov commented Mar 10, 2026

This branch won't build for Rawhide:

src/util/sss_ini.c:39:10: fatal error: ini_config.h: No such file or directory
   39 | #include "ini_config.h"

-- #8409 wasn't backported here.

But Fedora has it cherry-picked:
https://src.fedoraproject.org/rpms/sssd/c/5d1bfccea5c0ac728a37e6adac15a7442ddc28d9?branch=rawhide

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 backports the addition of support for the Plasma Login Manager (plasmalogin) as a supported PAM service. The changes consistently add plasmalogin to default service lists in GPO mapping, smartcard authentication, and related documentation and tests. The modifications are correct and maintain alphabetical ordering where it was already present. No high or critical severity issues were found by the code review. The security review agent encountered an error and was unable to provide specific security findings for this pull request.

Plasma Login Manager is the new recommended login manager for KDE Plasma,
and is used as the login manager for Fedora Linux 44 and higher for KDE
Plasma.

Reference: https://fedoraproject.org/wiki/Changes/PlasmaLoginManager

Resolves: SSSD#8490

Signed-off-by: Neal Gompa <ngompa@velocitylimitless.com>
Reviewed-by: Alexey Tikhonov <atikhono@redhat.com>
Reviewed-by: Sumit Bose <sbose@redhat.com>
(cherry picked from commit 5df3bff)
@sssd-bot
Copy link
Copy Markdown
Contributor Author

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


🟢 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-44-x86_64:upstream (success)
🔴 rpm-build:fedora-rawhide-x86_64:upstream (failure)
🟢 Build / freebsd (success)
🟢 Build / make-distcheck (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-pr8506-to-sssd-2-12 branch from 5c3aaa8 to 025952c Compare March 11, 2026 13:15
@alexey-tikhonov alexey-tikhonov merged commit 3cb7774 into SSSD:sssd-2-12 Mar 11, 2026
3 of 8 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