IBMNAS driver fails to resize volume after setting 600 or 660 permissions to volume

Bug #1333252 reported by Sasikanth Eda
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Undecided
Sasikanth Eda

Bug Description

Based on the discussion of drivers setting inappropriate permissions (https://wiki.openstack.org/wiki/OSSN/OSSN-0014), ibmnas driver inherits _set_rw_permissions_for_all() from nfs.py which sets 666 to the volumes.
Changes are expected to be made in nfs.py such that it sets 600 or 660 permissions, while doing so ibmnas driver fails performing the operation _resize_volume_file(). This can be fixed by performing resize operation using root user.

Changed in cinder:
assignee: nobody → Sasikanth Eda (sasikanth)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

Fix proposed to branch: master
Review: https://review.openstack.org/101919

Changed in cinder:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.openstack.org/101919
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=ea56cf255ac956d23eba5bb4f6e7312b944eb5fb
Submitter: Jenkins
Branch: master

commit ea56cf255ac956d23eba5bb4f6e7312b944eb5fb
Author: Sasikanth <email address hidden>
Date: Mon Jun 23 19:40:12 2014 +0530

    Update _resize_volume_file() to support appropriate permissions

    ibmnas driver inherits _set_rw_permissions_for_all() from nfs.py
    which sets 666 to the volumes. Changes are expected to be made in
    nfs.py such that it sets 600 or 660 permissions, by doing so
    ibmnas driver fails performing the operation _resize_volume_file()

    Adding run_as_root in resize_volume operation.

    Change-Id: Iecaf111f96f93b4e24bd81edd6aa82829113b4bb
    Closes-Bug: #1333252

Changed in cinder:
status: In Progress → Fix Committed
Changed in cinder:
milestone: none → juno-2
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in cinder:
milestone: juno-2 → 2014.2
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.