[Kimchi-devel] [PATCH 1/2] pep8 cleanup for featuretests.py

Rodrigo Trujillo rodrigo.trujillo at linux.vnet.ibm.com
Mon Dec 30 16:35:37 UTC 2013


Reviewed-by: Rodrigo Trujillo <rodrigo.trujillo at linux.vnet.ibm.com>

On 12/30/2013 10:53 AM, Aline Manera wrote:
> From: Aline Manera <alinefm at br.ibm.com>
>
> This patch cleans up pep8 style issue in featuretests.py
>
> Signed-off-by: Aline Manera <alinefm at br.ibm.com>
> ---
>   Makefile.am                |    1 +
>   src/kimchi/featuretests.py |    8 +++++---
>   2 files changed, 6 insertions(+), 3 deletions(-)
>
> diff --git a/Makefile.am b/Makefile.am
> index 0fd92c8..a9b2b9c 100644
> --- a/Makefile.am
> +++ b/Makefile.am
> @@ -43,6 +43,7 @@ PEP8_WHITELIST = \
>   	src/kimchi/cachebust.py \
>   	src/kimchi/config.py.in \
>   	src/kimchi/disks.py \
> +	src/kimchi/featuretests.py \
>   	src/kimchi/root.py \
>   	src/kimchi/server.py \
>   	plugins/__init__.py \
> diff --git a/src/kimchi/featuretests.py b/src/kimchi/featuretests.py
> index 4cabdc9..6843a26 100644
> --- a/src/kimchi/featuretests.py
> +++ b/src/kimchi/featuretests.py
> @@ -18,7 +18,7 @@
>   #
>   # You should have received a copy of the GNU Lesser General Public
>   # License along with this library; if not, write to the Free Software
> -# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301  USA
> +# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301 USA
>
>   import libvirt
>   import os
> @@ -67,6 +67,7 @@ ISO_STREAM_XML = """
>     </devices>
>   </domain>"""
>
> +
>   class FeatureTests(object):
>
>       @staticmethod
> @@ -85,7 +86,8 @@ class FeatureTests(object):
>
>       @staticmethod
>       def qemu_supports_iso_stream():
> -        cmd = "qemu-io http://127.0.0.1:8000/images/icon-fedora.png -c 'read -v 0 512'"
> +        cmd = "qemu-io http://127.0.0.1:8000/images/icon-fedora.png \
> +              -c 'read -v 0 512'"
>           proc = subprocess.Popen(cmd, stdout=subprocess.PIPE,
>                                   stderr=subprocess.PIPE, shell=True)
>           stdout, stderr = proc.communicate()
> @@ -98,7 +100,7 @@ class FeatureTests(object):
>                  "'read -v 0 512'"]
>           proc = subprocess.Popen(cmd, stdout=subprocess.PIPE,
>                                   stderr=subprocess.PIPE)
> -        thread = threading.Thread(target = proc.communicate)
> +        thread = threading.Thread(target=proc.communicate)
>           thread.start()
>           thread.join(5)
>




More information about the Kimchi-devel mailing list