URL: https://github.com/SSSD/sssd/pull/5782 Author: thalman Title: #5782: CONFDB: Change ownership of config.ldb Action: opened
PR body: """ Config database is owned by root. This prevents our socket activated services to start because they are started under the sssd user. Changing the ownership to sssd fixes the issue.
Resolves: https://github.com/SSSD/sssd/issues/5781 """
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5782/head:pr5782 git checkout pr5782
URL: https://github.com/SSSD/sssd/pull/5782 Title: #5782: CONFDB: Change ownership of config.ldb
pbrezina commented: """ Tests fails. """
See the full comment at https://github.com/SSSD/sssd/pull/5782#issuecomment-922846995
URL: https://github.com/SSSD/sssd/pull/5782 Title: #5782: CONFDB: Change ownership of config.ldb
Label: +Changes requested
URL: https://github.com/SSSD/sssd/pull/5782 Title: #5782: CONFDB: Change ownership of config.ldb
thalman commented: """
Tests fails.
I'm on it, I also found an issue in "upgrade" path. """
See the full comment at https://github.com/SSSD/sssd/pull/5782#issuecomment-922848191
URL: https://github.com/SSSD/sssd/pull/5782 Author: thalman Title: #5782: CONFDB: Change ownership of config.ldb Action: synchronized
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5782/head:pr5782 git checkout pr5782
URL: https://github.com/SSSD/sssd/pull/5782 Author: thalman Title: #5782: CONFDB: Change ownership of config.ldb Action: synchronized
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5782/head:pr5782 git checkout pr5782
URL: https://github.com/SSSD/sssd/pull/5782 Author: thalman Title: #5782: CONFDB: Change ownership of config.ldb Action: synchronized
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5782/head:pr5782 git checkout pr5782
URL: https://github.com/SSSD/sssd/pull/5782 Author: thalman Title: #5782: CONFDB: Change ownership of config.ldb Action: synchronized
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5782/head:pr5782 git checkout pr5782
URL: https://github.com/SSSD/sssd/pull/5782 Author: thalman Title: #5782: CONFDB: Change ownership of config.ldb Action: synchronized
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5782/head:pr5782 git checkout pr5782
URL: https://github.com/SSSD/sssd/pull/5782 Title: #5782: CONFDB: Change ownership of config.ldb
alexey-tikhonov commented: """
By the way, I haven't found why `debian10` is failing.
Isn't relevant to this PR (failed on attempt to setup machine before multihost tests that aren't run on this target anyway). """
See the full comment at https://github.com/SSSD/sssd/pull/5782#issuecomment-945603994
URL: https://github.com/SSSD/sssd/pull/5782 Title: #5782: CONFDB: Change ownership of config.ldb
Label: +Waiting for review
URL: https://github.com/SSSD/sssd/pull/5782 Title: #5782: CONFDB: Change ownership of config.ldb
Label: -Changes requested
URL: https://github.com/SSSD/sssd/pull/5782 Author: thalman Title: #5782: CONFDB: Change ownership of config.ldb Action: synchronized
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5782/head:pr5782 git checkout pr5782
URL: https://github.com/SSSD/sssd/pull/5782 Title: #5782: CONFDB: Change ownership of config.ldb
thalman commented: """
Just one last improvement before I accept the changes.
updated
By the way, I haven't found why `debian10` is failing.
This was CI problem. On the second run it worked.
"""
See the full comment at https://github.com/SSSD/sssd/pull/5782#issuecomment-946675753
URL: https://github.com/SSSD/sssd/pull/5782 Title: #5782: CONFDB: Change ownership of config.ldb
thalman commented: """
Just one last improvement before I accept the changes.
updated
By the way, I haven't found why `debian10` is failing.
This was CI problem. On the second run it worked.
"""
See the full comment at https://github.com/SSSD/sssd/pull/5782#issuecomment-946675753
URL: https://github.com/SSSD/sssd/pull/5782 Title: #5782: CONFDB: Change ownership of config.ldb
ikerexxe commented: """ LGTM! Thanks for the patches Tomas. """
See the full comment at https://github.com/SSSD/sssd/pull/5782#issuecomment-947383844
URL: https://github.com/SSSD/sssd/pull/5782 Title: #5782: CONFDB: Change ownership of config.ldb
Label: -Waiting for review
URL: https://github.com/SSSD/sssd/pull/5782 Title: #5782: CONFDB: Change ownership of config.ldb
Label: +Bugzilla
URL: https://github.com/SSSD/sssd/pull/5782 Title: #5782: CONFDB: Change ownership of config.ldb
Label: +Accepted
URL: https://github.com/SSSD/sssd/pull/5782 Title: #5782: CONFDB: Change ownership of config.ldb
Label: +Ready to push
URL: https://github.com/SSSD/sssd/pull/5782 Title: #5782: CONFDB: Change ownership of config.ldb
pbrezina commented: """ Pushed PR: https://github.com/SSSD/sssd/pull/5782
* `master` * 7db6cfd0674d45a4e769b0beeb551c89cc89f92f - CONFDB: Change ownership before dropping privileges * 92e1679943fd2a2a50c9e0e176a10a875cb3ac56 - CONFDB: Change ownership of config.ldb
"""
See the full comment at https://github.com/SSSD/sssd/pull/5782#issuecomment-948584573
URL: https://github.com/SSSD/sssd/pull/5782 Title: #5782: CONFDB: Change ownership of config.ldb
Label: +Pushed
URL: https://github.com/SSSD/sssd/pull/5782 Title: #5782: CONFDB: Change ownership of config.ldb
Label: -Accepted
URL: https://github.com/SSSD/sssd/pull/5782 Title: #5782: CONFDB: Change ownership of config.ldb
Label: -Ready to push
URL: https://github.com/SSSD/sssd/pull/5782 Author: thalman Title: #5782: CONFDB: Change ownership of config.ldb Action: closed
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5782/head:pr5782 git checkout pr5782
sssd-devel@lists.fedorahosted.org
-
alexey-tikhonov -
ikerexxe
-
pbrezina
-
thalman