-
Notifications
You must be signed in to change notification settings - Fork 245
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
DESKPROFILE: Document it doesn't work when run as unprivileged user #516
Conversation
src/man/sssd.conf.5.xml
Outdated
@@ -2461,6 +2461,11 @@ pam_account_locked_message = Account locked, please contact help desk. | |||
Default: <quote>id_provider</quote> is used if it | |||
is set and can perform session related tasks. | |||
</para> | |||
<note> | |||
In order to have this feature working as expected, | |||
SSSD must be running as "root" and not as the "sssd" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is just convention that unprivileged user is called sssd by default.
But it is configurable.
./configure --help | grep -- -user
--with-sssd-user=<user> User for running SSSD (root)
Therefore would be better to mention just unprivileged user.
6844c8a
to
aa179f6
Compare
Changes done according to your suggestion. |
retest this please |
LGTM from content point of view. @justin-stephenson could you check this PR as well? |
Okay from my side. |
Justin, |
I tried to figure out why centos CI is failing and I noticed that xml element
I can see that on other places in sssd.conf.5.xml we use either
or NOTE without emphasis
I'm sorry that I did not notice that earlier. |
aa179f6
to
3e96158
Compare
Resolves: https://pagure.io/SSSD/sssd/issue/3648 Signed-off-by: Fabiano Fidêncio <fidencio@redhat.com>
3e96158
to
20adffb
Compare
I think it's OK to add the Accepted label since the centos CI is passing now. |
master: |
No description provided.