From 53bb81c585ca9461f9fb5ea715927f6f93d0062e Mon Sep 17 00:00:00 2001
From: Prashanth Pai <ppai@redhat.com>
Date: Wed, 28 Aug 2013 16:12:00 +0530
Subject: [PATCH] Include extras/ dir in flake8 source code check
Moving forward all python scripts in extras/ are to
be checked for pep8 compliancy.
Change-Id: I4f20c1c97590049ba59fd432cbabba89df101e81
Signed-off-by: Prashanth Pai <ppai@redhat.com>
Reviewed-on: http://review.gluster.org/5714
Reviewed-by: Peter Portante <pportant@redhat.com>
Reviewed-by: Luis Pabon <lpabon@redhat.com>
Tested-by: Luis Pabon <lpabon@redhat.com>
---
extras/hook-scripts/S40ufo-stop.py | 4 +---
tox.ini | 2 +-
2 files changed, 2 insertions(+), 4 deletions(-)
diff --git a/extras/hook-scripts/S40ufo-stop.py b/extras/hook-scripts/S40ufo-stop.py
index d7d6756..92418df 100755
--- a/extras/hook-scripts/S40ufo-stop.py
+++ b/extras/hook-scripts/S40ufo-stop.py
@@ -16,7 +16,6 @@
# limitations under the License.
-import os
from optparse import OptionParser
if __name__ == '__main__':
@@ -25,14 +24,13 @@ if __name__ == '__main__':
from gluster.swift.common.Glusterfs import get_mnt_point, unmount
except ImportError:
import sys
- sys.exit("Openstack Swift does not appear to be installed properly");
+ sys.exit("Openstack Swift does not appear to be installed properly")
op = OptionParser(usage="%prog [options...]")
op.add_option('--volname', dest='vol', type=str)
op.add_option('--last', dest='last', type=str)
(opts, args) = op.parse_args()
-
mnt_point = get_mnt_point(opts.vol)
if mnt_point:
unmount(mnt_point)
diff --git a/tox.ini b/tox.ini
index 7793c83..7269319 100644
--- a/tox.ini
+++ b/tox.ini
@@ -21,7 +21,7 @@ downloadcache = ~/cache/pip
[testenv:pep8]
changedir = {toxinidir}
commands =
- flake8 gluster test
+ flake8 gluster test extras
[testenv:cover]
setenv = NOSE_WITH_COVERAGE=1