Skip to content

[autobackport: sssd-2-9] CONFIG: disable 'session_provider' by default#8260

Merged
sumit-bose merged 1 commit intoSSSD:sssd-2-9from
sssd-bot:SSSD-sssd-backport-pr8250-to-sssd-2-9
Dec 3, 2025
Merged

[autobackport: sssd-2-9] CONFIG: disable 'session_provider' by default#8260
sumit-bose merged 1 commit intoSSSD:sssd-2-9from
sssd-bot:SSSD-sssd-backport-pr8250-to-sssd-2-9

Conversation

@sssd-bot
Copy link
Contributor

@sssd-bot sssd-bot commented Dec 3, 2025

This is an automatic backport of PR#8250 CONFIG: disable 'session_provider' by default to branch sssd-2-9, created by @alexey-tikhonov.

Caution

@alexey-tikhonov The patches did not apply cleanly. It is necessary to resolve conflicts before merging this pull request. Commits that introduced conflict are marked with CONFLICT!.

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

Original commits
2a1048b - CONFIG: disable 'session_provider' by default

Backported commits

  • d022fac - CONFLICT! CONFIG: disable 'session_provider' by default

Conflicting Files Information (check for deleted and re-added files)

  • CONFLICT! CONFIG: disable 'session_provider' by default
On branch SSSD-sssd-backport-pr8250-to-sssd-2-9
You are currently cherry-picking commit 2a1048b59.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Changes to be committed:
	modified:   src/man/sssd.conf.5.xml

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   src/providers/data_provider/dp_targets.c```

---

**Original Pull Request Body**

:config: Default value of 'session_provider' option was changed to 'none' (i.e. disabled) doesn't matter what 'id_provider' used. Previously 'session_provider' was enabled by default for 'id_provider = ipa' case. But this didn't make sense because 'Fleet Commander', that is being integrated in this case, is long time obsolete.

:config: Default value of 'session_provider' option was changed to 'none'
(i.e. disabled) doesn't matter what 'id_provider' used. Previously
'session_provider' was enabled by default for 'id_provider = ipa' case.
But this didn't make sense because 'Fleet Commander', that is being
integrated in this case, is long time obsolete.

Reviewed-by: Pavel Březina <pbrezina@redhat.com>
Reviewed-by: Sumit Bose <sbose@redhat.com>
(cherry picked from commit 2a1048b)
@alexey-tikhonov alexey-tikhonov force-pushed the SSSD-sssd-backport-pr8250-to-sssd-2-9 branch from d022fac to 5330cb3 Compare December 3, 2025 11:07
@alexey-tikhonov alexey-tikhonov marked this pull request as ready for review December 3, 2025 11:33
@alexey-tikhonov alexey-tikhonov added no-backport This should go to target branch only. Waiting for review labels Dec 3, 2025
Copy link
Contributor

@sumit-bose sumit-bose left a comment

Choose a reason for hiding this comment

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

Hi,

thank you, backport looks good, ACK.

bye,
Sumit

@sumit-bose
Copy link
Contributor

Hi,

the issue in "tests/test_access_control_simple.py::test_access_control_simple__permits_user_login_based_on_group (samba)" is a known issue and not related to this PR, merging now.

bye,
Sumit

@sumit-bose sumit-bose merged commit 35519ca into SSSD:sssd-2-9 Dec 3, 2025
14 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-backport This should go to target branch only.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants