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
jewel: common: global-init: fixup chown of the run directory along with log and asok files #8754
Conversation
Signed-off-by: Karol Mroz <kmroz@suse.com> (cherry picked from commit aecab14)
Signed-off-by: Karol Mroz <kmroz@suse.com> (cherry picked from commit 860977b)
If init flags have changed between the time ceph context was instantiated and now, update them. This can happen, for example, if between manual global_pre_init() and global_init() calls, a daemon adds modifies it's init flags. Signed-off-by: Karol Mroz <kmroz@suse.com> (cherry picked from commit a0ad13d)
Signed-off-by: Karol Mroz <kmroz@suse.com> (cherry picked from commit 206585c)
Also performs log file chown() operations if permission drop has been deferred. Documents that admin socket chown() happen in the common init path. Signed-off-by: Karol Mroz <kmroz@suse.com> (cherry picked from commit 9d51379)
Signed-off-by: Karol Mroz <kmroz@suse.com> (cherry picked from commit 9c82d84)
Signed-off-by: Karol Mroz <kmroz@suse.com> (cherry picked from commit 088cc66)
fchown() returns success, but does not change asok file ownership. chown() does. Signed-off-by: Karol Mroz <kmroz@suse.com> (cherry picked from commit 3d051a5)
…directory along with log and asok files Reviewed-by: Loic Dachary <ldachary@redhat.com>
Reviewed-by: Loic Dachary <ldachary@redhat.com>
test this please |
1 similar comment
test this please |
…directory along with log and asok files Reviewed-by: Loic Dachary <ldachary@redhat.com>
test this please |
test this please (bluefs mount crash) |
test this please (log is no longer available) |
This seems to be failing make check. |
@dachary / @athanatos - Another bluefs mount crash. Performing a local run of make check, unittest_bluefs succeeds. |
test this please (bluefs mount crash) |
@dachary / @athanatos: |
…directory along with log and asok files Reviewed-by:
@liewegas does this backport look good to merge ? It passed the rados suite http://tracker.ceph.com/issues/16344#note-7 except for one failure which I believe is unrelated). |
http://tracker.ceph.com/issues/15608