[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/ +/871643 /opendev. org/openstack/ manila/ commit/ c61c1977c25e151 fd2839febd08a7f dac6b1f7b8
Committed: https:/
Submitter: "Zuul (22348)"
Branch: master
commit c61c1977c25e151 fd2839febd08a7f dac6b1f7b8
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
Change-Id: Ic5b5b42b882ce1