scality: volume driver should use distributed lock manager

Bug #1607059 reported by Szymon Wróblewski
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
New
Undecided
Unassigned

Bug Description

In attempt to make Cinder HA A/A ready ScalityDriver should start using RemoteFSSnapDriverDistributed instead of RemoteFSSnapDriver, this will change locked snapshot operations to use distributed lock manager.
Moreover remotefs.locked_volume_id_operation decorator should be switched to coordination.synchronized:

@coordination.synchronized('{self.driver_prefix}-{volume.id}')
def method_using_volume(self, volume, snapshot):

@coordination.synchronized('{self.driver_prefix}-{snapshot.volume.id}')
def method_using_snapshot(self, snapshot):

Driver should also switch utils.synchronized(..., external=True) decorator to coordination.synchronized unless decorated method could be executed at multiple nodes at the same time.

Tags: dlm
description: updated
description: updated
description: updated
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/348034

Changed in cinder:
assignee: nobody → Szymon Wróblewski (bluex)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on cinder (master)

Change abandoned by Sean McGinnis (<email address hidden>) on branch: master
Review: https://review.openstack.org/348034
Reason: This review is > 4 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

Revision history for this message
Sean McGinnis (sean-mcginnis) wrote : Bug Assignee Expired

Unassigning due to no activity for > 6 months.

Changed in cinder:
assignee: Szymon Wróblewski (bluex) → nobody
status: In Progress → New
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.