Comment 2 for bug 1221205

Revision history for this message
Ala Rezmerita (arezmerita) wrote :

For security resons, we can't change the export options used, but the user cinder has already read/write permissions :

# ls -la /var/lib/cinder/netapp-standard/e6403532daa49d571abc394dae312a63
total 1614188
drwxrwxrwx 2 root root 4096 Sep 5 11:17 .
drwxr-xr-x 3 cinder cinder 4096 Aug 27 12:10 ..
drwxrwxrwx 2 root root 4096 Aug 26 09:30 .snapshot
-rw-rw-rw- 1 nobody nogroup 1073741824 Sep 5 11:17 volume-4209e57d-cb63-426e-8932-25dd89f280e2
-rw-rw-rw- 1 nobody nogroup 1073741824 Aug 30 15:37 volume-97b3c2b7-3658-411f-bae4-2570c4f46f1f
-rw-rw-rw- 1 nobody nogroup 2147483648 Sep 4 15:42 volume-9ca4e129-bb07-4e2b-86dd-f4c1dec6c19a
-rw-rw-rw- 1 nobody nogroup 2147483648 Sep 4 14:29 volume-c690b911-3582-4704-9a7b-a59fbdd51798
root@d-ovcclc-0005:/usr/lib/python2.7/dist-packages/cinder#

So why change the ownership by default, and not try first to open the file and change the ownership only if the open fails.