[Cephfs] Fix erroneous share mode override on ensure_shares
In order to ensure that a share existed, we used to send the same
request to Ceph as we would do with the share creation. In that
request, we will set the share mode, which is a value set by
the deployer through the `cephfs_volume_mode` config option.
If the share was already created, Ceph would not override the value
we were forwarding for the permission mode. After a bugfix [1], the
value of mode started being reinforced when sent, even if the
subvolume was already created. If the value for
`cephfs_volume_mode` changed after the share was created, the mode
is now being overwritten after manila startup on ensure_shares.
Fixed this issue by modifying the ensure_shares to only confirm if
the export exists.
Reviewed: https:/ /review. opendev. org/c/openstack /manila/ +/874743 /opendev. org/openstack/ manila/ commit/ f320288f47b3f98 6a6df8349225035 27f446a728
Committed: https:/
Submitter: "Zuul (22348)"
Branch: stable/xena
commit f320288f47b3f98 6a6df8349225035 27f446a728
Author: silvacarloss <email address hidden>
Date: Tue Jan 24 11:49:14 2023 -0300
[Cephfs] Fix erroneous share mode override on ensure_shares
In order to ensure that a share existed, we used to send the same volume_ mode` config option.
request to Ceph as we would do with the share creation. In that
request, we will set the share mode, which is a value set by
the deployer through the `cephfs_
If the share was already created, Ceph would not override the value volume_ mode` changed after the share was created, the mode
we were forwarding for the permission mode. After a bugfix [1], the
value of mode started being reinforced when sent, even if the
subvolume was already created. If the value for
`cephfs_
is now being overwritten after manila startup on ensure_shares.
Fixed this issue by modifying the ensure_shares to only confirm if
the export exists.
[1] https:/ /tracker. ceph.com/ issues/ 54375
Closes-Bug: #2002394 346b4b46d1b29aa 31740933e0e fd2839febd08a7f dac6b1f7b8) ac0f86aa3fe1562 72deabb48e) a6f8e2983038a8c 035b6dc98a)
Change-Id: Ic5b5b42b882ce1
(cherry picked from commit c61c1977c25e151
(cherry picked from commit f8d0e0cad271baa
(cherry picked from commit 3d6fcc99458b07f