From: Guillaume Abrioux <gabrioux@redhat.com>
Date: Thu, 7 Oct 2021 14:23:07 +0000 (+0200)
Subject: cephadm: bootstrap --apply-spec shouldn't enforce :z
X-Git-Tag: v17.1.0~718^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F43450%2Fhead;p=ceph.git

cephadm: bootstrap --apply-spec shouldn't enforce :z

There's no need to mount this file with the `:z` option.
read-only access is enough for this file.

Fixes: https://tracker.ceph.com/issues/52855

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
---

diff --git a/src/cephadm/cephadm b/src/cephadm/cephadm
index fd64aef1129..265ed52e9f7 100755
--- a/src/cephadm/cephadm
+++ b/src/cephadm/cephadm
@@ -4614,7 +4614,7 @@ def command_bootstrap(ctx):
                         out, err, code = call_throws(ctx, ['sudo', '-u', ctx.ssh_user, 'ssh-copy-id', '-f', '-i', ssh_key, '-o StrictHostKeyChecking=no', '%s@%s' % (ctx.ssh_user, split[1])])
 
         mounts = {}
-        mounts[pathify(ctx.apply_spec)] = '/tmp/spec.yml:z'
+        mounts[pathify(ctx.apply_spec)] = '/tmp/spec.yml:ro'
 
         out = cli(['orch', 'apply', '-i', '/tmp/spec.yml'], extra_mounts=mounts)
         logger.info(out)