openstack
/
cinder
Code Issues Proposed changes

Pylint fixes for GlusterFS driver 

Remove unused 're' import
Remove unused parameter for _create_snapshot()

Change-Id: I40411b6206d282ae07c905faa44dcccecb45c120
This commit is contained in:
Eric Harney 2013-12-16 17:44:02 -05:00
parent f02b2dc71f
commit 8904259b94
1 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
cinder/volume/drivers/glusterfs.py
View File

@ -19,7 +19,6 @@ import errno
import hashlib
import json
import os
import re
import stat
import tempfile
import time
@ -453,8 +452,7 @@ class GlusterfsDriver(nfs.RemoteFsDriver):
LOG.debug(_('volume id: %s') % snapshot['volume_id'])
path_to_disk = self._local_path_volume(snapshot['volume'])
snap_id = snapshot['id']
self._create_snapshot(snapshot, path_to_disk, snap_id)
self._create_snapshot(snapshot, path_to_disk)
def _create_qcow2_snap_file(self, snapshot, backing_filename,
new_snap_path):
@ -483,7 +481,7 @@ class GlusterfsDriver(nfs.RemoteFsDriver):
new_snap_path]
self._execute(*command, run_as_root=True)
def _create_snapshot(self, snapshot, path_to_disk, snap_id):
def _create_snapshot(self, snapshot, path_to_disk):
"""Create snapshot (offline case)."""
# Requires volume status = 'available'