From 29d3eaa1a3c7c7ee9a7eb34eae9b3e6b2bd64ca9 Mon Sep 17 00:00:00 2001 From: Sage Weil Date: Thu, 23 Jan 2020 21:25:08 -0600 Subject: [PATCH] qa/tasks/ceph_manager: kludge around /var/log/ceph permissions The ceph.py task normally makes these permissive. But a package upgrade can reset the permissions so that we can't read and write the temp export files. (We put them in these dirs now because it's alreadly mapped out of cephadm containers to the host.) Signed-off-by: Sage Weil --- qa/tasks/ceph_manager.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/qa/tasks/ceph_manager.py b/qa/tasks/ceph_manager.py index 345466220d2..8441506c959 100644 --- a/qa/tasks/ceph_manager.py +++ b/qa/tasks/ceph_manager.py @@ -374,6 +374,9 @@ class OSDThrasher(Thrasher): # Copy export file to the other machine self.log("Transfer export file from {srem} to {trem}". format(srem=exp_remote, trem=imp_remote)) + # just in case an upgrade make /var/log/ceph unreadable by non-root, + exp_remote.run(args=['sudo', 'chmod', '777', '/var/log/ceph']) + imp_remote.run(args=['sudo', 'chmod', '777', '/var/log/ceph']) tmpexport = Remote.get_file(exp_remote, exp_host_path) Remote.put_file(imp_remote, tmpexport, exp_host_path) os.remove(tmpexport) -- 2.39.5